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

Reorganise quiz library code to take advantage of class auto-loading

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Critical
    • Resolution: Won't Fix
    • Affects Version/s: 2.3, 2.5
    • Fix Version/s: None
    • Component/s: Quiz
    • Labels:
    • Testing Instructions:
      Hide

      You reall need to do a quick test of all the quiz functionality, to ensure that nothing is broken.

      There is a reason this is being done early in the 2.6 development cycle

      Show
      You reall need to do a quick test of all the quiz functionality, to ensure that nothing is broken. There is a reason this is being done early in the 2.6 development cycle
    • Affected Branches:
      MOODLE_23_STABLE, MOODLE_25_STABLE
    • Pull from Repository:
    • Pull Master Branch:
    • Pull Master Diff URL:

      Description

      There are now lots of different classes defined in the quiz, and at the moment they are all mixed up in the mod/quiz folder with the front-end scripts like view.php and attempt.php.

      It would be better to move them into mod/quiz/classes folder, with one php file per class.

      At the same time, try to make it possible to use constants like quiz_attempt::FINISHED in lib.php without having to include all of locallib.php.

        Attachments

          Issue Links

          caused a regression

          Epic - Created by Jira Software - do not edit or delete. Issue type for a big user story that needs to be broken down. MDL-47779 Review TODO with associated MDL's

          • Minor - Minor loss of function where workaround is possible
          • Open
          has a non-specific relationship to

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-33048 Quiz with a close time causes a Fatal Error on MyMoodle

          • Major - Major loss of function, incorrect output
          • Closed

            Activity

              People

              Assignee:
              timhunt Tim Hunt
              Reporter:
              timhunt Tim Hunt
              Peer reviewer:
              Dan Poltawski
              Participants:
              Component watchers:
              Tim Hunt, Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              5 Vote for this issue
              Watchers:
              11 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: