don't bother ignoring notifications on iOS 10+

element-hq · Mar 13, 2019 · 93ce20e · 93ce20e
1 parent 2f1b615
commit 93ce20e
Showing 1 changed file with 2 additions and 9 deletions.
diff --git a/Riot/AppDelegate.m b/Riot/AppDelegate.m
@@ -1501,15 +1501,8 @@ - (void)handleLocalNotificationsForAccount:(MXKAccount*)account
         MXEvent *event;
 
         // Ignore event already notified to the user
-        if (@available(iOS 10, *))
-        {
-            if ([self displayedNotificationRequestForEvent:eventId andUser:account.mxCredentials.userId type:nil])
-            {
-                NSLog(@"[AppDelegate][Push] handleLocalNotificationsForAccount: Skip event already displayed in a notification. Event id: %@", eventId);
-                continue;
-            }
-        }
-        else if ([self displayedLocalNotificationForEvent:eventId andUser:account.mxCredentials.userId type:nil])
+        // only necessary on iOS 9, iOS 10 will just overwrite notifications with identical IDs
+        if (!@available(iOS 10, *) && [self displayedLocalNotificationForEvent:eventId andUser:account.mxCredentials.userId type:nil])
         {
             NSLog(@"[AppDelegate][Push] handleLocalNotificationsForAccount: Skip event already displayed in a notification. Event id: %@", eventId);
             continue;