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

SiteAction.buildContextForTemplate should send Eid to the CourseManagementService, not Id

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.1, 2.2.2, 2.2.3
    • Fix Version/s: 2.3.0
    • Component/s: Site Info
    • Labels:
      None

      Description

      lines 2548 and 2549 look like this:

      String userId = StringUtil.trimToZero(SessionManager.getCurrentSessionUserId());
      List courses = CourseManagementService.getInstructorCourses(userId, t.getYear(), t.getTerm());

      It passes an Id to the CourseManagementService, when it should be passing an Eid. The CourseManagementService delegates the call to the CourseManagementProvider, and the CourseManagementProvider should only have to deal with Eid's.

      Line 2548 should look like this:

      String userId = StringUtil.trimToZero(SessionManager.getCurrentSession().getUserEid());

        Gliffy Diagrams

          Attachments

            Activity

              People

              • Assignee:
                zach.thomas@txstate.edu Zach A. Thomas
                Reporter:
                zach.thomas@txstate.edu Zach A. Thomas
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Git Source Code