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

Assignments > missing permission check in AssignmentService.getSubmission(String)

    Details

    • Type: Bug
    • Status: RESOLVED
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 19.0
    • Fix Version/s: 19.0
    • Component/s: Assignments
    • Labels:
      None
    • Test Plan:
      Hide

      Please add a Test Plan here.

      Show
      Please add a Test Plan here.

      Description

      There are three overloads on AssignmentService.getSubmission(), each of which states that a PermissionException can be thrown. However, only two of the three overloads actually perform the permission checking and throw the exception.

      Linked PR adds the permission check and exception throw to the remaining method.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

                • Assignee:
                  bjones86 Brian Jones
                  Reporter:
                  bjones86 Brian Jones
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Git Source Code