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

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/6966
------------------------------------------------------------------------------
  Ticket             | 6966
  Created By         | sebastien.barre at univ-paris-diderot.fr
  Summary            | In lib/Driver/sql.php:listEvents (doesn't list future
                     | events in some context)
  Queue              | Kronolith
  Version            | 2.2
  Type               | Bug
  State              | Unconfirmed
  Priority           | 2. Medium
  Milestone          |
  Patch              |
  Owners             |
+New Attachment     | sql.php.patch
------------------------------------------------------------------------------


sebastien.barre at univ-paris-diderot.fr (2008-06-23 09:48) wrote:

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.