Merge branch '4.3.x' into 5.0.x

* 4.3.x:
  phpunit/phpunit (11.5.15 => 11.5.23) (#7006)
  Deprecate support for MariaDB 10.5 (#7000)
  Fix new and previously ignored PHPStan errors
  Ignore new PHPStan errors
  Bump dev tools (#7002)
  fix(PgSQL): Allow to pass IPv6 address in URI notation for postgres (#6344)
  Run tests on MySQL 9.3 instead of 9.1 (#7001)
  CI MariaDB: add 11.8 (#6991)

Author: derrabus

.github/workflows/continuous-integration.yml

@@ -142,6 +142,7 @@ jobs:
           - "10.6"  # LTS (Jul 2026) We have code specific to 10.6.0-10.10.0
           - "10.11" # LTS (Feb 2028) We have code specific to ^10.10
           - "11.4"  # LTS (May 2029)
+          - "11.8"  # LTS (Jun 2028)
         extension:
           - "mysqli"
           - "pdo_mysql"
@@ -170,7 +171,7 @@ jobs:
           - "8.4"
         mysql-version:
           - "8.0"
-          - "9.1"
+          - "9.3"
         extension:
           - "mysqli"
           - "pdo_mysql"

UPGRADE.md

@@ -348,6 +348,10 @@ all drivers and middleware.
 
 # Upgrade to 4.3
 
+## Deprecated support for MariaDB 10.5
+
+* Upgrade to MariaDB 10.6 or later.
+
 ## Deprecated `Column` methods
 
 The following `Column` methods have been deprecated:

composer.json

@@ -40,12 +40,12 @@
         "doctrine/coding-standard": "13.0.0",
         "fig/log-test": "^1",
         "jetbrains/phpstorm-stubs": "2023.2",
-        "phpstan/phpstan": "2.1.1",
+        "phpstan/phpstan": "2.1.17",
         "phpstan/phpstan-phpunit": "2.0.6",
         "phpstan/phpstan-strict-rules": "^2",
-        "phpunit/phpunit": "11.5.15",
+        "phpunit/phpunit": "11.5.23",
         "slevomat/coding-standard": "8.16.2",
-        "squizlabs/php_codesniffer": "3.12.0",
+        "squizlabs/php_codesniffer": "3.13.1",
         "symfony/cache": "^6.3.8|^7.0",
         "symfony/console": "^5.4|^6.3|^7.0"
     },

src/Driver/AbstractMySQLDriver.php

@@ -14,6 +14,7 @@
 use Doctrine\DBAL\Platforms\MariaDBPlatform;
 use Doctrine\DBAL\Platforms\MySQLPlatform;
 use Doctrine\DBAL\ServerVersionProvider;
+use Doctrine\Deprecations\Deprecation;
 
 use function preg_match;
 use function stripos;
@@ -42,6 +43,12 @@ public function getDatabasePlatform(ServerVersionProvider $versionProvider): Abs
                 return new MariaDB1060Platform();
             }
 
+            Deprecation::trigger(
+                'doctrine/dbal',
+                'https://github.com/doctrine/dbal/pull/6343',
+                'Support for MariaDB < 10.6.0 is deprecated and will be removed in DBAL 5',
+            );
+
             return new MariaDBPlatform();
         }
 

src/Driver/PgSQL/Connection.php

@@ -31,6 +31,7 @@ public function __construct(private PgSqlConnection $connection)
 
     public function __destruct()
     {
+        // @phpstan-ignore isset.initializedProperty
         if (! isset($this->connection)) {
             return;
         }

src/Driver/PgSQL/Driver.php

@@ -17,6 +17,7 @@
 use function func_get_args;
 use function implode;
 use function pg_connect;
+use function preg_match;
 use function restore_error_handler;
 use function set_error_handler;
 use function sprintf;
@@ -66,9 +67,18 @@ private function constructConnectionString(
         #[SensitiveParameter]
         array $params,
     ): string {
+        // pg_connect used by Doctrine DBAL does not support [...] notation,
+        // but requires the host address in plain form like `aa:bb:99...`
+        $matches = [];
+        if (isset($params['host']) && preg_match('/^\[(.+)\]$/', $params['host'], $matches) === 1) {
+            $params['hostaddr'] = $matches[1];
+            unset($params['host']);
+        }
+
         $components = array_filter(
             [
                 'host' => $params['host'] ?? null,
+                'hostaddr' => $params['hostaddr'] ?? null,
                 'port' => $params['port'] ?? null,
                 'dbname' => $params['dbname'] ?? 'postgres',
                 'user' => $params['user'] ?? null,

src/Driver/PgSQL/Statement.php

@@ -39,6 +39,7 @@ public function __construct(
 
     public function __destruct()
     {
+        // @phpstan-ignore isset.initializedProperty
         if (! isset($this->connection)) {
             return;
         }

src/Platforms/MariaDB1060Platform.php

@@ -8,6 +8,8 @@
 
 /**
  * Provides the behavior, features and SQL dialect of the MariaDB 10.6 database platform.
+ *
+ * @deprecated This class will be removed once support for MariaDB 10.5 is dropped.
  */
 class MariaDB1060Platform extends MariaDBPlatform
 {

src/Schema/TableEditor.php

@@ -43,17 +43,21 @@ final class TableEditor
     /** @internal Use {@link Table::editor()} or {@link Table::edit()} to create an instance */
     public function __construct()
     {
-        // @phpstan-ignore assign.propertyType (PHPStan doesn't infer the element type)
-        $this->columns = new UnqualifiedNamedObjectSet();
+        /** @var UnqualifiedNamedObjectSet<Column> $columns */
+        $columns       = new UnqualifiedNamedObjectSet();
+        $this->columns = $columns;
 
-        // @phpstan-ignore assign.propertyType
-        $this->indexes = new UnqualifiedNamedObjectSet();
+        /** @var UnqualifiedNamedObjectSet<Index> $indexes */
+        $indexes       = new UnqualifiedNamedObjectSet();
+        $this->indexes = $indexes;
 
-        // @phpstan-ignore assign.propertyType
-        $this->uniqueConstraints = new OptionallyUnqualifiedNamedObjectSet();
+        /** @var OptionallyUnqualifiedNamedObjectSet<UniqueConstraint> $uniqueConstraints */
+        $uniqueConstraints       = new OptionallyUnqualifiedNamedObjectSet();
+        $this->uniqueConstraints = $uniqueConstraints;
 
-        // @phpstan-ignore assign.propertyType
-        $this->foreignKeyConstraints = new OptionallyUnqualifiedNamedObjectSet();
+        /** @var OptionallyUnqualifiedNamedObjectSet<ForeignKeyConstraint> $foreignKeyConstraints */
+        $foreignKeyConstraints       = new OptionallyUnqualifiedNamedObjectSet();
+        $this->foreignKeyConstraints = $foreignKeyConstraints;
     }
 
     public function setName(OptionallyQualifiedName $name): self

tests/Driver/PgSQL/DriverTest.php

@@ -0,0 +1,49 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Doctrine\DBAL\Tests\Driver\PgSQL;
+
+use Doctrine\DBAL\Driver as DriverInterface;
+use Doctrine\DBAL\Driver\PgSQL\Driver;
+use Doctrine\DBAL\Tests\Driver\AbstractPostgreSQLDriverTestCase;
+use Doctrine\DBAL\Tests\TestUtil;
+
+use function in_array;
+
+class DriverTest extends AbstractPostgreSQLDriverTestCase
+{
+    protected function setUp(): void
+    {
+        parent::setUp();
+
+        if (isset($GLOBALS['db_driver']) && $GLOBALS['db_driver'] === 'pgsql') {
+            return;
+        }
+
+        self::markTestSkipped('Test enabled only when using pgsql specific phpunit.xml');
+    }
+
+    /**
+     * Ensure we can handle URI notation for IPv6 addresses
+     */
+    public function testConnectionIPv6(): void
+    {
+        if (! in_array($GLOBALS['db_host'], ['localhost', '127.0.0.1', '[::1]'], true)) {
+            // We cannot assume that every contributor runs the same setup as our CI
+            self::markTestSkipped('This test only works if there is a Postgres server listening on localhost.');
+        }
+
+        self::expectNotToPerformAssertions();
+
+        $params         = TestUtil::getConnectionParams();
+        $params['host'] = '[::1]';
+
+        $this->driver->connect($params);
+    }
+
+    protected function createDriver(): DriverInterface
+    {
+        return new Driver();
+    }
+}