Merge pull request #6 from goaop/fix/empty-reflection-type

Fix incorrect behavior for empty parameter types, resolves #5
goaop · Feb 20, 2016 · 872c383 · 872c383
2 parents 8c98194 + dc7f053
commit 872c383
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/src/ReflectionParameter.php b/src/ReflectionParameter.php
@@ -264,6 +264,8 @@ public function getType()
             $parameterType = $parameterType->toString();
         } elseif (is_string($parameterType)) {
             $isBuiltin = true;
+        } else {
+            return null;
         }
 
         return new ReflectionType($parameterType, $allowsNull, $isBuiltin);

diff --git a/tests/ReflectionParameterTest.php b/tests/ReflectionParameterTest.php
@@ -202,6 +202,11 @@ public function testGetTypeMethod()
                         $this->assertSame($originalReturnType->allowsNull(), $parsedReturnType->allowsNull());
                         $this->assertSame($originalReturnType->isBuiltin(), $parsedReturnType->isBuiltin());
                         $this->assertSame($originalReturnType->__toString(), $parsedReturnType->__toString());
+                    } else {
+                        $this->assertSame(
+                            $originalRefParameter->getType(),
+                            $refParameter->getType()
+                        );
                     }
                 }
             }