Tickets :: [#2709] Add event, division by zero

10/04/2005 03:10:30 PM	Chuck Hagenbuch	Comment #4 State ⇒ Resolved	Reply to this comment
It actually shouldn't be possible to set a recurInterval of 0 since the setRecurInterval() function checks for it explicitly. But this should be fixed now since nextRecurrence() immediately fails now if recurInterval is 0.

10/03/2005 12:57:16 PM	sgrondin (at) csbf (dot) qc (dot) ca	Comment #3	Reply to this comment
Yes, I have the latest CVS. Sorry, it's a warning: PHP Warning: Division by zero in /..../horde/kronolith/lib/Driver.php on line 1972 With a recur monthly same date. The line number change with the recurtype. Thanks

09/30/2005 08:59:17 PM	Chuck Hagenbuch	Comment #2 State ⇒ Feedback Priority ⇒ 2. Medium	Reply to this comment
Are you sure you're using latest CVS? If so, can you report the exact notice please?

09/29/2005 03:48:47 PM	sgrondin (at) csbf (dot) qc (dot) ca	Comment #1 State ⇒ Unconfirmed Priority ⇒ 3. High Type ⇒ Bug Summary ⇒ Add event, division by zero Queue ⇒ Kronolith	Reply to this comment
When adding an event, it's possible to put a zero (0) in the recur interval. For the Daily and Weekly, there's a single php notice. For the Monthly and Yearly, the php notice fills the php log file and never return the result. The http process take all the ressource until the partition that the php log file are in is filled. Thanks.