From ab339239f86c3338e7e5ccc6161dbfa581a9ff8f Mon Sep 17 00:00:00 2001
From: Rocky Duan <dementrock@gmail.com>
Date: Mon, 30 Jul 2012 13:44:00 -0400
Subject: [PATCH] use single js dependencies for discussion in forum /
 courseware

---
 lms/templates/courseware.html                 | 24 ++-------------
 .../discussion/_js_dependencies.html          | 30 +++++++++++++++++++
 lms/templates/discussion/index.html           | 28 +----------------
 3 files changed, 33 insertions(+), 49 deletions(-)
 create mode 100644 lms/templates/discussion/_js_dependencies.html

diff --git a/lms/templates/courseware.html b/lms/templates/courseware.html
index 78114d601f..eb8aed7e1b 100644
--- a/lms/templates/courseware.html
+++ b/lms/templates/courseware.html
@@ -28,28 +28,8 @@
   
    <%static:js group='courseware'/>
 
-  <script type="text/x-mathjax-config">
-    MathJax.Hub.Config({
-      tex2jax: {
-        inlineMath: [
-          ["$","$"],
-        ],
-        displayMath: [
-          ["$$","$$"],
-        ]
-      }
-    });
-  </script>
-
-  <!-- This must appear after all mathjax-config blocks, so it is after the imports from the other templates.
-       It can't be run through static.url because MathJax uses crazy url introspection to do lazy loading of
-       MathJax extension libraries -->
-  <script type="text/javascript" src="/static/js/vendor/mathjax-MathJax-c9db6ac/MathJax.js?config=TeX-MML-AM_HTMLorMML-full"></script>
-  <script type="text/javascript" src="${static.url('js/vendor/split.js')}"></script>
-  <script type="text/javascript" src="${static.url('js/vendor/jquery.ajaxfileupload.js')}"></script>
-  <script type="text/javascript" src="${static.url('js/vendor/Markdown.Converter.js')}"></script>
-  <script type="text/javascript" src="${static.url('js/vendor/Markdown.Sanitizer.js')}"></script>
-  <script type="text/javascript" src="${static.url('js/vendor/Markdown.Editor.js')}"></script>
+  <%include file="discussion/_js_dependencies.html" />
+  
 
   <!-- TODO: http://docs.jquery.com/Plugins/Validation -->
   <script type="text/javascript">
diff --git a/lms/templates/discussion/_js_dependencies.html b/lms/templates/discussion/_js_dependencies.html
new file mode 100644
index 0000000000..14f25e688d
--- /dev/null
+++ b/lms/templates/discussion/_js_dependencies.html
@@ -0,0 +1,30 @@
+
+<%namespace name='static' file='../static_content.html'/>
+
+<!-- The configuration below is different from the main mathjax config because dollar signs make it easier
+       to integrate with Markdown.  -->
+  <script type="text/x-mathjax-config">
+    MathJax.Hub.Config({
+      tex2jax: {
+        inlineMath: [
+          ["$","$"],
+        ],
+        displayMath: [
+          ["$$","$$"],
+        ]
+      }
+    });
+  </script>
+
+  ## This must appear after all mathjax-config blocks, so it is after the imports from the other templates.
+  ## It can't be run through static.url because MathJax uses crazy url introspection to do lazy loading of MathJax extension libraries
+  <script type="text/javascript" src="/static/js/vendor/mathjax-MathJax-c9db6ac/MathJax.js?config=TeX-MML-AM_HTMLorMML-full"></script>
+  <script type="text/javascript" src="${static.url('js/vendor/split.js')}"></script>
+  <script type="text/javascript" src="${static.url('js/vendor/jquery.ajaxfileupload.js')}"></script>
+  <script type="text/javascript" src="${static.url('js/vendor/Markdown.Converter.js')}"></script>
+  <script type="text/javascript" src="${static.url('js/vendor/Markdown.Sanitizer.js')}"></script>
+  <script type="text/javascript" src="${static.url('js/vendor/Markdown.Editor.js')}"></script>
+  <script type="text/javascript" src="${static.url('js/vendor/jquery.autocomplete.js')}"></script>
+  <script type="text/javascript" src="${static.url('js/vendor/jquery.tagsinput.js')}"></script>
+  <link href="${static.url('css/vendor/jquery.tagsinput.css')}" rel="stylesheet" type="text/css">
+  <link href="${static.url('css/vendor/jquery.autocomplete.css')}" rel="stylesheet" type="text/css">
diff --git a/lms/templates/discussion/index.html b/lms/templates/discussion/index.html
index 6a1ab7d130..cb5b42333c 100644
--- a/lms/templates/discussion/index.html
+++ b/lms/templates/discussion/index.html
@@ -8,33 +8,7 @@
 </%block>
 
 <%block name="js_extra">
-  <!-- The configuration below is different from the main mathjax config because dollar signs make it easier
-       to integrate with Markdown.  -->
-  <script type="text/x-mathjax-config">
-    MathJax.Hub.Config({
-      tex2jax: {
-        inlineMath: [
-          ["$","$"],
-        ],
-        displayMath: [
-          ["$$","$$"],
-        ]
-      }
-    });
-  </script>
-
-  ## This must appear after all mathjax-config blocks, so it is after the imports from the other templates.
-  ## It can't be run through static.url because MathJax uses crazy url introspection to do lazy loading of MathJax extension libraries
-  <script type="text/javascript" src="/static/js/vendor/mathjax-MathJax-c9db6ac/MathJax.js?config=TeX-MML-AM_HTMLorMML-full"></script>
-  <script type="text/javascript" src="${static.url('js/vendor/split.js')}"></script>
-  <script type="text/javascript" src="${static.url('js/vendor/jquery.ajaxfileupload.js')}"></script>
-  <script type="text/javascript" src="${static.url('js/vendor/Markdown.Converter.js')}"></script>
-  <script type="text/javascript" src="${static.url('js/vendor/Markdown.Sanitizer.js')}"></script>
-  <script type="text/javascript" src="${static.url('js/vendor/Markdown.Editor.js')}"></script>
-  <script type="text/javascript" src="${static.url('js/vendor/jquery.autocomplete.js')}"></script>
-  <script type="text/javascript" src="${static.url('js/vendor/jquery.tagsinput.js')}"></script>
-  <link href="${static.url('css/vendor/jquery.tagsinput.css')}" rel="stylesheet" type="text/css">
-  <link href="${static.url('css/vendor/jquery.autocomplete.css')}" rel="stylesheet" type="text/css">
+<%include file="_js_dependencies.html" />
 </%block>
 
 <%include file="../course_navigation.html" args="active_page='discussion'" />