Try to get event fastlane before sync on push

p1gp1g · Sep 16, 2021 · 6cd73da · 6cd73da
1 parent 03a4e75
commit 6cd73da
Showing 1 changed file with 9 additions and 13 deletions.
diff --git a/vector/src/main/java/im/vector/app/core/pushers/VectorMessagingReceiver.kt b/vector/src/main/java/im/vector/app/core/pushers/VectorMessagingReceiver.kt
@@ -219,29 +219,24 @@ val upHandler = object: MessagingReceiverHandler {
                 } else {
                     // Try to get the Event content faster
                     Timber.d("Requesting event in fast lane")
-                    getEventFastLane(session, notification.roomId, notification.eventId)
-
-                    Timber.d("Requesting background sync")
-                    session.requireBackgroundSync()
+                    if (!getEventFastLane(session, notification.roomId, notification.eventId)) {
+                        Timber.d("Requesting background sync")
+                        session.requireBackgroundSync()
+                    }
                 }
             }
         } catch (e: Exception) {
             Timber.e(e, "## onMessageReceivedInternal() failed")
         }
     }
 
-    private fun getEventFastLane(session: Session, roomId: String?, eventId: String?) {
-        roomId?.takeIf { it.isNotEmpty() } ?: return
-        eventId?.takeIf { it.isNotEmpty() } ?: return
+    private fun getEventFastLane(session: Session, roomId: String?, eventId: String?): Boolean {
+        roomId?.takeIf { it.isNotEmpty() } ?: return false
+        eventId?.takeIf { it.isNotEmpty() } ?: return false
 
         // If the room is currently displayed, we will not show a notification, so no need to get the Event faster
         if (notificationDrawerManager.shouldIgnoreMessageEventInRoom(roomId)) {
-            return
-        }
-
-        if (wifiDetector.isConnectedToWifi().not()) {
-            Timber.d("No WiFi network, do not get Event")
-            return
+            return true
         }
 
         coroutineScope.launch {
@@ -258,6 +253,7 @@ val upHandler = object: MessagingReceiverHandler {
                         notificationDrawerManager.refreshNotificationDrawer()
                     }
         }
+        return true
     }
 
     // check if the event was not yet received