Home → Problems →

More PHP wrapper issues?

• Edit

ra.keizer

25 May, 2012 04:19 PM

I have serious issues using the PHP wrapper.

e.g.:

$plan = $client->getPlan('FREE'); $allplans = $client->getPlans();

I have 2 plans configured, I would expect the second call return all plans instead of the plan previously fetched using the first call.
What's going on?

1. 1 Posted by ra.keizer on 25 May, 2012 04:36 PM

   

   I am using PHP Version 5.3.10

   • Edit

2. 2 Posted by ra.keizer on 25 May, 2012 07:23 PM

   

   I found the solution to this problem.
   There is a major bug in CurlAdapter.php.

   in the 'else' part where your're checking if the resource exists you forgot to pass the url

   curl_setopt($this->_resource, CURLOPT_URL, $url);

   the code should be:

       if (!$this->_resource) {
           $this->_resource = curl_init($url);
           $userAgent = (isset($_SERVER['SERVER_NAME'])) ? $_SERVER['SERVER_NAME'] . ' - CheddarGetter_Client PHP' : 'CheddarGetter_Client PHP';
           $options = array(
               CURLOPT_RETURNTRANSFER => true,
               CURLOPT_SSL_VERIFYPEER => false,
               CURLOPT_SSL_VERIFYHOST => false,
               CURLOPT_CONNECTTIMEOUT => 10,
               CURLOPT_TIMEOUT => 60,
               CURLOPT_USERAGENT => $userAgent,
               CURLOPT_USERPWD => $username . ':' . $password,
               CURLOPT_FOLLOWLOCATION => true,
               CURLOPT_MAXREDIRS => 10             
     );
           foreach ($options as $key=>$val) {
               curl_setopt($this->_resource, $key, $val);
           }
       } else {
           curl_setopt($this->_resource, CURLOPT_USERPWD, $username . ':' . $password);
           curl_setopt($this->_resource, CURLOPT_HTTPGET, true);
     curl_setopt($this->_resource, CURLOPT_URL, $url);
       }
   

   • Edit

3. 3 Posted by ra.keizer on 25 May, 2012 07:26 PM

   

   to be frank I think the code in there is a mess... excuse me...

   I think you should create the context without url parameter if the resource doesn't exist, then init the context with it's default parameters.
   After that you should pass the request specific parameters like the url, password, user and post parameters.

   • Edit

4. Support Staff 4 Posted by Marc Guyer on 25 May, 2012 09:34 PM

   

   I see. Thanks for pointing out the problem. The PHP wrapper is open source and the new curl adapter was contributed by the community in a recent version. Rather than making accusatory comments here or here (or anywhere), you might consider forking the repo and contributing the fix.

   We're here to help and make every effort to do so. We will continue to do so despite your inflammatory comments though it is my hope that our discourse proceeds in a more civil manner.

   • Edit

5. 5 Posted by ra.keizer on 25 May, 2012 09:42 PM

   

   I did not mean to be accusatory, i wanted to be helpfull.
   I found a nasty bug, you could be a little more thankfull.

   • Edit

6. ra.keizer closed this discussion on 26 May, 2012 03:12 PM.