Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.4.3
    • Fix Version/s: 2.4.5, 2.5.1
    • Component/s: Roles / Access
    • Labels:
    • Testing Instructions:
      Hide

      1. View a course page when logged in as admin. Verify that there are no errors and that you have full access to course options e.g. the 'turn editing on' button.

      2. Log in as a student and view the same page. Verify that there are no errors and that you do not have the 'turn editing on' button/option.

      Show
      1. View a course page when logged in as admin. Verify that there are no errors and that you have full access to course options e.g. the 'turn editing on' button. 2. Log in as a student and view the same page. Verify that there are no errors and that you do not have the 'turn editing on' button/option.
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE, MOODLE_25_STABLE
    • Pull Master Branch:
      MDL-39454-master

      Description

      In a typical page request, is_siteadmin gets called more than 200 times. In my profiling this takes about 5ms.

      The function is not very efficient. I am investigating two approaches to improve performance with the aim of saving a couple of milliseconds.

        Gliffy Diagrams

          Attachments

            Activity

              People

              • Votes:
                1 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  8/Jul/13