ZF-7702: E-Mail is sent to Reply-To if given
Description
Adding a reply-to address (via setReplyTo()) and sending the mail results in sending the mail to the given address additionaly to TO/CC/BCC.
Adding a reply-to address (via setReplyTo()) and sending the mail results in sending the mail to the given address additionaly to TO/CC/BCC.
Comments
Posted by Jachim Coudenys (coudenysj) on 2009-09-08T05:39:43.000+0000
The setReplyTo() function uses _addRecipientAndHeader, but should work like setFrom(), where only the header is set.
Posted by Jachim Coudenys (coudenysj) on 2009-09-08T06:40:39.000+0000
This is the first patch I make, so I hope it's a good one.
The patch fixes the the Reply-To header functions. It's acting like the ReturnPath functions.
Posted by Benjamin Eberlei (beberlei) on 2009-09-18T06:35:38.000+0000
Are you doing this on windows and using the Sendmail Transport.
Posted by Jachim Coudenys (coudenysj) on 2009-09-18T07:11:39.000+0000
I am, but this has nothing to do with the fact that the Reply-To address is added to the receivers list, I think.
Posted by Oliver Baltz (dewoob) on 2009-10-05T12:46:46.000+0000
I'm experiencing same bug with SMTP under Linux! The Reply-To address must not be added to the list of recipients, but just to the headers.
Posted by Dolf Schimmel (Freeaqingme) (freak) on 2009-10-15T19:17:20.000+0000
Issue fixed. Thank you for reporting.
Posted by Satoru Yoshida (satoruyoshida) on 2009-10-15T22:35:03.000+0000
Thank you, Dolf. And I add some fix at trunk SVN r18566: 1) from tab code to 4 spaces. 2) from $name = $this->_filterEmail($name); to $name = $this->_filterName($name);
Posted by Jachim Coudenys (coudenysj) on 2009-10-16T00:10:32.000+0000
Was the patch useful? Just to know if I should create other patches in the future...
Posted by Thomas Weidner (thomas) on 2009-10-16T00:40:55.000+0000
Patches are always usefull.
Even if te committer does not use it, he will always look at your sugestion and become an much better idea of what your problem and solution is.
It also gives you a better understanding of the framework.
And when you also add unittests to your patch, then the committer can use it completly without changes.
Posted by Dolf Schimmel (Freeaqingme) (freak) on 2009-10-16T10:04:24.000+0000
Yes, the patch certainly was useful as I hardly know anything about Zend_Mail (I did use it once though!). For some reason however your patch was pretty mangled up by which it was hard to see what actually did change, that is why I only used it as a reference. So please; do attach a patch too in future where possible, it is always appreciated!
Posted by Michael Sheakoski (michael) on 2009-12-27T23:02:21.000+0000
This issue still exists as of v1.9.6. Seems like the patch never made it in.
Posted by Satoru Yoshida (satoruyoshida) on 2010-01-04T20:27:12.000+0000
Hi, Michael, It seems that possibly you might misunderstand about the Next Minor Release, for me.
the Next Minor Release means not 1.9.x but 1.10.0 . I hope this comment could help you. ;-)