Details
-
Bug
-
Resolution: Unresolved
-
Minor
-
2.3.0.patch
-
None
Description
API patch 81 now checks the user's group/stem privileges, and only lets them mark as reviewed if they have update access. However, the daily mailer job that accumulates email addresses includes group readers as well as admins and updaters. It is probably better to take out the readers, since there is no action they can take on the email they get.
--- a/grouper/src/grouper/edu/internet2/middleware/grouper/app/attestation/GrouperAttestationJob.java
+++ b/grouper/src/grouper/edu/internet2/middleware/grouper/app/attestation/GrouperAttestationJob.java
@@ -630,9 +630,8 @@ public class GrouperAttestationJob extends OtherJobBase {
String[] emailAddresses = null;
if (StringUtils.isBlank(attestationEmailAddresses)) {
- // get the group's admins/updaters/readers
+ // get the group's admins/updaters
Set<Subject> groupMembers = group.getAdmins();
- groupMembers.addAll(group.getReaders());
groupMembers.addAll(group.getUpdaters());
Set<String> addresses = new HashSet<String>();