Always use document.write to output script tags, regardless of useragent.

inserting scripts into the dom breaks things when using horde's static javascript
caching.

diff --git a/horde/js/hordemap/map.js b/horde/js/hordemap/map.js
index a51169a..65f6503 100644 (file)
--- a/horde/js/hordemap/map.js
+++ b/horde/js/hordemap/map.js
@@ -72,25 +72,14 @@ HordeMap = {
     _includeScripts: function()
     {
         var files = this._includes;
-        var agent = navigator.userAgent;
-        var docWrite = (agent.match("MSIE") || agent.match("Safari"));
+
+        // Need to use document.write instead of inserting into DOM directly
+        // to play nice with horde's javascript caching/loading
         var writeFiles = [];
         for (var i = 0, len = files.length; i < len; i++) {
-            if (docWrite) {
-                writeFiles.push('<script src="' + files[i] + '"></script>');
-            } else {
-                var s = document.createElement("script");
-                s.src = files[i];
-                var h = document.getElementsByTagName("head").length ?
-                           document.getElementsByTagName("head")[0] :
-                           document.body;
-                h.appendChild(s);
-            }
-        }
-
-        if (docWrite) {
-            document.write(writeFiles.join(""));
+            writeFiles.push('<script src="' + files[i] + '"></script>');
         }
+        document.write(writeFiles.join(""));
     },
 
     _addScript: function(s)