Merge pull request #6413 from achterin/bugfix/foreign_key_name_change…

…_detection Fix foreign key name change detection
doctrine · Jun 1, 2024 · 080aab5 · 080aab5
2 parents a5d2baf + 422e459
commit 080aab5
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/src/Schema/Comparator.php b/src/Schema/Comparator.php
@@ -555,6 +555,10 @@ private function detectRenamedIndexes(array &$addedIndexes, array &$removedIndex
      */
     public function diffForeignKey(ForeignKeyConstraint $key1, ForeignKeyConstraint $key2)
     {
+        if (strtolower($key1->getName()) !== strtolower($key2->getName())) {
+            return true;
+        }
+
         if (
             array_map('strtolower', $key1->getUnquotedLocalColumns())
             !== array_map('strtolower', $key2->getUnquotedLocalColumns())

diff --git a/tests/Schema/AbstractComparatorTestCase.php b/tests/Schema/AbstractComparatorTestCase.php
@@ -652,7 +652,7 @@ public function testCompareColumnCompareCaseInsensitive(): void
         self::assertFalse($tableDiff);
     }
 
-    public function testCompareIndexBasedOnPropertiesNotName(): void
+    public function testDetectIndexNameChange(): void
     {
         $tableA = new Table('foo');
         $tableA->addColumn('id', Types::INTEGER);
@@ -672,7 +672,7 @@ public function testCompareIndexBasedOnPropertiesNotName(): void
         );
     }
 
-    public function testCompareForeignKeyBasedOnPropertiesNotName(): void
+    public function testDetectForeignKeyNameChange(): void
     {
         $tableA = new Table('foo');
         $tableA->addColumn('id', Types::INTEGER);
@@ -683,8 +683,9 @@ public function testCompareForeignKeyBasedOnPropertiesNotName(): void
         $tableB->addForeignKeyConstraint('bar', ['id'], ['id'], [], 'bar_constraint');
 
         $tableDiff = $this->comparator->diffTable($tableA, $tableB);
-
-        self::assertFalse($tableDiff);
+        self::assertNotFalse($tableDiff);
+        self::assertCount(1, $tableDiff->addedForeignKeys);
+        self::assertCount(1, $tableDiff->removedForeignKeys);
     }
 
     public function testCompareForeignKeyRestrictNoActionAreTheSame(): void