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

Move coursecat into autoloaded location

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      This patch does not change any behavior, it just moves classes around. phpunit/behat should be sufficient to ensure there are no regressions or debugging messages. This is a quick visual test:

      1. Edit frontpage settings and add to the frontpage all possible course lists (available, combo list, categories, etc). Do it for both logged in and guest users
      2. View frontpage and browse categories as guest and as logged in user, make sure courses are displayed
      3. As admin/manager go to management interface and make sure you can do operations with courses and categories
      4. Repeat in both Boost and Clean
      Show
      This patch does not change any behavior, it just moves classes around. phpunit/behat should be sufficient to ensure there are no regressions or debugging messages. This is a quick visual test: Edit frontpage settings and add to the frontpage all possible course lists (available, combo list, categories, etc). Do it for both logged in and guest users View frontpage and browse categories as guest and as logged in user, make sure courses are displayed As admin/manager go to management interface and make sure you can do operations with courses and categories Repeat in both Boost and Clean
    • Affected Branches:
      MOODLE_36_STABLE
    • Fixed Branches:
      MOODLE_36_STABLE
    • Pull Master Branch:
      wip-MDL-62742-master

      Description

      While looking at MDL-10965 I noticed that methods from coursecat class are used throughout the core but this class is not autoloaded. I suggest to move it into autoloaded location

        Attachments

          Issue Links

            Activity

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  3/Dec/18