jQuery Grid Plugin - jqGrid

Hello Tony,

I'm surprised, but $.jgrid.checkDate used in case of usage editrules: { date: true } works correctly only in case of very strict restrictions. For example if one use formatoptions.newformat or datefmt in form of $.jgrid.formatter.date.masks (like formatoptions: {newformat: "ShortDate"}) one get error message with correct formatted dates. The formats like "n/j/Y" (standard date format from grid.locale-en.js) don't work too (just try formatoptions: {newformat: "n/j/Y"}).

If seems that $.jgrid.checkDate woks only if the format of the date use "y", "m" and "d" only. The documentation describes that datefmt uses. On the other side the source code prefer formatoptions.newformat and uses datefmt only if formatoptions.newformat is not defined.

If would be good to use some common other existing code, like $.jqgrid.parseDate or $.fmatter.util.DateFormat. If would be good to move the common part of function $.jqgrid.parseDate and $.fmatter.util.DateFormat in separate place and make all thre methods $.jgrid.checkDate, $.jqgrid.parseDate and $.fmatter.util.DateFormat use the common parsing method.

In any way would be nice to fix the code of documantation to describe the current restrictions of editrules: { date: true }.

Best regards
Oleg