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

LASTATTEMPT quiz not using latest attempt to grade

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.0.3, 2.1.1, 2.2
    • 2.0.5, 2.1.2
    • Quiz
    • MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
    • MOODLE_20_STABLE, MOODLE_21_STABLE
    • Hide

      Testing is most needed in 2.0:

      1. Create a quiz set to grading method first, or last, attempt.
      2. Do lots of attempts as one student.
      3. Try to persuade your database to return records in an arbitrary order. I think Postgres does this after you have deleted enough rows from a table and vacuummed.
      4. Regrade the quiz, and make sure the final grade is computed correctly.

      Show
      Testing is most needed in 2.0: 1. Create a quiz set to grading method first, or last, attempt. 2. Do lots of attempts as one student. 3. Try to persuade your database to return records in an arbitrary order. I think Postgres does this after you have deleted enough rows from a table and vacuummed. 4. Regrade the quiz, and make sure the final grade is computed correctly.

    Description

      The quiz grade calculation in LASTATTEMPT mode uses the last on the list of attempts it is given, not the last by data.

      This is resolved by checking the attempt is newer than the best so far, so latest attempts are used regardless of order.

      Patch incoming with github link

      Attachments

        Issue Links

          is blocked by

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-28227 Regrade sometimes grades user attempts for any quiz not selected quiz

          • Minor - Minor loss of function where workaround is possible
          • Closed

          Activity

            People

              timhunt Tim Hunt
              tlevi Tony Levi
              Martin Dougiamas Martin Dougiamas
              Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
              Michael de Raadt Michael de Raadt
              Tim Hunt, Amaia Anabitarte, Bas Brands, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Kevin Percy, Laurent David, Mathew May, Mihail Geshoski, Sabina Abellan, Sara Arjona (@sarjona), Shamim Rezaie
              Votes:
              2 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                10/Oct/11