Extend AbstractMySQLPlatform::getColumnTypeSQLSnippets() deprecation …

…layer

UPGRADE.md

@@ -8,10 +8,12 @@ awareness about deprecated code.
 
 # Upgrade to 3.8
 
-## Deprecated `AbstractMySQLPlatform::getColumnTypeSQLSnippets()`
+## Deprecated `AbstractMySQLPlatform` methods
 
-`AbstractMySQLPlatform::getColumnTypeSQLSnippets()` has been deprecated.
-Use `AbstractMySQLPlatform::getColumnTypeSQLSnippet()` instead.
+* `AbstractMySQLPlatform::getColumnTypeSQLSnippets()` has been deprecated
+  in favor of `AbstractMySQLPlatform::getColumnTypeSQLSnippet()`.
+* `AbstractMySQLPlatform::getDatabaseNameSQL()` has been deprecated without replacement.
+* Not passing a database name to `AbstractMySQLPlatform::getColumnTypeSQLSnippet()` has been deprecated.
 
 ## Deprecated reset methods from `QueryBuilder`
 

psalm.xml.dist

@@ -511,6 +511,7 @@
                     TODO: remove in 4.0.0
                 -->
                 <referencedMethod name="Doctrine\DBAL\Platforms\AbstractMySQLPlatform::getColumnTypeSQLSnippets" />
+                <referencedMethod name="Doctrine\DBAL\Platforms\AbstractMySQLPlatform::getDatabaseNameSQL" />
 
                 <!-- TODO for PHPUnit 10 -->
                 <referencedMethod name="PHPUnit\Framework\MockObject\Builder\InvocationMocker::withConsecutive"/>

src/Platforms/AbstractMySQLPlatform.php

@@ -1422,8 +1422,16 @@ public function supportsColumnLengthIndexes(): bool
         return true;
     }
 
+    /** @deprecated Will be removed without replacement. */
     protected function getDatabaseNameSQL(?string $databaseName): string
     {
+        Deprecation::triggerIfCalledFromOutside(
+            'doctrine/dbal',
+            'TODO',
+            '%s is deprecated without replacement.',
+            __METHOD__,
+        );
+
         if ($databaseName !== null) {
             return $this->quoteStringLiteral($databaseName);
         }

src/Platforms/MariaDb1043Platform.php

@@ -3,6 +3,7 @@
 namespace Doctrine\DBAL\Platforms;
 
 use Doctrine\DBAL\Types\JsonType;
+use Doctrine\Deprecations\Deprecation;
 
 use function sprintf;
 
@@ -81,6 +82,16 @@ public function getColumnTypeSQLSnippet(string $tableAlias = 'c', ?string $datab
             return parent::getColumnTypeSQLSnippet($tableAlias, $databaseName);
         }
 
+        if ($databaseName === null) {
+            Deprecation::trigger(
+                'doctrine/dbal',
+                'TODO',
+                'Not passing a database name to methods "getColumnTypeSQLSnippet()", '
+                    . '"getColumnTypeSQLSnippets()", and "getListTableColumnsSQL()" of "%s" is deprecated.',
+                self::class,
+            );
+        }
+
         $databaseName = $this->getDatabaseNameSQL($databaseName);
 
         // The check for `CONSTRAINT_SCHEMA = $databaseName` is mandatory here to prevent performance issues