Created attachment 175073 [details] bug_repeated_Cell_Name_in_formula 1) I have a formula 2) I try to change with drug & drop using mouse the cell reference. 3) Cell name is repeared in formula as many times as you move it around. Attachment: a) In cell C6 press "F2" in order to access cell content b) Then, in Cell AA2, a "Red" Cell appears. c) Try to move this "Red" cell to a new cell d) Cell name is repeared in formula as many times as you move it around.
Created attachment 175074 [details] bug_repeated_Cell_Name_in_formula - tidier a more tidier example Cell a6, press F2, and then move C2 cell to new locations.
This seems to have begun at the below commit. Adding Cc: to Eike Rathke ; Could you possibly take a look at this one? Thanks 60af280fe15025d08d5f1ceecdb25d243c68f1b2 is the first bad commit commit 60af280fe15025d08d5f1ceecdb25d243c68f1b2 Author: Jenkins Build User <tdf@pollux.tdf> Date: Wed Jul 28 19:07:59 2021 +0200 source 516318113f0bd2b3c658aba9b285165e63a280e2 https://git.libreoffice.org/core/+/516318113f0bd2b3c658aba9b285165e63a280e2 Resolves: tdf#76310 Preserve whitespace TAB, CR, LF in formula expressions
Still repro Version: 7.5.2.1 (X86_64) / LibreOffice Community Build ID: e8bf3b441b8370f8440b0339fd9490765a8d57ca CPU threads: 16; OS: Linux 5.19; UI render: default; VCL: gtk3 Locale: ro-RO (ro_RO.UTF-8); UI: en-US Calc: threaded
Shouldn't the "Regression By" field be set to Eike Rathke?
Hmm... OTOH, the formula includes #REF! errors. Why anyone would care about solving something with a broken formula, before actually solving the formula itself first? Is this repeatable with a non-broken formula?
(In reply to ady from comment #4) There's no value in calling people out ;p
It's not even a regression because previously editing multi-line formulas wasn't possible at all. What happens seems to be that something blindly appends the new reference to the end of the first line instead of modifying the correct string position. (In reply to ady from comment #5) > Hmm... OTOH, the formula includes #REF! errors. Why anyone would care about > solving something with a broken formula, before actually solving the formula > itself first? Unrelated. > Is this repeatable with a non-broken formula? Yes. Just enter =B1 +B2 as multi-line formula in A1, then F2 to edit and drag&drop cell B2.
(In reply to Eike Rathke from comment #7) > Yes. Just enter > > =B1 > +B2 > > as multi-line formula in A1, then F2 to edit and drag&drop cell B2. That example also shows that cell B2 is marked (so it can be dragged), but B1 is not.
Eike Rathke committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/5f6d27711a297d1851dc6bc713fd16c44014cf5c Resolves: tdf#144547 ScRangeFindList: handle references in multi-line formula It will be available in 7.6.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.
Pending review https://gerrit.libreoffice.org/c/core/+/151010 for 7-5
(In reply to Commit Notification from comment #9) > It will be available in 7.6.0. Works as expected now in 7.6.alpha.
Eike Rathke committed a patch related to this issue. It has been pushed to "libreoffice-7-5": https://git.libreoffice.org/core/commit/f6a783af88b571ff1f3df9bda31eb0aac66a2c5f Resolves: tdf#144547 ScRangeFindList: handle references in multi-line formula It will be available in 7.5.4. 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.