Page 3 of 3
Re: 2k11 und Kommentare vom Autor
Posted: Mon Jul 22, 2013 11:20 am
by 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)
Re: 2k11 und Kommentare vom Autor
Posted: Wed Jul 24, 2013 2:47 pm
by 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.
Re: 2k11 und Kommentare vom Autor
Posted: Thu Jul 25, 2013 9:22 am
by garvinhicking
Timbalu wrote: 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
Re: 2k11 und Kommentare vom Autor
Posted: Thu Jul 25, 2013 9:27 am
by Timbalu
Re: 2k11 und Kommentare vom Autor
Posted: Thu Jul 25, 2013 9:31 am
by garvinhicking
Japp, hast Du dir die Datei mal angeschaut?
Re: 2k11 und Kommentare vom Autor
Posted: Thu Jul 25, 2013 9:50 am
by 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!)
Re: 2k11 und Kommentare vom Autor
Posted: Thu Jul 25, 2013 10:14 am
by garvinhicking
Timbalu wrote: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...
Re: 2k11 und Kommentare vom Autor
Posted: Thu Jul 25, 2013 10:23 am
by Timbalu
Eben! Das ist Bedingung für mattsches commit, siehe 1.Teil!
Re: 2k11 und Kommentare vom Autor
Posted: Fri Jul 26, 2013 12:48 pm
by 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?
Re: 2k11 und Kommentare vom Autor
Posted: Fri Jul 26, 2013 1:16 pm
by garvinhicking
Ich Depp.
Verstanden!
Re: 2k11 und Kommentare vom Autor
Posted: Fri Jul 26, 2013 1:38 pm
by Timbalu
Re: 2k11 und Kommentare vom Autor
Posted: Fri Jul 26, 2013 1:47 pm
by yellowled
mattsches wrote: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
Re: 2k11 und Kommentare vom Autor
Posted: Fri Jul 26, 2013 2:57 pm
by 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)