Yet more places.

Going forward, would make sense to explicitly cast this in Horde_Http_Client, but the phpdoc explicitly states it should be a string value, and that package also doesn't currently require Horde_Url as a dependency.
horde · Mar 4, 2015 · 601d336 · 601d336
1 parent 097b8a9
commit 601d336
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/framework/Service_Twitter/lib/Horde/Service/Twitter/Request/Oauth.php b/framework/Service_Twitter/lib/Horde/Service/Twitter/Request/Oauth.php
@@ -35,7 +35,7 @@ public function get($url, array $params = array())
         $url = ($url instanceof Horde_Url) ? $url : new Horde_Url($url);
         $url->add($params);
         try {
-            $response = $this->_twitter->getHttpClient()->get($url->setRaw(true), array('Authorization' => $request->buildAuthorizationHeader('Twitter API')));
+            $response = $this->_twitter->getHttpClient()->get((string)$url->setRaw(true), array('Authorization' => $request->buildAuthorizationHeader('Twitter API')));
         } catch (Horde_Http_Exception $e) {
             throw new Horde_Service_Twitter_Exception($e);
         }
@@ -70,7 +70,7 @@ public function post($url, array $params = array())
                        $this->_twitter->auth->getAccessToken($this->_request));
         $url = ($url instanceof Horde_Url) ? $url : new Horde_Url($url);
         try {
-            $response = $this->_twitter->getHttpClient()->post($url->setRaw(true), $params, array('Authorization' => $request->buildAuthorizationHeader('Twitter API')));
+            $response = $this->_twitter->getHttpClient()->post((string)$url->setRaw(true), $params, array('Authorization' => $request->buildAuthorizationHeader('Twitter API')));
         } catch (Horde_Http_Exception $e) {
             throw new Horde_Service_Twitter_Exception($e);
         }