Merge pull request #1925 from anoek/1692

Fixed duplicate entries in active_roles (again)
This commit is contained in:
Akita Noek 2016-05-16 11:23:58 -04:00
commit 2435de6261
2 changed files with 5 additions and 5 deletions

View File

@ -508,4 +508,4 @@ class ResourceAccessList(ListAPIView):
ancestors = set()
for r in roles:
ancestors.update(set(r.ancestors.all()))
return User.objects.filter(roles__in=list(ancestors))
return User.objects.filter(roles__in=list(ancestors)).distinct()

View File

@ -1573,12 +1573,12 @@ class ResourceAccessListElementSerializer(UserSerializer):
.exclude(id__in=indirect_team_roles)
ret['summary_fields']['direct_access'] \
= [format_role_perm(r) for r in direct_access_roles] \
+ [format_team_role_perm(r, direct_permissive_role_ids) for r in direct_team_roles]
= [format_role_perm(r) for r in direct_access_roles.distinct()] \
+ [format_team_role_perm(r, direct_permissive_role_ids) for r in direct_team_roles.distinct()]
ret['summary_fields']['indirect_access'] \
= [format_role_perm(r) for r in indirect_access_roles] \
+ [format_team_role_perm(r, all_permissive_role_ids) for r in indirect_team_roles]
= [format_role_perm(r) for r in indirect_access_roles.distinct()] \
+ [format_team_role_perm(r, all_permissive_role_ids) for r in indirect_team_roles.distinct()]
return ret