[VarExporter] fix proxy helper when a method returns null

nikophil · nikophil · commit 673bb6b3ff59 · 2024-05-06T17:18:16.000+02:00
diff --git a/src/Symfony/Component/VarExporter/ProxyHelper.php b/src/Symfony/Component/VarExporter/ProxyHelper.php
@@ -311,7 +311,7 @@ public static function exportType(\ReflectionFunctionAbstract|\ReflectionPropert
             return '';
         }
         if (null === $glue) {
-            return (!$noBuiltin && $type->allowsNull() && 'mixed' !== $name ? '?' : '').$types[0];
+            return (!$noBuiltin && $type->allowsNull() && 'null' !== $type->getName() && 'mixed' !== $name ? '?' : '').$types[0];
         }
         sort($types);
 
@@ -357,11 +357,11 @@ private static function exportDefault(\ReflectionParameter $param): string
                 'parent' => ($parent = $class->getParentClass()) ? '\\'.$parent->name : 'parent',
                 default => '\\'.$m[2],
             }
-            : fn ($m) => $m[1].match ($m[2]) {
-                'new', 'false', 'true', 'null', 'self', 'parent' => $m[2],
-                'NULL' => 'null',
-                default => '\\'.$m[2],
-            };
+        : fn ($m) => $m[1].match ($m[2]) {
+            'new', 'false', 'true', 'null', 'self', 'parent' => $m[2],
+            'NULL' => 'null',
+            default => '\\'.$m[2],
+        };
 
         return implode('', array_map(fn ($part) => match ($part[0]) {
             '"' => $part, // for internal classes only
diff --git a/src/Symfony/Component/VarExporter/Tests/ProxyHelperTest.php b/src/Symfony/Component/VarExporter/Tests/ProxyHelperTest.php
@@ -188,6 +188,28 @@ public function testCannotGenerateGhostForStringMagicGet()
         $this->expectException(LogicException::class);
         ProxyHelper::generateLazyGhost(new \ReflectionClass(StringMagicGetClass::class));
     }
+
+    /**
+     * @requires PHP 8.2
+     */
+    public function testNullStandaloneReturnType()
+    {
+        $classCode = <<<PHP
+        class TestNullStandaloneReturnType
+        {
+            public function foo(): null
+            {
+                return null;
+            }
+        }
+        PHP;
+        eval($classCode);
+
+        self::assertStringContainsString(
+            'public function foo(): null',
+            ProxyHelper::generateLazyProxy(new \ReflectionClass(\TestNullStandaloneReturnType::class))
+        );
+    }
 }
 
 abstract class TestForProxyHelper
