Adding checks to cope for foreign keys that are defined twice on the same set of columns.

We now do deduplication for those.

Author: moufmouf

src/SchemaAnalyzer.php

@@ -302,7 +302,8 @@ private function buildSchemaGraph()
 
         // Then, let's create all the edges
         foreach ($this->getSchema()->getTables() as $table) {
-            foreach ($table->getForeignKeys() as $fk) {
+            $fks = $this->removeDuplicates($table->getForeignKeys());
+            foreach ($fks as $fk) {
                 // Create an undirected edge, with weight = 1
                 $edge = $graph->getVertex($table->getName())->createEdge($graph->getVertex($fk->getForeignTableName()));
                 if (isset($this->alteredCosts[$fk->getLocalTable()->getName()][implode(',', $fk->getLocalColumns())])) {
@@ -340,6 +341,22 @@ private function buildSchemaGraph()
         return $graph;
     }
 
+    /**
+     * Remove duplicate foreign keys (assumes that all foreign yes are from the same local table).
+     *
+     * @param ForeignKeyConstraint[] $foreignKeys
+     * @return ForeignKeyConstraint[]
+     */
+    private function removeDuplicates(array $foreignKeys)
+    {
+        $fks = [];
+        foreach ($foreignKeys as $foreignKey) {
+            $fks[implode('__`__', $foreignKey->getLocalColumns())] = $foreignKey;
+        }
+
+        return array_values($fks);
+    }
+
     /**
      * Returns the schema (from the schema manager or the cache if needed).
      *
@@ -572,7 +589,8 @@ private function getChildrenRelationshipsWithoutCache($tableName)
             if ($table->getName() === $tableName) {
                 continue;
             }
-            foreach ($table->getForeignKeys() as $fk) {
+            $fks = $this->removeDuplicates($table->getForeignKeys());
+            foreach ($fks as $fk) {
                 if ($fk->getForeignTableName() === $tableName && $this->isInheritanceRelationship($fk)) {
                     $children[] = $fk;
                 }