Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

.GetOccurrences() only looking at first RecurrenceRules #559

Open
stephenhendry opened this issue Oct 11, 2022 · 2 comments
Open

.GetOccurrences() only looking at first RecurrenceRules #559

stephenhendry opened this issue Oct 11, 2022 · 2 comments

Comments

@stephenhendry
Copy link

I have the following coming from my ical feed:

BEGIN:VEVENT
DTSTART;TZID=Europe/London:20220101T180000
DTEND;TZID=Europe/London:20220101T230000
RRULE:FREQ=YEARLY;COUNT=3;INTERVAL=1;BYMONTH=10;BYDAY=1WE
RRULE:FREQ=YEARLY;COUNT=3;INTERVAL=1;BYMONTH=11;BYDAY=1WE
RRULE:FREQ=YEARLY;COUNT=3;INTERVAL=1;BYMONTH=1;BYDAY=1WE
RRULE:FREQ=YEARLY;COUNT=3;INTERVAL=1;BYMONTH=2;BYDAY=1WE
RRULE:FREQ=YEARLY;COUNT=3;INTERVAL=1;BYMONTH=3;BYDAY=1WE
RRULE:FREQ=YEARLY;COUNT=3;INTERVAL=1;BYMONTH=4;BYDAY=1WE
RRULE:FREQ=YEARLY;COUNT=3;INTERVAL=1;BYMONTH=5;BYDAY=1WE
RRULE:FREQ=YEARLY;COUNT=3;INTERVAL=1;BYMONTH=6;BYDAY=1WE
DTSTAMP:20221011T095103Z
UID:Ical4fa23491908fccd532b32da26bac4662
CREATED:19000101T120000Z
DESCRIPTION:this is a test
LAST-MODIFIED:20221003T191319Z
LOCATION:London
SEQUENCE:0
STATUS:CONFIRMED
SUMMARY: test
TRANSP:OPAQUE
END:VEVENT

When I call .GetOccurrences(start date, end date) it is only returning the first RRULE occurrences and not the others. Is it possible to look at all the rules?

@Wazmin
Copy link

Wazmin commented Dec 15, 2022

Hello stephenhendry,

in the RFC 5545 the section about RRULE (https://www.rfc-editor.org/rfc/rfc5545#section-3.8.5.3) says that RRULE can only be defined once.
So in the ical.net I thing it only consider the first one, I did not wen into the code but that does not matter since the best way is to follow the rfc first :)

It's a late answer but could be useful for other.

To me it's "not a defect" case :)

@gortok
Copy link

gortok commented Mar 16, 2024

it may not be a defect; but the specification doesn't disallow it (just says the behavior is undefined):

but it SHOULD NOT be specified more than once.
The recurrence set generated with multiple "RRULE" properties is
undefined.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants