Stop using the deprecated json_array type internally

lib/Doctrine/DBAL/Platforms/DB2Platform.php

@@ -876,6 +876,6 @@ public function supportsSavepoints()
      */
     protected function getReservedKeywordsClass()
     {
-        return 'Doctrine\DBAL\Platforms\Keywords\DB2Keywords';
+        return Keywords\DB2Keywords::class;
     }
 }

lib/Doctrine/DBAL/Platforms/DrizzlePlatform.php

@@ -20,9 +20,9 @@
 namespace Doctrine\DBAL\Platforms;
 
 use Doctrine\DBAL\Schema\Identifier;
-use Doctrine\DBAL\Schema\TableDiff;
 use Doctrine\DBAL\Schema\Index;
 use Doctrine\DBAL\Schema\Table;
+use Doctrine\DBAL\Schema\TableDiff;
 use Doctrine\DBAL\Types\BinaryType;
 
 /**
@@ -314,7 +314,7 @@ public function getListDatabasesSQL()
      */
     protected function getReservedKeywordsClass()
     {
-        return 'Doctrine\DBAL\Platforms\Keywords\DrizzleKeywords';
+        return Keywords\DrizzleKeywords::class;
     }
 
     /**

lib/Doctrine/DBAL/Platforms/MySQL57Platform.php

@@ -21,6 +21,7 @@
 
 use Doctrine\DBAL\Schema\Index;
 use Doctrine\DBAL\Schema\TableDiff;
+use Doctrine\DBAL\Types\Type;
 
 /**
  * Provides the behavior, features and SQL dialect of the MySQL 5.7 (5.7.9 GA) database platform.
@@ -79,7 +80,7 @@ protected function getRenameIndexSQL($oldIndexName, Index $index, $tableName)
      */
     protected function getReservedKeywordsClass()
     {
-        return 'Doctrine\DBAL\Platforms\Keywords\MySQL57Keywords';
+        return Keywords\MySQL57Keywords::class;
     }
 
     /**
@@ -89,6 +90,6 @@ protected function initializeDoctrineTypeMappings()
     {
         parent::initializeDoctrineTypeMappings();
 
-        $this->doctrineTypeMapping['json'] = 'json_array';
+        $this->doctrineTypeMapping['json'] = Type::JSON;
     }
 }

lib/Doctrine/DBAL/Platforms/MySqlPlatform.php

@@ -19,13 +19,13 @@
 
 namespace Doctrine\DBAL\Platforms;
 
+use Doctrine\DBAL\Connection;
 use Doctrine\DBAL\Schema\Identifier;
-use Doctrine\DBAL\Schema\TableDiff;
 use Doctrine\DBAL\Schema\Index;
 use Doctrine\DBAL\Schema\Table;
+use Doctrine\DBAL\Schema\TableDiff;
 use Doctrine\DBAL\Types\BlobType;
 use Doctrine\DBAL\Types\TextType;
-use Doctrine\DBAL\Connection;
 
 /**
  * The MySqlPlatform provides the behavior, features and SQL dialect of the
@@ -712,7 +712,7 @@ private function getPreAlterTableAlterPrimaryKeySQL(TableDiff $diff, Index $inde
             if (! $diff->fromTable->hasColumn($columnName)) {
                 continue;
             }
-          
+
             $column = $diff->fromTable->getColumn($columnName);
 
             if ($column->getAutoincrement() === true) {
@@ -1070,7 +1070,7 @@ public function getBinaryMaxLength()
      */
     protected function getReservedKeywordsClass()
     {
-        return 'Doctrine\DBAL\Platforms\Keywords\MySQLKeywords';
+        return Keywords\MySQLKeywords::class;
     }
 
     /**

lib/Doctrine/DBAL/Platforms/OraclePlatform.php

@@ -19,13 +19,13 @@
 
 namespace Doctrine\DBAL\Platforms;
 
+use Doctrine\DBAL\DBALException;
 use Doctrine\DBAL\Schema\ForeignKeyConstraint;
 use Doctrine\DBAL\Schema\Identifier;
 use Doctrine\DBAL\Schema\Index;
 use Doctrine\DBAL\Schema\Sequence;
 use Doctrine\DBAL\Schema\Table;
 use Doctrine\DBAL\Schema\TableDiff;
-use Doctrine\DBAL\DBALException;
 use Doctrine\DBAL\Types\BinaryType;
 
 /**
@@ -1156,7 +1156,7 @@ public function releaseSavePoint($savepoint)
      */
     protected function getReservedKeywordsClass()
     {
-        return 'Doctrine\DBAL\Platforms\Keywords\OracleKeywords';
+        return Keywords\OracleKeywords::class;
     }
 
     /**

lib/Doctrine/DBAL/Platforms/PostgreSQL91Platform.php

@@ -41,7 +41,7 @@ public function supportsColumnCollation()
      */
     protected function getReservedKeywordsClass()
     {
-        return 'Doctrine\DBAL\Platforms\Keywords\PostgreSQL91Keywords';
+        return Keywords\PostgreSQL91Keywords::class;
     }
 
     /**

lib/Doctrine/DBAL/Platforms/PostgreSQL92Platform.php

@@ -19,6 +19,8 @@
 
 namespace Doctrine\DBAL\Platforms;
 
+use Doctrine\DBAL\Types\Type;
+
 /**
  * Provides the behavior, features and SQL dialect of the PostgreSQL 9.2 database platform.
  *
@@ -61,7 +63,7 @@ public function hasNativeJsonType()
      */
     protected function getReservedKeywordsClass()
     {
-        return 'Doctrine\DBAL\Platforms\Keywords\PostgreSQL92Keywords';
+        return Keywords\PostgreSQL92Keywords::class;
     }
 
     /**
@@ -70,7 +72,8 @@ protected function getReservedKeywordsClass()
     protected function initializeDoctrineTypeMappings()
     {
         parent::initializeDoctrineTypeMappings();
-        $this->doctrineTypeMapping['json'] = 'json_array';
+
+        $this->doctrineTypeMapping['json'] = Type::JSON;
     }
 
     /**

lib/Doctrine/DBAL/Platforms/PostgreSQL94Platform.php

@@ -19,6 +19,8 @@
 
 namespace Doctrine\DBAL\Platforms;
 
+use Doctrine\DBAL\Types\Type;
+
 /**
  * Provides the behavior, features and SQL dialect of the PostgreSQL 9.4 database platform.
  *
@@ -45,7 +47,7 @@ public function getJsonTypeDeclarationSQL(array $field)
      */
     protected function getReservedKeywordsClass()
     {
-        return 'Doctrine\DBAL\Platforms\Keywords\PostgreSQL94Keywords';
+        return Keywords\PostgreSQL94Keywords::class;
     }
 
     /**
@@ -54,6 +56,7 @@ protected function getReservedKeywordsClass()
     protected function initializeDoctrineTypeMappings()
     {
         parent::initializeDoctrineTypeMappings();
-        $this->doctrineTypeMapping['jsonb'] = 'json_array';
+
+        $this->doctrineTypeMapping['jsonb'] = Type::JSON;
     }
 }

lib/Doctrine/DBAL/Platforms/PostgreSqlPlatform.php

@@ -1166,7 +1166,7 @@ public function getBinaryDefaultLength()
      */
     protected function getReservedKeywordsClass()
     {
-        return 'Doctrine\DBAL\Platforms\Keywords\PostgreSQLKeywords';
+        return Keywords\PostgreSQLKeywords::class;
     }
 
     /**

lib/Doctrine/DBAL/Platforms/SQLAnywhere11Platform.php

@@ -42,6 +42,6 @@ public function getRegexpExpression()
      */
     protected function getReservedKeywordsClass()
     {
-        return 'Doctrine\DBAL\Platforms\Keywords\SQLAnywhere11Keywords';
+        return Keywords\SQLAnywhere11Keywords::class;
     }
 }

