Tickets :: Comment on [#7035] listing alarms of recurring events which cross a day boundary

* Your Email Address
* Spam protection	Enter the letters below: . ..__ .__.. \ / \|\ \|[__)\| \|\| >< \| \\|\| \|__\\|___/ \
Comment	>> Please provide a test case to reproduce this. > > > > Suppose you have a recurring event each Monday, 16:00 to the > following Tuesday, 12:00 with an alarm of 15 minutes before start. > Let's assume it is Monday, 2008-07-07, 15:45, stored in $date. Then > the old code > > > > $start = new Horde_Date($next); > > $start->min -= $event->getAlarm(); > > $start->correct(); > > $end = new Horde_Date(array( > > 'year' => $next->year, > > 'month' => $next->month, > > 'mday' => $next->mday, > > 'hour' => $event->end->hour, > > 'min' => $event->end->min, > > 'sec' => $event->end->sec)); > > > > assigns "2008-07-07 15:45" to $start and "2008-07-07 12:00" to $end. > Now the second part of the following comparison > > > > if ($start->compareDateTime($date) <= 0 > > && $date->compareDateTime($end) <= -1) > > > > fails, as $date ("2008-07-07 15:45") does not come before $end > ("2008-07-07 12:00"). The patch fixes this by assigning the correct > event end "2008-07-08 12:00" to $end. > >
Attachment
Watch this ticket