Revert to this commit to test PR #432

Author: jleandroperez

WooCommerce/Classes/ViewRelated/Notifications/NotificationsViewController.swift

@@ -53,6 +53,8 @@ class NotificationsViewController: UIViewController {
     ///
     private let formatter = StringFormatter()
 
+    private var nextStatus = false
+
     /// UI Active State
     ///
     private var state: State = .results {
@@ -171,11 +173,7 @@ private extension NotificationsViewController {
     }
 
     @IBAction func markAllAsRead() {
-        let unread = resultsController.fetchedObjects.filter { $0.read == false }
-        if unread.isEmpty {
-            DDLogVerbose("# Every single notification is already marked as Read!")
-            return
-        }
+        let unread = resultsController.fetchedObjects
 
         markAsRead(notes: unread)
         hapticGenerator.notificationOccurred(.success)
@@ -191,13 +189,14 @@ private extension NotificationsViewController {
     ///
     func markAsRead(notes: [Note]) {
         let identifiers = notes.map { $0.noteId }
-        let action = NotificationAction.updateMultipleReadStatus(noteIds: identifiers, read: true) { error in
+        let action = NotificationAction.updateMultipleReadStatus(noteIds: identifiers, read: nextStatus) { error in
             if let error = error {
                 DDLogError("⛔️ Error marking notifications as read: \(error)")
             }
         }
 
         StoresManager.shared.dispatch(action)
+        nextStatus = !nextStatus
     }
 
     /// Synchronizes the Notifications associated to the active WordPress.com account.