From 8cda8fb53352df0da60dcfa491b4774037f10bdd Mon Sep 17 00:00:00 2001
From: Trenton H <797416+stumpylog@users.noreply.github.com>
Date: Thu, 12 Mar 2026 09:58:14 -0700
Subject: [PATCH] Use a single query and collapse the if statements

---
 src/paperless/consumers.py | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/src/paperless/consumers.py b/src/paperless/consumers.py
index 90f385135..fb745a9bc 100644
--- a/src/paperless/consumers.py
+++ b/src/paperless/consumers.py
@@ -20,11 +20,10 @@ class StatusConsumer(AsyncWebsocketConsumer):
         if user.is_superuser or user.id == owner_id or user.id in users_can_view:
             return True
 
-        for group_id in groups_can_view:
-            if await user.groups.filter(pk=group_id).aexists():
-                return True
-
-        return False
+        return (
+            groups_can_view
+            and await user.groups.filter(pk__in=groups_can_view).aexists()
+        )
 
     async def connect(self) -> None:
         if not self._authenticated():