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

Error on reviewing multiple answer question and disable numerical field during review.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.3, 2.1
    • Fix Version/s: 2.0.4
    • Component/s: Lesson
    • Labels:
      None
    • Testing Instructions:
      Hide

      1. login as teacher
      2. create a lesson and set 'Allow student review' to Yes and 'Maximum number of attempts' to a value greater than 1.
      3. add question and select multiple choice
      4. create new question and select the multiple-answer options.
      5. login as student and take the lesson
      6. attempt the lesson and at the end of lesson, select review lesson link.

      make sure it doesn't display any error.

      Show
      1. login as teacher 2. create a lesson and set 'Allow student review' to Yes and 'Maximum number of attempts' to a value greater than 1. 3. add question and select multiple choice 4. create new question and select the multiple-answer options. 5. login as student and take the lesson 6. attempt the lesson and at the end of lesson, select review lesson link. make sure it doesn't display any error.
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      Upon reviewing multiple answer question, notice message displayed for undefined variable and numerical question field need to be disable during review process.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            rwijaya Rossiani Wijaya added a comment -

            Submitting this to integration. Raj reviewed it earlier but I didn't press the 'request peer review' button.

            However, I received an approval from Eloy to submit for integration.

            Note for integrator: before applying this patch, please make sure you applied patch from MDL-27955.

            Thanks

            Show
            rwijaya Rossiani Wijaya added a comment - Submitting this to integration. Raj reviewed it earlier but I didn't press the 'request peer review' button. However, I received an approval from Eloy to submit for integration. Note for integrator: before applying this patch, please make sure you applied patch from MDL-27955 . Thanks
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Integrated thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Integrated thanks!
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Passing test without further action. Will be tested by MDLQA-1021 once this meets upstream.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Passing test without further action. Will be tested by MDLQA-1021 once this meets upstream.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            And this can be closed, thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - And this can be closed, thanks!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  1/Aug/11