So first, I had an error which sent $message->params = 'some_string' when I told messaging to queue the email. But, because of this, all hook_cron's after messaging were never run as messaging_store_unpack produced a php error if (!empty($params[$field]) would fail of course, if $params was a string.

So this simple patch adds an if (is_array($params)) before trying to do a bunch of things with it.

Small change and should prevent messaging from exploding when someone like me does bad things :-D

CommentFileSizeAuthor
#1 messaging_649540.patch2.49 KBScott Reynolds

Comments

Scott Reynolds’s picture

Scott Reynolds commented
StatusFileSize
new messaging_649540.patch2.49 KB

heres the patch

jose reyero’s picture

jose reyero commented

Hey Scott, since I'll be working for a while on the 4.x branch, you can go ahead and commit this and other bug fixing patches to 2.x.

This branch (2.x) just takes bug fixes form now on though.