|
|
php.net | support | documentation | report a bug | advanced search | search howto | statistics | random bug | login |
[2008-05-01 18:02 UTC] mike at silverorange dot com
Description: ------------ When decoding a SOAP response into zvals, the decoder sometimes gets confused decoding objects. We noticed this using the PayPal Express Checkout SOAP API. For one request type, the returned XML was correct; however, the decoded PHP objects that were returned were incorrect. One value that should have been a string was a string reference and an object that should have been an object was as string. Upon further investigation, we found other users had been experiencing this specific problem using PayPal's SOAP API with PHP since PHP 5.2.2. After a bit of CVS research, it seems the bug was introduced in 1.103.2.21.2.27 in soap_encoding.c to fix bug #37013. The patch at http://labs.silverorange.com/files/php-patches/soap-object-reference-bug-20080501.patch works around the bug for our needs, but it does not fix the underlying problem and probably reintroduces #37013. Reproduce code: --------------- I can't provide a runnable test case unless you have PayPal developer API credentials. If so, you can run the unit tests in the package at: http://labs.silverorange.com/files/Payment_PayPal_Soap/Payment_PayPal_Soap-0.0.4.tgz The second test will fail if the bug is present and pass if the bug is fixed. I've attached a var_dump of broken an working SOAP results to illustrate the broken 'PayerInfo' element parsing. Expected result: ---------------- object(stdClass)#114 (6) { ["Timestamp"]=> string(20) "2008-05-01T17:41:45Z" ["Ack"]=> string(7) "Success" ["CorrelationID"]=> string(13) "9d6aa8c0309df" ["Version"]=> string(8) "1.000000" ["Build"]=> string(6) "548868" ["GetExpressCheckoutDetailsResponseDetails"]=> object(stdClass)#113 (2) { ["Token"]=> string(20) "EC-6AU15743MX2745842" ["PayerInfo"]=> object(stdClass)#112 (6) { ["Payer"]=> string(0) "" ["PayerID"]=> string(0) "" ["PayerStatus"]=> string(10) "unverified" ["PayerName"]=> object(stdClass)#110 (5) { ["Salutation"]=> string(0) "" ["FirstName"]=> string(0) "" ["MiddleName"]=> string(0) "" ["LastName"]=> string(0) "" ["Suffix"]=> string(0) "" } ["PayerBusiness"]=> string(0) "" ["Address"]=> object(stdClass)#111 (9) { ["Name"]=> string(0) "" ["Street1"]=> string(0) "" ["Street2"]=> string(0) "" ["CityName"]=> string(0) "" ["StateOrProvince"]=> string(0) "" ["CountryName"]=> string(0) "" ["PostalCode"]=> string(0) "" ["AddressOwner"]=> string(6) "PayPal" ["AddressStatus"]=> string(4) "None" } } } } Actual result: -------------- object(stdClass)#114 (6) { ["Timestamp"]=> string(20) "2008-05-01T17:56:31Z" ["Ack"]=> string(7) "Success" ["CorrelationID"]=> string(13) "233fdab3c0758" ["Version"]=> string(8) "1.000000" ["Build"]=> string(6) "548868" ["GetExpressCheckoutDetailsResponseDetails"]=> object(stdClass)#113 (2) { ["Token"]=> &string(20) "EC-8JA130819B922424F" ["PayerInfo"]=> string(20) "EC-8JA130819B922424F" } } PatchesPull RequestsHistoryAllCommentsChangesGit/SVN commits
|
|||||||||||||||||||||||||||||||||||||
Copyright © 2001-2024 The PHP GroupAll rights reserved. |
Last updated: Thu Nov 21 11:01:29 2024 UTC |
Here is a test case that illustrates the bug, using the same technique the ext/soap test cases in the PHP distribution use. Please excuse the big block-o-base64, the namespace URIs in the XML were apparently triggering your spam filter. <?php class TestSoapClient extends SoapClient { public function __doRequest($req, $loc, $act, $ver, $oneway){ $data=<<<EOF nZdbd9o4EIDf8ys49BlkQ7IBH0KXOOluzoaEU0i33Zc9ij0k2tgWleQA++s7snzBlxQ3vCRIM99c NRKTj7sw6LyCkIxHF127b3U7EHncZ9HTRfdh9ak36n6cnkyW97NF7/rui3MdvULAN9BBvUg62fpF 91mpjUOI9J4hpLKP25LTTZ+LJ6L/IZAqkm5Z1W2jahzKVXeS5Vrb7ba/HSbCA8uyydf57TIB9Vgk FY08KLT841qFcHtZz7voxiJy4JHuHbph0nG5AJeHGx5BpFb7DchMdivjnwa8ldrIgFjnJFYsYGqf aUoaBsYOpxJtRDQE6SjPWc7mtw5WzqFSglBYyEzFfysIi1hjgjK+ZE8fMumQihdQ1VDmyWomQ2NP JZ1SFpqZ5dxV9j/IZ7YpizkLul/QYLa4IZlAH2uSK9UUzFq+/4J5rAnoxSK3Elol1x4QCR7s8rDg MaiS4fL2ksqiinkFI/lWWN3pSadTHJU/gfog9JpeBS8WWE0D+WUvJXNUEr6O0clgurO6JLXwGb7H IBUIV4CPDjMayMzaG/4ecDEDjhtLxcMMbtxPTBgLaOMIujlrJRsP2KI3/gJbdcuFX6IjX+/qvj5Q 2mGRlcDTnwWaCGaAo4LL+PE/8NTP5SbkILA0naQpn0mBSUOFi6pfcn/fYThp/rXSyCZ/gLrebQRI 6T6D98JjdQWKskB+BolpknCkTLmbK4ZHXtFw0zb5U2ykUc/6rTcYruxT58x2rNE/E5JzpicZeua9 tIYuY8/DYCYElYrW4EJAQPUYuLlqjaJg2UNqeZ6/todYhkNIjv5irqfW0DOrbyWfCUlVc9RlzAK/ PWg0Gp4iJdHKGUfLmX595wlpi6+enRV/gajCqoASEaN37fZs+9v5mT08Xw2G4/H4bOBiY2iB0jHb 43GN1rzCzdcrTmQqVTdCdHjm+9oXowE77L4Afk//9j0eTkiiWofpdqrPkCsDmj+MBn/PToffvs7H boooWqeALBVVsayHgUdM48y2y30wWHwRsTUDPyWa7Tr1rjyrEiZ2HI/0Ri03ekzQIFZJf7ftDlIh fGJCKmO3ZRdfmxxPSK6aH/uUOWe+H8CvQKte3dL3OpVpVhMVr9ds9y530ppVoaZiLo8jJfbVm8+s FvV/WKaUdKcOuowli7Cfj91Aeraazq/YLJ2HcvB3b12BWdo6efqayrlUAkDZzQR7MMy0O0uFl5kR njYhBseD05MfHwtHPcbZnorVLFEF92Ih+CvDx3ozZPaXdrQkWHWifV0bSloAEg8fIqbA7yQWZa7Q lOoFx3s00Pxmx8f6g62Ui1Wspk1wv41qQ/NwqwjAjCy8eg92m5mNE6+0V1BdHq2ZCPW8K0mUj1W6 dXA/kPwiKJ5Sbe+v7J11VD7J+cGrS7+w8Ccpqf0mnZ78AA== EOF; return gzinflate(base64_decode($data)); } } $client=new TestSoapClient("https://www.paypal.com/wsdl/PayPalSvc.wsdl"); var_dump($client->GetExpressCheckoutDetails(array('GetExpressCheckoutDetailsRequest'=>array('Version'=>50.0,'Token'=>'dummy')))); ?>