Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 0.14.0

Bug #375 error in WSDL.php in the SOAP_WSDL_Parser
Submitted: 2003-12-08 10:26 UTC
From: smith at backendmedia dot com Assigned: arnaud
Status: Closed Package: SOAP
PHP Version: Irrelevant OS: Windows
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 15 - 7 = ?

 
 [2003-12-08 10:26 UTC] smith at backendmedia dot com
Description: ------------ In SOAP_WSDL_Parser::startHandler method values for the operations get overwritten which result in an incomplete array. Here is a patch that fixed thigns for me in the current 0.8rc2 release.

Comments

 [2003-12-08 10:57 UTC] lsmith
err here is the patch: RCS file: /repository/pear/SOAP/WSDL.php,v retrieving revision 1.51 diff -u -r1.51 WSDL.php --- WSDL.php 8 Sep 2003 17:30:29 -0000 1.51 +++ WSDL.php 8 Dec 2003 10:59:59 -0000 @@ -1182,7 +1182,11 @@ // sect 3.4 // soap:operation, attributes: soapAction(required), style(optional, default = soap:binding:style) if (!isset($attrs['style'])) $attrs['style'] = $this->wsdl->bindings[$this->currentBinding]['style']; - $this->wsdl->bindings[$this->currentBinding]['operations'][$this->currentOperation] = $attrs; + if (isset($this->wsdl->bindings[$this->currentBinding]['operations'][$this->currentOperation])) { + $this->wsdl->bindings[$this->currentBinding]['operations'][$this->currentOperation] = array_merge($this->wsdl->bindings[$this->currentBinding]['operations'][$this->currentOperation],$attrs); + } else { + $this->wsdl->bindings[$this->currentBinding]['operations'][$this->currentOperation] = $attrs; + } break; case 'body': // sect 3.5
 [2003-12-18 21:18 UTC] arnaud
The line breaks broke the patch. Can you put it online ?
 [2004-04-03 12:09 UTC] arnaud
This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better.