Merge pull request #67 from e-sites/master

Added public urlArgs method (as suggested in #66)

Author: ded

src/script.js

@@ -15,6 +15,7 @@
     , delay = {}
     , scripts = {}
     , scriptpath
+    , urlArgs
 
   function every(ar, fn) {
     for (var i = 0, j = ar.length; i < j; ++i) if (!fn(ar[i])) return f
@@ -47,7 +48,7 @@
     setTimeout(function () {
       each(paths, function loading(path, force) {
         if (path === null) return callback()
-        path = !force && !/^https?:\/\//.test(path) && scriptpath ? scriptpath + path + '.js' : path
+        path = !force && path.indexOf('.js') === -1 && !/^https?:\/\//.test(path) && scriptpath ? scriptpath + path + '.js' : path
         if (scripts[path]) {
           if (id) ids[id] = 1
           if (scripts[path] == 2) callback()
@@ -73,7 +74,7 @@
       fn()
     }
     el.async = 1
-    el.src = path
+    el.src = urlArgs ? path + (path.indexOf('?') === -1 ? '?' : '&') + urlArgs : path;
     head.insertBefore(el, head.lastChild)
   }
 
@@ -89,6 +90,9 @@
   $script.path = function (p) {
     scriptpath = p
   }
+  $script.urlArgs = function (str) {
+    urlArgs = str;
+  }
   $script.ready = function (deps, ready, req) {
     deps = deps[push] ? deps : [deps]
     var missing = [];

tests/tests.js

@@ -90,6 +90,16 @@ script('../node_modules/domready/ready.js', function () {
           ok(typeof morpheus !== 'undefined', 'loaded morpheus.js from http')
         })
       })
+      
+      test('passing urlArgs', 2, function () {
+        script.urlArgs('key=value')
+        script(['../demos/js/foo.js'], function() {
+          ok(true, 'foo has been loaded with given urlArgs')
+        })
+        script(['../demos/js/bar.js?foo=bar'], function() {
+          ok(true, 'bar has been loaded and urlArgs have been added to existing query string')
+        })
+      })
 
       test('syncronous ordered loading', 2, function () {
         script.order(['order-a', 'order-b', 'order-c'], 'ordered-id', function () {