Currently, it is possible to add an evaluator to a matrix or a cell when the evaluator role does not have permission to evaluate the cell or matrix participant submissions. In order for an evaluator to evaluate a submission without this permission, the evaluator would need to be added to a group. So, let's display the following message to the coordinator when saving a matrix or a cell that has assigned evaluators who do not have permission to evaluate all participant submissions:
The evaluator role does not have permission to view all groups, so an evaluator will not be able to evaluate a submission unless you do one of the following:
1) Grant the evaluator role permission to view all groups by editing the permissions for this matrix
2) Add individual evaluators to the group of participants you want them to evaluate
Bryan Holladay added a comment - 23/Feb/10 10:25 AM
The warning text seems misleading since a selected evaluator (either role or users) doesn't have to be the "evaluator" role. The way I'm implementing it, is if any selected evaluator doesn't have the "VIEW_ALL_GROUPS" permission, then I through this warning.
Kristol Hancock added a comment - 23/Feb/10 11:09 AM
How about we just make the text more generic so that we can display it for any role that has permission to evaluate? Here is some suggested text, but you might want to run it by Lynn to see if she approves:
One or more evaluator(s) do not have permission to view any participants. Please check to ensure that each evaluator you assigned has been added to a group or the role(s) the evaluator(s) are assigned to have the 'View all groups' permission.
Bryan Holladay added a comment - 23/Feb/10 12:57 PM
Make sure to test adding a new Scaffolding and going to confirm page and clicking cancel.
Lynn Ward added a comment - 27/Apr/10 5:15 PM
The error message is appearing even when all evaluators are members of at least one group. The only time the error does not appear is if the selected roles have the "view all groups" permissions. It should not appear if either condition is met.
Lynn Ward added a comment - 27/Apr/10 6:14 PM
assigning back to Bryan.
Bryan Holladay added a comment - 28/Apr/10 1:11 PM
I have extended it so it will look for allGroups like before, but if that isn't set, then it will check that every selected user and every user in a selected group has at least 1 group assigned to them. If not either of these cases, then it will show the error.