Summary | improved sorting of recurring events |
Queue | Kronolith |
Queue Version | 2.2 |
Type | Bug |
State | Resolved |
Priority | 2. Medium |
Owners | jan (at) horde (dot) org |
Requester | develop (at) kristov (dot) de |
Created | 07/06/2008 (6198 days ago) |
Due | |
Updated | 07/06/2008 (6198 days ago) |
Assigned | |
Resolved | 07/06/2008 (6198 days ago) |
Github Issue Link | |
Github Pull Request | |
Milestone | |
Patch | Yes |
Assigned to Jan Schneider
State ⇒ Resolved
http://cvs.horde.org/diff.php/kronolith/docs/CHANGES?r1=1.449&r2=1.450&ty=u
http://cvs.horde.org/diff.php/kronolith/lib/Kronolith.php?r1=1.423&r2=1.424&ty=u
Priority ⇒ 2. Medium
New Attachment: horde-kronolith-2.2-sort.patch
Patch ⇒ Yes
Milestone ⇒
Queue ⇒ Kronolith
Summary ⇒ improved sorting of recurring events
Type ⇒ Bug
State ⇒ Unconfirmed
23:00-01:00. Assume further that you have a non-recurring event on a
Thursday from 10:00-11:00. In the summary block and in the month view,
the non-recurring event is listed before the recurring one. This is
because for sorting the events, only the event's start time is used
even if the event's start date is different (and 23:00 comes after
10:00). The attached patch fixes this by changing
Kronolith::_sortEventStartTime(): as listEvents() already returns
recurring events with corrected timestamps, the timestamps can be
directly used for sorting.