refactor: remove redundant deleted_at filter in permitted_object_ids

SoftDeleteManager's own get_queryset() already excludes soft-deleted
rows, so the extra deleted_at__isnull=True filter was dead code.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
stumpylog
2026-08-07 13:16:03 -07:00
committed by Trenton H
co-authored by Claude Sonnet 5
parent 5779fe4ade
commit 2676a70166
+1 -9
View File
@@ -183,15 +183,7 @@ def permitted_object_ids(
manager = (
model.global_objects if include_deleted and has_soft_delete else model.objects
)
if has_soft_delete:
base_qs = (
manager.all()
if include_deleted
else manager.filter(deleted_at__isnull=True)
)
else:
base_qs = manager.all()
base_qs = base_qs.only("id", "owner")
base_qs = manager.all().only("id", "owner")
if user is None or not getattr(user, "is_authenticated", False):
return base_qs.filter(owner__isnull=True).values_list("id", flat=True)