From cc001c98922f73fe46e9793fd77a82da45258b42 Mon Sep 17 00:00:00 2001 From: Vismay Golwala Date: Wed, 1 May 2019 13:12:52 -0400 Subject: [PATCH] Copy credential bug - add owner too While copying a credentials, we should set the new owner to person who copies it. There was a bug while trying to do so and this PR tries to address it. Link: #3750 Signed-off-by: Vismay Golwala --- awx/api/generics.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/awx/api/generics.py b/awx/api/generics.py index cb9350c6f9..2f2eca791b 100644 --- a/awx/api/generics.py +++ b/awx/api/generics.py @@ -982,7 +982,7 @@ class CopyAPIView(GenericAPIView): None, None, self.model, obj, request.user, create_kwargs=create_kwargs, copy_name=serializer.validated_data.get('name', '') ) - if hasattr(new_obj, 'admin_role') and request.user not in new_obj.admin_role: + if hasattr(new_obj, 'admin_role') and request.user not in new_obj.admin_role.members.all(): new_obj.admin_role.members.add(request.user) if sub_objs: permission_check_func = None