[PropertyInfo] Get short description from promoted properties in PhpD…

…ocExtractor

src/Symfony/Component/PropertyInfo/CHANGELOG.md

@@ -6,6 +6,7 @@ CHANGELOG
 
  * Add support for `non-positive-int`, `non-negative-int` and `non-zero-int` PHPStan types to `PhpStanExtractor`
  * Add `PropertyDescriptionExtractorInterface` to `PhpStanExtractor`
+ * Add support for getting description from constructor with promoted arguments to `PhpDocExtractor`
 
 7.1
 ---

src/Symfony/Component/PropertyInfo/Extractor/PhpDocExtractor.php

@@ -102,6 +102,18 @@ public function getShortDescription(string $class, string $property, array $cont
             }
         }
 
+        foreach ($docBlock->getTagsByName('param') as $param) {
+            if (!$param instanceof DocBlock\Tags\Param) {
+                continue;
+            }
+
+            $paramDescription = $param->getDescription()?->render();
+
+            if (null !== $paramDescription && '' !== $paramDescription) {
+                return $paramDescription;
+            }
+        }
+
         return null;
     }
 

src/Symfony/Component/PropertyInfo/Tests/Extractor/PhpDocExtractorTest.php

@@ -19,6 +19,7 @@
 use Symfony\Component\PropertyInfo\Tests\Fixtures\Dummy;
 use Symfony\Component\PropertyInfo\Tests\Fixtures\DummyCollection;
 use Symfony\Component\PropertyInfo\Tests\Fixtures\InvalidDummy;
+use Symfony\Component\PropertyInfo\Tests\Fixtures\Extractor\PromotedPropertiesWithDocBlock;
 use Symfony\Component\PropertyInfo\Tests\Fixtures\ParentDummy;
 use Symfony\Component\PropertyInfo\Tests\Fixtures\Php80Dummy;
 use Symfony\Component\PropertyInfo\Tests\Fixtures\PseudoTypeDummy;
@@ -889,6 +890,23 @@ public static function promotedPropertyProvider(): iterable
         yield ['promoted', null];
         yield ['promotedAndMutated', Type::string()];
     }
+
+    /**
+     * @dataProvider provideGetShortDescriptionOnPromotedPropertyTestCases
+     */
+    public function testGetShortDescriptionOnPromotedProperty(string $class, string $property, ?string $description)
+    {
+        $this->assertSame($description, $this->extractor->getShortDescription($class, $property));
+    }
+
+    public static function provideGetShortDescriptionOnPromotedPropertyTestCases(): array
+    {
+        return [
+            [PromotedPropertiesWithDocBlock::class, 'foo', 'Just a foo property'],
+            [PromotedPropertiesWithDocBlock::class, 'bar', 'A phpdoc without type'],
+            'Inlined phpdoc with @var is not compatible with the one above constructor' => [PromotedPropertiesWithDocBlock::class, 'baz', null],
+        ];
+    }
 }
 
 class EmptyDocBlock

src/Symfony/Component/PropertyInfo/Tests/Fixtures/Extractor/PromotedPropertiesWithDocBlock.php

@@ -0,0 +1,27 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\PropertyInfo\Tests\Fixtures\Extractor;
+
+class PromotedPropertiesWithDocBlock
+{
+    /**
+     * @param string $foo Just a foo property
+     * @param $bar A phpdoc without type
+     */
+    public function __construct(
+        public string $foo,
+        public int $bar,
+        /** @var string $baz This phpdoc isn't compatible with the one above this method, thus it won't appear */
+        public string $baz,
+    ) {
+    }
+}