Merge pull request #131 from jaapio/feature/param-type

Add parameter type resolving

Author: jaapio

src/phpDocumentor/Reflection/Php/Factory/Argument.php

@@ -18,8 +18,11 @@
 use phpDocumentor\Reflection\Php\ProjectFactoryStrategy;
 use phpDocumentor\Reflection\Php\StrategyContainer;
 use phpDocumentor\Reflection\PrettyPrinter;
+use phpDocumentor\Reflection\Type;
+use phpDocumentor\Reflection\TypeResolver;
 use phpDocumentor\Reflection\Types\Context;
 use PhpParser\Node\Expr\Variable;
+use PhpParser\Node\NullableType;
 use PhpParser\Node\Param;
 use Webmozart\Assert\Assert;
 
@@ -68,6 +71,24 @@ protected function doCreate($object, StrategyContainer $strategies, ?Context $co
             $default = $this->valueConverter->prettyPrintExpr($object->default);
         }
 
-        return new ArgumentDescriptor((string) $object->var->name, $default, $object->byRef, $object->variadic);
+        $argumentDescriptor = new ArgumentDescriptor((string) $object->var->name, $default, $object->byRef, $object->variadic);
+
+        if (!empty($object->type)) {
+            $argumentDescriptor->addType($this->createType($object));
+        }
+
+        return $argumentDescriptor;
+    }
+
+    private function createType(Param $arg, ?Context $context = null): Type
+    {
+        $typeResolver = new TypeResolver();
+        if ($arg->type instanceof NullableType) {
+            $typeString = '?' . $arg->type->type;
+        } else {
+            $typeString = (string) $arg->type;
+        }
+
+        return $typeResolver->resolve($typeString, $context);
     }
 }

tests/component/ProjectCreationTest.php

@@ -105,6 +105,7 @@ public function testWithNamespacedClass()
         );
 
         $this->assertEquals('style', $methods['\\Luigi\\Pizza::__construct()']->getArguments()[0]->getName());
+        $this->assertEquals([new Object_(new Fqsen('\\Luigi\\Pizza\Style'))], $methods['\\Luigi\\Pizza::__construct()']->getArguments()[0]->getTypes());
     }
 
     public function testDocblockOfMethodIsProcessed()