Only update notifications that are not already marked as read (#3202)

flarum · Dec 14, 2021 · 25dc26b · 25dc26b
1 parent 10c6694
commit 25dc26b
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/Notification/NotificationRepository.php b/src/Notification/NotificationRepository.php
@@ -50,6 +50,8 @@ public function findByUser(User $user, $limit = null, $offset = 0)
      */
     public function markAllAsRead(User $user)
     {
-        Notification::where('user_id', $user->id)->update(['read_at' => Carbon::now()]);
+        Notification::where('user_id', $user->id)
+            ->whereNull('read_at')
+            ->update(['read_at' => Carbon::now()]);
     }
 }