Merge pull request #3331 from michaelcullum/fetch-all

Fetch all should use the driver statement's fetchAll method
doctrine · Oct 25, 2018 · bef35e9 · bef35e9
2 parents 1b1adc2 + e35d5e1
commit bef35e9
Showing 1 changed file with 1 addition and 6 deletions.
diff --git a/lib/Doctrine/DBAL/Cache/ResultCacheStatement.php b/lib/Doctrine/DBAL/Cache/ResultCacheStatement.php
@@ -166,12 +166,7 @@ public function fetch($fetchMode = null, $cursorOrientation = PDO::FETCH_ORI_NEX
      */
     public function fetchAll($fetchMode = null, $fetchArgument = null, $ctorArgs = null)
     {
-        $rows = [];
-        while ($row = $this->fetch($fetchMode)) {
-            $rows[] = $row;
-        }
-
-        return $rows;
+        return $this->statement->fetchAll($fetchMode, $fetchArgument, $ctorArgs);
     }
 
     /**