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
undefined class added in addRowData (grid.base.js:1849-1905)
10/03/2010
13:10
Avatar
ben
Member
Members
Forum Posts: 9
Member Since:
19/10/2009
sp_UserOfflineSmall Offline

The cna variable (one dedicated to store the class for alternating rows color, if needed, and an empty string otherwise) is not always defined.

Indeed, line 1885 sets it but is under the condition that "aradd" evaluates to true on the line.

Therefore, line 1905:

     row = "<tr id=\""+rowid+"\" role=\"row\" class=\"ui-widget-content jqgrow ui-row-"+t.p.direction+" "+cna+"\">" + row+"</tr>";

sometimes results in "printing" undefined in the tr classes.

To solve the problem, either add a ternary operator to check whether cna is defined on line 1905 and use an empty string otherwise (i.e. instead of appending

cna

, append

cna ? cna : ""

)

or define cna with a default empty string value somewhere before this call.

Thanks for your time and consideration.

All lines numbers & code extracts were based on the repository version which commit hash is 2d3d98a72c3b39a047d88aa0c09916e020a36dd7.

11/03/2010
14:52
Avatar
tony
Sofia, Bulgaria
Moderator
Members

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

Hello,

Thanks. Fixed in GitHub

Best Regards

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