From 3b64e75ae2b64c61131a8cba91b85bfa48aae99d Mon Sep 17 00:00:00 2001
From: Edi Cziszter <eduard.cziszter@student.upt.ro>
Date: Thu, 28 Oct 2021 12:28:17 +0300
Subject: [PATCH] markRead method now has all notifications by default

---
 .../components/notifications/notifications.component.html   | 4 ++--
 .../app/components/notifications/notifications.component.ts | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/apps/red-ui/src/app/components/notifications/notifications.component.html b/apps/red-ui/src/app/components/notifications/notifications.component.html
index 6547b1e45..a7ef00a67 100644
--- a/apps/red-ui/src/app/components/notifications/notifications.component.html
+++ b/apps/red-ui/src/app/components/notifications/notifications.component.html
@@ -17,7 +17,7 @@
                 </div>
 
                 <div
-                    (click)="markRead($event, notification)"
+                    (click)="markRead($event, [notification.id])"
                     *ngFor="let notification of group.notifications | sortBy: 'desc':'creationDate'"
                     [class.unread]="!notification.readDate"
                     class="notification"
@@ -29,7 +29,7 @@
                         <div class="small-label mt-2">{{ notification.time }}</div>
                     </div>
                     <div
-                        (click)="markRead($event, notification, !notification.readDate)"
+                        (click)="markRead($event, [notification.id], !notification.readDate)"
                         class="dot"
                         matTooltip="{{ 'notifications.mark-as' | translate: { type: notification.readDate ? 'unread' : 'read' } }}"
                         matTooltipPosition="before"
diff --git a/apps/red-ui/src/app/components/notifications/notifications.component.ts b/apps/red-ui/src/app/components/notifications/notifications.component.ts
index ddaaf4938..01a4289f4 100644
--- a/apps/red-ui/src/app/components/notifications/notifications.component.ts
+++ b/apps/red-ui/src/app/components/notifications/notifications.component.ts
@@ -8,6 +8,7 @@ import { NotificationsService } from '@services/notifications.service';
 import { Notification } from '@red/domain';
 import { distinctUntilChanged, map, shareReplay } from 'rxjs/operators';
 import { BehaviorSubject, Observable } from 'rxjs';
+import { List } from '@iqser/common-ui';
 
 interface NotificationsGroup {
     date: string;
@@ -47,10 +48,9 @@ export class NotificationsComponent implements OnInit {
         await this._loadData();
     }
 
-    async markRead($event, notification?: Notification, isRead = true): Promise<void> {
+    async markRead($event, notifications: List<string> = this._notifications$.getValue().map(n => n.id), isRead = true): Promise<void> {
         $event.stopPropagation();
-        const body = notification ? [notification.id] : this._notifications$.getValue().map(n => n.id);
-        await this._notificationsService.toggleNotificationRead(body, isRead).toPromise();
+        await this._notificationsService.toggleNotificationRead(notifications, isRead).toPromise();
         await this._loadData();
     }