jQuery Grid Plugin - jqGrid

Hello Tony,

the file jquery.jqGrid-4.7.0.zip with the new version of jqGrid (from the official trirand download page) don't include source map file. Moreover I made some tests and could see small problem in generation of the source map: the sources property from the source map file. It seems to me that one should add small step to the distribution of jqGrid.

Let us we want that debugging of jquery.jqGrid.min.js in Google Chrome or Firefox would show the source lines of jquery.jqGrid.src.js. To implemnt this one could add the file jquery.jqGrid.min.map created by closure compiler. One need just verify that the last line of jquery.jqGrid.min.js contains the following

//@ sourceMappingURL=jquery.jqGrid.min.map

If one have jquery.jqGrid.min.map one should verify that and that the file contains the following lines 

{
"version":3,
"file":"jquery.jqGrid.min.js",
"sources":["jquery.jqGrid.src.js"],
...
}

It makes correct mapping from jquery.jqGrid.min.js to jquery.jqGrid.min.map and from jquery.jqGrid.min.map to jquery.jqGrid.src.js.

If the file of source entries have another values after working of closure compiler one can just make text replacement to modify jquery.jqGrid.min.map.

I'm not good in gradle but extending of doLast {...} of jqgrid-min task to the following work.

task 'jqgrid-min'(dependsOn: 'jqgrid') {
    description = "Minimize ..."

    def outputFile = new File(distdir, 'jquery.jqGrid.min.map')

    inputs.file jqGridFile
    outputs.file jqGridMinFile
    doLast {
        ant.taskdef(name: 'jscompile', ...
        ant.jscompile(
        ...
        }
        def fileContent = jqGridSrcMapFile.getText()
        fileContent = fileContent.replace('"sources":["dist/jquery.jqGrid.js"],', '"sources":["jquery.jqGrid.src.js"],')
        replace('"file":"jquery.jqGrid.map.js",', '"file":"jquery.jqGrid.min.js",')
        outputFile.write(fileContent)
    }
}

It doesn't matter how exactly jquery.jqGrid.min.js and jquery.jqGrid.min.map will be patched. it's just important that the described above entries are exist.

As an example one can try to debug in Google Chrome the demo, which uses jquery.jqGrid.min.js and jquery-1.11.1.min.js. One will sees the source code of jquery.jqGrid.src.js and jquery-1.11.1.js instead.

It's clear that source mapping is not perfect. The names of compiled/compressed variable will be not mapped to the names from the original source. Nevertheless it would be helpful to have jquery.jqGrid.min.map file which corresponds jquery.jqGrid.src.js to included in jquery.jqGrid-4.7.0.zip. Even without "//@ sourceMappingURL=..." line and without patched it jquery.jqGrid.min.map have some value for the users (like jQuery do this. jquery-1.11.1.min.js have no "//@ sourceMappingURL=..." statement, but jquery-1.11.1.min.map can be still used to make the reference to jquery-1.11.1.js). The users could makes the required modifications yourself if it will be difficult to automate the building process. One needs just to have jquery.jqGrid.min.map file which corresponds jquery.jqGrid.min.js and jquery.jqGrid.src.js. 

Best regards
Oleg

P.S. It would be helpful to publish the new 4.7.0 version of jqGrid on well known public places: nuget, the jQuery plugin registry, CDNJS CDN, jsDelivr CDN and other.