| Summary: | calc: editing: ui: weakness with named ranges, move of cell to another sheet destroys referencing to local name reference there | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | b. <newbie-02> |
| Component: | Calc | Assignee: | Eike Rathke <erack> |
| Status: | VERIFIED FIXED | ||
| Severity: | normal | CC: | erack, xiscofauli |
| Priority: | medium | Keywords: | bibisected, bisected, regression |
| Version: | 5.3 all versions | ||
| Hardware: | All | ||
| OS: | All | ||
| See Also: | https://bugs.documentfoundation.org/show_bug.cgi?id=130825 | ||
| Whiteboard: | target:7.0.0 target:6.4.2 target:6.3.6 | ||
| Crash report or crash signature: | Regression By: | ||
|
Description
b.
2020-02-02 23:03:51 UTC
I can reproduce it in Version: 7.0.0.0.alpha0+ Build ID: 28d844a589e52abfe62dc66b888e78665221ba28 CPU threads: 4; OS: Linux 4.19; UI render: default; VCL: gtk3; Locale: en-US (en_US.UTF-8); UI-Language: en-US Calc: threaded Steps to reproduce: 1. Open https://bugs.documentfoundation.org/attachment.cgi?id=102780 2. Cut B1 in sheet 2 3. Paste it to B1 in sheet 3 -> B3 changes to =#NAME? Regression introduced by: https://cgit.freedesktop.org/libreoffice/core/commit/?id=d3645961d98ce9f0cfb1ef7f0aba7bf020b90945 author Eike Rathke <erack@redhat.com> 2016-10-12 19:47:44 +0200 committer Eike Rathke <erack@redhat.com> 2016-10-12 19:48:55 +0200 commit d3645961d98ce9f0cfb1ef7f0aba7bf020b90945 (patch) tree 1c8abea94008f8a14b4da8b9fb3a73489a3f71d3 parent c76f90da1c54cd497c6dce6bdaeeb919f2cbe82b (diff) create NoName error token instead of setting error at the token array Bisected with: bibisect-linux-64-5.3 Adding Cc: to Eike Rathke Eike Rathke committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/180fef2a54207555c39fb63ead5e66ba3a71a0cf Resolves: tdf#130370 Increment sheet-local name only for moved cells It will be available in 7.0.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. The bisected commit isn't the culprit but revealed an existing bad handling elsewhere. Pending review https://gerrit.libreoffice.org/c/core/+/88946 for 6-4 Eike Rathke committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/0c6bf98cd94307ef7b0896e6643be6161691f9c9 Related: tdf#130370 Let cut&paste/move of a sheet-local name reference ... It will be available in 7.0.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. Eike Rathke committed a patch related to this issue. It has been pushed to "libreoffice-6-4": https://git.libreoffice.org/core/commit/bef90981e3f9f9dc3e4731c5bcbab315e52b9baf Resolves: tdf#130370 Do not increment sheet-local name on cut&paste / move It will be available in 6.4.2. 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. Verified in Version: 7.0.0.0.alpha0+ Build ID: bc994a4b01cf61444452bb377010c7c4cc377698 CPU threads: 4; OS: Linux 4.19; UI render: default; VCL: gtk3; Locale: en-US (en_US.UTF-8); UI-Language: en-US Calc: threaded @Eike, thanks for fixing this issue! Eike Rathke committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/4e5f875cdb81a1c1ca881ef8c90a63c2afd2ba78 Unit test tdf#130370 cut&paste / move with sheet-local named reference It will be available in 7.0.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. Eike Rathke committed a patch related to this issue. It has been pushed to "libreoffice-6-3": https://git.libreoffice.org/core/commit/461824307d2ee701f3026f75a9ee5a7865cd6e7a Resolves: tdf#130370 Do not increment sheet-local name on cut&paste / move It will be available in 6.3.6. 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. it's better in actual 7.0.0.0.a0+ master, thanks, |