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

Remove two uncessary yui2 requirements

          Details

          • Type: Bug
          • Status: Closed
          • Priority: Minor
          • Resolution: Fixed
          • Affects Version/s: 2.3
          • Fix Version/s: 2.3.1
          • Component/s: JavaScript
          • Labels:
            None
          • Testing Instructions:
            Hide
            1. Checking the code for M.util.set_user_preference you can see yui-connection is not used.

            Test that the set_user_preference is working fine:

            1. Go to a page with blocks
            2. Click the - hide button to keep that block hidden
            3. Reload the page
            4. VERIFY: that this block is hidden
            5. Go to my private files and bring up file picker
            6. Select a upload a file repository
            7. Chose a file
            8. Set license as 'creative commons - sharealike'
            9. Upload the file
            10. Save changes
            11. Bring up file picker again
            12. VERIFY: That the 'upload a file' repository is selected automatically
            13. VERIFY: That the 'creative commons - sharealike' license is selected automatically
            Show
            Checking the code for M.util.set_user_preference you can see yui-connection is not used. Test that the set_user_preference is working fine: Go to a page with blocks Click the - hide button to keep that block hidden Reload the page VERIFY: that this block is hidden Go to my private files and bring up file picker Select a upload a file repository Chose a file Set license as 'creative commons - sharealike' Upload the file Save changes Bring up file picker again VERIFY: That the 'upload a file' repository is selected automatically VERIFY: That the 'creative commons - sharealike' license is selected automatically
          • Affected Branches:
            MOODLE_23_STABLE
          • Fixed Branches:
            MOODLE_23_STABLE
          • Pull from Repository:
          • Pull Master Branch:
          • Pull Master Diff URL:

            Description

            Whilst looking at javascript included I noticed that even on plain moodle page there were some yui2 requirements which are not necessary.

            1. yui2 logger - we dont need this on every load YAHOO.log
            2. yui2-connection on user_preference_allow_ajax_update. user preference update now uses yui3 so this is not required

            By removing these uncessary requirements we shave 10ms off javascript loading (on my fast machine, even little helps)

              Gliffy Diagrams

                Attachments

                  Issue Links

                  will help resolve

                  Sub-task - Issues are sometimes broken into multiple sub-tasks. MDL-25292 get rid of old YUI2 includes on each page

                  • Minor - Minor loss of function where workaround is possible
                  • Closed

                    Activity

                    Ascending order - Click to sort in descending order
                    Hide
                    Permalink
                    samhemelryk Sam Hemelryk added a comment -

                    Aha good spotting Dan! has been integrated now and cherry-picked to 23.

                    Show
                    samhemelryk Sam Hemelryk added a comment - Aha good spotting Dan! has been integrated now and cherry-picked to 23.
                    Hide
                    Permalink
                    andyjdavis Andrew Davis added a comment -

                    No references to "yui-connection" found anywhere in core. All works as described. Passing.

                    Show
                    andyjdavis Andrew Davis added a comment - No references to "yui-connection" found anywhere in core. All works as described. Passing.
                    Hide
                    Permalink
                    samhemelryk Sam Hemelryk added a comment -

                    Congratulations your code is upstream - gold star for you!

                    This issue + 79 others made it in in time for the minor releases.
                    Thank you everyone involved for your exuberant efforts.

                    Show
                    samhemelryk Sam Hemelryk added a comment - Congratulations your code is upstream - gold star for you! This issue + 79 others made it in in time for the minor releases. Thank you everyone involved for your exuberant efforts.

                      People

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

                        Dates

                        • Created:
                          Updated:
                          Resolved:
                          Fix Release Date:
                          9/Jul/12