fix(dgsi/user): Flatten the values_list instead of iterating over it

2025-02-02 11:20:24 +01:00 · 2025-02-02 11:20:24 +01:00 · 8187c13db6
commit 8187c13db6
parent 750b85d792
1 changed files with 5 additions and 1 deletions
--- a/src/dgsi/models.py
+++ b/src/dgsi/models.py
@ -235,7 +235,11 @@ class User(AbstractUser):

        self.vlan_id = min(
            set(range(settings.VLAN_ID_MIN, settings.VLAN_ID_MAX))
-            - set(*User.objects.exclude(vlan_id__isnull=True).values_list("vlan_id"))
+            - set(
+                User.objects.exclude(vlan_id__isnull=True).values_list(
+                    "vlan_id", flat=True
+                )
+            )
        )

        # Preempt the vlan attribution