felixfoertsch/bundesmessenger-ios
1
0
Fork 0
mirror of https://gitlab.opencode.de/bwi/bundesmessenger/clients/bundesmessenger-ios.git synced 2026-04-17 23:18:27 +02:00
Code Issues Packages Projects Releases Wiki Activity

Handle app state better

Browse Source
This commit is contained in:
ismailgulek
2021-06-11 14:37:53 +03:00
parent afaebbcd85
commit bae3a6a1ea
1 changed files with 11 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
Riot/Managers/PushNotification/PushNotificationService.m
View File

@@ -548,13 +548,6 @@ Matrix session observer used to detect new opened sessions.
[[UNUserNotificationCenter currentNotificationCenter] removeUnwantedNotifications]; [[UNUserNotificationCenter currentNotificationCenter] removeUnwantedNotifications];
[[UNUserNotificationCenter currentNotificationCenter] removeCallNotificationsFor:roomId]; [[UNUserNotificationCenter currentNotificationCenter] removeCallNotificationsFor:roomId];
if ([UIApplication sharedApplication].applicationState == UIApplicationStateActive)
{
MXLogDebug(@"[PushNotificationService] didReceiveIncomingPushWithPayload: application is active. There is something wrong.");
completion();
return;
}
if (@available(iOS 13.0, *)) if (@available(iOS 13.0, *))
{ {
// for iOS 13, we'll just report the incoming call in the same runloop. It means we cannot call an async API here. // for iOS 13, we'll just report the incoming call in the same runloop. It means we cannot call an async API here.
@@ -565,7 +558,7 @@ Matrix session observer used to detect new opened sessions.
// when we have a VoIP push while the application is killed, session.callManager will not be ready yet. Configure it. // when we have a VoIP push while the application is killed, session.callManager will not be ready yet. Configure it.
[[AppDelegate theDelegate] configureCallManagerIfRequiredForSession:session]; [[AppDelegate theDelegate] configureCallManagerIfRequiredForSession:session];
MXLogDebug(@"[PushNotificationService] didReceiveIncomingPushWithPayload: callInvite: %@", callInvite); MXLogDebug(@"[PushNotificationService] didReceiveIncomingPushWithPayload: iOS 13+, callInvite: %@", callInvite);
if (callInvite) if (callInvite)
{ {
@@ -616,11 +609,20 @@ Matrix session observer used to detect new opened sessions.
else else
{ {
// It's a serious error. There is nothing to avoid iOS to kill us here. // It's a serious error. There is nothing to avoid iOS to kill us here.
MXLogDebug(@"[PushNotificationService] didReceiveIncomingPushWithPayload: iOS 13 and in bg, but we don't have the callInvite event for the eventId: %@. There is something wrong.", eventId); MXLogDebug(@"[PushNotificationService] didReceiveIncomingPushWithPayload: iOS 13+, but we don't have the callInvite event for the eventId: %@.", eventId);
} }
} }
else else
{ {
if ([UIApplication sharedApplication].applicationState == UIApplicationStateActive)
{
// below iOS 13, we don't have to report a call immediately.
// We can wait for a call invite from event stream and process.
MXLogDebug(@"[PushNotificationService] didReceiveIncomingPushWithPayload: Below iOS 13 and active app. Do nothing.");
completion();
return;
}
// below iOS 13, we can call an async API. After background sync, we'll hopefully fetch the call invite and report a new call to the CallKit. // below iOS 13, we can call an async API. After background sync, we'll hopefully fetch the call invite and report a new call to the CallKit.
[self launchBackgroundSync]; [self launchBackgroundSync];
} }