As mentioned in #1732474: Error while creating new nodes, sometimes reminders get saved to the node_reminder table with a NID of 0. I'm uncertain of the exact cause, but the end result of this is that reminder emails may be sent out with missing data, for example:

This message is sent based on the reminder at http://example.com/node/"

What is happening is this runs:

$tokens['reminder_url'] = url("node/{$object->reminder->nid}", array('absolute' => TRUE));

...when $object and/or $object/reminder are empty.

CommentFileSizeAuthor
#1 nid-0-reminders-2285379-1.patch403 bytesalexdmccabe

Comments

alexdmccabe’s picture

Status: Active » Needs review
StatusFileSize
new403 bytes

Here's a patch to fix the problem. Basically, it removes any reminder(s) with NID of 0 before sending any mails.

fizk’s picture

Status: Needs review » Needs work

Thanks for the patch alexdmccabe. I think we should try to figure out the root cause.