<table><tr><td style="">dcaliste created this revision.<br />dcaliste added a reviewer: vkrause.<br />Herald added a project: KDE PIM.<br />Herald added a subscriber: kde-pim.<br />dcaliste requested review of this revision.
</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/D21215">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>In various portion of the code, Duration( 0 ) is used to represent a zero delay duration (see alarm.cpp for examples). This is creating a Duration::Seconds type duration. But when reading an ical format, the zero delay duration is created with Duration( 0, Days ) which makes comparison always fail for zero delay durations…</p>

<p>This is part of two patches that have been added to the SailfishOS fork of KCalCore.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>The added testAlarm() function added by the patch in testicalformat.cpp is failing without the patch. This test is basically creating an alarm which triggers on time and export it to iCal format. Then the serialized data are parsed again and compared with initial alarm.</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/D21215">https://phabricator.kde.org/D21215</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/testicalformat.cpp<br />
autotests/testicalformat.h<br />
src/icalformat_p.cpp</div></div></div><br /><div><strong>To: </strong>dcaliste, vkrause<br /><strong>Cc: </strong>kde-pim, pvuorela, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>