Issues

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: - zf-simpy.18101.patch

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

<pre class="highlight">
/**
 * @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.

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