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

JoinSiteDelegate should use AuthzGroup.joinGroup()

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: RESOLVED
    • Priority: Major
    • Resolution: Won't Fix
    • Affects Version/s: 10.0
    • Fix Version/s: None
    • Component/s: Kernel
    • Labels:
      None
    • Previous Issue Keys:
      KNL-1203

      Description

      Looking at:

      org.sakaiproject.site.impl.JoinSiteDelegate#addJoinerToGroup(String)

      it seems todo it's own joining of site groups. The AuthzService already has code todo this in:

      org.sakaiproject.authz.api.AuthzGroupService#joinGroup(java.lang.String, java.lang.String)

      I'm not 100% sure about this change but it should be looked at. joinGroup() will also generates an event for the join.

        Gliffy Diagrams

          Zeplin

            Attachments

              Issue Links

                Activity

                  People

                  Assignee:
                  k1team KERNEL TEAM (Inactive)
                  Reporter:
                  buckett Matthew Buckett
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                    Dates

                    Created:
                    Updated:
                    Resolved:

                      Git Integration