- Issue created by @ergonlogic
- Status changed to Needs review
10 months ago 1:41am 26 January 2024 - last update
10 months ago 275 pass, 2 fail The last submitted patch, 2: eca-suppress_errors_when_processing_scheduled_tasks-3417097-2.patch, failed testing. View results →
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.- 🇩🇪Germany jurgenhaas Gottmadingen
This is an amazing finding, thank you so much @ergonlogic, your conclusion seems exactly what's going on here.
To move this forward, I'm changed the issue version to 2.0.x, which is the version where we always fix bugs. Then, we back port that to previous versions when that's relevant. Here, that's the case and that's why I've added the tag.
As for the code contribution, patches are going away (hopefully soon) and everything is being done with issue forks and merge requests. Would you mind turning this into that? That makes reviews and tests more convenient for all of us.
-
jurgenhaas →
committed 27cfe2c0 on 2.0.x
Issue #3417097 by ergonlogic: Errors thrown during cron when delayed...
-
jurgenhaas →
committed 27cfe2c0 on 2.0.x
-
jurgenhaas →
committed a13bac81 on 1.1.x
Issue #3417097 by ergonlogic: Errors thrown during cron when delayed...
-
jurgenhaas →
committed a13bac81 on 1.1.x
- Status changed to Fixed
9 months ago 9:32am 22 February 2024 Automatically closed - issue fixed for 2 weeks with no activity.
- 🇩🇪Germany jurgenhaas Gottmadingen
Just realized that we haven't fixed this issue in the best possible way and I've created a follow-up issue at 🐛 Delayed queue items should be requeued for later processing Needs review which needs a review before it can be back ported to 1.1.x