lib/Doctrine/DBAL/Platforms/SQLAnywhere12Platform.php

@@ -121,7 +121,7 @@ protected function getAdvancedIndexOptionsSQL(Index $index)
      */
     protected function getReservedKeywordsClass()
     {
-        return 'Doctrine\DBAL\Platforms\Keywords\SQLAnywhere12Keywords';
+        return Keywords\SQLAnywhere12Keywords::class;
     }
 
     /**

lib/Doctrine/DBAL/Platforms/SQLAnywhere16Platform.php

@@ -55,6 +55,6 @@ protected function getAdvancedIndexOptionsSQL(Index $index)
      */
     protected function getReservedKeywordsClass()
     {
-        return 'Doctrine\DBAL\Platforms\Keywords\SQLAnywhere16Keywords';
+        return Keywords\SQLAnywhere16Keywords::class;
     }
 }

lib/Doctrine/DBAL/Platforms/SQLAnywherePlatform.php

@@ -26,8 +26,8 @@
 use Doctrine\DBAL\Schema\ColumnDiff;
 use Doctrine\DBAL\Schema\Constraint;
 use Doctrine\DBAL\Schema\ForeignKeyConstraint;
-use Doctrine\DBAL\Schema\Index;
 use Doctrine\DBAL\Schema\Identifier;
