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

          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