Merge pull request #12198 from nextcloud/bugfix/noid/make-alluserids-…

…more-reliable

fix(signaling): Make alluserids more reliable for testing

lib/Signaling/BackendNotifier.php

@@ -176,6 +176,8 @@ public function roomInvited(Room $room, array $attendees): void {
 	 * @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) {
@@ -189,7 +191,7 @@ public function roomsDisinvited(Room $room, array $attendees): void {
 				'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),
 			],
 		]);
@@ -210,14 +212,16 @@ public function roomsDisinvited(Room $room, array $attendees): void {
 	 * @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',
 			'disinvite' => [
 				'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),
 			],
 		]);