From 812bf5c45d1eaca4dd8904c754460e1676a7dea9 Mon Sep 17 00:00:00 2001
From: Nicolas Grekas <nicolas.grekas@gmail.com>
Date: Tue, 19 Apr 2016 19:21:45 +0200
Subject: [PATCH] [VarDumper] Fix dumping type hints for non-existing parent
 classes

---
 .../Component/VarDumper/Caster/ReflectionCaster.php   | 11 +++++------
 .../VarDumper/Tests/Caster/ReflectionCasterTest.php   |  7 ++++---
 .../VarDumper/Tests/Fixtures/NotLoadableClass.php     |  7 +++++++
 3 files changed, 16 insertions(+), 9 deletions(-)
 create mode 100644 src/Symfony/Component/VarDumper/Tests/Fixtures/NotLoadableClass.php

diff --git a/src/Symfony/Component/VarDumper/Caster/ReflectionCaster.php b/src/Symfony/Component/VarDumper/Caster/ReflectionCaster.php
index cacd2113d7..94c1bd3a6b 100644
--- a/src/Symfony/Component/VarDumper/Caster/ReflectionCaster.php
+++ b/src/Symfony/Component/VarDumper/Caster/ReflectionCaster.php
@@ -171,12 +171,11 @@ class ReflectionCaster
                 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';
-            } elseif ($v = $c->getClass()) {
-                $a[$prefix.'typeHint'] = $v->name;
+            } else {
+                $v = explode(' ', $c->__toString(), 6);
+                if (isset($v[5]) && 0 === strspn($v[4], '.&$')) {
+                    $a[$prefix.'typeHint'] = $v[4];
+                }
             }
         } catch (\ReflectionException $e) {
             if (preg_match('/^Class ([^ ]++) does not exist$/', $e->getMessage(), $m)) {
diff --git a/src/Symfony/Component/VarDumper/Tests/Caster/ReflectionCasterTest.php b/src/Symfony/Component/VarDumper/Tests/Caster/ReflectionCasterTest.php
index 5c306a6710..e98642414f 100644
--- a/src/Symfony/Component/VarDumper/Tests/Caster/ReflectionCasterTest.php
+++ b/src/Symfony/Component/VarDumper/Tests/Caster/ReflectionCasterTest.php
@@ -12,6 +12,7 @@
 namespace Symfony\Component\VarDumper\Tests\Caster;
 
 use Symfony\Component\VarDumper\Test\VarDumperTestCase;
+use Symfony\Component\VarDumper\Tests\Fixtures\NotLoadableClass;
 
 /**
  * @author Nicolas Grekas <p@tchwork.com>
@@ -47,7 +48,7 @@ ReflectionClass {
     "export" => ReflectionMethod {
       +name: "export"
       +class: "ReflectionClass"
-      parameters: array:2 [
+%A    parameters: array:2 [
         "$%s" => ReflectionParameter {
 %A         position: 0
 %A      }
@@ -70,7 +71,7 @@ EOTXT
 ReflectionParameter {
   +name: "arg1"
   position: 0
-  typeHint: "Symfony\Component\VarDumper\Tests\Caster\NotExistingClass"
+  typeHint: "Symfony\Component\VarDumper\Tests\Fixtures\NotLoadableClass"
   default: null
 }
 EOTXT
@@ -121,6 +122,6 @@ EOTXT
     }
 }
 
-function reflectionParameterFixture(NotExistingClass $arg1 = null, $arg2)
+function reflectionParameterFixture(NotLoadableClass $arg1 = null, $arg2)
 {
 }
diff --git a/src/Symfony/Component/VarDumper/Tests/Fixtures/NotLoadableClass.php b/src/Symfony/Component/VarDumper/Tests/Fixtures/NotLoadableClass.php
new file mode 100644
index 0000000000..d8e0cb359f
--- /dev/null
+++ b/src/Symfony/Component/VarDumper/Tests/Fixtures/NotLoadableClass.php
@@ -0,0 +1,7 @@
+<?php
+
+namespace Symfony\Component\VarDumper\Tests\Fixtures;
+
+class NotLoadableClass extends NotLoadableClass
+{
+}