Hi, I've just encountered an issue where I have a custom token being used as the Send mail action. Sometimes this token evaluates as empty, but I've noticed that an email is still sent (somehow seems to default to my server email). This patch tries to solve this by checking before drupal_mail is called in the action.

There's probably a nicer way to do this but I needed a quick fix.

CommentFileSizeAuthor
dont-send-empty-recipient-email.patch643 bytesacbramley
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

TR’s picture

TR CreditAttribution: TR commented
Issue summary: View changes

Bump. Patch still applies.

TR’s picture

TR CreditAttribution: TR commented
Version: 7.x-2.x-dev » 8.x-3.x-dev
Category: Bug report » Feature request
Status: Needs review » Needs work
Issue tags: +Needs tests

Evidently, there's not much interest in this feature.

D7 Rules is basically in maintenance-only mode right now, as D7 is nearing its end-of-life. So I'm moving this to the 8.x-3.x branch for consideration.

This patch needs to be re-rolled for D8 and reviewed, and we need some test cases to demonstrate the patch works properly.