Tickets :: [#6966] In lib/Driver/sql.php:listEvents (doesn't list future events in some context)

6.0.0-alpha10

5/15/25

Summary	In lib/Driver/sql.php:listEvents (doesn't list future events in some context)
Queue	Kronolith
Queue Version	2.2
Type	Bug
State	Resolved
Priority	2. Medium
Owners	jan (at) horde (dot) org
Requester	sebastien.barre (at) univ-paris-diderot (dot) fr
Created	06/23/2008 (6170 days ago)
Due
Updated	07/13/2008 (6150 days ago)
Assigned	07/07/2008 (6156 days ago)
Resolved	07/13/2008 (6150 days ago)
Milestone	2.2.1
Patch	No

07/13/2008 09:17:47 AM	Jan Schneider	State ⇒ Resolved

07/07/2008 10:44:25 PM	CVS Commit	Comment #7	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/kronolith/docs/CHANGES?r1=1.165.2.212&r2=1.165.2.213&ty=u http://cvs.horde.org/diff.php/kronolith/lib/Driver/sql.php?r1=1.136.2.36&r2=1.136.2.37&ty=u http://cvs.horde.org/diff.php/kronolith/lib/FreeBusy.php?r1=1.11.2.5&r2=1.11.2.6&ty=u http://cvs.horde.org/diff.php/kronolith/lib/Kronolith.php?r1=1.263.2.69&r2=1.263.2.70&ty=u http://cvs.horde.org/diff.php/kronolith/lib/api.php?r1=1.126.2.48&r2=1.126.2.49&ty=u http://cvs.horde.org/diff.php/kronolith/po/de_DE.po?r1=1.43.2.35&r2=1.43.2.36&ty=u

07/07/2008 07:04:11 PM	Jan Schneider	Comment #6 State ⇒ Feedback Taken from Horde Developers	Reply to this comment
Please try what I committed.

07/07/2008 07:03:18 PM	CVS Commit	Comment #5	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/kronolith/docs/CHANGES?r1=1.450&r2=1.451&ty=u http://cvs.horde.org/diff.php/kronolith/lib/Driver/sql.php?r1=1.204&r2=1.205&ty=u http://cvs.horde.org/diff.php/kronolith/lib/FreeBusy.php?r1=1.15&r2=1.16&ty=u http://cvs.horde.org/diff.php/kronolith/lib/Kronolith.php?r1=1.424&r2=1.425&ty=u http://cvs.horde.org/diff.php/kronolith/lib/api.php?r1=1.233&r2=1.234&ty=u

07/06/2008 05:27:37 PM	Jan Schneider	Milestone ⇒ 2.2.1

06/26/2008 10:48:37 AM	Jan Schneider	Comment #4 Assigned to Horde Developers State ⇒ Assigned Assigned to Jan Schneider	Reply to this comment
Probably the same like ~~bug #6847~~.

06/26/2008 10:45:24 AM	Jan Schneider	Deleted Original Message

06/23/2008 02:36:43 PM	sebastien (dot) barre (at) univ-paris-diderot (dot) fr	Comment #3 New Attachment: sql.php.2.patch	Reply to this comment
Couldn't attach a patch with the same name, or mistake ? Here is the file.

06/23/2008 02:34:15 PM	sebastien (dot) barre (at) univ-paris-diderot (dot) fr	Comment #2	Reply to this comment
I attached a kind of patch, but I don't know if it's the best solution. Especially when it's wrong... here is something that works.

06/23/2008 01:48:40 PM	sebastien (dot) barre (at) univ-paris-diderot (dot) fr	Comment #1 Priority ⇒ 2. Medium State ⇒ Unconfirmed New Attachment: sql.php.patch Patch ⇒ Yes Milestone ⇒ Queue ⇒ Kronolith Summary ⇒ In lib/Driver/sql.php:listEvents (doesn't list future events in some context) Type ⇒ Bug	Reply to this comment
When doing a refresh-from-server sync with SyncML, we didn't get future events. I traced it back to a missing check in kronolith/lib/Driver/sql.php on line 227 (rev 1.204) Sometime $endDate is already set to 31/12/9999, so when Date_Calc::nextDay is called the resulting date is today. (It's not valid then reset to dateNow by Date_Calc::dateFormat). For example, kronolith/lib/api.php:658 (rev 1.230) sets $endstamp to 31/12/9999 I attached a kind of patch, but I don't know if it's the best solution.