[3.3] [CDB]Recent Topics NG

[LukeWCS]

Alles klar. Ich muss dann auch noch die neueste Version vorher testen, also dev21. Ich schau, dass ich das noch vor dem WE erledige und dir Bescheid gebe. Ich habe im Moment zuviele Baustellen, ich komm auch bei meinen eigenen Exts nicht rund.

[IMC]

Alles klar. Ich muss dann auch noch die neueste Version vorher testen, also dev21. Ich schau, dass ich das noch vor dem WE erledige

Kein Stress. Sonntag ist nicht fix. Später ist kein Problem. Ich warte deine Rückmeldung ab.

[LukeWCS]

Hey Thorsten

Hab dev21 gesichtet, die Erklärung macht sich prima, so hat der Admin ein paar Fragezeichen weniger.

Nur die ausgelagerte Serverlast Option finde ich nicht so ideal. Die Option passt zwar thematisch einwandfrei wo du sie platziert hast, aber ich dachte eher an eine eigene Options-Gruppe "Serverlast" in unserem eigenen ACP Modul, damit alles zusammen ist. Einstellungen einer Ext im ACP zu verteilen ist - aus meiner Sicht - unübersichtlich/umständlich und aufwendiger zu erklären bei Anfragen. Besser man findet alle ACP Einstellungen zu RTNG an einer Stelle.

Zumal dann auch die technische Realisierung einheitlich wäre, nämlich Template-basiert. Denn so wie bei dev21 hat man zwei Einstellungs-Modelle; Template und Listener, weil die Serverlast Option ja erst zur Laufzeit hinzugefügt wird. Was auch weitere Nachteile hat, wie z.B. fehlende Möglichkeit der direkten Adressierung durch Link und Anker. Eigene Module geben einem alle Möglichkeiten, aber Laufzeit-basierte Injektion von Einstellungen schränkt die Freiheit deutlich ein. Das wurde aus meiner Sicht seitens phpBB nicht so ganz durchdacht.

edit: Wenns auf der gleichen Seite wäre, könntest den Link zur Serverlast Gruppe sogar direkt realisieren, also mit Anker.

[IMC]

Nur die ausgelagerte Serverlast Option finde ich nicht so ideal. Die Option passt zwar thematisch einwandfrei wo du sie platziert hast ... Besser man findet alle ACP Einstellungen zu RTNG an einer Stelle.

Moin Patrick,

Ich würde die Einstellung in "Serverlast" belassen. Kann sie aber zusätzlich in den ACP Einstellungen von RTNG hinzufügen.

Um Benutzerfreundlich zu bleiben wird es jedoch etwas aufwendiger. Zur Zeit wird die Einstellung "Link des Thementitels zu ungelesenen Beiträgen" mit TWIG ausgeblendet, wenn diese nicht verfügbar ist. Dies müsste ich mit jQuery realisieren. Ich werde diese dann nicht ausblenden sondern auf disable setzen. Wenn dies nicht mit JS gelöst werden würde müsste der Admin das Formular zweimal senden um die Einstellung zu setzen.
Einmal um die Einstellung mit der Serverlast Einstellung verfügbar zu machen und ein zweitesmal um diese Einstellung zu setzen.

Dann gibt es im laufe der Woche einen dev22.

[LukeWCS]

Moin

Nee, Teile redundant anlegen ist unschön, sowas versuchen wir ja gerade zu vermeiden. Dann lass es wie es ist.

Okay, von meiner Seite kann Release 1.1.0 raus.

[IMC]

Nee, Teile redundant anlegen ist unschön, ... Dann lass es wie es ist.

Gut. Weil auch noch die Einstellung Serverseitige Gelesen-Markierung aktivieren in den Einstellungen zur Serverlast übergeordnet aktiv sein muss.
Edit: Das füge ich den erklärenden Text, in der Einstellung, in der Serverlast noch hinzu.

Das Release 1.1.0 mache heute abend fertig.

[LukeWCS]

Alles klar, dann fang ich heute auch mal mit meinen alljährlichen Updates an.

[IMC]

Version 1.1.0 wurde validiert. Download in der Customisation Database von phpBB.com.
Link im Startbeitrag.

[LukeWCS]

Servus Thorsten

Die Entwicklung von TC 1.3 und die Diskussion nebenan, werden auch kleinere Optimierungen bei RTNG bezüglich TC Schnittstelle bewirken:

• Der Twig Code im ACP Modul kann etwas abgespeckt werden, da es nicht mehr nötig ist, die TC Variable bei jedem einzelnen Makroaufruf per Parameter zu übergeben. Möglich wurde das durch die Möglichkeit eine echte globale Template Variable generieren zu können. Das ist eine Twig Eigenschaft die es schon seit Twig 1 gibt, die bei phpBB jedoch nicht "offiziell" zur Verfügung steht. Mir ist es dennoch gelungen, diese mit den Mitteln (Ext Strukturen) von phpBB nutzbar zu machen. Dadurch kann das Makro alleine die komplette Twig Steuerung bezüglich TC erledigen, zumindest im ACP Modul. So wollte ich das Makro eigentlich von Anfang an haben, aber dafür gab es bis jetzt technisch keine Möglichkeit.
• Dann würde auch die Generierung und Entfernung (je nach Modul) der Template Variable TOGGLECTRL_RTNG per PHP komplett entfallen. Das war eh nicht ganz ideal, da die Benennung der Variable vermuten lassen könnte, dass sie von TC generiert wird, was ja nicht der Fall ist. Das wird möglich, weil ich bei Twig eine weitere Eigenschaft entdeckt habe, die eine extra Template Variable in dem Fall obsolet machen kann. Das heisst es entfallen in 2 PHP Dateien je eine Zeile. Dafür sind dann 2 kleine Änderungen jeweils im ACP und UCP Modul notwendig. Die Lösung ist etwas eleganter, da sie komplett im Template stattfindet und somit die Implementierung der TC Schnittstelle mit purem Twig Code realisiert wird.

Ich würde auch die nächste Entwickler Version als dev/x.y.z Zweig oder simpel als dev/dev anlegen, damit schon unsere Entwickler Versionen direkt mit dem GH Test Framework geprüft werden. Für uns nicht zwingend, da wir eh EC nutzen, aber das Framework ist ja bei RTNG eingerichtet, also kann man das auch nutzen und das behindert ja unsere Arbeit nicht.

[IMC]

Hi Patrick,

da schein eine sehr Interessante Lösung zu sein, die man sicherlich an einigen anderen Stellen nutzen kann.

Das Problems mit der Extension "Separate Announcements and Stickies" ist jetzt auch gelöst. Stroker hat die, mit RTNG, kompatible Version 1.0.2 veröffentlicht.

Dieses Ereignis hat mich veranlasst die betroffene Codestelle in RTNG genauer zu betrachten. Ich habe unsere Methode fill_template() mehr an den Ablauf in der viewforum.php angpasst. Das heist ich habe ein Event hinzugefügt und ein Event um 2 Variablen erweitert. In dem Zusammenhang habe ich die Templatevariablen parent_forums als Array in das $tpl_ary gepackt. So hat man über das Event Zugriff auf alle Templatevariablen.
Zusätzlich habe ich den Code für die URL-Parameter der Pagination schlanker gestaltet.

Ich werde die Änderungen im Anschluss an deine Änderungen hochladen.