Email Benachrichtigung bei Kommentaren von Autoren
Re: Email Benachrichtigung bei Kommentaren von Autoren
Hi,
pull request habe ich erstellt. Ich höffe ich habs richtig gemacht
Danke nochmal.
Viele Grüße,
Robert
pull request habe ich erstellt. Ich höffe ich habs richtig gemacht
Danke nochmal.
Viele Grüße,
Robert
Re: Email Benachrichtigung bei Kommentaren von Autoren
Ja, hast du :)
Ich merke gerade das mir gar nicht klar war, von welcher Benachrichtigung wir reden. Und ich glaube – bin aber nicht wirklich sicher – das offenbart einen Fehler in deinem Patch. Das ist ja nicht eine Benachrichtigung von "Dein Kommentar wurde hinzugefügt", richtig? Das ist eine Benachrichtigung "Dein Artikel hat einen neuen Kommentar". Die natürlich wirklich unnötig ist, wenn der Kommentator der Autor ist, und vom Backend aus ist das garantiert. Aber das fehlt: Die Überprüfung, dass der Autor des Kommentars der Autor des Eintrags ist, auf den geantwortet wird.
Kann das sein?
Ich merke gerade das mir gar nicht klar war, von welcher Benachrichtigung wir reden. Und ich glaube – bin aber nicht wirklich sicher – das offenbart einen Fehler in deinem Patch. Das ist ja nicht eine Benachrichtigung von "Dein Kommentar wurde hinzugefügt", richtig? Das ist eine Benachrichtigung "Dein Artikel hat einen neuen Kommentar". Die natürlich wirklich unnötig ist, wenn der Kommentator der Autor ist, und vom Backend aus ist das garantiert. Aber das fehlt: Die Überprüfung, dass der Autor des Kommentars der Autor des Eintrags ist, auf den geantwortet wird.
Kann das sein?
Re: Email Benachrichtigung bei Kommentaren von Autoren
Ja genau es geht um die Mail "Neuer Kommentar für <Artikel>"
Verdammt Du hast vollkommen recht die Prüfung ob Kommentator = Autor habe ich vergessen. Ich habs bei mir bereits ergänzt. Muss ich einen neuen pull request erstellen oder kann man den bestehenden anpassen?
Verdammt Du hast vollkommen recht die Prüfung ob Kommentator = Autor habe ich vergessen. Ich habs bei mir bereits ergänzt. Muss ich einen neuen pull request erstellen oder kann man den bestehenden anpassen?
Re: Email Benachrichtigung bei Kommentaren von Autoren
Du kannst den bestehenden anpassen. Einfach in deinen fork commiten und nochmal auf den grünen Button klicken, Github führt dich da durch. Ich schau dann kurz drüber und würde es mergen.
Re: Email Benachrichtigung bei Kommentaren von Autoren
Okay habs im pull request geändert. Ich habe jetzt auf die Emailadresse geprüft, passt das oder hast du einen anderen Vorschlag?
Re: Email Benachrichtigung bei Kommentaren von Autoren
Ich hätte es über den Namen gemacht, aber email kann auch funktionieren. Würde es so belassen und später anpassen falls nötig.
Hast du meinen Kommentar gesehen? Da ist noch ein Fehler in der if-bedingung. Danach sollte es passen.
Hast du meinen Kommentar gesehen? Da ist noch ein Fehler in der if-bedingung. Danach sollte es passen.
Re: Email Benachrichtigung bei Kommentaren von Autoren
Ja hab ich gelesen. Ich verstehe was du meinst und ich passe es auch gerne an. Nur für mich zum Verständnis, technisch gesehen kommt doch, mal abgesehen von dem fehlenden Leerzeichen, bei
das selbe raus wie bei
oder habe ich einen Denkfehler?
Wenn $backend = true ergibt die Prüfung !$backend -> fals und wenn die Email Adressen unterschiedlich sind $email != $row['email'] -> true
false && true ergibt true und es wird eine Email gesendet, oder nicht?
Code: Select all
if(!$backend && $email != $row['email'])
Code: Select all
if (! ($backend && $email != $row['email']))
Wenn $backend = true ergibt die Prüfung !$backend -> fals und wenn die Email Adressen unterschiedlich sind $email != $row['email'] -> true
false && true ergibt true und es wird eine Email gesendet, oder nicht?
Re: Email Benachrichtigung bei Kommentaren von Autoren
Nehm an du bist im frontend. !$backend ist also wahr. Er geht zur nächsten Abfrage, und jetzt soll die email ja abgeschickt werden, egal welche Adresse das ist.
false && true ergibt false. false || true ergibt true. Ja, man könnte es als OR schreiben, also mit ||, aber das finde ich weniger klar.
false && true ergibt false. false || true ergibt true. Ja, man könnte es als OR schreiben, also mit ||, aber das finde ich weniger klar.
Re: Email Benachrichtigung bei Kommentaren von Autoren
Ja da hatte ich dann wirklich einen Denkfehler... Danke für die Aufklärung. Ich habs angepasst
Re: Email Benachrichtigung bei Kommentaren von Autoren
Wollen wir das ausweiten?
Wenn momentan ein eingeloggter Nutzer im Frontend auf einen eigenen Eintrag antwortet, kriegt er dann nicht auch eine Email?
Wenn momentan ein eingeloggter Nutzer im Frontend auf einen eigenen Eintrag antwortet, kriegt er dann nicht auch eine Email?
Re: Email Benachrichtigung bei Kommentaren von Autoren
Du meinst wenn Autor und Kommentator im Frontend ser selbe sind? Oder was meinst du mit einem eingeloggten Nutzer der auf einen eigenen Eintrag antwortet?
Oder meinst du Nutzer mit einer Subscription für neue Kommentare?
Oder meinst du Nutzer mit einer Subscription für neue Kommentare?
Re: Email Benachrichtigung bei Kommentaren von Autoren
Auch ne gute Idee! Wenn wir den klar identifizieren können (über die "Remember me?"-Funktion? Über die gleiche Email-Adresse?) wäre das eine weitere unnötige Mail, die wir weglassen können.Oder meinst du Nutzer mit einer Subscription für neue Kommentare?
Genau. Autor schreibt einen Eintrag, ist in s9y eingeloggt, schreibt einen Kommentar zu diesem Eintrag. Er würde momentan eine Mail bekommen, glaube ich. Das hatte ich sogar mal lokal gepatcht (für Bernd?), aber das ging glaube ich nie in den Core.Du meinst wenn Autor und Kommentator im Frontend ser selbe sind?
Was meinst du?
Re: Email Benachrichtigung bei Kommentaren von Autoren
Finde ich eine gute Idee, je weniger unnütze Mails umso besse
Können wir die Abfrage
nicht verallgemeinern und statt !$backend auf eine Session-Variable prüfen welcher Nutzer angemeldet ist? Dann spielt es gar keine Rolle ob aus dem Backend oder Frontend geantwortet wird.
Im selben Zug würde ich es ganz gut finden, wenn bei der Anmeldung im Frontend im Kontaktformular die Felder Name, URL und Email bereits automatisch vorausgefüllt werden. Im Backend stehen diese Werte bereits in der Form wenn man auf Antworten klickt.
Ausßerdem ist mit aufgefallen, dass ich im Backend immer zwei mal auf den Antwort-Button klicken muss... kannst du das bei dir nachstellen?
Können wir die Abfrage
Code: Select all
if (! ($backend && $email != $row['email']))
Im selben Zug würde ich es ganz gut finden, wenn bei der Anmeldung im Frontend im Kontaktformular die Felder Name, URL und Email bereits automatisch vorausgefüllt werden. Im Backend stehen diese Werte bereits in der Form wenn man auf Antworten klickt.
Ausßerdem ist mit aufgefallen, dass ich im Backend immer zwei mal auf den Antwort-Button klicken muss... kannst du das bei dir nachstellen?
Re: Email Benachrichtigung bei Kommentaren von Autoren
Ach wie gut, dass das außer mir noch jemanden stört, siehe:xoxyss wrote:Ausßerdem ist mit aufgefallen, dass ich im Backend immer zwei mal auf den Antwort-Button klicken muss... kannst du das bei dir nachstellen?
https://github.com/s9y/Serendipity/issues/387
Regards,
Ian
Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
Ian
Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
Re: Email Benachrichtigung bei Kommentaren von Autoren
Gut zu wissen dass es nicht nur bei mir auftritt also scheint es da ja ein generelles Problem zu geben