Moodle
  1. Moodle
  2. MDL-35744

Wrong review options for 'Never submitted' quiz attempts

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3.2, 2.4
    • Fix Version/s: 2.3.3
    • Component/s: Quiz
    • Labels:
    • Testing Instructions:
      Hide

      The unit tests cover this quite well, but ...

      1. Set up a quiz so that students are not allowed to review anything.

      2. Get a quiz attempt into the never submitted state.

      (When time expires -> Attempts must be submitted before time expires, and set a time limit. Start a quiz attempt as a student. Log out. Wait until time has expired. Log back in, and try to access the quiz attempt.)

      3. Ensure that the student cannot review the attempt.

      Show
      The unit tests cover this quite well, but ... 1. Set up a quiz so that students are not allowed to review anything. 2. Get a quiz attempt into the never submitted state. (When time expires -> Attempts must be submitted before time expires, and set a time limit. Start a quiz attempt as a student. Log out. Wait until time has expired. Log back in, and try to access the quiz attempt.) 3. Ensure that the student cannot review the attempt.
    • Affected Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
    • Rank:
      44493

      Description

      http://moodle.org/mod/forum/post.php?reply=927252

      It looks like the first line of quiz_attempt_state is wrong.

        Issue Links

          Activity

          Hide
          Tim Hunt added a comment -

          Submitting for integration.

          Show
          Tim Hunt added a comment - Submitting for integration.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Integrated (23 & master), thanks!

          Show
          Eloy Lafuente (stronk7) added a comment - Integrated (23 & master), thanks!
          Hide
          Andrew Davis added a comment -

          Im not able to start a quiz attempt. When I click "attempt quiz now" I get an overlay popup. When I click "start attempt" the following JS error is generated. Looking at the changes from this issue I suspect the breaking change may have been introduced by a different MDL.

          Timestamp: 03/10/12 08:59:29
          Error: TypeError: args.name is undefined
          Source File: http://localhost/moodle/int/master/lib/javascript.php/1349224881/lib/javascript-static.js
          Line: 93

          Show
          Andrew Davis added a comment - Im not able to start a quiz attempt. When I click "attempt quiz now" I get an overlay popup. When I click "start attempt" the following JS error is generated. Looking at the changes from this issue I suspect the breaking change may have been introduced by a different MDL. Timestamp: 03/10/12 08:59:29 Error: TypeError: args.name is undefined Source File: http://localhost/moodle/int/master/lib/javascript.php/1349224881/lib/javascript-static.js Line: 93
          Hide
          Andrew Davis added a comment -

          That JS error has been fixed in MDL-35754. Just waiting for that to be integrated.

          Show
          Andrew Davis added a comment - That JS error has been fixed in MDL-35754 . Just waiting for that to be integrated.
          Hide
          Dan Poltawski added a comment -

          The other bugfix has been integrated.

          Show
          Dan Poltawski added a comment - The other bugfix has been integrated.
          Hide
          Andrew Davis added a comment -

          Now seems to be working nicely.

          Show
          Andrew Davis added a comment - Now seems to be working nicely.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Closing as fixed, many thanks for your awesome collaboration.

          Show
          Eloy Lafuente (stronk7) added a comment - Closing as fixed, many thanks for your awesome collaboration.

            People

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

              Dates

              • Created:
                Updated:
                Resolved: