Do not query the participants for pushing call notifications

We can skip the participant check when we generate push notifications.
We just looped over the participants to create the notifications,
they can not be removed between these 2 steps, so we can save all those queries.

Signed-off-by: Joas Schilling <coding@schilljs.com>

1 files changed, 12 insertions, 5 deletions

diff --git a/lib/Notification/Notifier.php b/lib/Notification/Notifier.php
index 092cf186b..9ff13f7a6 100644
--- a/lib/Notification/Notifier.php
+++ b/lib/Notification/Notifier.php
@@ -179,11 +179,18 @@ class Notifier implements INotifier {
 			throw new AlreadyProcessedException();
 		}
 
-		try {
-			$participant = $room->getParticipant($userId);
-		} catch (ParticipantNotFoundException $e) {
-			// Room does not exist
-			throw new AlreadyProcessedException();
+		if ($this->notificationManager->isPreparingPushNotification() && $notification->getSubject() === 'call') {
+			// Skip the participant check when we generate push notifications
+			// we just looped over the participants to create the notification,
+			// they can not be removed between these 2 steps, but we can save
+			// n queries.
+		} else {
+			try {
+				$participant = $room->getParticipant($userId);
+			} catch (ParticipantNotFoundException $e) {
+				// Room does not exist
+				throw new AlreadyProcessedException();
+			}
 		}
 
 		$notification