Merge pull request #10593 from nextcloud/bugfix/noid/increase-user-st…

…atus-limitation feat(user_status): Increase participant limit to 1k users
nextcloud · Sep 26, 2023 · 49297fa · 49297fa
2 parents 7dadcc0 + b9f16f3
commit 49297fa
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/lib/Config.php b/lib/Config.php
@@ -43,6 +43,13 @@ class Config {
 	public const SIGNALING_TICKET_V1 = 1;
 	public const SIGNALING_TICKET_V2 = 2;
 
+	/**
+	 * Currently limiting to 1k users because the user_status API would yield
+	 * an error on Oracle otherwise. Clients should use a virtual scrolling
+	 * mechanism so the data should not be a problem nowadays
+	 */
+	public const USER_STATUS_INTEGRATION_LIMIT = 1000;
+
 	protected array $canEnableSIP = [];
 
 	public function __construct(

diff --git a/lib/Controller/RoomController.php b/lib/Controller/RoomController.php
@@ -688,7 +688,7 @@ protected function formatParticipantList(array $participants, bool $includeStatu
 
 		if ($this->userId !== null
 			&& $includeStatus
-			&& count($participants) < 100
+			&& count($participants) < Config::USER_STATUS_INTEGRATION_LIMIT
 			&& $this->appManager->isEnabledForUser('user_status')) {
 			$userIds = array_filter(array_map(static function (Participant $participant) {
 				if ($participant->getAttendee()->getActorType() === Attendee::ACTOR_USERS) {