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

M.util.get_string spawns needless YUI instances

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.4, 2.3.1
    • Fix Version/s: 2.2.5, 2.3.2
    • Component/s: AJAX and JavaScript
    • Labels:
      None

      Description

      Please see MDL-33448 for details.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            poltawski Dan Poltawski added a comment -

            Thanks Sam, i've integrated this now.

            Show
            poltawski Dan Poltawski added a comment - Thanks Sam, i've integrated this now.
            Hide
            abgreeve Adrian Greeve added a comment -

            I tested this in master to see how long it was taking for me to be able to access the Enrolled users page.
            I noticed that the patch does increase the speed with which I can access the page. On my machine I was only delayed for about one second after the page had loaded.
            Tested with:

            • Firefox
            • Chrome
            • IE
            Show
            abgreeve Adrian Greeve added a comment - I tested this in master to see how long it was taking for me to be able to access the Enrolled users page. I noticed that the patch does increase the speed with which I can access the page. On my machine I was only delayed for about one second after the page had loaded. Tested with: Firefox Chrome IE
            Hide
            nebgor Aparup Banerjee added a comment -

            yay, it works!

            This issue has been put through rigorous processes and finally swam upstream along with some 65 others this week.

            Thank you all for taking the time to get us here.

            cheers!

            Show
            nebgor Aparup Banerjee added a comment - yay, it works! This issue has been put through rigorous processes and finally swam upstream along with some 65 others this week. Thank you all for taking the time to get us here. cheers!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  10/Sep/12