Fix Camper News email notification logic

controllers/story.js

@@ -155,7 +155,7 @@ exports.returnIndividualStory = function(req, res, next) {
       title: story.headline,
       link: story.link,
       originalStoryLink: dashedName,
-      originalStoryAuthorEmail: story.author.email || "",
+      originalStoryAuthorEmail: story.author.email || '',
       author: story.author,
       description: story.description,
       rank: story.upVotes.length,
@@ -398,7 +398,7 @@ exports.storySubmission = function(req, res, next) {
     var comment = new Comment({
       associatedPost: data.associatedPost,
       originalStoryLink: data.originalStoryLink,
-      originalStoryAuthorEmail: req.user.email,
+      originalStoryAuthorEmail: data.originalStoryAuthorEmail,
       body: sanitizedBody,
       rank: 0,
       upvotes: 0,
@@ -496,30 +496,28 @@ exports.storySubmission = function(req, res, next) {
         return next(err);
       }
       try {
-        Context.find({'_id': comment.associatedPost}, function (err, associatedStory) {
+        // Based on the context retrieve the parent object of the comment (Story/Comment)
+        Context.find({'_id': data.associatedPost}, function (err, associatedContext) {
           if (err) {
             return next(err);
           }
-          associatedStory = associatedStory.pop();
+          associatedContext = associatedContext.pop();
-          if (associatedStory) {
+          if (associatedContext) {
-            associatedStory.comments.push(data._id);
+            associatedContext.comments.push(data._id);
-            associatedStory.save(function (err) {
+            associatedContext.save(function (err) {
               if (err) {
                 return next(err);
               }
               res.send(true);
             });
           }
-          User.findOne({'profile.username': associatedStory.author.username}, function(err, recipient) {
+          // Find the author of the parent object
+          User.findOne({'profile.username': associatedContext.author.username}, function(err, recipient) {
             if (err) {
               return next(err);
             }
-            var recipients = '';
+            // If the emails of both authors differ, only then proceed with email notification
-            if (data.originalStoryAuthorEmail && (data.originalStoryAuthorEmail !== recipient.email)) {
+            if (data.author.email && (data.author.email !== recipient.email)) {
-              recipients = data.originalStoryAuthorEmail + ',' + recipient.email;
-            } else {
-              recipients = recipient.email;
-            }
               var transporter = nodemailer.createTransport({
                 service: 'Mandrill',
                 auth: {
@@ -527,22 +525,25 @@ exports.storySubmission = function(req, res, next) {
                   pass: secrets.mandrill.password
                 }
               });
+
               var mailOptions = {
-              to: recipients,
+                to: recipient.email,
                 from: 'Team@freecodecamp.com',
-              subject: associatedStory.author.username + " replied to your post on Camper News",
+                subject: data.author.username + ' replied to your post on Camper News',
                 text: [
-                "Just a quick heads-up: " + associatedStory.author.username + " replied to you on Camper News.",
+                  'Just a quick heads-up: ' + data.author.username + ' replied to you on Camper News.',
-                "You can keep this conversation going.",
+                  'You can keep this conversation going.',
-                "Just head back to the discussion here: http://freecodecamp.com/stories/" + comment.originalStoryLink,
+                  'Just head back to the discussion here: http://freecodecamp.com/stories/' + data.originalStoryLink,
                   '- the Free Code Camp Volunteer Team'
                 ].join('\n')
               };
+
               transporter.sendMail(mailOptions, function (err) {
                 if (err) {
                   return err;
                 }
               });
+            }
           });
         });
       } catch (e) {

public/js/main.js

@@ -287,6 +287,8 @@ $(document).ready(function() {
       {
         data: {
           associatedPost: storyId,
+          originalStoryLink: originalStoryLink,
+          originalStoryAuthorEmail: originalStoryAuthorEmail,
           body: data
         }
       })

views/stories/comments.jade

@@ -108,6 +108,7 @@
                                             data: {
                                                 associatedPost: commentId,
                                                 originalStoryLink: originalStoryLink,
+                                                originalStoryAuthorEmail: originalStoryAuthorEmail,
                                                 body: $('#comment-to-comment-textinput').val(),
                                             }
                                         })

views/stories/show.jade

@@ -2,6 +2,7 @@
     script.
         var storyId = !{JSON.stringify(id)};
         var originalStoryLink = !{JSON.stringify(originalStoryLink)};
+        var originalStoryAuthorEmail = !{JSON.stringify(originalStoryAuthorEmail)};
         var comments = !{JSON.stringify(comments)};
         var upVotes = !{JSON.stringify(upVotes)};
         var image = !{JSON.stringify(image)};