Für ein Event brauchst du natürlich den Namen.
Der fängt mit
core. an und beinhaltet dann meist den Namen der Funktion beziehungsweise bei Modulen den Namen der Datei. Dann irgendwie die Stelle, aber das weißt du sicherlich schon.
Dann musst du gucken, welche Variablen sinnvoll wären. Also welche kann man gut gebrauchen und bei welchen macht es Sinn, diese zu verändern? Ich bin ein Inklusionist und würde daher eher immer zu viele Variablen übergeben als zu wenige...
Und dann fehlt bloß noch ein fetter Kommentarblock davor, wie du ihn sicherlich schon gesehen hast. Dieser wird
erzwungen. Gibt es ihn nicht, werden die Tests auf GitHub fehlschlagen und dein Event gelangt nicht in den Core.
D@ve hat geschrieben:Ich hab das zwar noch nie probiert, aber wenn ich das im Code richtig sehe ganz einfach mit
Das kann ich mir eigentlich nicht vorstellen, dass es so einfach ist...
Doch. Zur Abwechslung ist es mal wirklich so einfach.
Das Array muss soweit ich weiß zwar immer
$vars heißen, aber dann ist es lediglich diese Zeile danach:
extract($phpbb_dispatcher->trigger_event('core.mein_event', compact($vars))); und der ganze Dokumentationsblock davor.
Mehr braucht es nicht für ein Event mit Variablen.
Zu beachten ist jedoch, dass ab 7 oder 8 übergebenen Variablen das Array mehrzeilig geschrieben werden
muss, also pro Zeile eine Variable. Auch das wird durch die Tests erzwungen.
Ein Event ohne zu übergebende Variablen geht natürlich übrigens auch und sieht nur geringfügig anders aus, aber ich denke nicht, dass das hier für irgendjemanden von Bedeutung sein wird.
Wenn die Tests dann alle erfolgreich sind werden Events in der Regel einfach so durchgewunken. Natürlich gucken die da noch, ob das sinnvoll ist oder dadurch Sicherheitslücken entstehen - das Übliche halt.
Aber sonst drücken die quasi einfach auf "Merge pull request" und das Thema ist durch.
