Bug 72149 - Conditional formatting does not update icon sets when cells are previously copied and condition edited
Summary: Conditional formatting does not update icon sets when cells are previously co...
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
4.1.3.2 release
Hardware: Other All
: medium normal
Assignee: Markus Mohrhard
URL:
Whiteboard: target:4.3.0 target:4.2.0
Keywords:
Depends on:
Blocks:
 
Reported: 2013-11-29 13:18 UTC by Gerry
Modified: 2013-12-02 05:18 UTC (History)
0 users

See Also:
Crash report or crash signature:


Attachments
Cannot-change-icon-set-after-copying-cells.ods (8.99 KB, application/vnd.oasis.opendocument.spreadsheet)
2013-11-29 13:18 UTC, Gerry
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Gerry 2013-11-29 13:18:20 UTC
Created attachment 89994 [details]
Cannot-change-icon-set-after-copying-cells.ods

LibreOffice Calc does not update the icon set if the cells (containing conditional formatting) are previously copied and the user changes the icon set. However, the changed icon set is correctly applied when saving the file and opening the file again. I assume that it is some refreshing error.

How to reproduce:
* Open attached .ods file
* Copy cells A1:A10 to C1:C10
* Select the cells C1:C10 and go to format->conditional formatting -> icon sets and try to change the icon set.
* Nothing happens. It seems that the old icon set in C1:C10 cannot be changed.
* However, if you save the file, close the file and open it again, the change is applied.

System: Version: 4.1.3.2 Build ID: 410m0(Build:2) from PPA, Ubuntu 13.04
Comment 1 Markus Mohrhard 2013-12-02 03:11:11 UTC
Related to the changed copy paste code. I understand the problem but need to think about the correct solution.
Comment 2 Markus Mohrhard 2013-12-02 05:18:45 UTC
Sadly the bugzilla script does not work right now.

http://cgit.freedesktop.org/libreoffice/core/commit/?id=7c2936757ca10ccb692b05e6564783313f3576d1
http://cgit.freedesktop.org/libreoffice/core/commit/?id=a14cfd3d77104aee3e3c3d981a135161295197df

are the fix and the unit test for master. It has been pushed to 4-2 as well.