jQuery Grid Plugin - jqGrid - Topic: wrong definition of default values of addParams http://www.trirand.com/blog/?page_id=393/bugs/wrong-definition-of-default-values-of-addparams Simple:Press Version 5.7.5.3 tony on wrong definition of default values of addParams http://www.trirand.com/blog/?page_id=393/bugs/wrong-definition-of-default-values-of-addparams#p28695 Bugs http://www.trirand.com/blog/?page_id=393/bugs/wrong-definition-of-default-values-of-addparams#p28695 Thanks Oleg.

Fixed.

Tony

]]> Sun, 21 Apr 2013 11:04:28 +0300 OlegK on wrong definition of default values of addParams http://www.trirand.com/blog/?page_id=393/bugs/wrong-definition-of-default-values-of-addparams#p28684 Bugs http://www.trirand.com/blog/?page_id=393/bugs/wrong-definition-of-default-values-of-addparams#p28684 Hello Tony,

default values of inlineNav ar defined in the lines

o = $.extend({
    edit: true,
    ...
    addParams : {},
    ...
}, $.jgrid.nav, o ||{}); 

but one uses later (see the line)

if(!o.addParams.addRowParams.extraparam) {

and get null reference error (o.addParams.addRowParams is undefined) if inlineNav will be called with the simplest options: $("#grid").jqGrid("inlineNav", "#pager");

To fix the bug one can modify the code to 

o = $.extend(true, {
    edit: true,
    ...
    addParams : {addRowParams: {extraparam: {}}},
    ...
}, $.jgrid.nav, o ||{}); 

and removing now unneeded lines

if(!o.addParams.addRowParams.extraparam) {
    o.addParams.addRowParams.extraparam = {};
} 

Alternatively one can add lines

if(!o.addParams.addRowParams) {
    o.addParams.addRowParams = {};
}

Best regards
Oleg 

]]> Fri, 19 Apr 2013 10:43:58 +0300