Bug 138601 - Conditional formats of a XLSX-File are opened and saved in a different order than the original one. This breaks the documents appearance
Summary: Conditional formats of a XLSX-File are opened and saved in a different order ...
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
5.0.0.5 release
Hardware: All All
: medium enhancement
Assignee: Tibor Nagy
URL:
Whiteboard: target:7.6.0 target:7.5.2
Keywords:
: 91477 (view as bug list)
Depends on:
Blocks: XLSX-Conditional-Formatting
  Show dependency treegraph
 
Reported: 2020-12-01 14:31 UTC by Tim
Modified: 2024-03-14 04:54 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
original file (156.28 KB, application/vnd.openxmlformats-officedocument.spreadsheetml.sheet)
2020-12-01 14:34 UTC, Tim
Details
file which was opened and saved in LO and is broken now (141.17 KB, application/vnd.openxmlformats-officedocument.spreadsheetml.sheet)
2020-12-01 14:35 UTC, Tim
Details
The example file in Excel 13 and LO7.2 nightly (149.84 KB, image/png)
2020-12-01 15:00 UTC, NISZ LibreOffice Team
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Tim 2020-12-01 14:31:54 UTC
Description:
I created a template in MS Excel which relies heavily in conditional formats. The order of them is important, as they may overlap and are rendered in the order defined in Excel. 

If one user does open this file in LibreOffice and saves it, the conditional format is broken. 

Attached please find an example before and after opening it in LO. 

Steps to Reproduce:
1. View the original file (attached) in Excel
2. Open it in LO and see the difference here.
3. Save the file in LO 
4. Open it in Excel again and see that the original format is broken now.

Actual Results:
The conditional formatting is wrong after editing the file in LO

Expected Results:
The format should be untouched or at least the order of conditional formats should not be changed.


Reproducible: Always


User Profile Reset: Yes



Additional Info:
Version: 7.0.3.1 (x64)
Build ID: d7547858d014d4cf69878db179d326fc3483e082
CPU threads: 12; OS: Windows 10.0 Build 19042; UI render: Skia/Vulkan; VCL: win
Locale: de-DE (de_DE); UI: de-DE
Calc: threaded

------

> Is OpenGL enabled?

I was not able to check this in 7.0.
Comment 1 Tim 2020-12-01 14:34:23 UTC
Created attachment 167716 [details]
original file
Comment 2 Tim 2020-12-01 14:35:07 UTC
Created attachment 167717 [details]
file which was opened and saved in LO and is broken now
Comment 3 NISZ LibreOffice Team 2020-12-01 15:00:59 UTC
Created attachment 167719 [details]
The example file in Excel 13 and LO7.2 nightly

Repro with:

Version: 7.2.0.0.alpha0+ (x64)
Build ID: 4e63ec27b69fa01ff610c894c9fbf05c377a6179
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: default; VCL: win
Locale: en-US (hu_HU); UI: en-US
Calc: CL

Already bad by opening time. Saving also changes the order in Excel.
Comment 4 Justin L 2022-10-26 20:51:22 UTC
repro 7.5+
In column G, all the cells are plain white without gridlines, while it ought to have a green 90%, a yellow 25%, and a green 10%.
Comment 5 Justin L 2022-10-26 21:05:47 UTC
*** Bug 91477 has been marked as a duplicate of this bug. ***
Comment 6 Commit Notification 2023-02-15 10:17:59 UTC
Tibor Nagy committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/8c9a6abf30e9ff1ebd5647f7c271e0d64643860a

tdf#138601 XLSX import: fix priority of conditional formatting rules

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.
Comment 7 Commit Notification 2023-02-15 15:15:04 UTC
Tibor Nagy committed a patch related to this issue.
It has been pushed to "libreoffice-7-5":

https://git.libreoffice.org/core/commit/90f4ea09cbe34e26e473ed2cf1c3457cf0f41e5e

tdf#138601 XLSX import: fix priority of conditional formatting rules

It will be available in 7.5.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.
Comment 8 Tim 2023-02-16 15:01:31 UTC
I've just tested the nightly build and my issue seems to be fixed now. Thank you very much for your support. Looking forward for the stable release :-)
Comment 9 NISZ LibreOffice Team 2023-02-21 12:00:41 UTC
VERIFIED IN:
Version: 7.6.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 74dc2ac66f0130bcd77cf1bbe417b22865beb067
CPU threads: 8; OS: Windows 10.0 Build 19044; UI render: Skia/Vulkan; VCL: win
Locale: hu-HU (hu_HU); UI: hu-HU
Calc: CL threaded