jQuery Grid Plugin - jqGrid

Hello Tony,

You made last time many changes in the code and I decide that it can be useful to verify the code with respect of JSLint. I used current code from the github.

Here are the summary of the corresponding suggestions:

In the file of grid.common.js. Lines 100-104

Should be modified to the following

(semicolon at the end of the first an the third lines and two {} blocks)

In the file grid.formedit.js the variable copydata defined in the line 353 will be never used. The variable with the same name are defined in the line 625 will be used only. In the same way variable opt defined in the line 1186 will be also never used. Probably in some lines after the line 1186 instead of opt variable are used another variable.

At the end the line 1357 of the file grid.base.js are missing semicolon(;)

In the line 1718 and 1731 are used empty blocks. One can eliminate the code in a simple way.

In the file grid.setcolumns.js in the line 54 are missed var vor i. Currently i will be global.

In the line 60 are missing semicolon.

In the line 91 the statement after the if(p.closeAfterSubmit) should be enclosed in block {}.

In the line 31-33 of the file grid.treegrid.js are used empfy block which could be better removed. In the line 45 if(rd[loaded] != undefined) should be better replaced with if(rd[loaded] !== undefined) or if(typedef(rd[loaded]) !== 'undefined')

In the lines 398 and 403 of the file jquery.fmatter.js the statement after 'if' should be placed in the block {}

In the file JsonXml.js one should in the lines 61, 71, 100, 239.

Best regards
Oleg