From 59688bb26c937c44c8bfbef7990a7220c8bef091 Mon Sep 17 00:00:00 2001 From: Benjamin Dauvergne Date: Tue, 13 Aug 2019 16:56:51 +0200 Subject: [PATCH 2/5] a2_rbac: use bulk update of admin_role permissions (#35391) --- src/authentic2/a2_rbac/managers.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/authentic2/a2_rbac/managers.py b/src/authentic2/a2_rbac/managers.py index e94edcd2..2ec58849 100644 --- a/src/authentic2/a2_rbac/managers.py +++ b/src/authentic2/a2_rbac/managers.py @@ -71,9 +71,7 @@ class RoleManager(BaseRoleManager): self_perm = admin_role.add_self_administration() permissions.add(self_perm) if set(admin_role.permissions.all()) != permissions: - for permission in permissions: - admin_role.permissions.through.objects.get_or_create(role=admin_role, - permission=permission) + admin_role.permissions = permissions return admin_role def get_mirror_role(self, instance, name, slug, ou=None, -- 2.23.0.rc1