Merge branch '7.2' into 7.3

* 7.2:
  [VarExporter] Fix support for hooks and asymmetric visibility
  [TypeInfo] Fix promoted property with `@var` tag
  fix(process): use a pipe for stderr in pty mode to avoid mixed output between stdout and stderr
  [Cache] fix data collector

Author: nicolas-grekas

Tests/Fixtures/DummyWithPhpDoc.php

@@ -29,9 +29,18 @@ final class DummyWithPhpDoc
 
     /**
      * @param bool $promoted
+     * @param bool $promotedVarAndParam
      */
     public function __construct(
         public mixed $promoted,
+        /**
+         * @var string
+         */
+        public mixed $promotedVar,
+        /**
+         * @var string
+         */
+        public mixed $promotedVarAndParam,
     ) {
     }
 

Tests/TypeResolver/PhpDocAwareReflectionTypeResolverTest.php

@@ -41,6 +41,8 @@ public static function readPhpDocDataProvider(): iterable
 
         yield [Type::array(Type::object(Dummy::class)), $reflection->getProperty('arrayOfDummies')];
         yield [Type::bool(), $reflection->getProperty('promoted')];
+        yield [Type::string(), $reflection->getProperty('promotedVar')];
+        yield [Type::string(), $reflection->getProperty('promotedVarAndParam')];
         yield [Type::object(Dummy::class), $reflection->getMethod('getNextDummy')];
         yield [Type::object(Dummy::class), $reflection->getMethod('getNextDummy')->getParameters()[0]];
         yield [Type::int(), $reflection->getProperty('aliasedInt')];

TypeResolver/PhpDocAwareReflectionTypeResolver.php

@@ -64,36 +64,38 @@ public function resolve(mixed $subject, ?TypeContext $typeContext = null): Type
             throw new UnsupportedException(\sprintf('Expected subject to be a "ReflectionProperty", a "ReflectionParameter" or a "ReflectionFunctionAbstract", "%s" given.', get_debug_type($subject)), $subject);
         }
 
-        $docComment = match (true) {
-            $subject instanceof \ReflectionProperty => $subject->isPromoted() ? $subject->getDeclaringClass()?->getConstructor()?->getDocComment() : $subject->getDocComment(),
-            $subject instanceof \ReflectionParameter => $subject->getDeclaringFunction()->getDocComment(),
-            $subject instanceof \ReflectionFunctionAbstract => $subject->getDocComment(),
+        $typeContext ??= $this->typeContextFactory->createFromReflection($subject);
+
+        $docComments = match (true) {
+            $subject instanceof \ReflectionProperty => $subject->isPromoted()
+                ? ['@var' => $subject->getDocComment(), '@param' => $subject->getDeclaringClass()?->getConstructor()?->getDocComment()]
+                : ['@var' => $subject->getDocComment()],
+            $subject instanceof \ReflectionParameter => ['@param' => $subject->getDeclaringFunction()->getDocComment()],
+            $subject instanceof \ReflectionFunctionAbstract => ['@return' => $subject->getDocComment()],
         };
 
-        if (!$docComment) {
-            return $this->reflectionTypeResolver->resolve($subject);
-        }
+        foreach ($docComments as $tagName => $docComment) {
+            if (!$docComment) {
+                continue;
+            }
 
-        $typeContext ??= $this->typeContextFactory->createFromReflection($subject);
+            $tokens = new TokenIterator($this->lexer->tokenize($docComment));
+            $docNode = $this->phpDocParser->parse($tokens);
 
-        $tagName = match (true) {
-            $subject instanceof \ReflectionProperty => $subject->isPromoted() ? '@param' : '@var',
-            $subject instanceof \ReflectionParameter => '@param',
-            $subject instanceof \ReflectionFunctionAbstract => '@return',
-        };
+            foreach ($docNode->getTagsByName($tagName) as $tag) {
+                $tagValue = $tag->value;
 
-        $tokens = new TokenIterator($this->lexer->tokenize($docComment));
-        $docNode = $this->phpDocParser->parse($tokens);
+                if ('@var' === $tagName && $tagValue instanceof VarTagValueNode) {
+                    return $this->stringTypeResolver->resolve((string) $tagValue, $typeContext);
+                }
 
-        foreach ($docNode->getTagsByName($tagName) as $tag) {
-            $tagValue = $tag->value;
+                if ('@param' === $tagName && $tagValue instanceof ParamTagValueNode && '$'.$subject->getName() === $tagValue->parameterName) {
+                    return $this->stringTypeResolver->resolve((string) $tagValue, $typeContext);
+                }
 
-            if (
-                $tagValue instanceof VarTagValueNode
-                || $tagValue instanceof ParamTagValueNode && $tagName && '$'.$subject->getName() === $tagValue->parameterName
-                || $tagValue instanceof ReturnTagValueNode
-            ) {
-                return $this->stringTypeResolver->resolve((string) $tagValue, $typeContext);
+                if ('@return' === $tagName && $tagValue instanceof ReturnTagValueNode) {
+                    return $this->stringTypeResolver->resolve((string) $tagValue, $typeContext);
+                }
             }
         }