Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 2.0.0

Bug #18109 New "pipelining" parameter for Mail SMTP never used
Submitted: 2010-12-09 20:23 UTC
From: chrisdunavant2 Assigned: alec
Status: Closed Package: Mail (version 1.2.0)
PHP Version: 5.2.4 OS: Linux
Roadmaps: (Not assigned)    
Subscription  


 [2010-12-09 20:23 UTC] chrisdunavant2 (Chris Dunavant)
Description: ------------ There is a "pipelining" parameter that can be set for the mail package. However, that parameter is never passed along to the Net_SMTP package when the object is instantiated. This parameter is therefore never used by Mail SMTP. include_once 'Net/SMTP.php'; $this->_smtp = &new Net_SMTP($this->host, $this->port, $this->localhost); Net_SMTP expects the pipelining parameter to be the fourth parameter to the Class instantiation call. Test script: --------------- No code snippet. Expected result: ---------------- N/A

Comments

 [2011-01-08 18:42 UTC] alec (Aleksander Machniak)
There was support for this, but has been broken. This patch fixes this: --- smtp.php (wersja 307260) +++ smtp.php (kopia robocza) @@ -348,7 +348,8 @@ include_once 'Net/SMTP.php'; $this->_smtp = &new Net_SMTP($this->host, $this->port, - $this->localhost); + $this->localhost, + $this->pipelining); /* If we still don't have an SMTP object at this point, fail. */ if (is_object($this->_smtp) === false) {
 [2011-01-15 00:44 UTC] alec (Aleksander Machniak)
-Status: Open +Status: Closed -Assigned To: +Assigned To: alec -Roadmap Versions: +Roadmap Versions: 1.2.1
This bug has been fixed in SVN. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.