Tickets :: [#5229] Enhance recurrence API

6.0.0-beta1

7/23/25

Summary	Enhance recurrence API
Queue	Kronolith
Queue Version	HEAD
Type	Enhancement
State	Resolved
Priority	1. Low
Owners	jan (at) horde (dot) org
Requester	thomas.jarosch (at) intra2net (dot) com
Created	04/09/2007 (6680 days ago)
Due
Updated	05/14/2007 (6645 days ago)
Assigned
Resolved	04/17/2007 (6672 days ago)
Milestone
Patch	No

05/14/2007 09:01:46 AM	Jan Schneider	Comment #9	Reply to this comment
Thanks for your reply. So would it make sense to leave this bug open until Horde 3.2 is the offical standard so the patch won't get lost? No, BC is not going to break before Horde 4/Kronolith 3. And it's not really a critical patch.

05/14/2007 07:54:25 AM	thomas (dot) jarosch (at) intra2net (dot) com	Comment #8	Reply to this comment
Hello Chuck, With that change you make it impossible to use Kronolith with older Horde 3.0 releases. That's why it won't be committed. Thanks for your reply. So would it make sense to leave this bug open until Horde 3.2 is the offical standard so the patch won't get lost? Cheers, Thomas

05/12/2007 03:53:05 PM	Chuck Hagenbuch	Comment #7	Reply to this comment
With that change you make it impossible to use Kronolith with older Horde 3.0 releases. That's why it won't be committed.

05/12/2007 10:06:00 AM	thomas (dot) jarosch (at) intra2net (dot) com	Comment #6 New Attachment: kronolith-Recurrence-weekOfMonth.patch	Reply to this comment
Hello Jan, it looks like one small change from my original patch was lost during the merge. Here's the lost son. Thomas

04/17/2007 11:14:12 PM	Jan Schneider	Comment #5 State ⇒ Resolved	Reply to this comment
attached patches enhance the recurrence API. Committed, thanks. I have a question about the recurrence API: Why is "$end" in there? It gets set by the constructor but is never used. The real recurrence end is stored in $recurEnd. This was a left-over from the old recurrence code in Kronolith. I have removed it, thanks for the catch.

04/17/2007 11:06:16 PM	Jan Schneider	Comment #4	Reply to this comment
Patch for Horde_Date of the framework package. Committed, thanks.

04/09/2007 06:01:18 PM	Chuck Hagenbuch	Comment #3 Assigned to Jan Schneider State ⇒ Feedback	Reply to this comment
The Horde_Date patch can't be used for now since it breaks BC. I'll leave the rest of this to Jan for now.

04/09/2007 10:11:32 AM	thomas (dot) jarosch (at) intra2net (dot) com	Comment #2 New Attachment: framework-Date-weekOfMonth.patch	Reply to this comment
Patch for Horde_Date of the framework package.

04/09/2007 10:10:14 AM	thomas (dot) jarosch (at) intra2net (dot) com	Comment #1 Priority ⇒ 1. Low State ⇒ New New Attachment: kronolith-recurrence-API.patch Queue ⇒ Kronolith Summary ⇒ Enhance recurrence API Type ⇒ Enhancement	Reply to this comment
Hello, attached patches enhance the recurrence API. It also adds a new function to Horde_Date: weekOfMonth(). I needed this code in another place so I thought it might be good to add it to the API. I have a question about the recurrence API: Why is "$end" in there? It gets set by the constructor but is never used. The real recurrence end is stored in $recurEnd. Best regards, Thomas Jarosch