Portability\Statement incorrectly handles fetchAll(PDO::FETCH_COLUMN)

[morozov]

The Portability\Statement::fixRow() implementation doesn't take into account that mixed $row is not always iterable (particularly in the case if $fetchMode = PDO::FETCH_COLUMN).

use Doctrine\DBAL\Portability\Connection;
use Doctrine\DBAL\DriverManager;

$conn = DriverManager::getConnection(array(
    'driver' => 'ibm_db2',
    'wrapperClass' => Connection::class,
    'portability' => Connection::PORTABILITY_FIX_CASE,
    'fetch_case' => PDO::CASE_LOWER,
));

$conn->executeQuery('SELECT id FROM users')
    ->fetchAll(\PDO::FETCH_COLUMN);

Produces:

Warning: Invalid argument supplied for foreach() in vendor/doctrine/dbal/lib/Doctrine/DBAL/Portability/Statement.php on line 203

Besides that, the Portability\Statement::connect() implementation forcedly replaces the portability value from whatever provided in the connection parameters with PORTABILITY_DB2 which is inconsistent with the rest of the supported platforms.

[github-actions]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.