Migrator Bug in 3.3.16

[LukeWCS]

Durch meine Tests mit AnubisBB hat sich die folgende Vermutung als Fakt herausgestellt:

1. Das Migrator Tool permission.remove ist definitiv eine der möglichen Ursachen. Es könnte also auch noch andere Migrator Tools betreffen, das ist aber aktuell nur eine Vermutung.

Betroffen ist effektiv auch die Kombination revert_data + config.update.

[Kirk]

Marc hat die Änderung in der migrator.php wieder rückgängig gemacht: https://github.com/phpbb/phpbb/pull/6959/files
Diverse Tests haben ergeben das so keinerlei "Rückstände" bleiben.

[Crizzo]

Marc hat die Änderung in der migrator.php wieder rückgängig gemacht:

Das ist nicht ganz richtig. Der Pull Request würde das so machen, wenn er in der Art "gemerged" wird.

[chris1278]

Ich denke aber zumindest werden die das wohl mit dem nächsten updaste wieder korrigieren hoffe ich mal einfach. Egal in welche richtung hauptsache der fehler wird dann bereinigt.

[LukeWCS]

Der Pull Request würde das so machen, wenn er in der Art "gemerged" wird.

Also erstmal ein Zwischenstand. Aber dann siehts ja schon mal nicht schlecht aus.

Und den Migrator zu den Unit Tests hinzufügen ist prima, quasi als Prävention für künftige Änderungen am Migrator, weil das ist schon so ein kleines Biest. ^^

[LukeWCS]

Der Fix hat sich inzwischen weiter entwickelt und Udo und ich haben den neuesten Stand getestet mit positivem Ergebnis. Alle Details im zugehörigen PR:

[ticket/17650] Merge steps in try_revert and add test by marc1706 · Pull Request #6959 · phpbb/phpbb

Sobald das offiziell in die Codebase von 3.3.17 migriert wurde, passen wir das Thema [3.3] phpBB 3.3.16 - Wichtige Hinweise! an.

[LukeWCS]

Eine weitere Ext entdeckt die betroffen ist. Interessant ist hier, dass das Fehlerbild ganz anders ist, aber nach ersten Tests war es schnell klar, dass diese Ext exakt vom selben Migrator Bug betroffen ist.

Error when deleting data

Somit sind inzwischen 3 Migrator Konstellationen bekannt, die problematisch sind. Sobald ich alle Infos habe, aktualisiere ich den Tracker.