Forum


13:03

10/08/2009

Hallo Tony,
during answering on one question I found the bug in the line of code
(and this one too
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
In the case idn will be set to 0 (index of the first column which has key:true property). The line
will returns always undefined value and we will fall down in the case which I descrined above.
Best regards
Oleg
09:49

Moderators
30/10/2007

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.
Most Users Ever Online: 715
Currently Online:
49 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.comModerators: tony: 7721, Rumen[Trirand]: 81
Administrators: admin: 66