Merge pull request #802 from stof/fix_explain

Fix explaining queries
doctrine · Apr 20, 2018 · fbb8a34 · fbb8a34
2 parents 703fad3 + 6de4d03
commit fbb8a34
Showing 1 changed file with 16 additions and 2 deletions.
diff --git a/Controller/ProfilerController.php b/Controller/ProfilerController.php
@@ -8,6 +8,7 @@
 use Symfony\Component\DependencyInjection\ContainerInterface;
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\HttpKernel\Profiler\Profiler;
+use Symfony\Component\VarDumper\Cloner\Data;
 
 /**
  * ProfilerController.
@@ -77,14 +78,27 @@ private function explainSQLServerPlatform(Connection $connection, $query)
         } else {
             $sql = 'SET SHOWPLAN_TEXT ON; GO; SET NOEXEC ON; ' . $query['sql'] . '; SET NOEXEC OFF; GO; SET SHOWPLAN_TEXT OFF;';
         }
-        $stmt = $connection->executeQuery($sql, $query['params'], $query['types']);
+
+        $params = $query['params'];
+
+        if ($params instanceof Data) {
+            $params = $params->getValue(true);
+        }
+
+        $stmt = $connection->executeQuery($sql, $params, $query['types']);
         $stmt->nextRowset();
         return $stmt->fetchAll(\PDO::FETCH_ASSOC);
     }
 
     private function explainOtherPlatform(Connection $connection, $query)
     {
-        return $connection->executeQuery('EXPLAIN ' . $query['sql'], $query['params'], $query['types'])
+        $params = $query['params'];
+
+        if ($params instanceof Data) {
+            $params = $params->getValue(true);
+        }
+
+        return $connection->executeQuery('EXPLAIN ' . $query['sql'], $params, $query['types'])
             ->fetchAll(\PDO::FETCH_ASSOC);
     }
 }