Updated Reflection

[dg]

• Reflection uses ReflectionNamedType instead of ReflectionType since PHP 7.1
• added reflection for properties

[ondrejmirtes]

Hi, thanks for the PR! I disagree with the changes about ReflectionNamedType. I actually went the other way in 0.12.0: b447242

ReflectionNamedType is the only ReflectionType implementation in PHP 7.x, but that will stop being true in PHP 8 where there will also be ReflectionUnionType without the getName() method. So code that's relying on ReflectionNamedType will be broken on PHP 8. I recommend doing something like this instead in your code:

$returnType = $reflectionFunction->getReturnType(); if (!$returnType instanceof ReflectionNamedType) { throw new \Exception('PHP 8 is not yet supported.'); } 

Or maybe even code the support for union types today! :)

Otherwise the PR is fine.

[dg]

I see. So I'll turn it around and I'll move getName() and isBuiltin() from ReflectionType to ReflectionNamedType.

[ondrejmirtes]

Perfect, thank you!