Proofread upgrade guide

Fix headline in the upgrade docs

When using PDO, an exception is supposed to be thrown since we are using
the error mode that behaves that way. It only seems to be the case since
PHP 8 though.

Fix php8 mysql mariadb

Update PHP_CodeSniffer to 3.5.8

Store query params in driver exceptions

Remove warning comment from PDOOracle driver

Add missing instanceof check

…olon-prefix

Deprecate colon prefix for prepared statement parameters

…n-prefix

Remove support for colon prefix in statement parameters

Deprecate inappropriate usage of prepared statement parameters

This effectively prevented phpstan from inferring type of `T` template.

> Unable to resolve the template type T in call to method static method Doctrine\DBAL\DriverManager::getConnection()

…ache

ResultCacheStatement::fetchAllAssociative does not store results in cache

Testing Guidelines

Update .gitattributes

Add missing word

QueryBuilder must use 0-based positional parameter keys

Mark SQLParserUtils internal

This fixes the issue detailed in doctrine#4391, with SQL Server and SQL Anywhere setting WITH (NOLOCK) for LockMode::NONE, which effectively means using a READ UNCOMMITTED isolation level at table level, which is not the contract of LockMode::NONE.

LockMode::NONE should not set WITH (NOLOCK)

Throw exception on invalid LockMode

Port the SQL parser from PDO

Remove redundant phpstan param from DriverManager::getConnection()