Merge pull request #1310 from edx/db/requirejs-gettext

load gettext on every page

cms/djangoapps/contentstore/features/component.py

@@ -5,6 +5,7 @@ from lettuce import world, step
 from nose.tools import assert_true, assert_in, assert_equal  # pylint: disable=E0611
 from common import create_studio_user, add_course_author, log_into_studio
 
+
 @step(u'I am in Studio editing a new unit$')
 def add_unit(step):
     world.clear_courses()
@@ -18,7 +19,7 @@ def add_unit(step):
     add_course_author(user, course)
     log_into_studio()
     world.wait_for_requirejs([
-        "jquery", "js/models/course", "coffee/src/models/module",
+        "jquery", "gettext", "js/models/course", "coffee/src/models/module",
         "coffee/src/views/unit", "jquery.ui",
     ])
     world.wait_for_mathjax()
@@ -39,13 +40,13 @@ def add_a_single_step_component(step):
         css_selector = 'a[data-type="{}"]'.format(component.lower())
         world.css_click(css_selector)
 
-	# In the current implementation, all the "new component"
-	# buttons are handled by one BackBone.js view.
-	# If we click two buttons at super-human speed,
-	# the view will miss the second click while it's
-	# processing the first.
-	# To account for this, we wait for each component
-	# to be created before clicking the next component.
+        # In the current implementation, all the "new component"
+        # buttons are handled by one BackBone.js view.
+        # If we click two buttons at super-human speed,
+        # the view will miss the second click while it's
+        # processing the first.
+        # To account for this, we wait for each component
+        # to be created before clicking the next component.
         world.wait_for_visible('section.xmodule_{}Module'.format(component))
 
 

cms/static/coffee/src/main.coffee

@@ -1,5 +1,6 @@
-define ["jquery", "underscore.string", "backbone", "js/views/feedback_notification", "jquery.cookie"],
-($, str, Backbone, NotificationView) ->
+define ["domReady", "jquery", "underscore.string", "backbone", "gettext",
+        "js/views/feedback_notification", "jquery.cookie"],
+(domReady, $, str, Backbone, gettext, NotificationView) ->
   AjaxPrefix.addAjaxPrefix jQuery, ->
     $("meta[name='path_prefix']").attr('content')
 
@@ -36,5 +37,5 @@ define ["jquery", "underscore.string", "backbone", "js/views/feedback_notificati
     if onTouchBasedDevice()
       $('body').addClass 'touch-based-device'
 
-  $(main)
+  domReady(main)
   return main

cms/templates/base.html

@@ -159,8 +159,8 @@ var require = {
             deps: ["coffee/src/ajax_prefix"]
         }
     },
-    // load jquery automatically
-    deps: ["jquery"],
+    // load jquery and gettext automatically
+    deps: ["jquery", "gettext"],
     callback: function() {
         // load other scripts on every page, after jquery loads
         require(["js/base", "coffee/src/main", "coffee/src/logger", "datepair"]);

cms/templates/settings.html

@@ -17,6 +17,10 @@
   </script>
 
   <script type="text/javascript">
+window.CMS = window.CMS || {};
+CMS.URL = CMS.URL || {};
+CMS.URL.UPLOAD_ASSET = '${upload_asset_url}';
+
 require(["domReady!", "jquery", "js/models/settings/course_details", "js/views/settings/main"],
   function(doc, $, CourseDetailsModel, MainView) {
     // hilighting labels when fields are focused in
@@ -37,8 +41,6 @@ require(["domReady!", "jquery", "js/models/settings/course_details", "js/views/s
       },
       reset: true
     });
-
-    CMS.URL.UPLOAD_ASSET = '${upload_asset_url}';
 });
   </script>
 </%block>