Currently registration will log an error if the 'result' key in the array returned from drupal_mail() is NULL. That means if you are using registration in conjunction with a module like queue_mail, which sets the 'send' key of $message to FALSE and then queues the message to be sent on the next cron run, you'll end up with incorrect errors in watchdog.
This patch will only log the error if the 'result' key is explicitly false in accordance with this comment in mail.inc:
// The original caller requested sending. Sending was canceled by one or
// more hook_mail_alter() implementations. We set 'result' to NULL, because
// FALSE indicates an error in sending.
Comment | File | Size | Author |
---|---|---|---|
#2 | 2750457_11302987_improve_drupal_mail_test.patch | 492 bytes | bwood |
Comments
Comment #2
bwood CreditAttribution: bwood as a volunteer commentedHere's the patch.
Comment #3
bwood CreditAttribution: bwood as a volunteer commentedComment #4
bwood CreditAttribution: bwood as a volunteer commentedComment #5
asrobLooks good to me, successfully attached this patch.
Comment #8
gcb