2k11 und Kommentare vom Autor

[Timbalu]

Mattsches Lösung für zumindest diese eine Variable halte ich für pragmatisch und simpel, ich denke das könnte man schon gut einbasteln.

Stimmt! Jetzt habe ich das "// <--- NEU!!" in der query erst entdeckt... (- vor allem da wir auf pragmatisch und simpel stehen )
Hast du, Mattsches, nicht sowieso gerade die ID in 2.0 mit eingepflegt? (mir war so als ob ich sowas in den commits gelesen hätte)

[Timbalu]

Mattsches, willst du das nicht in die master (für upcoming 1.7.1) und in 2.0 committen?
Ich habe es gerade ausprobiert und es klappt wie gewünscht!
Dann müsste man nur noch suchen welches Template die comments_by_author bereitstellt und alle für das self... ändern.

[garvinhicking]

Mattsches, willst du das nicht in die master (für upcoming 1.7.1) und in 2.0 committen?
Ich habe es gerade ausprobiert und es klappt wie gewünscht!
Dann müsste man nur noch suchen welches Template die comments_by_author bereitstellt und alle für das self... ändern.

In der 1.7 release gibts keine comments_by_author.tpl die diese Variable abfragt...?! Wäre also dafür gar nicht relevant?

Grüße,
Garvin

[Timbalu]

https://github.com/s9y/Serendipity/blob ... author.tpl
Nicht?

[garvinhicking]

https://github.com/s9y/Serendipity/blob ... author.tpl
Nicht?

Japp, hast Du dir die Datei mal angeschaut?

[Timbalu]

Ja - denn ich habe ja getestet! und deshalb schrieb ich auch "...und alle für das self... ändern." Das wäre ja eine Kleinigkeit und würde eine Verbesserung bringen, auch wenn wohl kaum jemand diesen Kniff wirklich nutzt! (Dein Einwand würde auch für 2.0 gelten!)

[garvinhicking]

Ja - denn ich habe ja getestet! und deshalb schrieb ich auch "...und alle für das self... ändern." Das wäre ja eine Kleinigkeit und würde eine Verbesserung bringen, auch wenn wohl kaum jemand diesen Kniff wirklich nutzt! (Dein Einwand würde auch für 2.0 gelten!)

In der Datei kommt kein "self" vor...

[Timbalu]

Eben! Das ist Bedingung für mattsches commit, siehe 1.Teil!

[mattsches]

Also, ich habe gerade die Änderungen in der functions_comments.inc.php und in der 2k11/comments_by_author.tpl im 2.0-Branch und im master gepusht.

Für alle anderen Templates macht das erstmal keinen Unterschied, die können wir dann nach und nach anpassen, oder?

[garvinhicking]

Ich Depp. Verstanden!

[Timbalu]

[yellowled]

Für alle anderen Templates macht das erstmal keinen Unterschied, die können wir dann nach und nach anpassen, oder?

Code: Select all

matthias at yl-mba in ~/GitHub
$ find Serendipity -name comments_by_author.tpl
Serendipity/templates/2k11/comments_by_author.tpl
Serendipity/templates/default/comments_by_author.tpl
matthias at yl-mba in ~/GitHub
$ find additional_themes -name comments_by_author.tpl
matthias at yl-mba in ~/GitHub

Ich glaube, das wird nicht sooo lange dauern.

YL

[Timbalu]

Es wäre doch komisch, wenn jetzt das /default/comments_by_author.tpl auf irgendetwas warten müsste... so jedenfalls verstehe ich diesen hint... (gerade auch weil es das fallback tpl ist)