Require PHP 8.3

[morozov]

PHP 8.1 and 8.2 are no longer supported by the community. PHPUnit 12 also requires PHP 8.3 or newer (see #6888).

The code/phpDoc changes are needed to address the following coding standard violations:

FILE: src/Driver/PgSQL/Exception/UnexpectedValue.php
------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------
 23 | ERROR | [x] Method \Doctrine\DBAL\Driver\PgSQL\Exception\UnexpectedValue::getSQLState() has useless
    |       |     @return annotation. (SlevomatCodingStandard.TypeHints.ReturnTypeHint.UselessAnnotation)
------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------


FILE: src/Configuration.php
------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
------------------------------------------------------------------------------------------------------------
 137 | ERROR | [x] Method \Doctrine\DBAL\Configuration::getDisableTypeComments() has useless @return
     |       |     annotation. (SlevomatCodingStandard.TypeHints.ReturnTypeHint.UselessAnnotation)
 144 | ERROR | [x] Method \Doctrine\DBAL\Configuration::setDisableTypeComments() has useless @param
     |       |     annotation for parameter $disableTypeComments.
     |       |     (SlevomatCodingStandard.TypeHints.ParameterTypeHint.UselessAnnotation)
------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------

They are reported because the enableStandaloneNullTrueFalseTypeHints parameter defaults to true on PHP 8.2+. I don't think we want to turn it off just to retain the annotations, and we cannot modify the Configuration method signatures because it's not final. Modifying the return type in UnexpectedValue is fine because it is final.

[greg0ire]

This can help making a decision.

Apparently 25% of the 4.x userbase still uses 8.2. Only ~3% uses 8.1, so I'd say OK for dropping at least 8.1, but for 8.2, it still feels a bit early.

[morozov]

I think we agreed long ago on a policy that we support the supported PHP versions. The desire to upgrade DBAL will motivate users to upgrade their PHP. They are already using an unsupported PHP versions, and it's apparently not a problem.

[greg0ire]

Well OK then