Ensure a leading backslash (and only one) is returned #737
Conversation
…ype template - Fixes #736
|
@bpolaszek can you please add a test case that shows this behavior? An additional test asset should be sufficient. Did you by chance/accident run the tests on PHP 7.1.0-beta3? That beta is broken... |
|
@Ocramius Good catch, I was on PHP 7.1.0-beta3 indeed :) Btw, I don't think an additionnal test was needed now, since I thought only namespaced classes were impacted. Actually the tests couldn't run properly even with \stdClass. |
| @@ -1032,6 +1032,6 @@ private function getMethodReturnType(\ReflectionMethod $method) | |||
| return ': \\' . $method->getDeclaringClass()->getParentClass()->getName(); | |||
| } | |||
|
|
|||
| return ': \\' . (string) $returnType; | |||
| return ': \\' . ltrim((string) $returnType, '\\'); | |||
There was a problem hiding this comment.
The tests pass also without this change.
|
Closing then: 7.1.0-beta3 is bugged as per http://marc.info/?t=147145076000010&r=1&w=2, php/php-src#2068 and other reported bugs. We'll see how 7.1.0-RC1 goes. |
... in the return type template.
Fixes #736