actions_do makes it look like you don't have to pass in $context, but if you don't, a subsequent array_merge() causes errors.

CommentFileSizeAuthor
actions_do.diff555 bytesDave Cohen

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Title: actions_do signature not quite right » actions_do signature sets $context as null instead of empty array
Status: Needs review » Reviewed & tested by the community

Specifically the error is

array_merge() [function.array-merge]: Argument #1 is not an array in ... actions/actions.inc on line 98.

I applied this patch and it fixed the error for me.

To repeat the bug, create an action like system_send_email and call it with actions_do(1, $object);

I believe this is RTBC. Sorry if I'm overstepping my bounds by rtbc in your queue.

Dave Cohen’s picture

Dave Cohen commented
Priority: Normal » Critical

ping...

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

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.