Fix fk name change detection in schema comparator

As index renaming support was introduced a while back do the same for
foreign key name changes.

Signed-off-by: Christoph Krapp <[email protected]>

Author: Christoph Krapp

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())

tests/Schema/AbstractComparatorTestCase.php

@@ -683,8 +683,10 @@ public function testCompareForeignKeyBasedOnPropertiesNotName(): void
         $tableB->addForeignKeyConstraint('bar', ['id'], ['id'], [], 'bar_constraint');
 
         $tableDiff = $this->comparator->diffTable($tableA, $tableB);
+        self::assertNotFalse($tableDiff);
+        self::assertCount(1, $tableDiff->addedForeignKeys);
+        self::assertCount(1, $tableDiff->removedForeignKeys);
 
-        self::assertFalse($tableDiff);
     }
 
     public function testCompareForeignKeyRestrictNoActionAreTheSame(): void