<table><tr><td style="">dcaliste added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D26662">View Revision</a></tr></table><br /><div><div><p>The patch is not necessary the ideal one, but I would like to start a discussion on this matter. Currently, when calling <tt style="background: #ebebeb; font-size: 13px;">ICalFormat::fromString()</tt>, all created incidences feature a <tt style="background: #ebebeb; font-size: 13px;">created()</tt> and a <tt style="background: #ebebeb; font-size: 13px;">lastModified()</tt> properties. They either come from the values read from the ICS data string, or they are set to the current date time by <tt style="background: #ebebeb; font-size: 13px;">Incidence::recreate()</tt> from Incidence constructor.</p>

<p>RFC 5545 is specifying that <tt style="background: #ebebeb; font-size: 13px;">CREATED</tt> and <tt style="background: #ebebeb; font-size: 13px;">LAST-MODIFIED</tt> fields are not mandatory. With the incidence creator setting these two unconditionally, we cannot distinguish between ICS data with or without created and last modified fields set.</p>

<p>I wonder if we should on the contrary to the current patch, change <tt style="background: #ebebeb; font-size: 13px;">Incidence::Incidence()</tt> itself, not to call <tt style="background: #ebebeb; font-size: 13px;">Incidence::recreate()</tt> but just set the revision to 0 and set the scheduling id, but not set created and last modified. This would be more matching the RFC, but it is a huge change that could have unexpected consequences, since code may rely on the fact that the constructor of Incidence do set these two fields.</p>

<p>Finally, the patch itself may indeed suffer from the same unexpected consequences.</p>

<p>What do you think ? Should we ignore this, try to get incidence closer to the ICS data, or even don't set created and last modified field by default ?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R172 KCalendar Core</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26662">https://phabricator.kde.org/D26662</a></div></div><br /><div><strong>To: </strong>dcaliste, vkrause, dvratil, KDE PIM<br /><strong>Cc: </strong>kde-pim, fbampaloukas, dcaliste, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>