ZF-7852: Zend_Service_Simpy Http/Rest client clean up

Issue Type:
Patch
Created:
2009-09-15T22:06:01.000+0000
Last Updated:
2009-09-18T08:50:06.000+0000
Status:
Resolved
Fix version(s):
  • 1.9.3 (22/Sep/09)
Reporter:
Duo Zheng (duoduo)
Assignee:
Matthew Weier O'Phinney (matthew)
Tags:
  • Zend_Service_Simpy
Related issues:
Attachments:

Description

Removed unneccessary require_once of Zend_Rest_Client when it is never used. Zend_Rest_Client was being included in the constructor even when Zend_Http_Client is already included and is the only one being used.

Also renamed class property Zend_Service_Simpy::$_http to Zend_Service_Simpy::$_httpClient. I hope I am not going to far with that it just seemed that is the standard naming convention.

Patch file attached

Comments

Posted by Duo Zheng (duoduo) on 2009-09-15T22:07:21.000+0000

Patch file

Posted by Duo Zheng (duoduo) on 2009-09-16T11:46:24.000+0000

Matthew, The comment


/**
 * @see Zend_Service_Rest
 */

Is still in constructor.

Posted by Duo Zheng (duoduo) on 2009-09-18T08:50:06.000+0000

Matthew Turland has removed the stray code and comment. Resolving issue.