fields.dayOfWeek = [1,3,5] generates incorrect cron expression

[klumba12]

example: https://stackblitz.com/edit/typescript-playground-spjybj?file=index.ts

const at10amMonWenFri = parser.parseExpression('0 0 10 * 1,3,5');
const cron = parser.fieldsToExpression(at10amMonWenFri.fields).stringify(true);

// expected origin 0 0 10 * 1,3,5
// but cron equals to 0 0 10 * 1/2 
// when dow is 1/2  - Sundays are included in the generated dates
console.log(cron);

[abberadhi]

Hey, I can confirm I experience this issue as well. Feels like an issue related to the compactor. Please fix.

[harrisiirak]

Hi @klumba12 and @abberadhi!

Thanks for reporting and confirming this. It seems to be caused by the bugfix in #295. I'll revisit this and see if there is better fix for initial bug.

Best regards

[harrisiirak]

@klumba12 and @abberadhi created a quick fix Draft-PR: #312
Let me know how it working for you. Shortly, stringify should now retain original value where needed and only optimise multiple values based field when it makes sense.

[klumba12]

hi @harrisiirak, looks great! thank you

[d00rsfan]

Hi!
--there was notexisting bug report, sorry --

P.S. is it correct way to test with branche? just npm install https://github.com/harrisiirak/cron-parser#fix/stringify-range-step-handling