https://bugs.kde.org/show_bug.cgi?id=394423

Fabian Vogt <fab...@ritter-vogt.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://commits.kde.org/pla
                   |                            |sma-framework/d15f0fa8322dc
                   |                            |2632f15024457bea41e7dd058d7
         Resolution|UPSTREAM                    |FIXED

--- Comment #10 from Fabian Vogt <fab...@ritter-vogt.de> ---
Git commit d15f0fa8322dc2632f15024457bea41e7dd058d7 by Fabian Vogt.
Committed on 04/06/2018 at 08:33.
Pushed by fvogt into branch 'master'.

Use QDateTime for interfacing with QML

Summary:
QDate from/to JS Date has unexpected results, so use QDateTime to have a
timezone assigned to prevent conversions. That way the date values are
consistent.

The behaviour got changed with Qt 5.11 (see the linked bug report) which
lead to plasma showing the wrong dates in the calendar.

Test Plan:
Tested with and without this patch on Qt 5.10.1 and 5.11.0.
Now the correct date is displayed for -0004 and +0001 timezones.

Reviewers: #plasma, #frameworks, davidedmundson

Reviewed By: #plasma, davidedmundson

Subscribers: ngraham, Zren, sharvey, kde-frameworks-devel

Tags: #frameworks

Differential Revision: https://phabricator.kde.org/D13222

M  +13   -7    src/declarativeimports/calendar/calendar.cpp
M  +12   -5    src/declarativeimports/calendar/calendar.h

https://commits.kde.org/plasma-framework/d15f0fa8322dc2632f15024457bea41e7dd058d7

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to