Hi,

i reviewd cron control today. Here is a complete new appoch for managing cronjobs in Drupal 7. Drupal 7 provides hook_implements_alter(). Its possible to rewrite the order of hooks or unset certain implementions completely.

- Modifying module weight in hook_install is not necessary.
- Own implemention of hook_cron is not necessary, everything is done in hook_implements_alter().
- Disabled implementions are never called.

Patch is attached.

CommentFileSizeAuthor
#1 cron_control-hook_module_implements_alter-2.patch6.35 KBwebflo
cron_control-hook_module_implements_alter.patch5.69 KBwebflo

Comments

webflo’s picture

webflo commented
StatusFileSize
new cron_control-hook_module_implements_alter-2.patch6.35 KB

Here is an new patch. DrupalQueue is handled by drupal_cron_run() too.

mkalkbrenner’s picture

mkalkbrenner
Primary language German
Location 🇩🇪
commented
Status: Active » Needs review

I haven't reviewed your patch in detail yet. But your approach sounds promising. I hope to find the time to review and test your patch soon.

mkalkbrenner’s picture

mkalkbrenner
Primary language German
Location 🇩🇪
commented
Status: Needs review » Fixed

I switched to hook_implements_alter() as recommended. But in detail the patch looks different. (see git)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.