🐛 Oracle: fix comment retrieval on table with reserved keyword name

Author: homersimpsons

src/Schema/OracleSchemaManager.php

@@ -512,13 +512,13 @@ protected function fetchTableOptionsByTable(string $databaseName, ?string $table
 
         $sql .= ' FROM ALL_TAB_COMMENTS WHERE ' . implode(' AND ', $conditions);
 
-        /** @var array<string,array<string,mixed>> $metadata */
-        $metadata = $this->_conn->executeQuery($sql, $params)
-            ->fetchAllAssociativeIndexed();
+        /** @var array<array{TABLE_NAME: string, COMMENTS: string|null}> $metadata */
+        $metadata = $this->_conn->executeQuery($sql, $params)->fetchAllAssociative();
 
         $tableOptions = [];
-        foreach ($metadata as $table => $data) {
-            $data = array_change_key_case($data, CASE_LOWER);
+        foreach ($metadata as $data) {
+            $data  = array_change_key_case($data, CASE_LOWER);
+            $table = $this->_getPortableTableDefinition($data);
 
             $tableOptions[$table] = [
                 'comment' => $data['comments'],

tests/Functional/Schema/SchemaManagerFunctionalTestCase.php

@@ -1677,9 +1677,16 @@ public function testIntrospectReservedKeywordTableViaListTableDetails(): void
         $this->createReservedKeywordTables();
 
         $user = $this->schemaManager->introspectTable('"user"');
-        self::assertCount(2, $user->getColumns());
+        self::assertCount(3, $user->getColumns());
         self::assertCount(2, $user->getIndexes());
         self::assertCount(1, $user->getForeignKeys());
+
+        if ($this->connection->getDatabasePlatform() instanceof SqlitePlatform) {
+            return; // SQLite does not support table / column comment
+        }
+
+        self::assertSame('table comment', $user->getComment());
+        self::assertSame('column comment', $user->getColumn('user')->getComment());
     }
 
     public function testIntrospectReservedKeywordTableViaListTables(): void
@@ -1690,9 +1697,16 @@ public function testIntrospectReservedKeywordTableViaListTables(): void
 
         $user = $this->findTableByName($tables, 'user');
         self::assertNotNull($user);
-        self::assertCount(2, $user->getColumns());
+        self::assertCount(3, $user->getColumns());
         self::assertCount(2, $user->getIndexes());
         self::assertCount(1, $user->getForeignKeys());
+
+        if ($this->connection->getDatabasePlatform() instanceof SqlitePlatform) {
+            return; // SQLite does not support table / column comment
+        }
+
+        self::assertSame('table comment', $user->getComment());
+        self::assertSame('column comment', $user->getColumn('user')->getComment());
     }
 
     private function createReservedKeywordTables(): void
@@ -1705,7 +1719,9 @@ private function createReservedKeywordTables(): void
         $schema = new Schema();
 
         $user = $schema->createTable('user');
+        $user->setComment('table comment');
         $user->addColumn('id', Types::INTEGER);
+        $user->addColumn('user', Types::INTEGER)->setComment('column comment');
         $user->addColumn('group_id', Types::INTEGER);
         $user->setPrimaryKey(['id']);
         $user->addForeignKeyConstraint('group', ['group_id'], ['id']);