Wrong rule text when repeat by month and bysetpos is larger than 1

[Aaron-Lin-74]

Reporting an issue

• [x] Verify that you've looked through existing issues for duplicates before creating a new one yes
• [x] Code sample reproducing the issue. Be sure to include all input values you are using such as the exact RRule string and dates. new RRule({ freq: RRule.MONTHLY, interval: 1, byweekday: RRule.MO, bysetpos: [2] }) rule.toString() RRULE:FREQ=MONTHLY;INTERVAL=1;WKST=MO;BYDAY=MO;BYSETPOS=2
• [x] Expected output every month on the second Monday
• [x] Actual output every month on Monday
• [x] The version of rrule you are using 2.6.4
• [x] Your operating system windows 10
• [x] Your local timezone (run $ date from the command line of the machine showing the bug) Fri Jul 15 15:05:49 AEST 2022

[darkLord19]

@Aaron-Lin-74 did you find solution for this? I am also facing the same issue and for larger than 1 values rrule.after produces invalida date output

[smallst]

same issue. here's the demo output. the all() returns correct day while the nlp output text seems missing bysetpos param. don't very familiar with nlp. but it may not so hard to fix ?

[smallst]

i see. byweekday=2MO shows correct second monday whille bysetpos is currently ignored in nlp

[pcreehan]

Also seeing this, not just for bysetpos > 1 but also for bysetpos=-1. Seems the nlp isn't respecting bysetpos parameter at all