Merge pull request #3331 from michaelcullum/fetch-all

Fetch all should use the driver statement's fetchAll method

Merge pull request #3331 from michaelcullum/fetch-all
Fetch all should use the driver statement's fetchAll method
bef35e90 · Marco Pivetta · GitHub · 1b1adc2a · e35d5e12 · bef35e90
Unverified Commit bef35e90 authored Oct 25, 2018 by Marco Pivetta Committed by GitHub Oct 25, 2018
Show whitespace changes
Inline Side-by-side

Showing with 1 addition and 6 deletions

ResultCacheStatement.php lib/Doctrine/DBAL/Cache/ResultCacheStatement.php +1 -6

No files found.
--- a/lib/Doctrine/DBAL/Cache/ResultCacheStatement.php
+++ b/lib/Doctrine/DBAL/Cache/ResultCacheStatement.php
@@ -166,12 +166,7 @@ class ResultCacheStatement implements IteratorAggregate, ResultStatement
     */
    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);
    }

    /**