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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

greggles’s picture

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

Priority: Normal » Critical

ping...

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

jvandyk’s picture

Status: Reviewed & tested by the community » Fixed

Fixed. Thanks!

Anonymous’s picture

Status: Fixed » Closed (fixed)

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