peterbraden / ical.js

ical for javascript

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

RRule doesn't honor start date

jfrumar opened this issue · comments

On line 23 of node-ical.js:

curr['rrule'] = rrule.fromString(line.replace("RRULE:", ""));

A sample string is FREQ=YEARLY;BYMONTH=11;BYDAY=1SU

Since the start date is never passed along to the RRule library, it returns events in the past that shouldn't exist.

I just discovered just that right now so +1 from me!

Just made a pull request fixing this issue.