[3.3] [Fork] Recent Topics NG

[IMC]

Hi Patrick,

in Recent Topics werden noch zwei weitere veraltete Extensions unterstützt.
https://github.com/stathis/phpbb31-topicprefixes
https://github.com/imkingdavid/prefixed

Ich bin mir sicher das es auch in deinem Sinne ist wenn ich die ebenfalls im Code entferne.

[LukeWCS]

Ich bin mir sicher das es auch in deinem Sinne ist wenn ich die ebenfalls im Code entferne.

Jupp. Je mehr Abhängigkeiten wir in der Codebase haben, um so mehr haben wir an der Backe. Die beiden sind zudem noch aus der 3.1 Ära, die werden vermutlich eh nicht ohne Anpassung unter 3.3 laufen.

[IMC]

Moin Patrick,

pl16 habe ich jetzt auf GitHub.

Ich habe das Event für die Datenübernahme bei der Registrierung geändert.
Die Unterstützung für folgende Extension entfernt.
- nickvergessen/newspage
- part3/topicprefixes
- imkingdavid/prefixed
Und die Mindestanzahl der anzuzeigenden Themen beim Spinbutton im ACP & UCP von 0 auf 1 gesetzt.

Der Datensatz rt_on_newspage in der Migration release_2_0_5.php ist damit ohne Funktion. Der kann dann später weggelassen werden.

[LukeWCS]

Moin Thorsten

Grad pl16 gesichtet, das hat sich gelohnt, da fiel so einiges weg. Und mir war gar nicht aufgefallen, das die Anzahl Themen mit 0 gespeichert werden konnte.

[IMC]

Hi Patrick,
wenn du weiter nichts hast würde ich morgen Abend ein neues Release erstellen und anschließend unser neues Repositorie erstellen. Mit ca. einer Woche Verzögerung schiebe ich das alte in ein Public archive.

[LukeWCS]

nAbend

Warte noch, bei den Twig Makros hab ich noch was. Ich komm aber erst morgen dazu.

[LukeWCS]

Thorsten, pl17 ist jetzt im Repo. Ich habe das vorgezogen, weil ich nicht weiss, ob ich die nächsten Tage dazu gekommen wäre.

[IMC]

Hi Patrick,

der Extension Pre Validator hatte eben folgende Fehlermeldung geworfen.
Fatal error: Can not find '@event' information for event

Dies tut dieser wenn das Code Layout bei einem Core-Event nicht eine bestimmte Form hat. Unter anderem auch wenn in dem Kommentarblock zu dem Event nach dem @event ein TAB statt eines Leerzeichen folgt.

In den Coding Guidelines habe ich dazu nichts gefunden. Ich werde mich Morgen damit weiter auseinandersetzen. Solange muss das Release noch warten.

[LukeWCS]

Hast du ein neues Event eingebaut oder wie?

Wegen eigene Events, jupp, das Problem kennen wir. EPV ist da extrem pingelig, da muss alles perfekt passen, inklusive Kommentar Block. Stöber mal im EC Thema, hier mal ein Startpunkt:

viewtopic.php?p=1401849&hilit=event#p1401849

Und dummerweise haut einem EPV irgendwelchen Nonsens (was Events angeht) um die Ohren, anstatt klipp und klar mitzuteilen, was denn genau krumm ist.

[IMC]

Hast du ein neues Event eingebaut oder wie?

Nein, ich will die vorhandenen Events in die Form bringen wie es in den von dir verlinkten Docs beschrieben ist. Ich habe dort nur nicht gesucht, weil ich dachte das meine Syntax i.O. ist und nur die Formatierung den Fehler hervorruft.

Deine verlinkte Beschreibung war wie immer Top und hat meine durch "try & error" erlangten Erkenntnisse bestätigt.

Tatsächlich würde ich in Zukunft 2 Events rausschmeißen wollen. Dazu stimme ich mich noch mit dir ab wenn wir in dem neuen Repo arbeiten.