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

COUNT and weekstart format #39

Open
RIKALPHA opened this issue Feb 13, 2019 · 2 comments
Open

COUNT and weekstart format #39

RIKALPHA opened this issue Feb 13, 2019 · 2 comments

Comments

@RIKALPHA
Copy link

This is a standard ICS format that I have:
FREQ=DAILY;INTERVAL=1;COUNT=3;WKST=SU
can you/I help update the code so that it parses this format too please?

@RIKALPHA
Copy link
Author

or maybe make a function that just let's you set the string with no parsing or validation?

@jasvrcek
Copy link
Owner

@RIKALPHA If you're interested, code needs to be added for parsing and using the WKST attribute, and also parsing the COUNT attribute. This would all be done in https://github.com/jasvrcek/ICS/blob/master/src/Model/Recurrence/RecurrenceRule.php.

For WKST, add a $weekStart property to RecurrenceRule, and then update RecurrenceRule::__toString() and RecurrenceRule::parse() to handle the WKST attribute, in the same manner the other attributes are being handled.

For COUNT, you only need to update RecurrenceRule::parse(), since it's already taken care of in the RecurrenceRule::__toString() method.

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

2 participants