Notification: bug fix

Problem: Mentioned users in a comment were missing from the notification receivers. Solution: Find mantioned users in a comment and add them to receivers.
naver · Mar 12, 2015 · cd67cfb · cd67cfb
1 parent d969d62
commit cd67cfb
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/app/models/NotificationEvent.java b/app/models/NotificationEvent.java
@@ -544,8 +544,11 @@ public static NotificationEvent forComment(Comment comment, User author, EventTy
 
         NotificationEvent notiEvent = createFrom(author, comment);
         notiEvent.title = formatReplyTitle(post);
-        notiEvent.receivers = getReceivers(post, author);
         notiEvent.eventType = eventType;
+        Set<User> receivers = getReceivers(post, author);
+        receivers.addAll(getMentionedUsers(comment.contents));
+        receivers.remove(author);
+        notiEvent.receivers = receivers;
         notiEvent.oldValue = null;
         notiEvent.newValue = comment.contents;
         notiEvent.resourceType = comment.asResource().getType();