There is a bug in the add function in the Cron parser code. The problem is that nov is 11 and feb is 2 and it only adds constraints while min is less than or equal to max. Since 11 is already greater than 2, the constraints aren't getting added correctly. Since no constraint is added, it is treated as a 0 which to the scheduling engine means 'use the last month', hence you end up with December.

Shouldn't be a hard fix, basically you need to go from min to max modulus the field max which is found in the FIELDS struct. Unfortunately I don't have time at the moment to make and test the fix, but hopefully that's enough info if you would like to make a fix.

would it make sense to modify that function so min is always the lesser of the two values ?

No, because that would be the opposite of what you want. You don't want the months between 2 and 11 (this would be feb-nov), you want the months between 11 and 2 (nov-feb).

Same issue exists for days : sat-mon returns empty, as does any range that passes sunday

It's been awhile but the pull request I have outstanding fixed the issue
for me. Give that a shot.

On Wed, Sep 7, 2016 at 1:43 PM Emily [email protected] wrote:

Same issue exists for days : sat-mon returns empty, as does any range that
passes sunday

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#48 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA0WkOR5yUSpDlQZ3D8vxkJrf5DTnPAGks5qnyH8gaJpZM4B5ARy
.

Do you know if that will also fix the days wrapping? Or do I have to apply the fix elsewhere?

Thanks! Confirmed it's working, used this file: https://raw.githubusercontent.com/solomon23/later/070b7ed2ad70e5dc4fb344299ba7fb67874e2b2c/later.js

@bunkat Any update on this? I have a requirement where I had to run a task for every 45 days and the current version of laterjs is not returning the expected next occurrence. It returns the first day of the next month as the next occurrence. I tried @solomon23's fix. The recur parser seems to work fine but the text parser is not returning the expected dates.