Viewer group for provisioner checks for view privilege in the group, not membership

XMLWordPrintable

    • Type: Bug
    • Resolution: Fixed
    • Priority: Minor
    • 4.1.1
    • Affects Version/s: 2.6.14
    • Component/s: provisioning
    • None

      Is this change correct? You should need membership in the ACL group, not a view privilege, right?

      commit 95bac8e577cf8edb211d7f55dec5acf96f557874
      Date: Tue Aug 9 22:56:19 2022 -0700
       
      Show provisioning button for viewers as well
       
      — a/grouper/src/grouper/edu/internet2/middleware/grouper/app/provisioning/GrouperProvisioningService.java
      +++ b/grouper/src/grouper/edu/internet2/middleware/grouper/app/provisioning/GrouperProvisioningService.java
       
      @@ -1624,7 +1625,7 @@ public class GrouperProvisioningService {
                 }
               }
               
      - return group.hasMember(subject);
      + return group.hasPrivilege(subject, AccessPrivilege.VIEW.getName());
             }
      

            Assignee:
            Chris Hyzer (upenn.edu)
            Reporter:
            Chad Redman
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: