[Kde-pim] Review Request 114882: calendarviews: make MonthItem respect thisAndFuture flag

Commit Hook null at kde.org
Sun Jan 12 01:20:55 GMT 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/114882/#review47242
-----------------------------------------------------------


This review has been submitted with commit 934db392dde2ec5e4a9eaf189e751e79bf29b84e by Sergio Martins on behalf of Justus Matthiesen to branch KDE/4.12.

- Commit Hook


On Jan. 12, 2014, 1:20 a.m., Justus Matthiesen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/114882/
> -----------------------------------------------------------
> 
> (Updated Jan. 12, 2014, 1:20 a.m.)
> 
> 
> Review request for KDEPIM and KDEPIM-Libraries.
> 
> 
> Repository: kdepim
> 
> 
> Description
> -------
> 
> If a recurrence contains an exception with thisAndFuture set, all subsequent incidences will inherit the exception's data. Thus, recurs() will return false for all subsequent incidences causing IncidenceMonthItem to ignore the offset.
> 
> However, since startDate and recurStartDate will be identical in all other cases, it is safe to remove the check.
> 
> 
> Diffs
> -----
> 
>   calendarviews/month/monthitem.cpp 5b2d246812f9ef6e1d7ad0b1983be91c801eba3a 
> 
> Diff: https://git.reviewboard.kde.org/r/114882/diff/
> 
> 
> Testing
> -------
> 
> The following calendar file now displays correctly in month view.
> 
> BEGIN:VCALENDAR
> VERSION:2.0
> 
> BEGIN:VEVENT
> UID:notveryunique
> RECURRENCE-ID;RANGE=THISANDFUTURE:20131204T100000
> DTSTART:20131203T110000
> DTEND:20131203T120000
> SEQUENCE:5
> END:VEVENT
> 
> BEGIN:VEVENT
> UID:notveryunique
> RECURRENCE-ID;RANGE=THISANDFUTURE:20131203T100000
> DTSTART:20131204T120000
> DTEND:20131204T130000
> SEQUENCE:4
> END:VEVENT
> 
> BEGIN:VEVENT
> UID:notveryunique
> RRULE:FREQ=DAILY;COUNT=7
> DTSTART:20131202T100000
> DTEND:20131202T110000
> END:VEVENT
> 
> END:VCALENDAR
> 
> 
> Thanks,
> 
> Justus Matthiesen
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/



More information about the kde-pim mailing list