Forum

November 2nd, 2014
A A A
Avatar

Lost password?
Advanced Search

— Forum Scope —




— Match —





— Forum Options —





Minimum search word length is 3 characters - maximum search word length is 84 characters

The forums are currently locked and only available for read only access
sp_Feed Topic RSS sp_Related Related Topics sp_TopicIcon
wrong definition of default values of addParams
Tags: inlineNav
19/04/2013
10:43
Avatar
OlegK
Germany
Member
Members
Forum Posts: 1255
Member Since:
10/08/2009
sp_UserOfflineSmall Offline

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 

21/04/2013
11:04
Avatar
tony
Sofia, Bulgaria
Moderator
Members

Moderators
Forum Posts: 7721
Member Since:
30/10/2007
sp_UserOfflineSmall Offline

Thanks Oleg.

Fixed.

Tony

For professional UI suites for Java Script and PHP visit us at our commercial products site - guriddo.net - by the very same guys that created jqGrid.

Forum Timezone: Europe/Sofia

Most Users Ever Online: 715

Currently Online:
31 Guest(s)

Currently Browsing this Page:
1 Guest(s)

Top Posters:

OlegK: 1255

markw65: 179

kobruleht: 144

phicarre: 132

YamilBracho: 124

Renso: 118

Member Stats:

Guest Posters: 447

Members: 11373

Moderators: 2

Admins: 1

Forum Stats:

Groups: 1

Forums: 8

Topics: 10592

Posts: 31289

Newest Members:

, razia, Prankie, psky, praveen neelam, greg.valainis@pa-tech.com

Moderators: tony: 7721, Rumen[Trirand]: 81

Administrators: admin: 66

Comments are closed.
Privacy Policy   Terms and Conditions   Contact Information