phpBB.de - Die deutsche phpBB-Community
https://www.phpbb.de/community/
Code: Alles auswählen
[phpBB Debug] PHP Notice: in file [ROOT]/ext/imcger/recenttopicsng/event/main_listener.php on line 107: Only variables should be passed by referenceend(). Der passende Doku Abschnitt:Anders ausgedrückt: die Funktion liefert nicht nur das letzte Array Element, sondern manipuliert auch den Array Pointer. Letzteres kann aber nur funktionieren, wenn das Array auch als dediziertes Datenobjekt (sprich Variable) vorhanden ist. Direkt auf eine Funktions-Rückgabe anwenden geht nicht.Das Array. Das Array wird per Referenz übergeben, weil es von der Funktion bearbeitet wird. Das bedeutet, dass der Funktion eine echte Variable übergeben werden muss und nicht beispielsweise der Rückgabewert einer Funktion.
Code: Alles auswählen
$session_page = explode('/', $event['row']['session_page']);
$site = $session_page[array_key_last($session_page)];
end() ist eleganter und der Pointer kann uns dabei auch nicht in die Quere kommen, weil wir ihn gar nicht brauchen.
Ich habe das bei mir nochmal getestet um zu sehen ob mir da etwas durch gegangen ist. Einstellung in meineLukeWCS hat geschrieben: 10.06.2025 16:13 Das funktioniert zwar, triggert aber ein NOTICE, ... Dabei wird der Pointer nicht verändert.
php.ini E_ALL, die php Version 8.4.7. Ich habe keine Meldung im Error-Log-File. Ein guter Anlass mal meinen Einstellungen zu testen. Damit ich die Notize auch bekomme.Mache ich fertig wenn du alles hochgeladen hast.Ich mach heute Abend nochmal ein Update mit dem hier Besprochenen und dann kannst du dich ans Release machen.
Finde ich gut. Ich habe mich damit noch nicht im Detail auseinander gesetzt. Ein guter Anlass mir das dann genauer anzuschauen.würde ich auch bei RNTG das offizielle phpBB GitHub Actions Framework einrichten
Im Log habe ich das auch nicht, sondern als direkte Anzeige in phpBB. Dazu muss aber der Debug Modus von phpBB aktiv sein, da sonst NOTICE Meldungen unterdrückt werden.IMC hat geschrieben: 10.06.2025 18:41 Einstellung in meinephp.iniE_ALL, die php Version 8.4.7. Ich habe keine Meldung im Error-Log-File.
Falls du damit den Code Block meinst aus meinem vorletzten Beitrag; das ist nur eine Alternative zu meiner tatsächlichen Lösung die ich erst noch hochladen muss. Die entspricht dann genau deinem Code, nur minimal anders notiert.Deinen Änderungsvorschlag finde ich gut.
Das Framework in der zweiten Generation ist sehr einfach einzubauen. Und schon die erste Generation war erheblich einfacher einzubauen als noch das ganz alte System "Travis CI". Das war vielleicht ein Krampf, weil man da z.B. auch in der Shell erst ein Skript startfähig machen musste und ähnliche Scherze. Das war wirklich nur was für Nerds und alles andere als Praxis-orientiert.Finde ich gut. Ich habe mich damit noch nicht im Detail auseinander gesetzt. Ein guter Anlass mir das dann genauer anzuschauen.
Das ist noch besser. Ist wohl auch nur eine Zeile mehr. Das werde ich dann sehen.LukeWCS hat geschrieben: 10.06.2025 19:50 Die entspricht dann genau deinem Code, nur minimal anders notiert.
Jupp, MattF hat das deutlich abgespeckt was von unserer Seite vorhanden sein muss. Die erste Generation war da noch erheblich unübersichtlicher mit mehr Stolperfallen.IMC hat geschrieben: 10.06.2025 21:01 GH Actions sieht nicht so kompliziert aus wie ich dachte. Da hat sich einiges verbessert. Wie du auch schon schriebst.
tests.yml gepusht, ich hatte da noch eine ältere Fassung. MattF hatte das zwischenzeitlich aber noch weiter entschlackt und zusätzliche Kommentare eingebaut. Diese aktuelle Fassung habe ich zwar schon hier im parallelen GH Actions Thema, aber die habe ich noch nicht bei meinen Exts und von EMP hatte ich das für RTNG kopiert. In beiden Branches von RTNG ist jetzt aber die aktuellste Fassung.Alles klar.Die NOTICE Meldung konnte nachstellen und in einer weiteren Extension beseitigen.