[3.3] [CDB]Recent Topics NG

[IMC]

edit: Was RTNG angeht, wenns ohne grosse Umbauten ginge, würde ich auf assign_block_vars() verzichten wollen.

Kein Problem. Werde ich mit Umsetzen.

Editing arrays is a quite trivial task for any piece of code.

Ich habe auch noch S_FIRST_ROW und ähnliche Einträge im HTML-Code gefunden und durch Twig ersetzt. Ich bin mir fast sicher, dass wir jetzt zu 100 % Twig-Code haben.

ich hatte vor vielen Jahren mal das Problem, das ich bei einer Ext nachträglich ein Template Array auslesen

Kann ich nachvollziehen. Ich hatte vor kurzen die Methode retrieve_block_vars() getestet. Da hatte ich auch nicht das gewünschte Ergebnis bekommen.

[LukeWCS]

Ich habe auch noch S_FIRST_ROW und ähnliche Einträge im HTML-Code gefunden und durch Twig ersetzt.

Ahja okay, Twig hat für solche Abfragen ja eh native loop. Variablen.

Ich bin mir fast sicher, dass wir jetzt zu 100 % Twig-Code haben.

Und selbst wenn nicht; solange wir phpBB 3.3 haben, ist eh noch der Twig Konverter im Core, der die alte Syntax behandelt.

Ich muss mich da übrigens korrigieren:

Bei Exts schon lange nicht mehr gesehen, ich dachte wir hätten den ganzen alten Kram entsorgt.

Bei EMP hatte ich loops. bis 1.0.8 (2023) verwendet, dann entfernt. Seit 3.0.0 habe ich das aber versehentlich wieder drin, weil ich das Template mit dem Twig Converter vom original phpBB Template exportiert hatte. Dann habe ich das komplette Template bezüglich Twig modernisiert und an EMP angepasst und dabei dann offensichtlich dieses Detail übersehen. Heute entdeckt.

https://github.com/LukeWCS/ext-mgr-plus ... .html#L102

Und noch eine weitere Zeile.

Da hatte ich auch nicht das gewünschte Ergebnis bekommen.

Jupp, dann bist du ja im Boot. Es geht mit dieser Funktion, aber da muss vorher noch was anderes erledigt werden und selbst dann, muss man jedes Element einzeln extrahieren. Man kann also nicht das ganze Array auf einmal extrahieren. Ich sagte ja: umständlich.

[IMC]

Man kann also nicht das ganze Array auf einmal extrahieren. Ich sagte ja: umständlich.

Ja, bei mir kam nur das letzte Element vom Array. Die Beschreibung "empty array gets all vars" stimmte so nicht. War auch nur zum Testen. Hatte weiter keinen Grund.

assign_block_vars() werde ich in der RTNG Funktion fill_template() nicht weg bekommen. In der Schleife werden für größere Topics Paginationen erzeugt. Wenn es irgendwie funktioniert ist mit Sicherheit der Aufwand zu hoch.

[LukeWCS]

Ach da hängt die Pagination von phpBB dran? Okay, dann natürlich nicht, dann hat sich mein Einwand auch schon erledigt. Die Funktion tut ja, ich mag sie nur nicht. Eben aus den genannten Gründen.

Ich hab mich jetzt auch schon mal bezüglich meiner Änderungen bei RTNG "warm" gemacht, also gesichtet, wo ich was einbauen und ändern muss. Sobald du deine genannten Änderungen im Dev Branch hast, würde ich weitermachen. Damit wir uns nicht gegenseitig in die Parade fahren und wir nicht unnötig Merge Konflikte beheben müssen.

[IMC]

Ich babe meine Änderungen in den Dev Branch geschoben.

• PHP min aufgrund der Verwendung von str_starts_with() auf v8.0 geändert.
• „rtng_user_data” in Array zwischengespeicht um DB-Abfragen zu reduzieren.
• SQL-Abfrage für die Topic-Zählung verbessert, um eine bessere Leistung zu erzielen.
• assign_block_vars() in main_listener ersetzt.
• assign_block_vars() in page_controller ersetzt.
• phpBB Template-Code durch Twig-Syntax ersetzt.