Keep NVARCHAR(MAX) length as -1

doctrine · Jan 3, 2024 · 62b57ae · 62b57ae
1 parent 8bc0d9a
commit 62b57ae
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 1 deletion.
diff --git a/src/Schema/SQLServerSchemaManager.php b/src/Schema/SQLServerSchemaManager.php
@@ -83,12 +83,20 @@ protected function _getPortableTableColumnDefinition(array $tableColumn): Column
 
         switch ($dbType) {
             case 'nchar':
-            case 'nvarchar':
             case 'ntext':
                 // Unicode data requires 2 bytes per character
                 $length /= 2;
                 break;
 
+            case 'nvarchar':
+                if ($length === -1) {
+                    break;
+                }
+
+                // Unicode data requires 2 bytes per character
+                $length /= 2;
+                break;
+
             case 'varchar':
                 // TEXT type is returned as VARCHAR(MAX) with a length of -1
                 if ($length === -1) {

diff --git a/tests/Functional/Schema/SchemaManagerFunctionalTestCase.php b/tests/Functional/Schema/SchemaManagerFunctionalTestCase.php
@@ -1409,6 +1409,19 @@ protected function findTableByName(array $tables, string $name): ?Table
 
         return null;
     }
+
+    public function testNvarcharMaxIsLengthMinus1(): void
+    {
+        $createTableSQL = 'CREATE TABLE test_nvarchar_max (
+            col_nvarchar_max NVARCHAR(MAX),
+            col_nvarchar NVARCHAR(128)
+        )';
+        $this->connection->executeStatement($createTableSQL);
+
+        $table = $this->schemaManager->introspectTable('test_nvarchar_max');
+        self::assertSame(-1, $table->getColumn('col_nvarchar_max')->getLength());
+        self::assertSame(128, $table->getColumn('col_nvarchar')->getLength());
+    }
 }
 
 interface ListTableColumnsDispatchEventListener