CoreInstaller.php: drop foreign keys instead of ignoring them

More cross-platform solution.
ibexa · Feb 20, 2025 · 1a22a87 · 1a22a87
1 parent 7c87e0f
commit 1a22a87
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/src/bundle/RepositoryInstaller/Installer/CoreInstaller.php b/src/bundle/RepositoryInstaller/Installer/CoreInstaller.php
@@ -98,10 +98,20 @@ protected function getDropSqlStatementsForExistingSchema(
         AbstractPlatform $databasePlatform
     ): array {
         $existingSchema = $this->db->getSchemaManager()->createSchema();
-        $statements = ['SET FOREIGN_KEY_CHECKS=0'];
+        $statements = [];
         // reverse table order for clean-up (due to FKs)
         $tables = array_reverse($newSchema->getTables());
-        // cleanup pre-existing database
+        if ($databasePlatform->supportsForeignKeyConstraints()) {
+            // cleanup pre-existing database: drop foreign keys
+            foreach ($tables as $table) {
+                if ($existingSchema->hasTable($table->getName())) {
+                    foreach($this->db->getSchemaManager()->listTableForeignKeys($table->getName()) as $foreignKeyConstraint) {
+                        $statements[] = $databasePlatform->getDropForeignKeySQL($foreignKeyConstraint->getName(), $table->getName());
+                    }
+                }
+            }
+        }
+        // cleanup pre-existing database: drop tables
         foreach ($tables as $table) {
             if ($existingSchema->hasTable($table->getName())) {
                 $statements[] = $databasePlatform->getDropTableSQL($table);