From 7f1c784629b7f98763b8467d27693e9b170bfef3 Mon Sep 17 00:00:00 2001
From: louyihua <SuperMouseLyh@hotmail.com>
Date: Tue, 15 Jul 2014 11:34:09 +0800
Subject: [PATCH] Add defininition of window.location.origin in both CMS/LMS

The same bug in studio was fixed in #3893, and as TinyMCE is also used in LMS, it should be also fixed in LMS.
Here a common javascript file called 'ie_shim.js' is added, so that in the future it can be used for other IE compatibility purposes too.
---
 cms/templates/base.html         | 11 +++--------
 common/static/js/src/ie_shim.js | 16 ++++++++++++++++
 lms/envs/common.py              |  1 +
 3 files changed, 20 insertions(+), 8 deletions(-)
 create mode 100644 common/static/js/src/ie_shim.js

diff --git a/cms/templates/base.html b/cms/templates/base.html
index e4bd066d24..cae3e8f319 100644
--- a/cms/templates/base.html
+++ b/cms/templates/base.html
@@ -1,4 +1,4 @@
-## -*- coding: utf-8 -*-
+## -*- coding: utf-8 -*-
 <%! from django.utils.translation import ugettext as _ %>
 <%namespace name='static' file='static_content.html'/>
 
@@ -39,12 +39,6 @@
 
     <script type="text/javascript">
     window.baseUrl = "${settings.STATIC_URL}";
-    // Internet Explorer does not have built-in property 'window.location.origin',
-    // we need to create one here as some vendor code such as TinyMCE uses this.
-    if (!window.location.origin) {
-        window.location.origin = window.location.protocol + "//" + window.location.hostname
-                               + (window.location.port ? ':' + window.location.port : '');
-    }
     var require = {
         baseUrl: baseUrl,
         waitSeconds: 60,
@@ -86,6 +80,7 @@
             "accessibility": "js/src/accessibility_tools",
             "draggabilly": "js/vendor/draggabilly.pkgd",
             "URI": "js/vendor/URI.min",
+            "ieshim": "js/src/ie_shim",
 
             // externally hosted files
             "tender": [
@@ -239,7 +234,7 @@
         deps: ["jquery", "gettext"],
         callback: function() {
             // load other scripts on every page, after jquery loads
-            require(["js/base", "coffee/src/main", "coffee/src/logger", "datepair", "accessibility"]);
+            require(["js/base", "coffee/src/main", "coffee/src/logger", "datepair", "accessibility", "ieshim"]);
             // we need "datepair" because it dynamically modifies the page
             // when it is loaded -- yuck!
         }
diff --git a/common/static/js/src/ie_shim.js b/common/static/js/src/ie_shim.js
new file mode 100644
index 0000000000..cfd6f75c6f
--- /dev/null
+++ b/common/static/js/src/ie_shim.js
@@ -0,0 +1,16 @@
+/*
+ * This file is used for keeping compatibility with Internet Explorer.
+ * As starting with IE10, the conditional comments are not supported, this file
+ * will always be loaded whether the browser is IE or not. Therefore, the code
+ * here should not make any assumption and should always detect the execution
+ * conditions itself.
+ */
+
+// Shim name: Create the attribute of 'window.location.origin'
+// IE version: 11 or earlier, 12 or later not tested
+// Internet Explorer does not have built-in property 'window.location.origin',
+// we need to create one here as some vendor code such as TinyMCE uses this.
+if (!window.location.origin) {
+    window.location.origin = window.location.protocol + "//" + window.location.hostname
+                           + (window.location.port ? ':' + window.location.port : '');
+}
diff --git a/lms/envs/common.py b/lms/envs/common.py
index d1d764341d..05f1b53229 100644
--- a/lms/envs/common.py
+++ b/lms/envs/common.py
@@ -1002,6 +1002,7 @@ PIPELINE_JS = {
             'js/query-params.js',
             'js/src/utility.js',
             'js/src/accessibility_tools.js',
+            'js/src/ie_shim.js',
         ],
         'output_filename': 'js/lms-application.js',