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

question_type::export_to_xml misses files in answers & feedback when using extra_answer_fields

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.5, 2.2.2
    • Fix Version/s: 2.1.6, 2.2.3
    • Component/s: Questions
    • Labels:
    • Testing Instructions:
      Hide

      To test this, you need to install a third-party question type that relies on the extra_answer_fields functionality, and the question type base class implementation of export_to_xml. I will use qtype_varnumericset here.

      0. Type
      git clone git://github.com/moodleou/moodle-qtype_varnumericset.git question/type/varnumericset
      or
      git clone -b MOODLE_21_STABLE git://github.com/moodleou/moodle-qtype_varnumericset.git question/type/varnumericset
      in the root of your Moodle install, as applicable. The go to admin to let the plugin install itself.

      1. Create a new varnumericset question in the question bank:
      Question name: VNS
      Question text: What is [[a]] + [[b]]?
      Variable 1: Predefined variable
      Name: a
      Value for variant 1: 1
      Value for variant 2: 2
      Variable 2: Predefined variable
      Name: b
      Value for variant 1: 3
      Value for variant 2: 4
      Answer 1: a + b
      Answer 1 grade: 100%
      Answer 1 feedback: Well done and then add an image in this HTML editor.
      then Save changes.

      1.5 You should probably also create some standard Moodle question in the same category, to make sure there are no regressions there.

      2. Export the category containing this question as Moodle XML.

      3. Import the XML file into another course.

      4. Edit the imported question, and make sure the image is still there in the answer feedback.

      4.5 If you made extra questions in step 1.5, check those too.

      Show
      To test this, you need to install a third-party question type that relies on the extra_answer_fields functionality, and the question type base class implementation of export_to_xml. I will use qtype_varnumericset here. 0. Type git clone git://github.com/moodleou/moodle-qtype_varnumericset.git question/type/varnumericset or git clone -b MOODLE_21_STABLE git://github.com/moodleou/moodle-qtype_varnumericset.git question/type/varnumericset in the root of your Moodle install, as applicable. The go to admin to let the plugin install itself. 1. Create a new varnumericset question in the question bank: Question name: VNS Question text: What is [ [a] ] + [ [b] ]? Variable 1: Predefined variable Name: a Value for variant 1: 1 Value for variant 2: 2 Variable 2: Predefined variable Name: b Value for variant 1: 3 Value for variant 2: 4 Answer 1: a + b Answer 1 grade: 100% Answer 1 feedback: Well done and then add an image in this HTML editor. then Save changes. 1.5 You should probably also create some standard Moodle question in the same category, to make sure there are no regressions there. 2. Export the category containing this question as Moodle XML. 3. Import the XML file into another course. 4. Edit the imported question, and make sure the image is still there in the answer feedback. 4.5 If you made extra questions in step 1.5, check those too.
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      Actually, there was a TODO in the code saying that this code should be refactored to use $format->write_answer, and that will fix this problem.

        Gliffy Diagrams

          Activity

          Hide
          timhunt Tim Hunt added a comment -

          I will cherry-pick to other branches once this has been peer-reviewed.

          Show
          timhunt Tim Hunt added a comment - I will cherry-pick to other branches once this has been peer-reviewed.
          Hide
          timhunt Tim Hunt added a comment -

          Oleg, any chance you could have a look at these changes. This is an area of the code you know quite well. Thanks.

          Show
          timhunt Tim Hunt added a comment - Oleg, any chance you could have a look at these changes. This is an area of the code you know quite well. Thanks.
          Hide
          timhunt Tim Hunt added a comment -

          I am confident this fix is right, so submitting for integration.

          Show
          timhunt Tim Hunt added a comment - I am confident this fix is right, so submitting for integration.
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

          TIA and ciao

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
          Hide
          poltawski Dan Poltawski added a comment -

          Thanks Tim - this has been integrated now.

          Show
          poltawski Dan Poltawski added a comment - Thanks Tim - this has been integrated now.
          Hide
          poltawski Dan Poltawski added a comment -

          Pheww,

          tested on all 3 branches and looks good

          Show
          poltawski Dan Poltawski added a comment - Pheww, tested on all 3 branches and looks good
          Hide
          nebgor Aparup Banerjee added a comment -

          The code here has been spread to upstream moodle repositories and mirrors for anyone to use .

          Closing, have a good weekend!

          Show
          nebgor Aparup Banerjee added a comment - The code here has been spread to upstream moodle repositories and mirrors for anyone to use . Closing, have a good weekend!

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                14/May/12