Serendipity Forums

Hi,
pull request habe ich erstellt. Ich höffe ich habs richtig gemacht

Danke nochmal.

Viele Grüße,
Robert

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?

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?

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.

Okay habs im pull request geändert. Ich habe jetzt auf die Emailadresse geprüft, passt das oder hast du einen anderen Vorschlag?

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.

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?

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.

Ja da hatte ich dann wirklich einen Denkfehler... Danke für die Aufklärung. Ich habs angepasst

Wollen wir das ausweiten?

Wenn momentan ein eingeloggter Nutzer im Frontend auf einen eigenen Eintrag antwortet, kriegt er dann nicht auch eine Email?

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?

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.

Du meinst wenn Autor und Kommentator im Frontend ser selbe sind?

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.

Was meinst du?

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?

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?

Ach wie gut, dass das außer mir noch jemanden stört, siehe:
https://github.com/s9y/Serendipity/issues/387

Gut zu wissen dass es nicht nur bei mir auftritt also scheint es da ja ein generelles Problem zu geben