[VarDumper] Fix return type dumping

nicolas-grekas · nicolas-grekas · commit c5b67519d244 · 2015-10-19T19:15:56.000+02:00
diff --git a/src/Symfony/Component/VarDumper/Caster/ReflectionCaster.php b/src/Symfony/Component/VarDumper/Caster/ReflectionCaster.php
@@ -114,6 +114,9 @@ public static function castFunctionAbstract(\ReflectionFunctionAbstract $c, arra
             'this' => 'getClosureThis',
         ));
 
+        if (isset($a[$prefix.'returnType'])) {
+            $a[$prefix.'returnType'] = (string) $a[$prefix.'returnType'];
+        }
         if (isset($a[$prefix.'this'])) {
             $a[$prefix.'this'] = new CutStub($a[$prefix.'this']);
         }
diff --git a/src/Symfony/Component/VarDumper/Tests/Caster/ReflectionCasterTest.php b/src/Symfony/Component/VarDumper/Tests/Caster/ReflectionCasterTest.php
@@ -60,4 +60,25 @@ public function testReflectionCaster()
             , $var
         );
     }
+
+    /**
+     * @requires PHP 7.0
+     */
+    public function testReturnType()
+    {
+        $f = eval('return function ():int {};');
+
+        $this->assertDumpMatchesFormat(
+            <<<'EOTXT'
+Closure {
+  returnType: "int"
+  class: "Symfony\Component\VarDumper\Tests\Caster\ReflectionCasterTest"
+  this: Symfony\Component\VarDumper\Tests\Caster\ReflectionCasterTest { …}
+  file: "%sReflectionCasterTest.php(69) : eval()'d code"
+  line: "1 to 1"
+}
+EOTXT
+            , $f
+        );
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -114,6 +114,9 @@ public static function castFunctionAbstract(\ReflectionFunctionAbstract $c, arra`
`114`	`114`	`'this' => 'getClosureThis',`
`115`	`115`	`));`
`116`	`116`
	`117`	`+ if (isset($a[$prefix.'returnType'])) {`
	`118`	`+ $a[$prefix.'returnType'] = (string) $a[$prefix.'returnType'];`
	`119`	`+ }`
`117`	`120`	`if (isset($a[$prefix.'this'])) {`
`118`	`121`	`$a[$prefix.'this'] = new CutStub($a[$prefix.'this']);`
`119`	`122`	`}`