DueTask scheduled inconsistent

[guikingone]

DueTask scheduled inconsistent

[Guikingone]

Actually, there might be a solution (note that I use "might" and not "there's a solution" 😄):

• When retrieving tasks using getDueTasks(), we can filter the results using last_execution option, this one is set using a DatetimeImmutable each time that the task is executed.

Maybe that we can do something like this:

Given the current date (using a specific format)
When I retrieve the due tasks 
And that I filter each one using CronExpression()->isDue()
And that I check if the current date (close to the current minute) does not equal the last execution of the task
Then I should find a set of tasks (or none if tasks have already been executed

WDYT?

[jvancoillie]

after search, it is a limit of the cron-expression parser, the isDue() method does not take seconds into account.

/**
 * Determine if the cron is due to run based on the current date or a
 * specific date.  This method assumes that the current number of
 * seconds are irrelevant, and should be called once per minute.
 *
 * @param string|\DateTimeInterface $currentTime Relative calculation date
 * @param null|string               $timeZone    TimeZone to use instead of the system default
 *
 * @return bool Returns TRUE if the cron is due to run or FALSE if not
 */

So, when I run several scheduler:consume in the same minute the task is executed each time.

(and should be called once per minute)

Is it a problem ? do we need to add a check for this ?

[Guikingone]

Hum, currently and apart from the scheduler:consume command, I don't know how to bypass this limitation, maybe we can pause the worker (thanks to sleep) until the next minute but the issue is located in the scheduler, not the worker.

Regarding the getDueTasks() method, I don't know how to add a check in it, maybe via a flag (maybe hasBeenCalled?) or an attribute but for me, the issue is related to the external library, not the bundle 🙁

[jvancoillie]

yes i agree with you i don't see how to improve that. you can close this request if you wish

[Guikingone]

Hi @jvancoillie

I'm gonna merge the fix this morning, should be released in 0.4.5 🙂

[jvancoillie]

great new ! will test this soon 😉