phpBB.de

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"?

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.

Die Prüfroutine empfiehlt ja den Einbau von 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

Hi Paul,

defaults muss einen Unterstrich am Anfang habe.

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

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.

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.

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?

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

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

Ich habe es mal getestet. Hier die Fehlermeldung: Der Test bei der Extensions .yml Files Syntax Check meldet dann:

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