New fetch() and fetchAll() method signatures

[codebarista]

What is this feature about (expected vs actual behaviour)?

MINOR BC BREAK: fetch() and fetchAll() method signatures in Doctrine\DBAL\Driver\ResultStatement

How can I reproduce it?

Composer update doctrine to version 2.6

Does it take minutes, hours or days to fix?

Minutes.

Any additional information?

Changes in Statement.php line 214 & 224

/**
 * {@inheritdoc}
 *
 * @SuppressWarnings("PHPMD.StaticAccess")
 */
public function fetch($fetchMode = NULL, $cursorOrientation = \PDO::FETCH_ORI_NEXT, $cursorOffset = 0) {
    $fetchMode = empty($fetchMode) ? $this->fetchMode : $fetchMode;
    Assertions::assertSupportedFetchMode($fetchMode);

    return count($this->result) === 0 ? false : array_pop($this->result);
}

/**
 * {@inheritdoc}
 */
public function fetchAll($fetchMode = NULL, $fetchArgument = NULL, $ctorArgs = NULL) {
    $result    = [];
    $fetchMode = empty($fetchMode) ? $this->fetchMode : $fetchMode;

    while (($row = $this->fetch($fetchMode))) array_push($result, $row);

    return $result;
}

https://github.com/doctrine/dbal/blob/master/UPGRADE.md