Incorrect cron description

[calebdw]

Hello!!

Given the following cron:

CronTranslator::translate('3-59/5 * * * *');

This package outputs 57 times every 5 minutes instead of the crontab guru

At every 5th minute from 3 through 59.

Thanks!

[lorisleiva]

Hey, thanks for raising that!

You're right that is incorrect.

That's because, this package simplifies ranges (which can be pretty lengthy e.g. 1,3,12-27,30-40,41) to the amount of times they would be triggered so that the output is shorter and less confusing to read.

However, when combined with frequencies (e.g. /5), the result makes little sense and doesn't convey what's actually happening.

Because this package is multi-language I suspect this would actually need a decent refactoring first, allowing different sentence templates so each language can decide how to construct these more complex sentences.

I won't have time to dedicate to this for a while but I'd be more than happy to accept/reviews PRs for it.