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

Upgrade bug for numerical qtype

    Details

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

      This is such a non-critical bug, and such a pain to test, that I would not bother, but if you are prepared to test an upgrade 1.9 -> 2.0.3+ including numerical and calculated questions, then that would be good.

      Show
      This is such a non-critical bug, and such a pain to test, that I would not bother, but if you are prepared to test an upgrade 1.9 -> 2.0.3+ including numerical and calculated questions, then that would be good.
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE
    • Pull from Repository:

      Description

      Eloy spotted that this line is wrong: https://github.com/moodle/moodle/blob/MOODLE_20_STABLE/question/type/numerical/db/upgrade.php#L74

      This is not a very critical bug. That field is new in 2.0, so there is no problem for people upgrading 1.9 -> 2.0, just 2.0 dev -> 2.0

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              timhunt Tim Hunt added a comment -

              Note that this fix is included in MDL-206363 for integration into master. Only 2.0 needs to be done here.

              Show
              timhunt Tim Hunt added a comment - Note that this fix is included in MDL-206363 for integration into master. Only 2.0 needs to be done here.
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Thanks Tim, this has been integrated now.

              Show
              samhemelryk Sam Hemelryk added a comment - Thanks Tim, this has been integrated now.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Passing without testing. It's trivial and obvious.

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Passing without testing. It's trivial and obvious.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              This is now upstream, yay! Many thanks!

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - This is now upstream, yay! Many thanks!

                People

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

                  Dates

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