Moodle
  1. Moodle
  2. MDL-28115

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

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor 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:
    • Rank:
      17735

      Description

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

        Issue Links

          Activity

          Hide
          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
          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
          Eloy Lafuente (stronk7) added a comment -

          Integrated thanks!

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

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

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

          And this can be closed, thanks!

          Show
          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: