Database PHP API - Foreign Key löschen

Hallo,

Quote from Hanashi

Mein Problem ist nun, dass das Update ewig läuft und nie zum Abschluss kommt. Bei Webspaces würde das dann wahrscheinlich sogar zu einem PHP Timeout kommen.

Nein, vermutlich sind das unendlich viele Einzelrequests, weil die Änderungen nicht konvergieren. Kannst du die zum Update gehörende Log-Datei aus dem log/-Ordner zur Verfügung stellen? Kannst du ggf. ein simples Plugin zum reproduzieren zur Verfügung stellen (eine Version zum installieren des „alten“ Datenbankzustands, eine weitere Version mit dem versuchten Update)?

Hallo,

danke. Ich hab das Thema mal in den Fehlerbereich verschoben und prüfe das bei Gelegenheit.

Hallo,

folgende Änderung korrigiert den Fehler. Der gleiche Bug existiert aber vermutlich generell, wenn array_diff verwendet wird. Das muss ich noch im Detail prüfen.

diff --git i/wcfsetup/install/files/lib/system/database/table/DatabaseTableChangeProcessor.class.php w/wcfsetup/install/files/lib/system/database/table/DatabaseTableChangeProcessor.class.php
index 6ee6fa6085..36b3b68d26 100644
--- i/wcfsetup/install/files/lib/system/database/table/DatabaseTableChangeProcessor.class.php
+++ w/wcfsetup/install/files/lib/system/database/table/DatabaseTableChangeProcessor.class.php
@@ -444,7 +444,7 @@ protected function calculateChanges()
                 foreach ($table->getForeignKeys() as $foreignKey) {
                     $matchingExistingForeignKey = null;
                     foreach ($existingForeignKeys as $existingForeignKey) {
-                        if (empty(\array_diff($foreignKey->getDiffData(), $existingForeignKey->getDiffData()))) {
+                        if (empty(\array_diff_assoc($foreignKey->getDiffData(), $existingForeignKey->getDiffData()))) {
                             $matchingExistingForeignKey = $existingForeignKey;
                             break;
                         }

Hallo,

die ersten beiden Korrekturen sind hier zu finden:

PHP DDL Fixes by TimWolla · Pull Request #4433 · WoltLab/WCF

The use of array_diff() in the PHP based DDL API is bogus. This PR fixes the first two locations where a bug could clearly be reproduced. The reproducer is…

github.com

Hallo,

mittlerweile wurden alle Vorkommnisse von array_diff entsprechend angepasst: https://github.com/WoltLab/WCF/issues/4434

Die Korrekturen sind über mehrere Versionen verteilt, deswegen nur ein allgemeines „Behoben“-Label ohne Versionslabels.

Hallo,

Quote from MysteryCode

Tim Düsterhus wäre es nicht sinnvoll die höchste Version zu labeln, damit man weiß, welche Version man voraussetzen muss?

Wir unterstützen generell nur die jeweils neuste Version eines Branches. Die Änderungen lassen sich aber so zusammenfassen:

• Änderungen wurden fehlerhaft angewendet, ohne eine Fehlermeldung: 5.2.x (weil dieser Fehler potentiell Datenverlust bedeutet).
• Änderungen wurden fehlerhaft zurückgewiesen, obwohl sie zulässig wären: 5.4.x (hier kracht es dann potentiell beim Update, aber das bemerkt man leicht während der Entwicklung).

Viele der Korrekturen sind aber auch eher theoretischer Natur, da man beispielsweise eine Spalte namens UNIQUE in einer Tabelle benötigt und dann einen UNIQUE INDEX auf dieser Spalte in einen regulären INDEX verwandeln möchte (https://github.com/WoltLab/WCF/co…15946c164d9f30a)

Ich habe die einzelnen Korrekturen sehr sorgfältig innerhalb der jeweiligen Commit-Nachrichten erklärt. Ich empfehle einfach mal selbst nachzusehen:

History for wcfsetup/install/files/lib/system/database/table/DatabaseTableChangeProcessor.class.php - WoltLab/WCF

WoltLab Suite Core (previously WoltLab Community Framework) - History for…

github.com