Tickets :: [#11181] kronolith swallows recurring events on 5th weekday

6.0.0-beta1

7/10/25

Summary	kronolith swallows recurring events on 5th weekday
Queue	Horde Framework Packages
Queue Version	Git master
Type	Bug
State	Resolved
Priority	1. Low
Owners	jan (at) horde (dot) org
Requester	steffo76 (at) gmx (dot) de
Created	05/09/2012 (4810 days ago)
Due
Updated	03/04/2013 (4511 days ago)
Assigned	10/17/2012 (4649 days ago)
Resolved	02/13/2013 (4530 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

03/04/2013 01:04:14 PM	steffo76 (at) gmx (dot) de	Comment #8	Reply to this comment
Sorry, the previous comment was totally bogus. I thought Horde_Date 2.0.4 was already out and installed.

03/04/2013 01:00:45 PM	steffo76 (at) gmx (dot) de	Comment #7	Reply to this comment
I just created an event on March 29th which happens to be the fifth friday in March. The event was created to recur monthly on the same weekday- in April it doesn't show up but instead it shows up twice in May (May 3rd and May 31st). It doesn't show up in June. I don't think this is really fixed.

02/13/2013 09:45:31 AM	Jan Schneider	Comment #6 Taken from Horde Developers State ⇒ Resolved Assigned to Jan Schneider	Reply to this comment
Fixed for Horde_Date 2.0.4.

02/13/2013 09:45:06 AM	Git Commit	Comment #5	Reply to this comment
Changes have been made in Git (master): commit e85da36beacb62f0e4df3e25385ed9949b3cf67b Author: Jan Schneider <jan@horde.org> Date: Wed Feb 13 10:42:19 2013 +0100 [jan] Fix recurring dates on the 5th weekday of a month (~~Bug #11181~~). framework/Date/lib/Horde/Date/Recurrence.php \| 4 ++++ framework/Date/package.xml \| 4 ++-- 2 files changed, 6 insertions(+), 2 deletions(-) http://git.horde.org/horde-git/-/commit/e85da36beacb62f0e4df3e25385ed9949b3cf67b

02/13/2013 09:43:26 AM	Jan Schneider	Version ⇒ Git master Queue ⇒ Horde Framework Packages

10/17/2012 08:16:31 PM	Michael Rubinsky	Assigned to Horde Developers State ⇒ Assigned

05/09/2012 08:03:49 PM	Git Commit	Comment #4	Reply to this comment
Changes have been made in Git (develop): commit 73257bf039e5631554918fd27f024ff8a77a6644 Author: Jan Schneider <jan@horde.org> Date: Wed May 9 16:11:46 2012 +0200 Add failing test for ~~bug #11181~~. framework/Date/test/Horde/Date/RecurrenceTest.php \| 18 ++++++++++++++++++ 1 files changed, 18 insertions(+), 0 deletions(-) http://git.horde.org/horde-git/-/commit/73257bf039e5631554918fd27f024ff8a77a6644

05/09/2012 02:29:11 PM	Git Commit	Comment #3	Reply to this comment
Changes have been made in Git (master): commit 73257bf039e5631554918fd27f024ff8a77a6644 Author: Jan Schneider <jan@horde.org> Date: Wed May 9 16:11:46 2012 +0200 Add failing test for ~~bug #11181~~. framework/Date/test/Horde/Date/RecurrenceTest.php \| 18 ++++++++++++++++++ 1 files changed, 18 insertions(+), 0 deletions(-) http://git.horde.org/horde-git/-/commit/73257bf039e5631554918fd27f024ff8a77a6644

05/09/2012 01:05:30 PM	steffo76 (at) gmx (dot) de	Comment #2	Reply to this comment
The Bugtracker swolled my greater than sign. It should have been "If the day has a number greater or equal 29".

05/09/2012 01:02:09 PM	steffo76 (at) gmx (dot) de	Comment #1 Priority ⇒ 1. Low State ⇒ Unconfirmed New Attachment: kronolith_recurrence.png Patch ⇒ No Milestone ⇒ Queue ⇒ Kronolith Summary ⇒ kronolith swallows recurring events on 5th weekday Type ⇒ Bug	Reply to this comment
Hi, I noticed the following in traditional view: If I choose 'Recurs every X month(s) on the same weekday' (HORDE_DATE_RECUR_MONTHLY_WEEKDAY) and this weekday is the fifth of its kind it doesn't show up in months where there are only four of its kind. For example: May 31st 2012 is the 5th Thursday. In the .ics this shows up as BYDAY=5TH. I'm not sure how clients should react if the day doesn't exist. What is really confusing that with this particular event it sometimes shows where it shouldn't. Example: Viewing July kronolith shows this event on the 2nd of August. Switching to August there is no event on the 2nd. I guess in iCalendar terms this should be: If the day has a number ;=29 set the recurrence to the last of the month (BYDAY=-1TH in this case).