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

DTEND seems to be an invalid property #322

Closed
imrealashu opened this issue Feb 14, 2019 · 4 comments
Closed

DTEND seems to be an invalid property #322

imrealashu opened this issue Feb 14, 2019 · 4 comments

Comments

@imrealashu
Copy link

FREQ=MONTHLY;DTSTART=20181114T110000;DTEND=20191114T140000;INTERVAL=1 this seems to be invalid saying .
Error: Unknown RRULE property 'DTEND'
I checked the ical rules and seems to be present in the ruleset https://www.kanzaki.com/docs/ical/rrule.html

Even I'm using one PHP version which does letting me parse it properly https://github.com/simshaun/recurr

@davidgoli
Copy link
Collaborator

davidgoli commented Feb 16, 2019

That's correct, it is a property that's present in the spec, but not supported by this library (currently). The reason is that this library does not support the concept of "duration" which is what DTEND represents, and it's not clear how to add support for duration without a major overhaul of the library. See discussion here: rlanvin/php-rrule#4

Note that the rrule properties as you have them are invalid. According to the spec, DTSTART and DTEND do not appear within the RRULE: property string, but at the top level as peers of the RRULE:. Supported properties (all of which this library supports):

       recur-rule-part = ( "FREQ" "=" freq )
                       / ( "UNTIL" "=" enddate )
                       / ( "COUNT" "=" 1*DIGIT )
                       / ( "INTERVAL" "=" 1*DIGIT )
                       / ( "BYSECOND" "=" byseclist )
                       / ( "BYMINUTE" "=" byminlist )
                       / ( "BYHOUR" "=" byhrlist )
                       / ( "BYDAY" "=" bywdaylist )
                       / ( "BYMONTHDAY" "=" bymodaylist )
                       / ( "BYYEARDAY" "=" byyrdaylist )
                       / ( "BYWEEKNO" "=" bywknolist )
                       / ( "BYMONTH" "=" bymolist )
                       / ( "BYSETPOS" "=" bysplist )
                       / ( "WKST" "=" weekday )

@pantonis
Copy link

pantonis commented Jun 2, 2020

This is a must!!!!!!!!!!!!!!!

@espen
Copy link
Collaborator

espen commented Jun 2, 2020

@pantonis feel free to send a pull request for this to get it included.

@pantonis
Copy link

pantonis commented Jun 2, 2020

@espen I am not a JavaScript dev. I would create more problems rather than solve. :D If I knew be sure that I would do create a pull request.

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

4 participants