phpBB Ext Check - Diskussion bezüglich Prozedur und Reports

[LukeWCS]

Ergänzung: siehe meinen Edit im Beitrag von 11:50.

Es ist wohl derzeit nicht erforderlich die vorgeschlagene Änderung in den Dateien einzupflegen.

Ich denke darin besteht Konsens.

Die sonstigen Prüfungen von "Extensions .yml check" sind nach wie vor gültig und sollten beachtet werden, sofern das dafür notwendige Hintergrundwissen vorhanden ist, oder aber wenn es sich um Meldungen handelt, die problemlos umsetzbar sind, wie z.B. das Problem mit fehlenden einfachen Anführungszeichen. Bei phpBB Ext Check wird ebenfalls alles geprüft was "Extensions .yml check" auch prüft, inklusive weiterer Dinge und exklusive der "public" Prüfung. Solange es keine offizielle Verlautbarung seitens phpBB.com gibt, werde ich daran auch nichts ändern.

[SeewolfPK]

Ergänzung: siehe meinen Edit im Beitrag von 11:50

Dann habe ich vermutlich genau den gleichen Fehler gemacht. :D :grin:
Habe es jetzt mal bei dieser EXT ergänzt und es ist fehlerfrei.

[LukeWCS]

ping

Hey Karsten

Ich bin gerade dabei EC auf PHP 8 umzustellen. Das wäre prinzipiell relativ schmerzfrei, aber leider ist EPV mit PHP 8 inkompatibel. Darum muss ich EVP jetzt als isoliertes Composer Projekt laufen lassen, damit ich EPV alleine mit PHP 7.3 fahren kann. Und da hätte ich ein kleines Anliegen an dich, weil ich in Erinnerung hab, das du auch einen lokalen EPV eingerichtet hattest.

Könntest du mal bei Gelegenheit etwas testen? Normal sollte EPV auch mit PHP 7.4 laufen, aber in Kombination mit bestimmten Exts, ist eine Prüfung nicht möglich und EPV steigt aus mit einem mehrere MB grossen Fehler-Log.

Testbedingungen wären:

1. Ext "Teamspeak-Viewer": https://github.com/Tecwar/Teamspeak-Viewer_EXT
2. Windows Umgebung
3. PHP 7.4

Das Fehler-Log selber ist nicht relevant, die Ext auch nicht. Mir gehts nur darum, ob bei dir EPV das prüfen kann oder nicht. Wenn ich auf 7.3 schalte, packt EPV das problemlos.

[oxpus]

Ich habe EPV schon lange nicht mehr lokal im Einsatz, da ich nur noch über Deine Testsuite prüfe.
Sorry, aber hier kann ich aktuell nichts zu sagen.

[LukeWCS]

Alles klar Karsten, danke für Rückmeldung. Okay, ich belasse EPV dann einfach bei 7.3 um sicher zu gehen.

[oxpus]

Ich bekomme EPV noch nicht einmal neu installiert.
Das Paket scheint hoffnungslos veraltet zu sein.

[LukeWCS]

Das Paket scheint hoffnungslos veraltet zu sein.

Veraltet; ja. Hoffnungslos; ich hoffe nicht.

EPV ist nach wie vor eine der wichtigsten Prüfungen. Allerdings haben diejenigen Ext Coder die EPV lokal installieren wohl keine grosse Relevanz. Anders kann ich mir es nicht erklären, dass EPV 3 reproduzierbare Windows Bugs hat, bzw. hatte. Einen hab ich bereits 2019 gemeldet, der wurde auch rasend schnell von kasimi behoben, noch am selben Tag. Später fand ich 2 weitere Windows Bugs. Den ersten davon selber gefixt und den Fix auf Wunsch per PR eingereicht der seitdem vor sich hin rostet und nicht übernommen wird. Den dritten (ebenfalls selber gefixt) hab ich gar nicht erst gemeldet. Eine Verbesserung von Mike ist auch immer noch nicht übernommen worden, obwohl ebenfalls fixfertig als PR eingereicht.

[oxpus]

Und damit werden Extensions vorgeprüft.
Tztztz...

[LukeWCS]

Und damit werden Extensions vorgeprüft.
Tztztz...

Auf .com hat man damit ja keine Probleme, da läuft ja nicht Windows.

Die erwähnten Windows Bugs sind keine PHP Fehler die sich direkt und deutlich bemerkbar machen würden, in Form von FATALS oder WARNINGS. Das könnte man ja prima handhaben und dann wüsste man wenigstens das etwas nicht stimmt und wo.

Die Bugs sind viel perfider: sie unterdrücken schlicht die Prüfung bestimmter Eigenschaften einer Ext. Die Ursache für beide Bugs zu finden hat sehr viele Stunden gekostet. Durch Bug #2 werden gleich alle Prüfungen für die Sprachpakete "übersprungen". Durch Bug #3 wird eine spezifische Prüfung für die Migration übersprungen. Das heisst effektiv: EPV unter Windows wiegt den Benutzer u.U. in falscher Sicherheit, dass mit der geprüften Ext alles palletti ist. Lädt man dann die Ext in die CDB hoch, kann es passieren, dass man eine unschöne Überraschung erlebt.

[BNa]

Jetzt seien Sie mal nicht zu kritisch, Mr. Oxpus