Details

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

      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

        Issue Links

          Activity

          Hide
          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
          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
          Sam Hemelryk added a comment -

          Thanks Tim, this has been integrated now.

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

          Passing without testing. It's trivial and obvious.

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

          This is now upstream, yay! Many thanks!

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