Uploaded image for project: 'Grouper'
  1. Grouper
  2. GRP-3544

edu.internet2.middleware.grouper.*Save API's should have constructor/methods to set the session to use for the operation.

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Won't Fix
    • Minor
    • 2.5.55
    • 2.5.52
    • None
    • None

    Description

      Please review all of theĀ edu.internet2.middleware.grouper.*Save.java classes.

      They all could use a constructor that passes in the session for the object to use.
      *Save(GrouperSession session_to_use) {...}

      These already support it, but not all of them do.
      StemSave, GroupSave
      https://github.com/Internet2/grouper/blob/99cd2a64a853fea5c8a672e282115dcb76bee978/grouper/src/grouper/edu/internet2/middleware/grouper/StemSave.java#L135

      https://github.com/Internet2/grouper/blob/99cd2a64a853fea5c8a672e282115dcb76bee978/grouper/src/grouper/edu/internet2/middleware/grouper/GroupSave.java#L243

      Likewise the pattern of .assignRunAsRoot(boolean) would be good to also replicate to all of these classes.

      Attachments

        Activity

          People

            chris.hyzer@at.internet2.edu Chris Hyzer (upenn.edu)
            carey.black@at.internet2.edu Carey Black (osu.edu)
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: