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: JavaScript
    • Labels:
      None

      Description

      Please see MDL-33448 for details.

        Gliffy Diagrams

          Attachments

            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