Reduce duplication of collation declaration code

It is safe to quote collation in the default implementation of
getColumnCollationDeclarationSQL() since those platforms where the case
sensitivity of an identifier depends on whether it's quoted (Oracle,
IBM DB2) do not support column collation anyways.

Author: morozov

src/Platforms/AbstractMySQLPlatform.php

@@ -1028,16 +1028,6 @@ public function getColumnCharsetDeclarationSQL($charset)
         return 'CHARACTER SET ' . $charset;
     }
 
-    /**
-     * {@inheritDoc}
-     *
-     * @internal The method should be only used from within the {@see AbstractPlatform} class hierarchy.
-     */
-    public function getColumnCollationDeclarationSQL($collation)
-    {
-        return 'COLLATE ' . $this->quoteSingleIdentifier($collation);
-    }
-
     /**
      * {@inheritDoc}
      *

src/Platforms/AbstractPlatform.php

@@ -3321,7 +3321,7 @@ public function getColumnCharsetDeclarationSQL($charset)
      */
     public function getColumnCollationDeclarationSQL($collation)
     {
-        return $this->supportsColumnCollation() ? 'COLLATE ' . $collation : '';
+        return $this->supportsColumnCollation() ? 'COLLATE ' . $this->quoteSingleIdentifier($collation) : '';
     }
 
     /**

src/Platforms/PostgreSQLPlatform.php

@@ -1302,16 +1302,6 @@ public function supportsColumnCollation()
         return true;
     }
 
-    /**
-     * {@inheritdoc}
-     *
-     * @internal The method should be only used from within the {@see AbstractPlatform} class hierarchy.
-     */
-    public function getColumnCollationDeclarationSQL($collation)
-    {
-        return 'COLLATE ' . $this->quoteSingleIdentifier($collation);
-    }
-
     /**
      * {@inheritdoc}
      */

tests/Platforms/SqlitePlatformTest.php

@@ -787,14 +787,6 @@ public function testSupportsColumnCollation(): void
         self::assertTrue($this->platform->supportsColumnCollation());
     }
 
-    public function testColumnCollationDeclarationSQL(): void
-    {
-        self::assertSame(
-            'COLLATE NOCASE',
-            $this->platform->getColumnCollationDeclarationSQL('NOCASE'),
-        );
-    }
-
     public function testGetCreateTableSQLWithColumnCollation(): void
     {
         $table = new Table('foo');
@@ -804,7 +796,7 @@ public function testGetCreateTableSQLWithColumnCollation(): void
         self::assertSame(
             [
                 'CREATE TABLE foo (no_collation VARCHAR(255) NOT NULL, '
-                    . 'column_collation VARCHAR(255) NOT NULL COLLATE NOCASE)',
+                    . 'column_collation VARCHAR(255) NOT NULL COLLATE "NOCASE")',
             ],
             $this->platform->getCreateTableSQL($table),
         );