Email rerouting should be disabled if no email address is configured

ulf1 - July 31, 2008 - 16:54
Project:Reroute Email
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

Hi,

If reroute email module is enabled, but the rerouting emailadress is empty (I selected "restore default button), emails are not being sent to original recipient, because in your code you do

$message['to'] = variable_get(REROUTE_EMAIL_ADDRESS, ini_get('sendmail_from'));

despite the fact that REROUTE_EMAIL_ADDRESS is empty.

It should not be necessary to enable/disable the module all the time.
All configuration should be done from within the configuration dialog. If the reroute field is empty I would expect the email to be sent to its original recipients.

Thanks,
Ulf

#1

ceardach - May 20, 2009 - 18:26
Title:emails via contact form can not be sent when reroute email module is enabled» Email rerouting should be disabled if no email address is configured

I had this same problem and hacked the module (simplified patch attached). This allows me to disable rerouting by setting this in my $conf array in settings.php:

<?php
  $conf
= array(
   
'reroute_email_address' => '',
  );
?>

In a distributed development environment, it'd also make it easy for each dev to set a different email address to reroute to.

AttachmentSize
reroute_email.patch 242 bytes

#2

kbahey - May 20, 2009 - 18:32
Status:active» fixed

#3

System Message - June 3, 2009 - 18:40
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.