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

Boost admin renderer hard codes admin path

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.2
    • Fix Version/s: 3.2
    • Component/s: Themes
    • Labels:
    • Testing Instructions:
      Hide
      1. Start with a Moodle 3.1 installation.
      2. Rename the 'admin' directory to something else such as 'myadmin'.
      3. In config.php change the admin setting $CFG->admin = 'myadmin';.
      4. Upgrade to master. There should be no errors during the upgrade.
      Show
      Start with a Moodle 3.1 installation. Rename the 'admin' directory to something else such as 'myadmin'. In config.php change the admin setting $CFG->admin = 'myadmin'; . Upgrade to master. There should be no errors during the upgrade.
    • Affected Branches:
      MOODLE_32_STABLE
    • Fixed Branches:
      MOODLE_32_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      wip-MDL-56788-master

      Description

      The require in theme/boost/classes/output/core/admin_renderer.php hard codes admin in the path.

      This just needs to be changed to use the config setting such as:

      require_once($CFG->dirroot . '/' . $CFG->admin . '/renderer.php');
      

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              abgreeve Adrian Greeve
              Reporter:
              abgreeve Adrian Greeve
              Peer reviewer:
              cameron1729
              Integrator:
              Dan Poltawski
              Tester:
              Ankit Agarwal
              Participants:
              Component watchers:
              Bas Brands
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                5/Dec/16