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

Move coursecat into autoloaded location

    XMLWordPrintable

Details

    • MOODLE_36_STABLE
    • MOODLE_36_STABLE
    • wip-MDL-62742-master
    • 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

    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

              marina Marina Glancy
              marina Marina Glancy
              Sara Arjona (@sarjona) Sara Arjona (@sarjona)
              Andrew Lyons Andrew Lyons
              CiBoT CiBoT
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Laurent David, Raquel Ortega, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                3/Dec/18