Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Pretty simple idea. The Cron class could do with a bit of a tidy, everything just lives in the run() method. As well as just having a general spring clean, splitting some of the logic out to separate methods will allow better DX for people that want to swap this class out for their own class, extending the Drupal\Core\Cron class.
Proposed resolution
Separate some logic out into separate methods, maybe something like the addition of:
- setCronLastTime
- processQueues
- invokeCronHandlers
Remaining tasks
Reviews, consensus.
User interface changes
None
API changes
None
Comment | File | Size | Author |
---|---|---|---|
#1 | 2249899.patch | 3.68 KB | damiankloip |
Comments
Comment #1
damiankloip CreditAttribution: damiankloip commentedSomething like this.
Comment #3
damiankloip CreditAttribution: damiankloip commented1: 2249899.patch queued for re-testing.
Comment #4
dawehnernice!
Comment #5
catchCommitted/pushed to 8.x, thanks!