phpBB Ext Check - Diskussion bezüglich Prozedur und Reports

[Mike-on-Tour]

Funzt wunderbar

Habe bei der Usermap mal die ungepackte js-Datei von leaflet unter usermap/thirdparty statt usermap/vendor gespeichert und die gepackte Version unter usermap/styles/all/template/thirdparty statt usermap/styles/all/template abgespeichert und jetzt meckert der PSSE auch nicht mehr, dass die leaflet.js "minified" sei und nicht gelesen werden kann.
Und nach der Anpassung der "INCLUDEJS"-Anweisungen funzt mit den neuen Verzeichnissen die Usermap wie gewohnt, also alles in Butter mit dem EC. Ein großes Danke an LukeWCS.

[LukeWCS]

Habe bei der Usermap mal die ungepackte js-Datei von leaflet unter usermap/thirdparty statt usermap/vendor gespeichert und die gepackte Version unter usermap/styles/all/template/thirdparty statt usermap/styles/all/template abgespeichert und jetzt meckert der PSSE auch nicht mehr, dass die leaflet.js "minified" sei und nicht gelesen werden kann.

Stimmt ja, an die Minify Version hab ich gar nicht mehr gedacht. Jupp, das bietet sich an, quasi wie bei Chris. Hat jetzt auch den Vorteil, das die Drittanbieter-Komponente quasi ihren eigenen Ordner bei Usermap hat. So kann man auch rein organisatorisch ein bisschen trennen.

Und nach der Anpassung der "INCLUDEJS"-Anweisungen funzt mit den neuen Verzeichnissen die Usermap wie gewohnt, also alles in Butter mit dem EC. Ein großes Danke an LukeWCS.

Das Feature fällt unter die gleiche Kategorie wie oxpus' Wunsch bezüglich PHPC-AUTO: das Coder Leben einfacher machen und sei es nur auf Seiten des EC Berichts. ^^

Danke für die Rückmeldung.

[oxpus]

Auch vom mir zunächst ein großes Danke für das Update.

Die Download Extension habe ich zunächst deutlich verschlankt - ich habe alle Vorschaubilder für prettyPhoto entfernt (ansonsten hätte ich die GA nicht mehr fehlerfrei ausführen können) - und den verbleibenden Part der Libary in den Ordner 3rdparty unter assets/ gepackt.
Also an die gleiche Stelle, wie auch im phpBB diverse JS-Ergänzungen bereitgehalten werden.

Der Test der Extension lief damit auch erwartungsgemäß fehlerfrei ab, also im gleichen Zuge wie vorher, eben halt auch mit der geänderten Paketstruktur.

Kurzum: Alles Prima, saubere Arbeit.

[LukeWCS]

Die Download Extension habe ich zunächst deutlich verschlankt - ich habe alle Vorschaubilder für prettyPhoto entfernt (ansonsten hätte ich die GA nicht mehr fehlerfrei ausführen können) - und den verbleibenden Part der Libary in den Ordner 3rdparty unter assets/ gepackt.
Also an die gleiche Stelle, wie auch im phpBB diverse JS-Ergänzungen bereitgehalten werden.

Habs mir grad auf Github angeschaut. Jupp, auch eine Variante. Was ist mit "GA" gemeint?

Danke für deine Rückmeldung.

[oxpus]

GA = GitHub Actions, also die Testsuite...

[LukeWCS]

GA = GitHub Actions, also die Testsuite...

Ah *patsch*... Kaffee wirkt nicht mehr. Zickt das bei Bildern rum oder wie?

[oxpus]

Ja, die Tests können die "ICC Profile" der Bilder nicht laden und dabei werden lediglich falsche Pfade verwendet (der Extension-Hauptpfad - hier ocpus/dlext/ - wird jeweils verdoppelt, was dann wiederum falsch ist).
Daher habe ich die großen Bilder, die lediglich für die enthaltene Beispielseite vorhanden sind, entfernt.
Dadurch wird die Extension als gepacktes Paket auch mal eben um rund 700 kb verkleinert (von 1003 kb runter auf 337 kb als 7z-File)

*hallo-wach-tropfen-reicht*

[LukeWCS]

Hmm, da ich keine Bilder in meinen Exts habe, kenne ich das Problem nicht. Das heisst GA selbst produziert den Fehler mit den verdoppelten Pfaden?

GA scheint schon 'ne Diva zu sein. Ich hatte erst vor ein paar Monaten ein weiteres Problem mit GA gemeldet, darauf hat bis heute keiner reagiert. https://www.phpbb.com/community/viewtop ... #p15706491

[oxpus]

Ja, die Fehler scheinen von der GA selber zu kommen oder das Repository wäre defekt, was ich mir aber nicht vorstellen kann.
Stört mich an dieser Stelle aber nicht weiter, da die besagten Bilder ja nicht benötigt werden.

[LukeWCS]

Ja, die Fehler scheinen von der GA selber zu kommen oder das Repository wäre defekt, was ich mir aber nicht vorstellen kann.

Hab jetzt zum testen einfach mal dein Repo Lokal bei mir geklont, das hat problemlos funktioniert. Repo scheint also okay. Das besagte Problem dürfte dann einfach eine weitere Macke von GA sein. Travis CI war robuster hab ich den Eindruck, diese beiden Probleme gabs da jedenfalls nicht.