Merge #77 - Fix typehint resolution - stop prefixing unions with namespace

Author: williamdes

src/Parser/NodeVisitor.php

@@ -165,6 +165,30 @@ protected function addFunction(FunctionNode $node, ?string $namespace = null)
         }
     }
 
+    /**
+     * @param \PhpParser\Node\ComplexType|\PhpParser\Node\Identifier|\PhpParser\Node\Name|NullableType|UnionType|IntersectionType|null $type Type declaration
+     */
+    protected function typeToArray($type): array
+    {
+        $typeArray = [];
+        if ($type !== null && ! ($type instanceof NullableType || $type instanceof UnionType || $type instanceof IntersectionType)) {
+            $typeAsStr = $type->__toString();
+            if ($type instanceof FullyQualified && 0 !== strpos($typeAsStr, '\\')) {
+                $typeAsStr = '\\' . $typeAsStr;
+            }
+            $typeArray[] = [$typeAsStr, false];
+        } elseif ($type instanceof NullableType) {
+            $typeArray = array_merge($typeArray, $this->typeToArray($type->type));
+            $typeArray[] = ['null', false];
+        } elseif ($type instanceof UnionType || $type instanceof IntersectionType) {
+            foreach ($type->types as $subType) {
+                $typeArray = array_merge($typeArray, $this->typeToArray($subType));
+            }
+        }
+
+        return $typeArray;
+    }
+
     /**
      * @param \PhpParser\Node\ComplexType|\PhpParser\Node\Identifier|\PhpParser\Node\Name|NullableType|UnionType|IntersectionType|null $type Type declaration
      */
@@ -439,20 +463,14 @@ protected function manageHint($type, Reflection $object): void
 
             $typeArr = [];
             foreach ($type->types as $type) {
-                $typeStr   = $this->typeToString($type);
-                $typeArr[] = [$typeStr, false];
+                $typeArr = array_merge($typeArr, $this->typeToArray($type));
             }
 
             $object->setHint($this->resolveHint($typeArr));
         } else {
-            $typeStr = $this->typeToString($type);
-
-            if (null !== $typeStr) {
-                $typeArr = [[$typeStr, false]];
+            $typeArr = $this->typeToArray($type);
 
-                if ($type instanceof NullableType) {
-                    $typeArr[] = ['null', false];
-                }
+            if (!empty($typeArr)) {
                 $object->setHint($this->resolveHint($typeArr));
             }
         }