When sending a confirmation email, the from email uses either "smtp_from" or "site_mail" as the default. However, on the config form there is no default provided. If an admin edits the config form and does not provide a from email address, this results in the email being set to "", which will be set into the variable and the default would be ignored.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

BassistJimmyJam’s picture

Status: Active » Needs review
FileSize
2.31 KB

Patch attached.

jaydub’s picture

Thanks for the patch, committed with some minor changes and some additional form field descriptions.

jaydub’s picture

Version: 7.x-1.1 » 7.x-1.x-dev
Status: Needs review » Fixed

  • jaydub committed 0022ad7 on 7.x-1.x
    Issue #2307949 by BassistJimmyJam, jaydub - Fix potentially empty value...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.