[doctrineGH-4645] Fix SQLServerPlatform::quoteIdentifier()

morozov · May 13, 2021 · fde42dd · fde42dd
1 parent b728198
commit fde42dd
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 1 deletion.
diff --git a/lib/Doctrine/DBAL/Platforms/SQLServerPlatform.php b/lib/Doctrine/DBAL/Platforms/SQLServerPlatform.php
@@ -1610,7 +1610,7 @@ protected function getReservedKeywordsClass()
      */
     public function quoteSingleIdentifier($str)
     {
-        return '[' . str_replace(']', '][', $str) . ']';
+        return '[' . str_replace(']', ']]', $str) . ']';
     }
 
     /**

diff --git a/tests/Doctrine/Tests/DBAL/Functional/Platform/QuotingTest.php b/tests/Doctrine/Tests/DBAL/Functional/Platform/QuotingTest.php
@@ -4,6 +4,8 @@
 
 use Doctrine\Tests\DbalFunctionalTestCase;
 
+use function key;
+
 class QuotingTest extends DbalFunctionalTestCase
 {
     /**
@@ -29,4 +31,30 @@ public static function stringLiteralProvider(): iterable
             'single-quote' => ["'"],
         ];
     }
+
+    /**
+     * @dataProvider identifierProvider
+     */
+    public function testQuoteIdentifier(string $identifier): void
+    {
+        $platform = $this->connection->getDatabasePlatform();
+        $query    = $platform->getDummySelectSQL(
+            'NULL AS ' . $platform->quoteIdentifier($identifier)
+        );
+
+        self::assertSame($identifier, key($this->connection->fetchAssociative($query)));
+    }
+
+    /**
+     * @return iterable<string,array{0:string}>
+     */
+    public static function identifierProvider(): iterable
+    {
+        return [
+            '[' => ['['],
+            ']' => [']'],
+            '"' => ['"'],
+            '`' => ['`'],
+        ];
+    }
 }