+use Doctrine\DBAL\Schema\Index;
 use Doctrine\DBAL\Schema\Table;
 use Doctrine\DBAL\Schema\TableDiff;
 
@@ -1429,7 +1429,7 @@ protected function getRenameIndexSQL($oldIndexName, Index $index, $tableName)
      */
     protected function getReservedKeywordsClass()
     {
-        return 'Doctrine\DBAL\Platforms\Keywords\SQLAnywhereKeywords';
+        return Keywords\SQLAnywhereKeywords::class;
     }
 
     /**

lib/Doctrine/DBAL/Platforms/SQLServer2005Platform.php

@@ -58,6 +58,6 @@ public function getClobTypeDeclarationSQL(array $field)
      */
     protected function getReservedKeywordsClass()
     {
-        return 'Doctrine\DBAL\Platforms\Keywords\SQLServer2005Keywords';
+        return Keywords\SQLServer2005Keywords::class;
     }
 }

lib/Doctrine/DBAL/Platforms/SQLServer2008Platform.php

@@ -114,6 +114,6 @@ protected function initializeDoctrineTypeMappings()
      */
     protected function getReservedKeywordsClass()
     {
-        return 'Doctrine\DBAL\Platforms\Keywords\SQLServer2008Keywords';
+        return Keywords\SQLServer2008Keywords::class;
     }
 }

lib/Doctrine/DBAL/Platforms/SQLServer2012Platform.php

