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

Quiz statistics report fails when all first attempts require manual grading

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.3, 2.2
    • Fix Version/s: 2.1.4, 2.2.1
    • Component/s: Quiz
    • Labels:
      None
    • Testing Instructions:
      Hide

      1. Create a quiz with one essay question.
      2. Make two attempts as a student, putting an answer into the essay question.
      3. As teacher, go to the quiz grades report, and verify that both attempts require grading.
      4. Manually grade the second attempt.
      5. Go to the quiz statistics report. Verify that there is no fatal error, and that it says
      Number of complete graded first attempts 0
      Total number of complete graded attempts 1

      Show
      1. Create a quiz with one essay question. 2. Make two attempts as a student, putting an answer into the essay question. 3. As teacher, go to the quiz grades report, and verify that both attempts require grading. 4. Manually grade the second attempt. 5. Go to the quiz statistics report. Verify that there is no fatal error, and that it says Number of complete graded first attempts 0 Total number of complete graded attempts 1
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Gliffy Diagrams

        Attachments

          Activity

          Hide
          samhemelryk Sam Hemelryk added a comment -

          Thanks Tim - this has been integrated now.

          Show
          samhemelryk Sam Hemelryk added a comment - Thanks Tim - this has been integrated now.
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Thanks Tim, due to HQ not getting through testing last week Eloy and myself have decided to integrate and test issues ourselves as we go. Slower but at least we can still roll the weeklies.

          This has been tested thoroughly on all branches. All works perfectly.

          Cheers
          Sam

          Show
          samhemelryk Sam Hemelryk added a comment - Thanks Tim, due to HQ not getting through testing last week Eloy and myself have decided to integrate and test issues ourselves as we go. Slower but at least we can still roll the weeklies. This has been tested thoroughly on all branches. All works perfectly. Cheers Sam
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Whoever decided one week was worth 14 days had really one bad idea. Anyway, the nightmare is over, so thanks for your, once again, amazing contributions. Many, many thanks!

          Now... disconnect, relax and enjoy the next days, yay!

          Closing...ciao

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Whoever decided one week was worth 14 days had really one bad idea. Anyway, the nightmare is over, so thanks for your, once again, amazing contributions. Many, many thanks! Now... disconnect, relax and enjoy the next days, yay! Closing...ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                9/Jan/12