fix(signaling): Make alluserids more reliable for testingbugfix/noid/make-alluserids-more-reliable

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

1 files changed, 6 insertions, 2 deletions

diff --git a/lib/Signaling/BackendNotifier.php b/lib/Signaling/BackendNotifier.php
index 0b784f0f5..b735487a9 100644
--- a/lib/Signaling/BackendNotifier.php
+++ b/lib/Signaling/BackendNotifier.php
@@ -193,6 +193,8 @@ class BackendNotifier {
 	 * @throws \Exception
 	 */
 	public function roomsDisinvited(Room $room, array $attendees): void {
+		$allUserIds = $this->participantService->getParticipantUserIds($room);
+		sort($allUserIds);
 		$userIds = [];
 		foreach ($attendees as $attendee) {
 			if ($attendee->getActorType() === Attendee::ACTOR_USERS) {
@@ -206,7 +208,7 @@ class BackendNotifier {
 				'userids' => $userIds,
 				// TODO(fancycode): We should try to get rid of 'alluserids' and
 				// find a better way to notify existing users to update the room.
-				'alluserids' => $this->participantService->getParticipantUserIds($room),
+				'alluserids' => $allUserIds,
 				'properties' => $room->getPropertiesForSignaling('', false),
 			],
 		]);
@@ -227,6 +229,8 @@ class BackendNotifier {
 	 * @throws \Exception
 	 */
 	public function roomSessionsRemoved(Room $room, array $sessionIds): void {
+		$allUserIds = $this->participantService->getParticipantUserIds($room);
+		sort($allUserIds);
 		$start = microtime(true);
 		$this->backendRequest($room, [
 			'type' => 'disinvite',
@@ -234,7 +238,7 @@ class BackendNotifier {
 				'sessionids' => $sessionIds,
 				// TODO(fancycode): We should try to get rid of 'alluserids' and
 				// find a better way to notify existing users to update the room.
-				'alluserids' => $this->participantService->getParticipantUserIds($room),
+				'alluserids' => $allUserIds,
 				'properties' => $room->getPropertiesForSignaling('', false),
 			],
 		]);