From 83f4d940546cc37e812035f6690fb380a771891c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20P=C3=A9ters?= Date: Mon, 13 Aug 2018 09:22:19 +0200 Subject: [PATCH] provisionning: use explicit __in lookup (#25646) --- hobo/agent/authentic2/provisionning.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/hobo/agent/authentic2/provisionning.py b/hobo/agent/authentic2/provisionning.py index 8373cac..f21dbda 100644 --- a/hobo/agent/authentic2/provisionning.py +++ b/hobo/agent/authentic2/provisionning.py @@ -105,7 +105,7 @@ class Provisionning(object): # Find roles giving a superuser attribute # If there is any role of this kind, we do one provisionning message for each user and # each service. - roles_with_attributes = (self.Role.objects.filter(members=users) + roles_with_attributes = (self.Role.objects.filter(members__in=users) .parents(include_self=True) .filter(attributes__name='is_superuser') .exists()) @@ -115,10 +115,10 @@ class Provisionning(object): roles = dict((r.id, r) for r in all_roles) user_roles = {} parents = {} - for rp in self.RoleParenting.objects.filter(child=all_roles): + for rp in self.RoleParenting.objects.filter(child__in=all_roles): parents.setdefault(rp.child.id, []).append(rp.parent.id) Through = self.Role.members.through - for u_id, r_id in Through.objects.filter(role__members=users).values_list('user_id', + for u_id, r_id in Through.objects.filter(role__members__in=users).values_list('user_id', 'role_id'): user_roles.setdefault(u_id, set()).add(roles[r_id]) for p_id in parents.get(r_id, []): -- 2.18.0