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

Include YUI_config in js debug

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.6
    • Component/s: JavaScript
    • Labels:
    • Testing Instructions:
      Hide

      There should be no changes in non-developer modes.

      • Set $CFG->jsrev = -1;
      • Open some pages in Moodle
        • Confirm that all JS works as expected
        • Confirm that there were no JavaScript errors in the console
      • In the console type:

        YUI_config.groups.moodle.modules
        

        • Confirm that a list of modules was shown
      • In config.php, remove the $CFG->jsrev = 1; and add $CFG>cachejs = false;
      • Rinse, wash, repeat.
      Show
      There should be no changes in non-developer modes. Set $CFG->jsrev = -1; Open some pages in Moodle Confirm that all JS works as expected Confirm that there were no JavaScript errors in the console In the console type: YUI_config.groups.moodle.modules Confirm that a list of modules was shown In config.php, remove the $CFG->jsrev = 1; and add $CFG >cachejs = false; Rinse, wash, repeat.
    • Affected Branches:
      MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE
    • Pull Master Branch:

      Description

      There are times when it would be really helpful to include module metadata in development mode, just like we do when development is not set.
      This is calculated on each page load so there is an added expense, but it is only when $CFG->jsrev = 1, or $CFG>cachejs = false.

      This will allow us to do more useful things with YUI submodules and break up our JS into more logical blocks.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    18/Nov/13