Issue 124197 - Link button in Hyperlink Toolbar Works Only First Time in Document
Summary: Link button in Hyperlink Toolbar Works Only First Time in Document
Status: CLOSED FIXED
Alias: None
Product: Writer
Classification: Application
Component: ui (show other issues)
Version: 4.0.1
Hardware: PC Windows 7
: P3 Normal (vote)
Target Milestone: 4.1.0
Assignee: Armin Le Grand
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-02-06 19:55 UTC by B. Jerew
Modified: 2017-05-20 10:35 UTC (History)
3 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: 4.1.0-dev
Developer Difficulty: ---


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description B. Jerew 2014-02-06 19:55:02 UTC
For example, I want to link the text "Text" to "http://www.text.com" Highlighting "Text" automatically loads "Text" in the first dropdown box. I click and drag the URL to the URL box, which populates that box. Then, the Link button is enabled. The first time I do this in a document, it works flawlessly, assigning the URL to the text in one click.

However, this is where the bug comes in, any subsequent attempt to use the link button, on another text that I want to assign a URL, nothing happens. I have to click the link drop button and select "As Text" in order to complete the link.
Comment 1 Edwin Sharp 2014-02-06 20:28:31 UTC
Please attach example.
Comment 2 B. Jerew 2014-02-06 20:44:17 UTC
will have to try a desktop video capture…
Comment 3 B. Jerew 2014-02-07 12:05:52 UTC
http://youtu.be/v5d-nIbCFts
Comment 4 Edwin Sharp 2014-02-07 12:42:26 UTC
Thank you bnjroo
Confirmed with
AOO410m1(Build:9750)  -  Rev. 1562493
Rev.1562493
Win 7
Comment 5 Armin Le Grand 2014-02-07 16:39:05 UTC
I took a look. This happens because there is a member 'bSend' which is set to true when the URL gets send to the app. Irritatingly there is no place where it gets reset to false, except that the toolber gets reconstructed, that cannot be intended. Tried to find info about 'bSend' in the history, but no hints. It seems to be intended to not send the URL to the app twice, but even that works well (stepping over setting it). When thinking about at which conditions to reset it to false its pretty much every one, so the question is why it exists at all.
Tried to watch changes to the URL string itself, but that is ineffective. Even when resetting every time SvxHyperlinkDlg::EnableLink() is called and enables the Link button it will be reset pretty much all the time.
It may have been used to prevent unwanted endless loops when SvxHyperlinkDlg::SendToApp triggers something that indirectly triggers some updates to the URL toolbar and may trigger SvxHyperlinkDlg::SendToApp again, but then it should be sloved locally.
I tend to remove the bool and ensure that SvxHyperlinkDlg::SendToApp will not run recursively, testing that...
Comment 6 Armin Le Grand 2014-02-07 16:47:05 UTC
Works as expected and will not be more risky than before, preparing commit...
Comment 7 SVN Robot 2014-02-07 16:52:19 UTC
"alg" committed SVN revision 1565724 into trunk:
i124197 allow URL Toolbar to send URLs to the app more than once
Comment 8 Armin Le Grand 2014-02-07 16:52:38 UTC
Okay, done.
Comment 9 B. Jerew 2014-02-07 22:02:42 UTC
Noted status change to resolved/fixed. Any timeframe on when this update will be available to end user or how to patch?

Thank you.
Comment 10 Edwin Sharp 2014-02-08 16:16:29 UTC
(In reply to bnjroo from comment #9)
> Noted status change to resolved/fixed. Any timeframe on when this update
> will be available to end user or how to patch?
> 
> Thank you.

Release date of 4.1 unknown to me.
Snapshots are available here:
http://ci.apache.org/projects/openoffice/#win
Comment 11 B. Jerew 2014-02-08 20:11:22 UTC
Good job ALG & ES
Thanks for your help.
-LW