phpBB Ext Check - Diskussion bezüglich Prozedur und Reports

In diesem Forum gibt es Starthilfe zum neuen Extension-System von phpBB 3.1/3.2. Fragen zur Entwicklung von Extensions und zur Konvertierung von phpBB 3.0.x MODs sind ebenfalls willkommen.
Benutzeravatar
Mike-on-Tour
Supporter
Supporter
Beiträge: 1063
Registriert: 13.01.2020 21:09
Kontaktdaten:

Re: phpBB Ext Check - Diskussion bezüglich Prozedur und Reports

Beitrag von Mike-on-Tour »

LukeWCS hat geschrieben: 21.11.2022 18:18 Dann frage ich mich, warum diese Prüfung überhaupt eingebaut wurde. Denn sie hat ja wohl weder jetzt noch zukünftig Relevanz.
Vielleicht nach dem Motto:"Lieber eine Prüfung zu viel als eine zu wenig"?
Benutzeravatar
LukeWCS
Supporter
Supporter
Beiträge: 2091
Registriert: 15.12.2014 10:19
Kontaktdaten:

Re: phpBB Ext Check - Diskussion bezüglich Prozedur und Reports

Beitrag von LukeWCS »

Mike-on-Tour hat geschrieben: 21.11.2022 19:16 Vielleicht nach dem Motto:"Lieber eine Prüfung zu viel als eine zu wenig"?
Wäre denkbar. :D
Möge das Backup mit dir sein. Immer.

Erweiterungen - Infos zur artgerechten Haltung
phpBB Ext Check - Analysesystem für phpBB Erweiterungen (Entwickler Werkzeug)
Benutzeravatar
SeewolfPK
Mitglied
Beiträge: 408
Registriert: 11.05.2010 21:45
Kontaktdaten:

Re: phpBB Ext Check - Diskussion bezüglich Prozedur und Reports

Beitrag von SeewolfPK »

Die Prüfroutine empfiehlt ja den Einbau von

Code: Alles auswählen

defaults:
        public: true
Wenn ich das z.B. bei Mikes mot/usermap oder Thanks for posts gfksx/thanksforposts
einbaue, dann bekomme ich einen Systemabsturz.

Das ist äußerst tückisch, wenn man nicht weiß, wie man da wieder raus kommt :D
Gruß Paul (Freunde nennen mich auch Paulchen Panther) https://www.reisemobiltreff.de https://www.seewolfpk.de
Benutzeravatar
IMC
Mitglied
Beiträge: 538
Registriert: 25.11.2018 20:32
Wohnort: Lüneburg
Kontaktdaten:

Re: phpBB Ext Check - Diskussion bezüglich Prozedur und Reports

Beitrag von IMC »

Hi Paul,

defaults muss einen Unterstrich am Anfang habe.

Code: Alles auswählen

   _defaults:
        public: true
Gruß, Thorsten
Benutzeravatar
LukeWCS
Supporter
Supporter
Beiträge: 2091
Registriert: 15.12.2014 10:19
Kontaktdaten:

Re: phpBB Ext Check - Diskussion bezüglich Prozedur und Reports

Beitrag von LukeWCS »

SeewolfPK hat geschrieben: 22.11.2022 09:36 Wenn ich das z.B. bei Mikes mot/usermap oder Thanks for posts gfksx/thanksforposts
einbaue, dann bekomme ich einen Systemabsturz.
Sprich ein FATAL.

Danke für den Hinweis Seewolf. Ich hatte das Thema nach den Antworten von Mike schon wieder abgehakt gehabt und mich nicht weiter damit befasst, da ich die Relevanz nicht erkennen kann, die das Ganze haben soll. Offensichtlich fehlen mir hier Hintergrundinfos und solange mir die fehlen, werde ich bei unserem phpBB33YAMLcheck auch nichts ändern. Und darum ging es mir halt, ob wir bei phpBB Ext Check nachrüsten müssen.
Das ist äußerst tückisch, wenn man nicht weiß, wie man da wieder raus kommt :D
Jupp. Dazu kommt noch, das die Ext von david63 explizit für Endbenutzer konzipiert wurde. Und als ein solches Tool sollte es keinerlei Dinge prüfen und melden, die beim Endbenutzer für Irritationen sorgen können, wie wir ja jetzt sehen. Nicht nur du bist verunsichert, auch wir als Ext Coder sind es. Ich zumindest. :wink:

Hier bei phpBB Ext Check geht es ja um ein hochgradig spezialisiertes Analysetool für Entwickler. Das heisst der Bericht wird von Leuten ausgewertet (und bewertet), die auch die nötigen Kenntnisse dazu haben. Aber selbst da überlege ich mir immer sehr genau was ich prüfen lasse und wie ich das Ergebnis präsentiere.

Ein Analysetool soll Klarheit schaffen was nicht passt, aber nicht Verwirrung stiften. Ich vermute dem neuen zuständigen Coder ist nicht bewusst gewesen, welche Zielgruppe bei davids Tool adressiert wurde.
Möge das Backup mit dir sein. Immer.

Erweiterungen - Infos zur artgerechten Haltung
phpBB Ext Check - Analysesystem für phpBB Erweiterungen (Entwickler Werkzeug)
Benutzeravatar
SeewolfPK
Mitglied
Beiträge: 408
Registriert: 11.05.2010 21:45
Kontaktdaten:

Re: phpBB Ext Check - Diskussion bezüglich Prozedur und Reports

Beitrag von SeewolfPK »

