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

qformat_xml::format() should be public

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.2, 2.2
    • Fix Version/s: 2.1.3
    • Component/s: Questions
    • Labels:
      None
    • Testing Instructions:
      Hide

      Install this qtype: https://github.com/jamiepratt/moodle-qtype_varnumericset

      1. In a course, create some questions including at least one varnumericset, and some of the standard qtypes.

      2. Export these questions and Moodle XML format, then re-import them into a new course.

      3. Verify that all the details of the questions have been copied accurately.

      Show
      Install this qtype: https://github.com/jamiepratt/moodle-qtype_varnumericset 1. In a course, create some questions including at least one varnumericset, and some of the standard qtypes. 2. Export these questions and Moodle XML format, then re-import them into a new course. 3. Verify that all the details of the questions have been copied accurately.
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      See MDL-28426. Question types need to be able to use this method.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              timhunt Tim Hunt added a comment -

              Actually, this does not affect 2.0.x

              Show
              timhunt Tim Hunt added a comment - Actually, this does not affect 2.0.x
              Hide
              timhunt Tim Hunt added a comment -

              Jamie, it would be great if you could edit the testing instructions to suggest a question type that could be used to test this properly.

              Show
              timhunt Tim Hunt added a comment - Jamie, it would be great if you could edit the testing instructions to suggest a question type that could be used to test this properly.
              Hide
              jamiesensei Jamie Pratt added a comment -

              Hi,

              Export of all the built in question types is done in the xml format class.

              My varnumericset question type uses answers and uses the code in the question type base to export them. You can get it here :

              https://github.com/jamiepratt/moodle-qtype_varnumericset

              Jamie

              Show
              jamiesensei Jamie Pratt added a comment - Hi, Export of all the built in question types is done in the xml format class. My varnumericset question type uses answers and uses the code in the question type base to export them. You can get it here : https://github.com/jamiepratt/moodle-qtype_varnumericset Jamie
              Hide
              jamiesensei Jamie Pratt added a comment -

              Ah! Looks like we submitted our edited testing instructions at the same time. I was just about to re edit my not very clear instructions and I saw you also had submitted instructions (which are much clearer).

              Show
              jamiesensei Jamie Pratt added a comment - Ah! Looks like we submitted our edited testing instructions at the same time. I was just about to re edit my not very clear instructions and I saw you also had submitted instructions (which are much clearer).
              Hide
              nebgor Aparup Banerjee added a comment -

              Thanks, this has been integrated into master and 2.1.x.

              Test instructions are at the ready!

              Show
              nebgor Aparup Banerjee added a comment - Thanks, this has been integrated into master and 2.1.x. Test instructions are at the ready!
              Hide
              rajeshtaneja Rajesh Taneja added a comment -

              Works Great
              Thanks for fixing this Tim, and Jamie for easy test instructions.

              Show
              rajeshtaneja Rajesh Taneja added a comment - Works Great Thanks for fixing this Tim, and Jamie for easy test instructions.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              And this has landed upstream, just on time for the upcoming new releases week. Thanks for it!

              Ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - And this has landed upstream, just on time for the upcoming new releases week. Thanks for it! Ciao

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    28/Nov/11