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. ^^