The value of the site_mail was not retrieved correctly if no other default "from" option was specified.

CommentFileSizeAuthor
smtp.mail_.inc_.patch759 byteslucor
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Fabianx’s picture

Status: Needs review » Reviewed & tested by the community

Reviewed and Tested.

site_email does not exist as variable, it should really be site_mail.

Please apply.

franz’s picture

Status: Reviewed & tested by the community » Fixed

Thanks for the catch. Next time, please read http://drupal.org/patch to roll a patch properly.

Status: Fixed » Closed (fixed)

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

Ryan Palmer’s picture

Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Closed (fixed) » Active

Also an issue in the D6 branch. All instances of site_email in smtp.module should be changed to site_mail.

Fabianx’s picture

Status: Active » Patch (to be ported)
wundo’s picture

Status: Patch (to be ported) » Fixed

Commited, thanks!

Status: Fixed » Closed (fixed)

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