I find that cron actions are not triggered without the attached patch. Just me?

CommentFileSizeAuthor
actions-cron.diff495 bytesDave Cohen

Comments

Dave Cohen’s picture

Dave Cohen commented
Status: Active » Needs review
shariharan’s picture

shariharan commented

Thank you.

Dave Cohen’s picture

Dave Cohen commented
Priority: Normal » Critical

ping...

This, along with http://drupal.org/node/260957 are required for cron actions to work.

Dave Cohen’s picture

Dave Cohen commented
Status: Needs review » Reviewed & tested by the community

whoops, forgot to update status.

jvandyk’s picture

jvandyk commented
Status: Reviewed & tested by the community » Fixed

Fixed. Thanks!

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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

NigelC’s picture

NigelC commented
Status: Closed (fixed) » Active

Any reason why this wasn't implemented the way Dave proposed?

I see (in 2.6) that the following change wasn't made.

function actions_cron() {
- $aids = _actions_get_hook_aids('cron');
+ $aids = _actions_get_hook_aids('cron', 'run');

jvandyk’s picture

jvandyk commented
Status: Active » Fixed

Uh...that would be user error. On my part. Fixed, will appear in 5.x-2.7.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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