NotificationMail: Fix incorrect References header

A message id of a notification mail for new comment thread should be the message id of the comment thread's first comment.
naver · Jan 27, 2015 · 6299412 · 6299412
1 parent 157cefe
commit 6299412
Showing 1 changed file with 12 additions and 1 deletion.
diff --git a/app/models/NotificationMail.java b/app/models/NotificationMail.java
@@ -187,7 +187,18 @@ public void addReferences() {
             Resource container = resource.getContainer();
 
             if (container != null) {
-                addHeader("References", container.getMessageId());
+                String reference;
+                switch (container.getType()) {
+                    case COMMENT_THREAD:
+                        CommentThread thread =
+                                CommentThread.find.byId(Long.valueOf(container.getId()));
+                        reference = thread.getFirstReviewComment().asResource().getMessageId();
+                        break;
+                    default:
+                        reference = container.getMessageId();
+                        break;
+                }
+                addHeader("References", reference);
             }
         }
     }