Bug 165310 - Calc opens settings for new conditional formatting instead of settings for existing one
Summary: Calc opens settings for new conditional formatting instead of settings for ex...
Status: RESOLVED DUPLICATE of bug 160252
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
25.2.0.3 release
Hardware: x86-64 (AMD64) All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, regression
Depends on:
Blocks:
 
Reported: 2025-02-18 07:24 UTC by Johannes
Modified: 2025-02-19 09:49 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
Example file (34.99 KB, application/vnd.oasis.opendocument.spreadsheet)
2025-02-18 07:25 UTC, Johannes
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Johannes 2025-02-18 07:24:57 UTC
Description:
Opening the settings for editing an existing conditional formatting after editing another conditional formatting, gives the settings for a new conditional formatting instead of the possibility to edit the existing one.

Steps to Reproduce:
1. Open attached example file
2. Click on Format > Conditional > Manage...
3. Select one of the conditional formats (doesn't matter)
4. Click on Edit...
5. Click on OK or Cancel (doesn't matter)
6. Select one of the conditional formats (doesn't matter)
7. Click on Edit...

Actual Results:
Settings for new conditional formatting for the selected cell/range is shown.

Expected Results:
Settings for the existing selected conditional formatting is shown.


Reproducible: Always


User Profile Reset: Yes

Additional Info:
Exists in 25.2.0.3, but also in:

Version: 25.8.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: f4cf9971ceaf72a630fe1d1dd8154326daca9832
CPU threads: 4; OS: Linux 6.13; UI render: default; VCL: kf5 (cairo+wayland)
Locale: nl-NL (nl_NL.UTF-8); UI: en-US
Calc: threaded
Comment 1 Johannes 2025-02-18 07:25:32 UTC
Created attachment 199267 [details]
Example file
Comment 2 Xisco Faulí 2025-02-18 09:46:50 UTC
Reproduced in

Version: 25.8.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 7da1497aa462e2b719aa9b308a749caf7b9a19b1
CPU threads: 8; OS: Linux 6.1; UI render: default; VCL: gtk3
Locale: es-ES (es_ES.UTF-8); UI: en-US
Calc: threaded
Comment 3 Xisco Faulí 2025-02-18 10:19:30 UTC
The dialog changed in bdd06e255cf89cbc1ee100306d27739dbb5290a4 and then when the commit was reverted in 529524cdb1847f40e0aebaa904f58e3f8a1bdde2 it was already happening. The revert is not 1 to 1 equal to the original commit so something might be wrong in the revert
Comment 4 Xisco Faulí 2025-02-18 13:00:55 UTC
so in my local build I did:
git checkout bdd06e255cf89cbc1ee100306d27739dbb5290a4 and build it -> issue reproduced
git reset --hard HEAD~1 and build -> issue not reproduced

So issue introduced by

commit bdd06e255cf89cbc1ee100306d27739dbb5290a4	[log]
author	Pranam Lashkari <lpranam@collabora.com>	Wed Jun 26 09:16:43 2024 +0100
committer	Caolán McNamara <caolan.mcnamara@collabora.com>	Thu Aug 15 18:40:21 2024 +0200
tree ed1b148992af52e5370863d7872d9b6b23c4501e
parent fcd18913d5e8450681522c85cf5313f1d3f43aa9 [diff]

tdf#162475 sc: now conditions can be edited from easy conditional format dialog

meaning the revert is incorrect...
Comment 5 Xisco Faulí 2025-02-18 14:58:38 UTC
(In reply to Xisco Faulí from comment #3)
> The dialog changed in bdd06e255cf89cbc1ee100306d27739dbb5290a4 and then when
> the commit was reverted in 529524cdb1847f40e0aebaa904f58e3f8a1bdde2 it was
> already happening. The revert is not 1 to 1 equal to the original commit so
> something might be wrong in the revert

of course the revert commit is 36108acc124efd73a5e69f5780e8e8220cf0875c, then one in master, not in co-24.04
Comment 6 Xisco Faulí 2025-02-18 15:18:29 UTC
Not reproduced with Collabora Online

Version: 24.04.13.20250218 (X86_64)
Build ID: 190593257ad7636a9f88ff765ab66ff7307b9dca
CPU threads: 4; OS: Windows 10.0 Build 19045; UI render: Skia/Raster; VCL: win
Locale: es-ES (es_ES); UI: en-US
Calc: threaded
Comment 7 Xisco Faulí 2025-02-18 15:18:50 UTC
Sorry, Collabora Office
Comment 8 Xisco Faulí 2025-02-19 09:49:09 UTC
Actually this is a duplicate of bug 160252

*** This bug has been marked as a duplicate of bug 160252 ***