Fix for Issue doctrine#1487

cristi-contiu · Feb 28, 2025 · e48897d · e48897d
1 parent a76ce4d
commit e48897d
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 2 deletions.
diff --git a/src/Generator/DiffGenerator.php b/src/Generator/DiffGenerator.php
@@ -146,6 +146,10 @@ private function createToSchema(): Schema
      */
     private function resolveTableName(string $name): string
     {
+        if ($this->platform->supportsSchemas()) {
+            return $name;
+        }
+
         $pos = strpos($name, '.');
 
         return $pos === false ? $name : substr($name, $pos + 1);

diff --git a/tests/Generator/DiffGeneratorTest.php b/tests/Generator/DiffGeneratorTest.php
@@ -184,7 +184,12 @@ public function testGenerateFromEmptySchema(): void
         self::assertSame('path2', $this->migrationDiffGenerator->generate('2345', null, false, 120, true, true));
     }
 
-    public function testGenerateAppliesFilterOnMappedSchema(): void
+    /**
+     * @param array<int, string> $expectedTables
+     *
+     * @dataProvider getGenerateAppliesFilterOnMappedSchemaData
+     */
+    public function testGenerateAppliesFilterOnMappedSchema(bool $platformSupportsSchemas, array $expectedTables): void
     {
         // a standard Regex SchemaAssetsFilter already registered on the DBAL
         $dbalSchemaAssetsFilter = static function ($assetName): bool {
@@ -197,6 +202,10 @@ public function testGenerateAppliesFilterOnMappedSchema(): void
         $toTable2 = new Table('some_schema.table2');
         $toSchema = new Schema([$toTable1, $toTable2]);
 
+        $this->platform->expects(self::atLeast(1))
+            ->method('supportsSchemas')
+            ->willReturn($platformSupportsSchemas);
+
         $this->schemaManager->expects(self::once())
             ->method('introspectSchema')
             ->willReturn($fromSchema);
@@ -224,7 +233,16 @@ public function testGenerateAppliesFilterOnMappedSchema(): void
 
         $filteredTableNames = array_map(static fn (Table $table) => $table->getName(), $toSchema->getTables());
 
-        self::assertSame(['some_schema.table1', 'some_schema.table2'], array_values($filteredTableNames));
+        self::assertSame($expectedTables, array_values($filteredTableNames));
+    }
+
+    /** @return array<string, array<int, bool|array<int, string>>> */
+    public static function getGenerateAppliesFilterOnMappedSchemaData(): array
+    {
+        return [
+            'platform without schemas supports' => [false, []],
+            'platform with schema schemas support' => [true, ['some_schema.table1', 'some_schema.table2']],
+        ];
     }
 
     protected function setUp(): void