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_TopicIcon
small bug with reading id=0
25/03/2013
13:03
Avatar
OlegK
Germany
Member
Members
Forum Posts: 1255
Member Since:
10/08/2009
sp_UserOfflineSmall Offline

Hallo Tony,

during answering on one question I found the bug in the line of code

idr = ccur !== undefined ? ccur[idn] || idr : idr;

(and this one too

idr = ccur2[idn] || idr;

later in the code). The problem one have if ccur[idn] (or ccur2[idn]) is 0 (the id of the row is numeric 0). In the case it will be used idr instead of ccur[idn] (or ccur2[idn]) which can follow to id dupplicates.

To fix the problem one should test for ccur[idn] !== undefined instead of usage || operation. Additionally one can change the first test ccur !== undefined to ccur != null, to be sure that ccur is neither undefined nor null before usage of ccur[idn].

By the way to reproduce the proglem one can add key: true to some colum and use standard format where cell array is defined in JSON input data. For example

{"rows":[{"cell":[0,...]}, {"cell":[1,...]}, ...]}

In the case idn will be set to 0 (index of the first column which has key:true property). The line

idr = $.jgrid.getAccessor(cur,idn);

will returns always undefined value and we will fall down in the case which I descrined above.

Best regards
Oleg 

27/03/2013
09:49
Avatar
tony
Sofia, Bulgaria
Moderator
Members

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

Hello Oleg,

Thanks.

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:
73 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