search

humanmade / Cavalcade

A better wp-cron. Horizontally scalable, works perfectly with multisite.
https://engineering.hmn.md/projects/cavalcade/
Other
517 stars 46 forks source link

Don't include failed jobs in the WordPress cron array. #27

Closed dd32 closed 7 years ago

dd32 commented 7 years ago

Don't include failed jobs in the WordPress cron array. This allows WordPress to re-queue a recurring cron task after it's failed - See #23

This has potential back-compat issues for anything calling Job::get_by_site() and expecting failed jobs in the resulting data. The parameter could be switched to default to true, with the WP compat shim explicitly passing false if that's a concern.

dd32 commented 7 years ago

@rmccue Fair, it's probably about time that core was to handle a list syntax..

Updated with your request.