jQuery Grid Plugin - jqGrid

Hi,

I needed to combine loading of the included JS into a single request to reduce load time & requests against our environment.

Atteched is a unified diff of the changes to jquery.jqGrid.js:

function jqGridInclude()
 {
-    var pathtojsfiles = "/java/js/"; // need to be ajusted
+    var pathtojsfiles = "js/"; // need to be ajusted
     // if you do not want some module to be included
     // set include to false.
     // by default all modules are included.
     var minver = true;
+    var combineIntoOne = true;
+    var combinedInclude = new Array();
+    var combinedIncludeURL = "/static/combine.php?type=javascript&files=";
+   
     var modules = [
         { include: true, incfile:'grid.base.js',minfile: 'min/grid.base-min.js'}, // jqGrid base
         { include: true, incfile:'grid.formedit.js',minfile: 'min/grid.formedit-min.js' }, // jqGrid Form editing
@@ -14,13 +18,27 @@
         { include: true, incfile:'grid.custom.js',minfile: 'min/grid.custom-min.js'}, //jqGrid custom
         { include: true, incfile:'grid.postext.js',minfile: 'min/grid.postext-min.js'} //jqGrid postext
     ];
-    for(var i=0;i<modules.length; i++)
-    {
+    for(var i=0;i<modules.length; i++) {
+        var filename = "";
         if(modules[i].include === true) {
-            if (minver !== true) IncludeJavaScript(pathtojsfiles+modules[i].incfile,CallMe);
-            else IncludeJavaScript(pathtojsfiles+modules[i].minfile,CallMe);
+            if (minver !== true) {
+            filename = pathtojsfiles+modules[i].incfile;
+          } else {
+            filename = pathtojsfiles+modules[i].minfile;
+          }
+          if (combineIntoOne !== true) {
+            IncludeJavaScript(pathtojsfiles+modules[i].incfile,CallMe);
+          } else {
+            combinedInclude[combinedInclude.length] = filename;
+          }
         }
     }
+
+    if ((combineIntoOne === true) && (combinedInclude.length>0) ) {
+      var fileList = implode(",",combinedInclude);
+      IncludeJavaScript(combinedIncludeURL+fileList,CallMe);
+    }
+       
     function CallMe() {
         return true;
     }
@@ -41,6 +59,15 @@
         };
         return false;
     }
+    function implode( glue, pieces ) {
+      // http://kevin.vanzonneveld.net
+      // +   original by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
+      // +   improved by: _argos
+      // *     example 1: implode(' ', ['Kevin', 'van', 'Zonneveld']);
+      // *     returns 1: 'Kevin van Zonneveld'
+  
+      return ( ( pieces instanceof Array ) ? pieces.join ( glue ) : pieces );
+    }
 }
 
 jqGridInclude();