Meteor-Community-Packages/meteor-roles

[v2.0] support onlyAssigned option for method getUsersInRole

Open

#177 aberto em 1 de fev. de 2016

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)JavaScript (919 stars) (161 forks)batch import
enhancementgood first issuehelp wanted

Description

Use case In my model, 'project.member' is a sub-role of 'project.manager'. If I want a list of all project members that are not managers, AFAIK I cannot yet. Well yes, I can filter the output of getUsersInRole but that's not convenient.

Solution The method getRolesForUser already support this option :

onlyAssigned: return only assigned roles and not automatically inferred (like subroles)

So basically, the idea is to filter users which role param has an assigned:true field, perhaps using Roles._onlyAssignedMatcher method.

Guia do colaborador

Pilha de tecnologia
javascript
Domain
authorization
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
mostly clear
Prerequisites
familiarity with Meteor roles package
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60
Direção de pesquisa
Check the implementation of `getRolesForUser` to see how the `onlyAssigned` option is handled, especially the use of `Roles. onlyAssignedMatcher`. Apply a similar pattern to `getUsersInRole` method in the same source file. Also update relevant tests to cover the new option.