Issues

ZF-7702: E-Mail is sent to Reply-To if given

Issue Type: Bug Created: 2009-08-27T09:08:52.000+0000 Last Updated: 2010-01-04T20:27:16.000+0000 Status: Resolved Fix version(s): - 1.10.0 (27/Jan/10)

Reporter: Marius Mühlberger (tracer) Assignee: Dolf Schimmel (Freeaqingme) (freak) Tags: - Zend_Mail

Related issues: Attachments: - replyTo.patch

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.

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. ;-)

Have you found an issue?

See the Overview section for more details.

Copyright

© 2006-2016 by Zend, a Rogue Wave Company. Made with by awesome contributors.

This website is built using zend-expressive and it runs on PHP 7.

Contacts