From 00698b82fca08899bd0a500002b1e78525da0aad Mon Sep 17 00:00:00 2001 From: Wayne Witzel III Date: Fri, 10 Jun 2016 17:03:55 -0400 Subject: [PATCH] fix issue with Team.roles and ActivityStream --- awx/main/signals.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/awx/main/signals.py b/awx/main/signals.py index 40382c98b4..7f3f900db8 100644 --- a/awx/main/signals.py +++ b/awx/main/signals.py @@ -158,13 +158,17 @@ def org_admin_edit_members(instance, action, model, reverse, pk_set, **kwargs): def rbac_activity_stream(instance, sender, **kwargs): user_type = ContentType.objects.get_for_model(User) + team_type = ContentType.objects.get_for_model(Team) # Only if we are associating/disassociating if kwargs['action'] in ['pre_add', 'pre_remove']: # Only if this isn't for the User.admin_role if hasattr(instance, 'content_type'): if instance.content_type in [None, user_type]: return - role = instance + elif instance.content_type == team_type: + role = kwargs['model'].objects.filter(pk__in=kwargs['pk_set']).first() + else: + role = instance instance = instance.content_object else: role = kwargs['model'].objects.filter(pk__in=kwargs['pk_set']).first() @@ -192,6 +196,7 @@ post_save.connect(emit_ad_hoc_command_event_detail, sender=AdHocCommandEvent) m2m_changed.connect(rebuild_role_ancestor_list, Role.parents.through) m2m_changed.connect(org_admin_edit_members, Role.members.through) m2m_changed.connect(rbac_activity_stream, Role.members.through) +m2m_changed.connect(rbac_activity_stream, Role.parents.through) post_save.connect(sync_superuser_status_to_rbac, sender=User) post_save.connect(create_user_role, sender=User) pre_delete.connect(cleanup_detached_labels_on_deleted_parent, sender=UnifiedJob)