[#ZF-3876] getMessage() should be getMessages() in Zend/Mail/Protocol/Smtp.php - Zend Framework Issue Tracker

Details

Type: Bug
Status: Resolved
Priority: Major
Resolution: Fixed
Affects Version/s: 1.6.0RC1
Fix Version/s: 1.6.2
Component/s: Zend_Mail
Labels:
None

Fix Version Priority:
Should Have

Description

In lib/Zend/Mail/Protocol/Smtp.php there appears to be a typo: in the helo() function when the client hostname fails to validate, the exception string fails to be built because the method to get the error messages is incorrect (missing an "s").

throw new Zend_Mail_Protocol_Exception(join(', ', $this->_validHost->getMessage()));

should read

throw new Zend_Mail_Protocol_Exception(join(', ', $this->_validHost->getMessages()));

Activity

Hide

Permalink

Satoru Yoshida added a comment - 01/Sep/08 6:02 PM

Solved in SVN r11196.
The getMessage() is not found, but getMessages() function exists in Zend_Validate that was included by Zend/Mail/Protocol/Abstract.php.

Abstract.php is parent of Smtp.php.

Show

Satoru Yoshida added a comment - 01/Sep/08 6:02 PM Solved in SVN r11196. The getMessage() is not found, but getMessages() function exists in Zend_Validate that was included by Zend/Mail/Protocol/Abstract.php. Abstract.php is parent of Smtp.php.

People

Assignee:

Satoru Yoshida

Reporter:

Sebastian

Vote (0)

Watch (1)

Dates

Created:

06/Aug/08 6:53 PM

Updated:

21/Mar/12 11:04 AM

Resolved:

01/Sep/08 6:03 PM