From: Dessalines <tyhou13@gmx.com>
Date: Fri, 3 Jan 2020 18:12:19 +0000 (-0500)
Subject: Don't send email notification for self replies.
X-Git-Url: http://these/git/%7B%60/feeds/u/%7BpictshareAvatarThumbnail%28?a=commitdiff_plain;h=c252785632f91d5c819a723d6cfbbf0512e40c06;p=lemmy.git

Don't send email notification for self replies.

- Fixes #401
---

diff --git a/server/src/api/comment.rs b/server/src/api/comment.rs
index 5c3417dd..62759578 100644
--- a/server/src/api/comment.rs
+++ b/server/src/api/comment.rs
@@ -137,43 +137,47 @@ impl Perform<CommentResponse> for Oper<CreateComment> {
     match data.parent_id {
       Some(parent_id) => {
         let parent_comment = Comment::read(&conn, parent_id)?;
-        let parent_user = User_::read(&conn, parent_comment.creator_id)?;
-        if parent_user.send_notifications_to_email {
-          if let Some(comment_reply_email) = parent_user.email {
-            let subject = &format!(
-              "{} - Reply from {}",
-              Settings::get().hostname,
-              claims.username
-            );
-            let html = &format!(
-              "<h1>Comment Reply</h1><br><div>{} - {}</div><br><a href={}/inbox>inbox</a>",
-              claims.username, comment_form.content, hostname
-            );
-            match send_email(subject, &comment_reply_email, &parent_user.name, html) {
-              Ok(_o) => _o,
-              Err(e) => eprintln!("{}", e),
-            };
+        if parent_comment.creator_id != user_id {
+          let parent_user = User_::read(&conn, parent_comment.creator_id)?;
+          if parent_user.send_notifications_to_email {
+            if let Some(comment_reply_email) = parent_user.email {
+              let subject = &format!(
+                "{} - Reply from {}",
+                Settings::get().hostname,
+                claims.username
+              );
+              let html = &format!(
+                "<h1>Comment Reply</h1><br><div>{} - {}</div><br><a href={}/inbox>inbox</a>",
+                claims.username, comment_form.content, hostname
+              );
+              match send_email(subject, &comment_reply_email, &parent_user.name, html) {
+                Ok(_o) => _o,
+                Err(e) => eprintln!("{}", e),
+              };
+            }
           }
         }
       }
       // Its a post
       None => {
-        let parent_user = User_::read(&conn, post.creator_id)?;
-        if parent_user.send_notifications_to_email {
-          if let Some(post_reply_email) = parent_user.email {
-            let subject = &format!(
-              "{} - Reply from {}",
-              Settings::get().hostname,
-              claims.username
-            );
-            let html = &format!(
-              "<h1>Post Reply</h1><br><div>{} - {}</div><br><a href={}/inbox>inbox</a>",
-              claims.username, comment_form.content, hostname
-            );
-            match send_email(subject, &post_reply_email, &parent_user.name, html) {
-              Ok(_o) => _o,
-              Err(e) => eprintln!("{}", e),
-            };
+        if post.creator_id != user_id {
+          let parent_user = User_::read(&conn, post.creator_id)?;
+          if parent_user.send_notifications_to_email {
+            if let Some(post_reply_email) = parent_user.email {
+              let subject = &format!(
+                "{} - Reply from {}",
+                Settings::get().hostname,
+                claims.username
+              );
+              let html = &format!(
+                "<h1>Post Reply</h1><br><div>{} - {}</div><br><a href={}/inbox>inbox</a>",
+                claims.username, comment_form.content, hostname
+              );
+              match send_email(subject, &post_reply_email, &parent_user.name, html) {
+                Ok(_o) => _o,
+                Err(e) => eprintln!("{}", e),
+              };
+            }
           }
         }
       }