Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-31905

Cannot add multiple providers when creating or updating groups

    Details

    • 11 status:
      Verified
    • Conversion Script Required:
      Yes
    • Test Plan:
      Hide

      From a course site
      1) Site Info -> Manage Groups -> Create New Group
      2) Assign more than one roster to the site.
      3) Site Info -> Manage Groups -> Create New Group
      4) Assign a couple of roles.

      The result should be a new group with members from the multiple providers and instead you see 0 members.
      There is a second problem in that the changes don't show up in the UI straight away. If you wait for a message in the logs from RefreshAuthzGroupTask.run then you should see the changes.

      Show
      From a course site 1) Site Info -> Manage Groups -> Create New Group 2) Assign more than one roster to the site. 3) Site Info -> Manage Groups -> Create New Group 4) Assign a couple of roles. The result should be a new group with members from the multiple providers and instead you see 0 members. There is a second problem in that the changes don't show up in the UI straight away. If you wait for a message in the logs from RefreshAuthzGroupTask.run then you should see the changes.

      Description

      Regression from SAK-29402

      SiteGroupHelper is using a different separator than CourseManagementGroupProvider

      So when calling pack or unpack there are issues.

      The result of this is you cannot add multiple providers when creating or updating groups.

        Gliffy Diagrams

          Zeplin

            Attachments

              Issue Links

                Activity

                  People

                  Assignee:
                  buckett Matthew Buckett
                  Reporter:
                  ern Earle R Nietzel
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  6 Start watching this issue

                    Dates

                    Created:
                    Updated:
                    Resolved:

                      Git Integration