Summary: | Hyperlink to Draw page does not update after change in page position, since LO 6.4 | ||
---|---|---|---|
Product: | LibreOffice | Reporter: | Stéphane Guillou (stragu) <stephane.guillou> |
Component: | Draw | Assignee: | Hossein <hossein> |
Status: | VERIFIED FIXED | ||
Severity: | normal | CC: | kara, yeohakoon |
Priority: | medium | Keywords: | bibisected, bisected, regression |
Version: | 6.4.0.3 release | ||
Hardware: | x86-64 (AMD64) | ||
OS: | All | ||
See Also: |
https://bugs.documentfoundation.org/show_bug.cgi?id=112126 https://bugs.documentfoundation.org/show_bug.cgi?id=105282 https://bugs.documentfoundation.org/show_bug.cgi?id=139472 |
||
Whiteboard: | target:24.8.0 target:24.2.3 target:7.6.7 | ||
Crash report or crash signature: | Regression By: | Muhammet Kara | |
Bug Depends on: | |||
Bug Blocks: | 107733 |
Description
Stéphane Guillou (stragu)
2024-03-12 07:56:55 UTC
*** Bug 160110 has been marked as a duplicate of this bug. *** If you try renaming the slides to "P1, P2, and P3" and then remove "P2", you will see that the link is no longer updated, but as the page number is also no longer updated, the link will continue to work. So, there are 2 things here. When a page is renamed and/or removed, or a new page is created in the middle of others: 1. Renaming the pages is done to update its name according to its position, which is only done when original name is preserved. Here, "Page" is used instead of "Slide", and renaming works. If you remove "Page 2", "Page 3" becomes "Page 2". 2. Changing the links to keep them up-to-date. This is also done only when the name "Slide" is preserved. It does not currently work with "Page". I am changing SdDrawDocument::UpdatePageRelativeURLs() to fix this issue. Another issue, I see many duplicate names of "Default" in the latest version when trying to insert links via "Target in Document": Notes > Default Handouts > Default Master Page > Default Thanks for looking into it, Hossein. Please note relevant comments in associated bugs: - bug 139472 comment 4 (internal link broken by change in UI language, inherited) - bug 105282 comment 15 (form control internal link doesn't update when it should, inherited) (In reply to Hossein from comment #2) > Notes > Default > Handouts > Default > Master Page > Default Noticed that as well, something for another report. Notes and Handouts are not relevant to Draw so shouldn't even be listed. And also, the section with the pages is still called "Slides". (In reply to Stéphane Guillou (stragu) from comment #3) > Thanks for looking into it, Hossein. > Please note relevant comments in associated bugs: > - bug 139472 comment 4 (internal link broken by change in UI language, > inherited) > - bug 105282 comment 15 (form control internal link doesn't update when it > should, inherited) You're welcome! The below patch should fix the specific issue discussed here: tdf#160162 Fix hyperlink update in Draw https://gerrit.libreoffice.org/c/core/+/164716 For the others, more changes will be needed. Hossein committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/c1f097854f89f35cfc0f097a10384014042b95d6 tdf#160162 Fix hyperlink update in Draw It will be available in 24.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Thanks Hossein! Verified the fix in my own build on Linux. Follow-up report, while you're looking at that function: bug 160174. Hossein committed a patch related to this issue. It has been pushed to "libreoffice-24-2": https://git.libreoffice.org/core/commit/2ad14fdade187579b2abf3c76a08780b77614662 tdf#160162 Fix hyperlink update in Draw It will be available in 24.2.3. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Hossein committed a patch related to this issue. It has been pushed to "libreoffice-7-6": https://git.libreoffice.org/core/commit/58352309d1299cbbaf465991f5cec17c2ae32cee tdf#160162 Fix hyperlink update in Draw It will be available in 7.6.7. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. |