|  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #54684 SoapServer->handle() breaks type hinting
Submitted: 2011-05-07 02:55 UTC Modified: 2021-01-13 11:23 UTC
Avg. Score:4.2 ± 0.8
Reproduced:4 of 4 (100.0%)
Same Version:0 (0.0%)
Same OS:0 (0.0%)
From: luke at cywh dot com Assigned: cmb (profile)
Status: Closed Package: SOAP related
PHP Version: 5.3.6 OS: Mac OS X 10.6
Private report: No CVE-ID: None
View Add Comment Developer Edit
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:
Solve the problem:
20 + 20 = ?
Subscribe to this entry?

 [2011-05-07 02:55 UTC] luke at cywh dot com
SoapServer's handle() function breaks PHPs type-hinting for function parameters.

Note that if you remove "$server->handle()" you get the expected result. If you leave it there you get the actual result.

If this is intended it might have to do with SoapServer passing back stdClass as function arguments. But please note that this test code is not even used by SoapServer. I merely call the handle() function and every function call is effected, not just those invoked by SoapServer. Also for functions invoked by SoapServer, shouldn't a string fail because it's a non-object?

If this isn't going to be fixed because it's a "feature" then we have a documentation problem. There needs to be a warning on the handle() documentation that states type-hinting is broken globally.

If this was intended, I honestly don't think this should happen on a global scope. Only functions called by SoapServer should be effected, and even then the type should be at least restricted to an object.

Test script:
$server = new \SoapServer(NULL, array('uri' => 'http://localhost'));

class test
	public function sayHello(Blah $one, $two)
		return $one;

$test = new test;
var_dump($test->sayHello('one', 'two'));

Expected result:
Catchable fatal error: Argument 1 passed to test::sayHello() must be an instance of Blah, string given

Actual result:
string(3) "one"


Add a Patch

Pull Requests

Add a Pull Request


AllCommentsChangesGit/SVN commitsRelated reports
 [2011-09-15 15:29 UTC] henning dot panke at erento dot com
Can reproduce this issue on a gentoo system with php 5.2.17.
 [2021-01-13 11:23 UTC]
-Status: Open +Status: Closed -Assigned To: +Assigned To: cmb
 [2021-01-13 11:23 UTC]
This has apparently been fixed in the meantime; at least I cannot
reproduce the reported behavior with PHP-7.3 nor PHP-7.4.
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Tue May 28 10:01:31 2024 UTC