Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-38557

Access to the quiz manual grading 'report' should be controlled by mod/quiz:grade, not mod/quiz:viewreports

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3.6, 2.4.3, 2.5
    • Fix Version/s: 2.3.7, 2.4.4
    • Component/s: Quiz
    • Labels:
    • Testing Instructions:
      Hide

      Ideally, this needs to be tested in both an upgrade an a newly installed site on each branch.

      1. You need to create users with 4 possible combinations of with/without mod/quiz:viewreports and mod/quiz:grade. (Perhaps easiest using permission overrides?)

      2. Check that mod/quiz:viewreports controls access to the Grades and Responses quiz report

      • In the navigation.
      • Which report the 'Attempts: 123' message links to.
      • Whether you can acutally use the report, once you have gone there in the navigation.
      Show
      Ideally, this needs to be tested in both an upgrade an a newly installed site on each branch. 1. You need to create users with 4 possible combinations of with/without mod/quiz:viewreports and mod/quiz:grade. (Perhaps easiest using permission overrides?) 2. Check that mod/quiz:viewreports controls access to the Grades and Responses quiz report In the navigation. Which report the 'Attempts: 123' message links to. Whether you can acutally use the report, once you have gone there in the navigation.
    • Affected Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      I actually thought it worked like that, but it seems I was wrong.

      I think it makes more sense to use the mod/quiz:grade capability to control access to this bit of UI, rather than mod/quiz:viewreports.

      All we need to do is to change the value in the quiz_reports.capability column.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              timhunt Tim Hunt
              Reporter:
              timhunt Tim Hunt
              Integrator:
              Eloy Lafuente (stronk7)
              Tester:
              Rajesh Taneja
              Participants:
              Component watchers:
              Tim Hunt, Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                13/May/13