Skip to content

Commit 872c383

Browse files
committed
Merge pull request #6 from goaop/fix/empty-reflection-type
Fix incorrect behavior for empty parameter types, resolves #5
2 parents 8c98194 + dc7f053 commit 872c383

File tree

2 files changed

+7
-0
lines changed

2 files changed

+7
-0
lines changed

src/ReflectionParameter.php

+2
Original file line numberDiff line numberDiff line change
@@ -264,6 +264,8 @@ public function getType()
264264
$parameterType = $parameterType->toString();
265265
} elseif (is_string($parameterType)) {
266266
$isBuiltin = true;
267+
} else {
268+
return null;
267269
}
268270

269271
return new ReflectionType($parameterType, $allowsNull, $isBuiltin);

tests/ReflectionParameterTest.php

+5
Original file line numberDiff line numberDiff line change
@@ -202,6 +202,11 @@ public function testGetTypeMethod()
202202
$this->assertSame($originalReturnType->allowsNull(), $parsedReturnType->allowsNull());
203203
$this->assertSame($originalReturnType->isBuiltin(), $parsedReturnType->isBuiltin());
204204
$this->assertSame($originalReturnType->__toString(), $parsedReturnType->__toString());
205+
} else {
206+
$this->assertSame(
207+
$originalRefParameter->getType(),
208+
$refParameter->getType()
209+
);
205210
}
206211
}
207212
}

0 commit comments

Comments
 (0)