| Summary: | Wrong result of Conditional formatting with type Date: condition "This week" shows 9 days in week instead 7 days | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | Roman Kuznetsov <79045_79045> |
| Component: | Calc | Assignee: | Eike Rathke <erack> |
| Status: | VERIFIED FIXED | ||
| Severity: | normal | CC: | erack, marcoscesteves, marina.latini, xiscofauli |
| Priority: | medium | Keywords: | bibisected, bisected, regression |
| Version: | 6.0.0.0.alpha1+ | ||
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | target:6.2.0 target:6.1.1 target:6.0.7 | ||
| Crash report or crash signature: | Regression By: | ||
| Bug Depends on: | |||
| Bug Blocks: | 87351 | ||
| Attachments: |
wrong CF with Date (this week)
wrong CF with Date (this week).ods in LibreOffice Calc 6.0.4 wrong CF with Date (this week).ods in LibreOffice Calc 5.4.7 |
||
|
Description
Roman Kuznetsov
2018-08-21 14:44:01 UTC
Created attachment 144348 [details]
wrong CF with Date (this week)
bug repro in Version: 6.0.4.2 (x64) Build ID: 9b0d9b32d5dcda91d2f1a96dc04c645c450872bf CPU threads: 4; OS: Windows 10.0; UI render: GL; Locale: ru-RU (ru_RU); Calc: CL and doesn't repro in Version: 5.4.7.2 (x64) Build ID: c838ef25c16710f8838b1faec480ebba495259d0 CPU threads: 4; OS: Windows 6.19; UI render: GL; Locale: ru-RU (ru_RU); Calc: CL regression Created attachment 144350 [details]
wrong CF with Date (this week).ods in LibreOffice Calc 6.0.4
Created attachment 144351 [details]
wrong CF with Date (this week).ods in LibreOffice Calc 5.4.7
Regression introduced by: https://cgit.freedesktop.org/libreoffice/core/commit/?id=667aa6ade94a4dee441868d21db037e42c871932 author Eike Rathke <erack@redhat.com> 2017-07-19 19:10:32 +0200 committer Eike Rathke <erack@redhat.com> 2017-07-19 20:13:58 +0200 commit 667aa6ade94a4dee441868d21db037e42c871932 (patch) tree 148c76752126ba4b9322300c3d0b7c8b9d248f82 parent ef8f13e08bccb7c082d3c5b19cce8a7a63c0b2fa (diff) Avoid temporary Date and normalization Rather than possibly calculating Date=((Date=(Date+int))+int) with a normalization involved for each operator+() force Date=(Date+(int+int)) Bisected with: bibisect-linux64-6.0 Adding Cc: to Eike Rathke grml.. stupid me. Eike Rathke committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=cadd6c595b250b3ce0025a2a09c160eaae483d5a Resolves: tdf#119402 +- typo sign flip It will be available in 6.2.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Pending review https://gerrit.libreoffice.org/59475 for 6-0 https://gerrit.libreoffice.org/59476 for 6-1 Eike Rathke committed a patch related to this issue. It has been pushed to "libreoffice-6-1": http://cgit.freedesktop.org/libreoffice/core/commit/?id=40a7c1c1ef133950ddb4c2052d460a9ce2b02b15&h=libreoffice-6-1 Resolves: tdf#119402 +- typo sign flip It will be available in 6.1.1. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Eike Rathke committed a patch related to this issue. It has been pushed to "libreoffice-6-0": http://cgit.freedesktop.org/libreoffice/core/commit/?id=6d6136398afd1581c6292ba8091c3c5186eaf4b1&h=libreoffice-6-0 Resolves: tdf#119402 +- typo sign flip It will be available in 6.0.7. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. *** Bug 119494 has been marked as a duplicate of this bug. *** verified in LO 6.1.1.1 Thank you, Eike |