Migrator Bug in 3.3.16

[Kirk]

Hallo
Beim testen meiner EXT Average Daily Posts and Topics on Index unter 3.3.16 ist mir aufgefallen dass wenn man die Arbeitsdaten löscht, das ACP Modul in der Modul-Konfiguration im ACP weiterhin vorhanden ist. Dieses Verhalten konnte ich bei WWH auch nachvollziehen.
Ich hab mir mal die Code Changes angesehen, da wurde in der migrator.php das hier geändert. Macht man diese Änderung rückgängig, dann wird das Modul auch korrekt entfernt. Des weiteren ist mir aufgefallen da ich jenes public function revert_data() auch in anderen EXTs drin habe, dort beim löschen der Arbeitsdaten nichts zurück bleibt. Nur wenn dieses oben genannte in einer migrations Datei vorhanden ist wo die ACP Module module.add erstellt werden, kommt es zu diesem Verhalten. Dies ist z. B. in dieser sowie bei WWH EXT der Fall.

[LukeWCS]

Ich bin noch nicht dazu gekommen das alles selber anzuschauen, aber hier mal vorab einer meiner Beiträge von unserem privaten Bastelforum als zusätzliche Info:

Okay, das würde heissen, dass das dort auftritt, wo Rechte manuell entfernt werden. Und genau das war mal irgendwann notwendig, weil der Migrator das nicht selbst machte. Das hatte ich ja mal festgestellt und hier gemeldet. Irgendwann konnte der Migrator das dann selbst, aber das ist ja eben trotzdem noch in unseren Exts vorhanden teilweise. Das hiesse also, das primär sehr alte Exts betroffen sind, die noch zu 3.1 und 3.2 Zeiten entstanden sind.

Nach meiner Einschätzung dürfte dieses Problem bei allen Exts auftreten, wo Rechte manuell per Migration entfernt werden. Mir ist aber noch der Zusammenhang mit den Modulen unklar. Da fehlt mir noch das sichten aller Infos und auch Tests.

edit: Aufgrund von Udos Erkenntnissen, habe ich den verantwortlichen Commit im phpBB Repo ermittelt:

[ticket/17533] Fix reverting migrations logic

[Kirk]

Diesbezüglich hab ich im Tracker noch jenes gefunden: https://tracker.phpbb.com/browse/PHPBB- ... lter=16890