Bug 145032 - Conditional Formatting breaks cell formatting
Summary: Conditional Formatting breaks cell formatting
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
7.0.4.2 release
Hardware: x86-64 (AMD64) Linux (All)
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, regression
Depends on:
Blocks: Conditional-Formatting
  Show dependency treegraph
 
Reported: 2021-10-10 08:13 UTC by armin
Modified: 2023-04-22 12:43 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
ODS file demonstrating the bug (14.72 KB, application/vnd.oasis.opendocument.spreadsheet)
2021-10-10 08:13 UTC, armin
Details
Screenshot how the bug looks in LO 7.0.4.2 (11.16 KB, image/png)
2021-10-10 08:15 UTC, armin
Details
Screenshow how the file looks in LO 6.1.5.2 (11.87 KB, image/png)
2021-10-10 08:18 UTC, armin
Details

Note You need to log in before you can comment on or make changes to this bug.
Description armin 2021-10-10 08:13:05 UTC
Created attachment 175614 [details]
ODS file demonstrating the bug

The error occurred in an ODS file being edited the last time with LO 6.1.5 when the behavior was as I expected.
I have a list of data where in the first column are timestamps and other data right to it. I created a conditional format to create a border on the top if the difference of the timestamp in the first column to the one in the row above is above a limit (to make lines between the days).
The timestamps are all formatted with "DD.MM.YY hh:mm" (done by LO automatically on input). In the first cell of a day, starting with LO 7.0.4.2 (not having tried any version between 6.1.5 and 7.0.4.2!) the cell formatting is ignored and the value is displayed as a number.
Comment 1 armin 2021-10-10 08:15:57 UTC
Created attachment 175615 [details]
Screenshot how the bug looks in LO 7.0.4.2
Comment 2 armin 2021-10-10 08:18:29 UTC
Created attachment 175616 [details]
Screenshow how the file looks in LO 6.1.5.2
Comment 3 raal 2022-07-29 21:41:09 UTC
This seems to have begun at the below commit.
Adding Cc: to Eike Rathke; Could you possibly take a look at this one?
Thanks
 c3dc960a90a62f0343f99cc63f4e446ea8292426 is the first bad commit
commit c3dc960a90a62f0343f99cc63f4e446ea8292426
Author: Jenkins Build User <tdf@pollux.tdf>
Date:   Sun Oct 27 00:18:08 2019 +0200

    source 2b0626161d3ef7c4a51007018d13ec391d3a2b04

https://git.libreoffice.org/core/+/2b0626161d3ef7c4a51007018d13ec391d3a2b04
Comment 4 ady 2023-04-22 12:43:40 UTC
In attachment 175614 [details] from comment 0, the conditional format uses the "Untitled2" style, which has the "General" number format.

Before LO 7.0, the base direct format is modified only by applying the border in the "Untitled2" style.

Since LO 7.0, the base direct format is basically avoided/ignored when the conditional format is applied, and so the number format from the "Untitled2" style completely overrides the "date" direct format.

IDK whether this was done on purpose for LO 7.0, or this is some unwanted consequence of some other decision/change.

I guess only Eike can answer that(?).