toText does not display the bysetpos values about rrule HOT 7 CLOSED

+1 Same here.

from rrule.

+1
ByDay can achieve the same result :

RRule.fromString('FREQ=MONTHLY;BYDAY=+2MO,-1MO').toText();
// every month on the 2nd Monday and last Monday

from rrule.

The support is incomplete (as mentioned here), but there should be a warning in the string returned by toText() that the textual representation is approximate.

It looks like isFullyConvertibleToText() is broken and always returns true.

var r = RRule.fromString('FREQ=MONTHLY;BYDAY=MO;BYSETPOS=2,-1')
r.isFullyConvertibleToText()
true  // Should be false

from rrule.

+1 Same here.

from rrule.

I have fixed this issue.
Please find below lines.

         if(typeof options.bysetpos != 'undefined' && options.bysetpos != '') {
            n = Number(options.bysetpos)
            wday = day;
            wday = RRule[wday].weekday
            options.byweekday.push(new Weekday(wday, n))
          } else {
            wday = RRule[day] // wday instanceof Weekday
            options.byweekday.push(wday)
          }

Line Number 1331 (rrule.js file)
https://github.com/rameshsrinivasan/rrule/blob/master/lib/rrule.js#L1331

from rrule.

I have added for only BYSETPOS, U can use it and fix it accordingly.

from rrule.

+1
I have the same issue.

from rrule.