[PropertyInfo] Fixed promoted property type detection.

LastDragon-ru · LastDragon-ru · commit f7fd692a7571 · 2023-11-22T13:14:42.000+04:00
diff --git a/src/Symfony/Component/PropertyInfo/Extractor/PhpStanExtractor.php b/src/Symfony/Component/PropertyInfo/Extractor/PhpStanExtractor.php
@@ -227,22 +227,27 @@ private function getDocBlockFromProperty(string $class, string $property): ?arra
             return null;
         }
 
+        // Type can be inside promoted property comment as `@var`
+        $rawDocNode = $reflectionProperty->getDocComment();
+        $phpDocNode = $rawDocNode ? $this->getPhpDocNode($rawDocNode) : null;
         $source = self::PROPERTY;
 
-        if ($reflectionProperty->isPromoted()) {
+        if ($phpDocNode && [] === $phpDocNode->getTagsByName('@var')) {
+            $phpDocNode = null;
+        }
+
+        // or in the constructor comment as `@param`.
+        if (!$phpDocNode && $reflectionProperty->isPromoted()) {
             $constructor = new \ReflectionMethod($class, '__construct');
             $rawDocNode = $constructor->getDocComment();
+            $phpDocNode = $rawDocNode ? $this->getPhpDocNode($rawDocNode) : null;
             $source = self::MUTATOR;
-        } else {
-            $rawDocNode = $reflectionProperty->getDocComment();
         }
 
-        if (!$rawDocNode) {
+        if (!$phpDocNode) {
             return null;
         }
 
-        $phpDocNode = $this->getPhpDocNode($rawDocNode);
-
         return [$phpDocNode, $source, $reflectionProperty->class];
     }
 
diff --git a/src/Symfony/Component/PropertyInfo/Tests/Extractor/PhpStanExtractorTest.php b/src/Symfony/Component/PropertyInfo/Tests/Extractor/PhpStanExtractorTest.php
@@ -465,6 +465,8 @@ public function testExtractPhp80Type(string $class, $property, array $type = nul
     public function php80TypesProvider()
     {
         return [
+            [Php80Dummy::class, 'promotedWithDocCommentAndType', [new Type(Type::BUILTIN_TYPE_INT)]],
+            [Php80Dummy::class, 'promotedWithDocComment', [new Type(Type::BUILTIN_TYPE_STRING)]],
             [Php80Dummy::class, 'promotedAndMutated', [new Type(Type::BUILTIN_TYPE_STRING)]],
             [Php80Dummy::class, 'promoted', null],
             [Php80Dummy::class, 'collection', [new Type(Type::BUILTIN_TYPE_ARRAY, collection: true, collectionValueType: new Type(Type::BUILTIN_TYPE_STRING))]],
diff --git a/src/Symfony/Component/PropertyInfo/Tests/Fixtures/Php80Dummy.php b/src/Symfony/Component/PropertyInfo/Tests/Fixtures/Php80Dummy.php
@@ -17,9 +17,23 @@ class Php80Dummy
 
     /**
      * @param string $promotedAndMutated
+     * @param string $promotedWithDocComment
+     * @param string $promotedWithDocCommentAndType
      * @param array<string> $collection
      */
-    public function __construct(private mixed $promoted, private mixed $promotedAndMutated, private array $collection)
+    public function __construct(
+        private mixed $promoted,
+        private mixed $promotedAndMutated,
+        /**
+         * Comment without @var.
+         */
+        private mixed $promotedWithDocComment,
+        /**
+         * @var int
+         */
+        private mixed $promotedWithDocCommentAndType,
+        private array $collection,
+    )
     {
     }
 

Original file line number	Diff line number	Diff line change
`@@ -465,6 +465,8 @@ public function testExtractPhp80Type(string $class, $property, array $type = nul`
`465`	`465`	`public function php80TypesProvider()`
`466`	`466`	`{`
`467`	`467`	`return [`
	`468`	`+ [Php80Dummy::class, 'promotedWithDocCommentAndType', [new Type(Type::BUILTIN_TYPE_INT)]],`
	`469`	`+ [Php80Dummy::class, 'promotedWithDocComment', [new Type(Type::BUILTIN_TYPE_STRING)]],`
`468`	`470`	`[Php80Dummy::class, 'promotedAndMutated', [new Type(Type::BUILTIN_TYPE_STRING)]],`
`469`	`471`	`[Php80Dummy::class, 'promoted', null],`
`470`	`472`	`[Php80Dummy::class, 'collection', [new Type(Type::BUILTIN_TYPE_ARRAY, collection: true, collectionValueType: new Type(Type::BUILTIN_TYPE_STRING))]],`