@@ -101,7 +101,7 @@ public function supportsSequences()
      */
     protected function getReservedKeywordsClass()
     {
-        return 'Doctrine\DBAL\Platforms\Keywords\SQLServer2012Keywords';
+        return Keywords\SQLServer2012Keywords::class;
     }
 
     /**
@@ -122,7 +122,7 @@ protected function doModifyLimitQuery($query, $limit, $offset = null)
         if ($matchesCount > 0) {
             $orderByPos = $matches[0][($matchesCount - 1)][1];
         }
-        
+
         if ($orderByPos === false
             || substr_count($query, "(", $orderByPos) - substr_count($query, ")", $orderByPos)
         ) {

lib/Doctrine/DBAL/Platforms/SQLServerPlatform.php

@@ -22,11 +22,11 @@
 use Doctrine\DBAL\LockMode;
 use Doctrine\DBAL\Schema\Column;
 use Doctrine\DBAL\Schema\ColumnDiff;
-use Doctrine\DBAL\Schema\Identifier;
-use Doctrine\DBAL\Schema\TableDiff;
 use Doctrine\DBAL\Schema\ForeignKeyConstraint;
+use Doctrine\DBAL\Schema\Identifier;
 use Doctrine\DBAL\Schema\Index;
 use Doctrine\DBAL\Schema\Table;
+use Doctrine\DBAL\Schema\TableDiff;
 
 /**
  * The SQLServerPlatform provides the behavior, features and SQL dialect of the
@@ -1512,7 +1512,7 @@ public function getForUpdateSQL()
      */
     protected function getReservedKeywordsClass()
     {
-        return 'Doctrine\DBAL\Platforms\Keywords\SQLServerKeywords';
+        return Keywords\SQLServerKeywords::class;
     }
 
     /**

lib/Doctrine/DBAL/Platforms/SqlitePlatform.php

@@ -21,12 +21,12 @@
 
 use Doctrine\DBAL\DBALException;
 use Doctrine\DBAL\Schema\Column;
-use Doctrine\DBAL\Schema\TableDiff;
-use Doctrine\DBAL\Schema\Table;
+use Doctrine\DBAL\Schema\Constraint;
 use Doctrine\DBAL\Schema\ForeignKeyConstraint;
-use Doctrine\DBAL\Schema\Index;
 use Doctrine\DBAL\Schema\Identifier;
-use Doctrine\DBAL\Schema\Constraint;
+use Doctrine\DBAL\Schema\Index;
+use Doctrine\DBAL\Schema\Table;
+use Doctrine\DBAL\Schema\TableDiff;
 
 /**
  * The SqlitePlatform class describes the specifics and dialects of the SQLite
@@ -633,7 +633,7 @@ protected function initializeDoctrineTypeMappings()
      */
     protected function getReservedKeywordsClass()
     {
-        return 'Doctrine\DBAL\Platforms\Keywords\SQLiteKeywords';
+        return Keywords\SQLiteKeywords::class;
     }
 
     /**

lib/Doctrine/DBAL/Schema/PostgreSqlSchemaManager.php

@@ -444,7 +444,7 @@ protected function _getPortableTableColumnDefinition($tableColumn)
             $column->setPlatformOption('collation', $tableColumn['collation']);
         }
 
-        if ($column->getType()->getName() === 'json_array') {
+        if (in_array($column->getType()->getName(), [Type::JSON_ARRAY, Type::JSON], true)) {
             $column->setPlatformOption('jsonb', $jsonb);
         }
 

tests/Doctrine/Tests/DBAL/Functional/Schema/PostgreSqlSchemaManagerTest.php

@@ -28,7 +28,7 @@ public function testGetSearchPath()
         $params = $this->_conn->getParams();
 
         $paths = $this->_sm->getSchemaSearchPaths();
-        $this->assertEquals(array($params['user'], 'public'), $paths);
+        $this->assertEquals([$params['user'], 'public'], $paths);
     }
 
     /**
@@ -40,7 +40,7 @@ public function testGetSchemaNames()
 
         $this->assertInternalType('array', $names);
         $this->assertTrue(count($names) > 0);
-        $this->assertTrue(in_array('public', $names), "The public schema should be found.");
+        $this->assertContains('public', $names, 'The public schema should be found.');
     }
 
     /**
@@ -366,22 +366,33 @@ public function testPartialIndexes()
         $this->assertSame('(id IS NULL)', $onlineTable->getIndex('simple_partial_index')->getOption('where'));
     }
 
-    public function testJsonbColumn()
+    /**
+     * @dataProvider jsonbColumnTypeProvider
+     */
+    public function testJsonbColumn(string $type): void
     {
         if (!$this->_sm->getDatabasePlatform() instanceof PostgreSQL94Platform) {
             $this->markTestSkipped("Requires PostgresSQL 9.4+");
             return;
         }
 
         $table = new Schema\Table('test_jsonb');
-        $table->addColumn('foo', 'json_array')->setPlatformOption('jsonb', true);
+        $table->addColumn('foo', $type)->setPlatformOption('jsonb', true);
         $this->_sm->dropAndCreateTable($table);
 
         /** @var Schema\Column[] $columns */
         $columns = $this->_sm->listTableColumns('test_jsonb');
 
-        $this->assertEquals('json_array', $columns['foo']->getType()->getName());
-        $this->assertEquals(true, $columns['foo']->getPlatformOption('jsonb'));
+        $this->assertSame(TYPE::JSON, $columns['foo']->getType()->getName());
+        $this->assertTrue(true, $columns['foo']->getPlatformOption('jsonb'));
+    }
+
+    public function jsonbColumnTypeProvider(): array
+    {
+        return [
+            [Type::JSON],
+            [Type::JSON_ARRAY],
+        ];
     }
 
     /**

tests/Doctrine/Tests/DBAL/Platforms/MySQL57PlatformTest.php

@@ -3,6 +3,7 @@
 namespace Doctrine\Tests\DBAL\Platforms;
 
 use Doctrine\DBAL\Platforms\MySQL57Platform;
+use Doctrine\DBAL\Types\Type;
 
 class MySQL57PlatformTest extends AbstractMySQLPlatformTestCase
 {
@@ -27,7 +28,7 @@ public function testReturnsJsonTypeDeclarationSQL()
     public function testInitializesJsonTypeMapping()
     {
         $this->assertTrue($this->_platform->hasDoctrineTypeMappingFor('json'));
-        $this->assertSame('json_array', $this->_platform->getDoctrineTypeMapping('json'));
+        $this->assertSame(Type::JSON, $this->_platform->getDoctrineTypeMapping('json'));
     }
 
     /**