Remove support for Postgres 10 and 11

.github/workflows/continuous-integration.yml

@@ -216,7 +216,7 @@ jobs:
         php-version:
           - "8.1"
         postgres-version:
-          - "10"
+          - "12"
           - "15"
           - "16"
         extension:

UPGRADE.md

@@ -13,10 +13,11 @@ awareness about deprecated code.
 A new method `getColumnName()` has been added to the `Result` interface and must be implemented by
 all drivers and middleware.
 
-## BC BREAK: Removed support for MariaDB 10.4 and MySQL 5.7
+## BC BREAK: Removed support for MariaDB 10.4, MySQL 5.7 and Postgres 10 + 11
 
 * Upgrade to MariaDB 10.5 or later.
 * Upgrade to MySQL 8.0 or later.
+* Upgrade to Postgres 12 or later.
 
 # Upgrade to 4.1
 

docs/en/reference/platforms.rst

@@ -56,8 +56,7 @@ Microsoft SQL Server
 PostgreSQL
 ^^^^^^^^^^
 
--  ``PostgreSQLPlatform`` for version 10.0 and above.
--  ``PostgreSQL120Platform`` for version 12.0 and above.
+-  ``PostgreSQLPlatform`` for version 12.0 and above.
 
 IBM DB2
 ^^^^^^^

psalm.xml.dist

@@ -42,15 +42,6 @@
                 <file name="src/Driver/OCI8/ConvertPositionalToNamedPlaceholders.php"/>
             </errorLevel>
         </ConflictingReferenceConstraint>
-        <DeprecatedClass>
-            <errorLevel type="suppress">
-                <!--
-                    See https://github.com/doctrine/dbal/pull/6343
-                    TODO: remove in 5.0.0
-                -->
-                <referencedClass name="Doctrine\DBAL\Platforms\PostgreSQL120Platform" />
-            </errorLevel>
-        </DeprecatedClass>
         <DeprecatedMethod>
             <errorLevel type="suppress">
                 <!--

src/Driver/AbstractPostgreSQLDriver.php

@@ -7,14 +7,8 @@
 use Doctrine\DBAL\Driver;
 use Doctrine\DBAL\Driver\API\ExceptionConverter as ExceptionConverterInterface;
 use Doctrine\DBAL\Driver\API\PostgreSQL\ExceptionConverter;
-use Doctrine\DBAL\Platforms\Exception\InvalidPlatformVersion;
-use Doctrine\DBAL\Platforms\PostgreSQL120Platform;
 use Doctrine\DBAL\Platforms\PostgreSQLPlatform;
 use Doctrine\DBAL\ServerVersionProvider;
