Switch condition order to support PHP 8

[jdreesen]

In PHP 8, ReflectionType::isBuiltin() has been moved to ReflectionNamedType::isBuiltin() and thus should be called after asserting that it's a named type.

[mnapoli]

Any idea why it's currently working fine in PHP 8? I'm confused how it didn't break before

[jdreesen]

As ReflectionType is now abstract, $parameter->getType() will either return null (if there's no type), ReflectionNamedType (if it has one type) or ReflectionUnionType (if it has multiple types).

So it works fine as long as there are only "named" (single) types, but breaks for union types.

See: https://3v4l.org/baloT

[mnapoli]

Thank you!