Skip to content

Commit 4f66ec2

Browse files
committed
lint
1 parent 2169aab commit 4f66ec2

File tree

1 file changed

+15
-18
lines changed

1 file changed

+15
-18
lines changed

src/backend/core/api/viewsets/maildomain.py

Lines changed: 15 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -182,24 +182,21 @@ def get_permissions(self):
182182
def get_queryset(self):
183183
maildomain_pk = self.kwargs.get("maildomain_pk")
184184
user = self.request.user
185-
return (
186-
models.Mailbox.objects.filter(domain_id=maildomain_pk)
187-
.annotate(
188-
# Annotate user role for get_abilities() optimization
189-
user_role=Subquery(
190-
models.MailboxAccess.objects.filter(
191-
mailbox=OuterRef("pk"), user=user
192-
).values("role")[:1]
193-
),
194-
# Annotate domain admin status to avoid N+1 queries in get_abilities()
195-
is_domain_admin=Exists(
196-
models.MailDomainAccess.objects.filter(
197-
user=user,
198-
maildomain=OuterRef("domain"),
199-
role=models.MailDomainAccessRoleChoices.ADMIN,
200-
)
201-
),
202-
)
185+
return models.Mailbox.objects.filter(domain_id=maildomain_pk).annotate(
186+
# Annotate user role for get_abilities() optimization
187+
user_role=Subquery(
188+
models.MailboxAccess.objects.filter(
189+
mailbox=OuterRef("pk"), user=user
190+
).values("role")[:1]
191+
),
192+
# Annotate domain admin status to avoid N+1 queries in get_abilities()
193+
is_domain_admin=Exists(
194+
models.MailDomainAccess.objects.filter(
195+
user=user,
196+
maildomain=OuterRef("domain"),
197+
role=models.MailDomainAccessRoleChoices.ADMIN,
198+
)
199+
),
203200
)
204201

205202
@extend_schema(

0 commit comments

Comments
 (0)