Moodle
  1. Moodle
  2. MDL-32322

Code cleanup requried in the quiz reports

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.5, 2.2.2
    • Fix Version/s: 2.3
    • Component/s: Quiz
    • Labels:
    • Testing Instructions:
      Hide

      Testing instructions:

      Well, really, we need a full round of QA testing on the quiz reports. Fortunately, we have one of those coming up.

      I would say, don't spend too much time testing this now, because I am about to make a lot more changes for MDL-3030. Let us wait for those changes to go in, then test the lot.

      On the other hand, it would be worth doing some quiz testing now.

      Find (or create) a course with

      • groups
      • a quiz with attempts by various students
      • who are in different groups
      • ensure the quiz is set to separate groups mode.

      Then, look at each report, for All students, and some groups, and for various options, and make sure nothing is obviously broken.

      Show
      Testing instructions: Well, really, we need a full round of QA testing on the quiz reports. Fortunately, we have one of those coming up. I would say, don't spend too much time testing this now, because I am about to make a lot more changes for MDL-3030 . Let us wait for those changes to go in, then test the lot. On the other hand, it would be worth doing some quiz testing now. Find (or create) a course with groups a quiz with attempts by various students who are in different groups ensure the quiz is set to separate groups mode. Then, look at each report, for All students, and some groups, and for various options, and make sure nothing is obviously broken.
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
    • Rank:
      39131

      Description

      Things got a lot better in Moodle 2.1, but there is still a lot of duplication between mod/quiz/report/overview and mod/quiz/report/responses.

      I just got into a terrible mess trying to clean this up as part of MDL-3030, so now I am making a separate branch, so I can clean this up separately.

        Issue Links

          Activity

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: