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

When upgrading from 2.0 to 2.2, cannot limit quiz attempt upgrades with tool_qeupgradehelper_get_quizzes_to_upgrade

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.1, 2.3
    • Fix Version/s: 2.2.2
    • Component/s: Questions
    • Labels:
      None
    • Testing Instructions:
      Hide

      See Colin Campbell's comment. Do you want more testing than that?

      1. Install Moodle < 2.1
      2. Create some quizzes and some quiz attempts.
      3. Copy the 2.2.x code into place.
      4. Follow the instructions in admin/tool/qeupgradehelp/partialupgrade-example.php
      5. Run the upgrade.
      6. Make sure that only the expected quiz attempts have been upgraded during the upgrade process, and that the rest can be completed using qeupgradehelper.

      Show
      See Colin Campbell's comment. Do you want more testing than that? 1. Install Moodle < 2.1 2. Create some quizzes and some quiz attempts. 3. Copy the 2.2.x code into place. 4. Follow the instructions in admin/tool/qeupgradehelp/partialupgrade-example.php 5. Run the upgrade. 6. Make sure that only the expected quiz attempts have been upgraded during the upgrade process, and that the rest can be completed using qeupgradehelper.
    • Affected Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      In 2.2, the qeupgradehelper is installed in admin/tool/qeupgradehelper, but get_quiz_ids() in question/engine/upgrade/upgradelib.php looks in local/qeupgradehelper for the qeupgradehelper code. Consequently, implementing a tool_qeupgradehelper_get_quizzes_to_upgrade in admin/tool/qeupgradehelper/partialupgrade.php does not limit the quiz attempts to upgrade during the instance upgrade.

        Attachments

          Activity

            People

            Assignee:
            timhunt Tim Hunt
            Reporter:
            colin Colin Campbell
            Peer reviewer:
            Colin Campbell
            Integrator:
            Aparup Banerjee
            Tester:
            Adrian Greeve
            Participants:
            Component watchers:
            Tim Hunt, Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              12/Mar/12