[VarDumper] Fix PHP7 type-hints compat

nicolas-grekas · nicolas-grekas · commit edd563337485 · 2015-11-06T15:49:25.000+01:00
diff --git a/src/Symfony/Component/VarDumper/Caster/ReflectionCaster.php b/src/Symfony/Component/VarDumper/Caster/ReflectionCaster.php
@@ -167,7 +167,11 @@ public static function castParameter(\ReflectionParameter $c, array $a, Stub $st
         ));
 
         try {
-            if ($c->isArray()) {
+            if (method_exists($c, 'hasType')) {
+                if ($c->hasType()) {
+                    $a[$prefix.'typeHint'] = $c->getType()->__toString();
+                }
+            } elseif ($c->isArray()) {
                 $a[$prefix.'typeHint'] = 'array';
             } elseif (method_exists($c, 'isCallable') && $c->isCallable()) {
                 $a[$prefix.'typeHint'] = 'callable';
diff --git a/src/Symfony/Component/VarDumper/Tests/Caster/ReflectionCasterTest.php b/src/Symfony/Component/VarDumper/Tests/Caster/ReflectionCasterTest.php
@@ -78,20 +78,41 @@ public function testReflectionParameter()
         );
     }
 
+    /**
+     * @requires PHP 7.0
+     */
+    public function testReflectionParameterScalar()
+    {
+        $f = eval('return function (int $a) {};');
+        $var = new \ReflectionParameter($f, 0);
+
+        $this->assertDumpMatchesFormat(
+            <<<'EOTXT'
+ReflectionParameter {
+  +name: "a"
+  position: 0
+  typeHint: "int"
+}
+EOTXT
+            , $var
+        );
+    }
+
     /**
      * @requires PHP 7.0
      */
     public function testReturnType()
     {
         $f = eval('return function ():int {};');
+        $line = __LINE__ - 1;
 
         $this->assertDumpMatchesFormat(
-            <<<'EOTXT'
+            <<<EOTXT
 Closure {
   returnType: "int"
   class: "Symfony\Component\VarDumper\Tests\Caster\ReflectionCasterTest"
   this: Symfony\Component\VarDumper\Tests\Caster\ReflectionCasterTest { …}
-  file: "%sReflectionCasterTest.php(86) : eval()'d code"
+  file: "%sReflectionCasterTest.php($line) : eval()'d code"
   line: "1 to 1"
 }
 EOTXT