Das schlimme ist, die sehen das nicht einmal ein.
David selber antwortete mir
Post by david63 » 22 Nov 2022 11:24
Seewolfpk wrote: ↑22 Nov 2022 09:45
Please fix the checkup routine.
There is nothing to fix.
First of all anything that is reported by this extension is only advisory - it is up to you whether you apply the changes or not.
Secondly if the change is applied correctly, and the cache purged after making the changes, then it will not "shoot down your whole board".
Lastly this will be a requirement in a future version of Symfony.

---
PS: phpBB33YAMLcheck wo gibt es den?
Gruß Paul (Freunde nennen mich auch Paulchen Panther) https://www.reisemobiltreff.de https://www.seewolfpk.de
Benutzeravatar
LukeWCS
Supporter
Supporter
Beiträge: 2091
Registriert: 15.12.2014 10:19
Kontaktdaten:

Re: phpBB Ext Check - Diskussion bezüglich Prozedur und Reports

Beitrag von LukeWCS »

IMC hat geschrieben: 22.11.2022 12:18 defaults muss einen Unterstrich am Anfang habe.

Code: Alles auswählen

   _defaults:
        public: true
Das mit dem Unterstrich Ist mir nicht aufgefallen. :oops: Dann gibts keine Fehlermeldung?
SeewolfPK hat geschrieben: 22.11.2022 12:28 Das schlimme ist, die sehen das nicht einmal ein.
Keine Überraschung. Ich hatte in den vergangenen Jahren selbst schon mehrere Male das Vergnügen bei verschiedenen Leuten, wenn ich zu einem Thema spezifischer nachgehakt habe.

Mit ein Grund warum ich hier bei uns was eigenes haben wollte. phpBB Ext Check ist für Entwickler geschrieben worden, aber nicht nur für die routinemäßigen (beruflichen) Coder, sondern auch für die weniger versierten Coder, weil gerade die Unterstützung brauchen können bei der Entwicklung. Ich für meinen Teil wäre 2018 - als ich mit Ext Programmierung anfing - überglücklich gewesen, wenn es da phpBB Ext Check schon gegeben hätte.
Post by david63 » 22 Nov 2022 11:24
Lastly this will be a requirement in a future version of Symfony.
Das bestätigt meine bereits geäusserte Vermutung:
LukeWCS hat geschrieben: 21.11.2022 12:33 Das heisst für mich wiederum, dass es hier nicht um eine gewollte Änderung bei phpBB geht, sondern um eine notwendige Änderung bei Symfony.
PS: phpBB33YAMLcheck wo gibt es den?
Ist nicht öffentlich verfügbar. Dabei handelt es sich um ein reines CLI Tool ohne jegliche GUI. Es ist ein Modul von phpBB Ext Check. Hier im Startbeitrag gibts 2 Demo Berichte ganz am Anfang, da siehst auch wie phpBB33YAMLcheck Fehler meldet innerhalb eines Berichts.
Möge das Backup mit dir sein. Immer.

Erweiterungen - Infos zur artgerechten Haltung
phpBB Ext Check - Analysesystem für phpBB Erweiterungen (Entwickler Werkzeug)
Benutzeravatar
SeewolfPK
Mitglied
Beiträge: 408
Registriert: 11.05.2010 21:45
Kontaktdaten:

Re: phpBB Ext Check - Diskussion bezüglich Prozedur und Reports

Beitrag von SeewolfPK »

LukeWCS hat geschrieben: 22.11.2022 13:22 Das mit dem Unterstrich Ist mir nicht aufgefallen. Dann gibts keine Fehlermeldung?
Doch, den Unterstrich habe ich immer drin gehabt,

Die EXT bietet ja sogar Copy an. Habe mich weiter oben verschrieben.
_defaults:
public: true

Copy to clipboard
Fehler kommt trotzdem. Es darf halt

_defaults:
public: true

nicht eingebaut werden.

Ist ja auch nur eine Empfehlung laut David´s Antwort. Hihihi
Gruß Paul (Freunde nennen mich auch Paulchen Panther) https://www.reisemobiltreff.de https://www.seewolfpk.de
Benutzeravatar
IMC
Mitglied
Beiträge: 538
Registriert: 25.11.2018 20:32
Wohnort: Lüneburg
Kontaktdaten:

Re: phpBB Ext Check - Diskussion bezüglich Prozedur und Reports

Beitrag von IMC »

Ich habe es mal getestet. Hier die Fehlermeldung:

Code: Alles auswählen

Fehler: Attribute "public" on service "gfksx.thanksforposts.notification.type.thanks" cannot be inherited from "_defaults" when a "parent" is set.
Move your child definitions to a separate file or define this attribute explicitly.
Der Test bei der Extensions .yml Files Syntax Check meldet dann: :wink:

Code: Alles auswählen

The routing.yml file is valid
The services.yml file is valid
The tables.yml file is valid 
Gruß, Thorsten
Benutzeravatar
LukeWCS
Supporter
Supporter
Beiträge: 2091
Registriert: 15.12.2014 10:19
Kontaktdaten:

Re: phpBB Ext Check - Diskussion bezüglich Prozedur und Reports

Beitrag von LukeWCS »

SeewolfPK hat geschrieben: 22.11.2022 12:28 David selber antwortete mir
Apropos, wo hat david63 das geschrieben gehabt, was du zitiert hast? Gibts da vielleicht weitere Infos die interessant sind?
IMC hat geschrieben: 22.11.2022 17:31 Ich habe es mal getestet. Hier die Fehlermeldung:
In dem Zusammenhang, gerade entdeckt:

https://www.phpbb.com/community/viewtop ... #p15883611
Möge das Backup mit dir sein. Immer.

Erweiterungen - Infos zur artgerechten Haltung
phpBB Ext Check - Analysesystem für phpBB Erweiterungen (Entwickler Werkzeug)
Antworten

Zurück zu „Extension Bastelstube“