D26662: Don't set created and last modified fields when not present in ICS data.

Damien Caliste noreply at phabricator.kde.org
Tue Jan 14 13:02:23 GMT 2020


dcaliste created this revision.
dcaliste added reviewers: vkrause, dvratil.
Herald added a project: KDE PIM.
Herald added a subscriber: kde-pim.
dcaliste requested review of this revision.

REVISION SUMMARY
  CREATED and LAST-MODIFIED fields are not mandatory in RFC5545.
  Unfortunately, Incidence::recreate() is unconditionally
  setting a created and a last modified time stamps. This causes
  all read events or todos from ICS data to always have a
  created date and a last modified one.

TEST PLAN
  Added a test in testicalformat to check that unspecified created
  or last modified dates results in invalid created and lastModified
  properties.

REPOSITORY
  R172 KCalendar Core

BRANCH
  master

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

AFFECTED FILES
  autotests/testicalformat.cpp
  autotests/testicalformat.h
  src/icalformat_p.cpp

To: dcaliste, vkrause, dvratil
Cc: kde-pim, fbampaloukas, dcaliste, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20200114/4b85feb3/attachment.html>


More information about the kde-pim mailing list