ZF-6334: Zend_Mail_Transport_Smtp constructor leaves $_name property same

Issue Type: Docs: Problem Created: 2009-04-17T05:48:47.000+0000 Last Updated: 2009-10-31T04:53:14.000+0000 Status: Resolved Fix version(s): - 1.9.6 (24/Nov/09)

Reporter: Alex Scherbakov (lexabug) Assignee: Satoru Yoshida (satoruyoshida) Tags: - Zend_Mail

Related issues: Attachments:


When I try Example 33.4. Sending Multiple Mails per SMTP Connection from… first of all I get a warning message "inet_pton(): Unrecognized address localhost" and if I change the $host parameter of constructor to different IP ADDRESS NOT HOSTNAME, the warning message continues to emerge.

Also as I noticed in code of Zend_Mail_Transport_Smtp->_sendMail method it tries to execute next line $this->_connection->helo($this->_name);. Is it proper line or it should be like this $this->_connection->helo($this->_host);}?

P.S. I think for loops in examples 33.4 and 33.5 have mistakes for ($i = 0; $i > 5; $i++) { it will never do any iteration because of condition $i > 5. It should be $i < 5


Posted by Marcos Gil Fuertes (marcis) on 2009-05-15T04:41:08.000+0000

That's TRUE. What is the "_name" property used for?

Apparently, only for that call. It's set in the construct method from $config['name'] value (if set). By default, its value is "localhost".

Does it make sense to duplicate the 'host' value? Or are we missing something about "helo" method?


Posted by Satoru Yoshida (satoruyoshida) on 2009-10-31T03:41:21.000+0000

Hi, Marcos.

The $_name is used for sender hostname. It is different from $_host which means receiver's one.

I think the document seems to be bit difficult to understand how to use helo() function.

So, I will update the document.

Posted by Satoru Yoshida (satoruyoshida) on 2009-10-31T04:53:13.000+0000

Solved in SVN r18755 of trunk, r18756 of 1.9 branch.

Have you found an issue?

See the Overview section for more details.


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

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