jQuery Grid Plugin - jqGrid - Topic: multiple select problem when row id has whitespace

tony on multiple select problem when row id has whitespace

Fri, 18 Sep 2009 10:48:40 +0300

Thanks adige,

Thanks Mark, as usuall good catch, but I think that we should follow some standarts about what are allowed characters for the id

http://www.w3.org/TR/REC-xml/#NT-Name

It seems that XHTL is a subset of HTML specs,

So I think we should apply this for the following chars (finally): [,],-,_,:,.

I will apply your fix where is no need to make a series of replace calls.

This should remember me that I must correct the id's in the grid where we should not have duplicate id.

Tony

markw65 on multiple select problem when row id has whitespace

Fri, 18 Sep 2009 08:57:54 +0300

According to the jquery page, here is the complete list of chars that need quoting: #;&,.+*~':"!^$[]()=>|/

So clearly there's a few more that need adding 🙂

Also, rather than adding a series of replace calls, it could be written: sid.replace(/([\\#\\;\\&\\,\\.\\+\\*\\~\\'\\:\\"\\!\\^\\$\\[\\]\$\$\\=\\>\\|\\/])/g,"\\\\$1");

Or maybe just quote everything but the "usual selector chars": sid.replace(/([^-a-zA-Z_0-9])/g, "\\\\$1");

adige on multiple select problem when row id has whitespace

Fri, 18 Sep 2009 07:06:47 +0300

Hi,

If row id has whitespace than grid select doesn't work properly. When select changes the old selected row still looks selected (multiple select).

in grid.base.js (jqGrid 5.3.3) line 64 i added bold part, now it works fine even if id has whitespace:

return sid.replace(/:/g,"\\\\:").replace(/\\./g,"\\\\.").replace(/\\[/g,"\\\\[").replace(/\\]/g,"\\\\]").replace(/ /g,"\\\\ ");