The reminder watchdog entry is always inserted, even when the message has not actually been sent due to an error or a message alteration (see #800434: drupal_mail, allow hook_mail_alter implementation to cancel mail).

CommentFileSizeAuthor
#1 signup-1297808-1.D6.patch1.07 KBplach
#1 signup-1297808-1.patch1.07 KBplach
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
CreditAttribution: plach commented
Status: Active » Needs review
Issue tags: +Needs backport to D6, +Quick fix
FileSize
signup-1297808-1.patch1.07 KB
signup-1297808-1.D6.patch1.07 KB

Testing the message result should be enough. Attached also a backported patch to 6.x.

sgabe’s picture

sgabe CreditAttribution: sgabe commented
Status: Needs review » Reviewed & tested by the community

The patch in #1 applies fine and fixes the issue.

sgabe’s picture

sgabe CreditAttribution: sgabe commented
Component: Code » Email

Changing component.

sgabe’s picture

sgabe CreditAttribution: sgabe commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Reviewed & tested by the community » Needs review

Committed to D7, thanks!