6.0.0-git
2019-03-24

[#5229] Enhance recurrence API
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 2007-04-09 (4367 days ago)
Due
Updated 2007-05-14 (4332 days ago)
Assigned
Resolved 2007-04-17 (4359 days ago)
Milestone
Patch No

History
2007-05-14 09:01:46 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.
2007-05-14 07:54:25 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


2007-05-12 15:53:05 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.
2007-05-12 10:06:00 thomas (dot) jarosch (at) intra2net (dot) com Comment #6
New Attachment: kronolith-Recurrence-weekOfMonth.patch Download
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


2007-04-17 23:14:12 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.
2007-04-17 23:06:16 Jan Schneider Comment #4 Reply to this comment
Patch for Horde_Date of the framework package.
Committed, thanks.
2007-04-09 18:01:18 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.
2007-04-09 10:11:32 thomas (dot) jarosch (at) intra2net (dot) com Comment #2
New Attachment: framework-Date-weekOfMonth.patch Download
Reply to this comment
Patch for Horde_Date of the framework package.


2007-04-09 10:10:14 thomas (dot) jarosch (at) intra2net (dot) com Comment #1
Type ⇒ Enhancement
State ⇒ New
Priority ⇒ 1. Low
Summary ⇒ Enhance recurrence API
Queue ⇒ Kronolith
New Attachment: kronolith-recurrence-API.patch Download
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


Saved Queries