From 8aafd69db10141bcf39788bc5c4f4159bb0ae706 Mon Sep 17 00:00:00 2001 From: jasquat Date: Tue, 27 Dec 2022 16:44:58 -0500 Subject: [PATCH] do not delete groups that were just added w/ burnettk --- .../services/authorization_service.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/spiffworkflow_backend/services/authorization_service.py b/src/spiffworkflow_backend/services/authorization_service.py index 3c42b24a..5ee1bf62 100644 --- a/src/spiffworkflow_backend/services/authorization_service.py +++ b/src/spiffworkflow_backend/services/authorization_service.py @@ -757,20 +757,22 @@ class AuthorizationService: desired_group_identifiers = result["group_identifiers"] for group in group_info: + group_identifier = group["name"] for username in group["users"]: GroupService.add_user_to_group_or_add_to_waiting( - username, group["name"] + username, group_identifier ) + desired_group_identifiers.add(group_identifier) for permission in group["permissions"]: for crud_op in permission["actions"]: desired_permission_assignments.extend( cls.add_permission_from_uri_or_macro( - group_identifier=group["name"], + group_identifier=group_identifier, target=permission["uri"], permission=crud_op, ) ) - desired_group_identifiers.add(group["name"]) + desired_group_identifiers.add(group_identifier) for ipa in initial_permission_assignments: if ipa not in desired_permission_assignments: