FIX: badge notifications are marked read by the system after opening

discourse · Oct 2, 2014 · 37af13f · 37af13f
1 parent 72a28ae
commit 37af13f
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/app/models/user.rb b/app/models/user.rb
@@ -255,8 +255,12 @@ def unread_notifications
   end
 
   def saw_notification_id(notification_id)
-    User.where(["id = ? and seen_notification_id < ?", id, notification_id])
+    User.where("id = ? and seen_notification_id < ?", id, notification_id)
         .update_all ["seen_notification_id = ?", notification_id]
+
+    # mark all badge notifications read
+    Notification.where('user_id = ? AND NOT read AND notification_type = ?', id, Notification.types[:granted_badge])
+        .update_all ["read = ?", true]
   end
 
   def publish_notifications_state