Merge pull request #11878 from edx/PERF-279

Use a module/path mapping for RequireJS overrides instead of just paths.

common/djangoapps/pipeline_mako/__init__.py

@@ -128,17 +128,17 @@ def render_require_js_path_overrides(path_overrides):  # pylint: disable=invalid
     </script>'''
 
     new_paths = []
-    for url_path in path_overrides:
+    for module in path_overrides:
         # Calculate the full URL, including any hashes added to the filename by the pipeline.
         # This will also include the base static URL (for example, "/static/") and the
         # ".js" extension.
-        actual_url = staticfiles_storage.url(url_path)
+        actual_url = staticfiles_storage.url(path_overrides[module])
 
         # RequireJS assumes that every file it tries to load has a ".js" extension, so
         # we need to remove ".js" from the module path.
         # RequireJS also already has a base URL set to the base static URL, so we can remove that.
         path = actual_url.replace('.js', '').replace(django_settings.STATIC_URL, '')
 
-        new_paths.append("'{module}': '{path}'".format(module=url_path.replace('.js', ''), path=path))
+        new_paths.append("'{module}': '{path}'".format(module=module, path=path))
 
     return html.format(overrides=',\n'.join(new_paths))

common/djangoapps/pipeline_mako/tests/test_render.py

@@ -7,20 +7,20 @@ from pipeline_mako import render_require_js_path_overrides
 class RequireJSPathOverridesTest(TestCase):
     """Test RequireJS path overrides. """
 
-    OVERRIDES = [
-        'js/vendor/jquery.min.js',
-        'js/vendor/backbone-min.js',
-        'js/vendor/text.js'
-    ]
+    OVERRIDES = {
+        'jquery': 'js/vendor/jquery.min.js',
+        'backbone': 'js/vendor/backbone-min.js',
+        'text': 'js/vendor/text.js'
+    }
 
     OVERRIDES_JS = [
         "<script type=\"text/javascript\">",
         "(function (require) {",
         "require.config({",
         "paths: {",
-        "'js/vendor/jquery.min': 'js/vendor/jquery.min',",
-        "'js/vendor/backbone-min': 'js/vendor/backbone-min',",
-        "'js/vendor/text': 'js/vendor/text'",
+        "'jquery': 'js/vendor/jquery.min',",
+        "'text': 'js/vendor/text',",
+        "'backbone': 'js/vendor/backbone-min'",
         "}",
         "});",
         "}).call(this, require || RequireJS.require);",

lms/envs/common.py

@@ -1636,14 +1636,18 @@ REQUIRE_ENVIRONMENT = "node"
 # If you want to load JavaScript dependencies using RequireJS
 # but you don't want to include those dependencies in the JS bundle for the page,
 # then you need to add the js urls in this list.
-REQUIRE_JS_PATH_OVERRIDES = [
-    'js/bookmarks/views/bookmark_button.js',
-    'js/views/message_banner.js',
-    'js/vendor/moment.min.js',
-    'js/vendor/url.min.js',
-    'js/courseware/course_home_events.js',
-    'js/courseware/toggle_element_visibility.js'
-]
+REQUIRE_JS_PATH_OVERRIDES = {
+    'js/bookmarks/views/bookmark_button': 'js/bookmarks/views/bookmark_button.js',
+    'js/views/message_banner': 'js/views/message_banner.js',
+    'moment': 'js/vendor/moment.min.js',
+    'jquery.url': 'js/vendor/url.min.js',
+    'js/courseware/course_home_events': 'js/courseware/course_home_events.js',
+    'js/courseware/toggle_element_visibility': 'js/courseware/toggle_element_visibility.js',
+    'js/student_account/logistration_factory': 'js/student_account/logistration_factory.js',
+    'js/student_profile/views/learner_profile_factory': 'js/student_profile/views/learner_profile_factory.js',
+    'js/bookmarks/bookmarks_factory': 'js/bookmarks/bookmarks_factory.js',
+    'js/groups/views/cohorts_dashboard_factory': 'js/groups/views/cohorts_dashboard_factory.js'
+}
 ################################# CELERY ######################################
 
 # Celery's task autodiscovery won't find tasks nested in a tasks package.