-use Doctrine\Deprecations\Deprecation;
-
-use function preg_match;
-use function version_compare;
 
 /**
  * Abstract base implementation of the {@see Driver} interface for PostgreSQL based drivers.
@@ -23,30 +17,6 @@ abstract class AbstractPostgreSQLDriver implements Driver
 {
     public function getDatabasePlatform(ServerVersionProvider $versionProvider): PostgreSQLPlatform
     {
-        $version = $versionProvider->getServerVersion();
-
-        if (preg_match('/^(?P<major>\d+)(?:\.(?P<minor>\d+)(?:\.(?P<patch>\d+))?)?/', $version, $versionParts) === 0) {
-            throw InvalidPlatformVersion::new(
-                $version,
-                '<major_version>.<minor_version>.<patch_version>',
-            );
-        }
-
-        $majorVersion = $versionParts['major'];
-        $minorVersion = $versionParts['minor'] ?? 0;
-        $patchVersion = $versionParts['patch'] ?? 0;
-        $version      = $majorVersion . '.' . $minorVersion . '.' . $patchVersion;
-
-        if (version_compare($version, '12.0', '>=')) {
-            return new PostgreSQL120Platform();
-        }
-
-        Deprecation::trigger(
-            'doctrine/dbal',
-            'https://github.com/doctrine/dbal/pull/6495',
-            'Support for Postgres < 12 is deprecated and will be removed in DBAL 5',
-        );
-
         return new PostgreSQLPlatform();
     }
 

src/Platforms/PostgreSQLPlatform.php

@@ -677,19 +677,6 @@ public function getTruncateTableSQL(string $tableName, bool $cascade = false): s
         return $sql;
     }
 
-    /**
-     * Get the snippet used to retrieve the default value for a given column
-     */
-    public function getDefaultColumnValueSQLSnippet(): string
-    {
-        return <<<'SQL'
-             SELECT pg_get_expr(adbin, adrelid)
-             FROM pg_attrdef
-             WHERE c.oid = pg_attrdef.adrelid
-                AND pg_attrdef.adnum=a.attnum
-        SQL;
-    }
-
     protected function initializeDoctrineTypeMappings(): void
     {
         $this->doctrineTypeMapping = [

src/Schema/PostgreSQLSchemaManager.php

@@ -417,7 +417,7 @@ protected function selectTableColumns(string $databaseName, ?string $tableName =
             $sql .= 'c.relname AS table_name, n.nspname AS schema_name,';
         }
 
-        $sql .= sprintf(<<<'SQL'
+        $sql .= <<<'SQL'
             a.attnum,
             quote_ident(a.attname) AS field,
             t.typname AS type,
@@ -434,7 +434,14 @@ protected function selectTableColumns(string $databaseName, ?string $tableName =
                 AND pg_index.indkey[0] = a.attnum
                 AND pg_index.indisprimary = 't'
             ) AS pri,
-            (%s) AS default,
+            (SELECT
+                CASE
+                    WHEN a.attgenerated = 's' THEN NULL
+                    ELSE pg_get_expr(adbin, adrelid)
+                END
+             FROM pg_attrdef
+             WHERE c.oid = pg_attrdef.adrelid
+                AND pg_attrdef.adnum=a.attnum) AS default,
             (SELECT pg_description.description
                 FROM pg_description WHERE pg_description.objoid = c.oid AND a.attnum = pg_description.objsubid
             ) AS comment
@@ -449,7 +456,7 @@ protected function selectTableColumns(string $databaseName, ?string $tableName =
                     ON d.objid = c.oid
                         AND d.deptype = 'e'
                         AND d.classid = (SELECT oid FROM pg_class WHERE relname = 'pg_class')
-            SQL, $this->platform->getDefaultColumnValueSQLSnippet());
+            SQL;
 
         $conditions = array_merge([
             'a.attnum > 0',

tests/Driver/VersionAwarePlatformDriverTest.php

@@ -12,7 +12,6 @@
 use Doctrine\DBAL\Platforms\MariaDBPlatform;
 use Doctrine\DBAL\Platforms\MySQL84Platform;
 use Doctrine\DBAL\Platforms\MySQLPlatform;
-use Doctrine\DBAL\Platforms\PostgreSQL120Platform;
 use Doctrine\DBAL\Platforms\PostgreSQLPlatform;
 use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
@@ -64,11 +63,9 @@ public function testPDOPgSQL(string $version, string $expectedClass): void
     public static function postgreSQLVersionProvider(): array
     {
         return [
-            ['10.0', PostgreSQLPlatform::class],
-            ['11.0', PostgreSQLPlatform::class],
-            ['12.0', PostgreSQL120Platform::class],
-            ['13.16', PostgreSQL120Platform::class],
-            ['16.4', PostgreSQL120Platform::class],
+            ['12.0', PostgreSQLPlatform::class],
+            ['13.16', PostgreSQLPlatform::class],
+            ['16.4', PostgreSQLPlatform::class],
         ];
     }
 

tests/Functional/Schema/PostgreSQLSchemaManagerTest.php

@@ -5,7 +5,6 @@
 namespace Doctrine\DBAL\Tests\Functional\Schema;
 
 use Doctrine\DBAL\Platforms\AbstractPlatform;
-use Doctrine\DBAL\Platforms\PostgreSQL120Platform;
 use Doctrine\DBAL\Platforms\PostgreSQLPlatform;
 use Doctrine\DBAL\Schema\ForeignKeyConstraint;
 use Doctrine\DBAL\Schema\Schema;
@@ -293,10 +292,6 @@ public function testBooleanDefault(): void
 
     public function testGeneratedColumn(): void
     {
-        if (! $this->connection->getDatabasePlatform() instanceof PostgreSQL120Platform) {
-             self::markTestSkipped('Generated columns are not supported in Postgres 11 and earlier');
-        }
-
         $table = new Table('ddc6198_generated_always_as');
         $table->addColumn('id', Types::INTEGER);
         $table->addColumn(