Bug 124412 - Conditional formatting screen always shows 'Date is today' as conditional when inserting date conditional UI
Summary: Conditional formatting screen always shows 'Date is today' as conditional whe...
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
4.4.7.2 release
Hardware: All All
: medium trivial
Assignee: Andreas Heinisch
URL:
Whiteboard: target:7.4.0 target:7.3.0.0.beta2
Keywords: topicUI
Depends on:
Blocks: Conditional-Formatting-Editing
  Show dependency treegraph
 
Reported: 2019-03-28 23:17 UTC by matjojo
Modified: 2021-12-14 18:04 UTC (History)
1 user (show)

See Also:
Crash report or crash signature:


Attachments
conditional format date (10.00 KB, application/vnd.oasis.opendocument.spreadsheet)
2019-03-30 14:50 UTC, Oliver Brinzing
Details

Note You need to log in before you can comment on or make changes to this bug.
Description matjojo 2019-03-28 23:17:42 UTC
Description:
Making conditions based on date always show as 'Date is today' even if the condition is not 'Date is today'.

See here: https://i.imgur.com/DLok4qV.png
And here you see condition 2 isn't really Date is today: https://i.imgur.com/aXzUHDI.png

Steps to Reproduce:
1. select a tile
2. Right mouse button on the selection, open conditional formatting
3. Create a formatting rule based on the 'Date is' version
4. Make a rule based on anything
5. Make a rule based on anything that is not 'Date is today'
6. Select the bottom condition


Actual Results:
See that the top condition now says: 'Condition # Date is today'

Expected Results:
See that the top condition now says: 'Condition # Actual condition'


Reproducible: Always


User Profile Reset: No



Additional Info:
Comment 1 Oliver Brinzing 2019-03-30 14:49:35 UTC
reproducible with:

Version: 4.4.7.2
Build-ID: f3153a8b245191196a4b6b9abd1d0da16eead600
Gebietsschema: de_DE
Comment 2 Oliver Brinzing 2019-03-30 14:50:03 UTC
Created attachment 150412 [details]
conditional format date
Comment 3 QA Administrators 2021-04-16 03:56:13 UTC Comment hidden (obsolete)
Comment 4 matjojo 2021-04-16 09:42:46 UTC
Bug is still present.
Version info from help:

Version: 7.1.2.2 (x64) / LibreOffice Community
Build ID: 8a45595d069ef5570103caea1b71cc9d82b2aae4
CPU threads: 12; OS: Windows 10.0 Build 19042; UI render: Skia/Raster; VCL: win
Locale: en-US (en_NL); UI: en-GB
Calc: threaded

Steps to reproduce:
1. select a tile
2. Click Format
3. Click Conditional
4. Click Manage
5. Click Add
6. Select condition type: "Date is"
7. Select condition: anything but "Today"
8. Click Add

Actual Results:
See that the top condition now says: 'Condition 1     Date is today'

Expected Results:
See that the top condition now says: 'Condition 1     Actual condition'
Comment 5 Commit Notification 2021-12-02 19:47:04 UTC
Andreas Heinisch committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/dfdf33a34729de7f256ed396be93fb96b4844404

tdf#124412 - Set actual condition for an inactive date entry

It will be available in 7.4.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 6 Commit Notification 2021-12-14 17:11:49 UTC
Andreas Heinisch committed a patch related to this issue.
It has been pushed to "libreoffice-7-3":

https://git.libreoffice.org/core/commit/968437cc61cc45ea3e15cabf2c3f67360c243173

tdf#124412 - Set actual condition for an inactive date entry

It will be available in 7.3.0.0.beta2.

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 2021-12-14 18:04:26 UTC
Xisco Fauli committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/d3f0aa90ce30c2566f0c7ad775c4355b939f338c

tdf#124412: sc: Add UItest

It will be available in 7.4.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.