D13222: Use QDateTime for interfacing with QML

[Fabian Vogt]

fvogt created this revision.
fvogt added reviewers: Plasma, Frameworks.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: kde-frameworks-devel.
fvogt requested review of this revision.

REVISION 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.
  
  BUG: 394423

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.

REPOSITORY
  R242 Plasma Framework (Library)

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D13222

AFFECTED FILES
  src/declarativeimports/calendar/calendar.cpp
  src/declarativeimports/calendar/calendar.h

To: fvogt, #plasma, #frameworks
Cc: sharvey, kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180530/ee6f5246/attachment.html>