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

Eliminate mod/quiz/editlib.php

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.8
    • Fix Version/s: 2.8
    • Component/s: Quiz
    • Labels:
    • Testing Instructions:
      Hide

      Running all the quiz PHPunit and Behat tests are a sufficient test of almost all of this. It is just a code clean-up, not a functional change.

      Except that, when doing this, I noticed something that was wrong and so fixed it:

      • You need a course with at least one question in the question bank.
      • Add a new quiz.
      • Go to the Edit quiz page.
      • Choose Add -> from the question bank.
      • In the pop-up, verify that the Question column shows both he question name in bold, and the start of the question text.
      • Click the column headings to try sorting on different combinations of columns.
      Show
      Running all the quiz PHPunit and Behat tests are a sufficient test of almost all of this. It is just a code clean-up, not a functional change. Except that, when doing this, I noticed something that was wrong and so fixed it: You need a course with at least one question in the question bank. Add a new quiz. Go to the Edit quiz page. Choose Add -> from the question bank. In the pop-up, verify that the Question column shows both he question name in bold, and the start of the question text. Click the column headings to try sorting on different combinations of columns.
    • Affected Branches:
      MOODLE_28_STABLE
    • Fixed Branches:
      MOODLE_28_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      The classes should move to /classes. The functions can probalby move to locallib.php.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              timhunt Tim Hunt
              Reporter:
              timhunt Tim Hunt
              Peer reviewer:
              Eloy Lafuente (stronk7)
              Integrator:
              Sam Hemelryk
              Tester:
              Frédéric Massart
              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:
                10/Nov/14