jQuery Grid Plugin - jqGridForum

Topic RSS

json bug with advanced search

04/12/2009
13:01

spider

Member

Members

Forum Posts: 5

Member Since:
09/09/2009

Offline

problem:

Advance searching does not escape strings,

thats why a \\ or a " will break the json format.

reproduce:

- go to http://trirand.com/jqgrid/jqgrid.html

- open New in version 3.5 -> Advanced Searching

- click on the search button

- insert

<< Inv No equal \\ >> or

<< InvNo equal " >>

(you can search for any column..)

--> you will see the whole list instead off the ones equal to \\ or "

this all hapens in jquery.searchFilter.js in line 438

solution:

\\ or " needs to be escaped with an additional \\

05/12/2009
12:55

tony

Sofia, Bulgaria

Moderator

Members

Moderators

Forum Posts: 7721

Member Since:
30/10/2007

Offline

Hello,

Thank you for this. This is true.

I'm not sure, but maybe it is a good idea to use encodeURIComponent.

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.

07/12/2009
14:07

spider

Member

Members

Forum Posts: 5

Member Since:
09/09/2009

Offline

There is a problem with encodeURIComponent:

After your server side script decoded the json data, it must make a "decodeURIComponent".

Depending on what language you use, this could be a pain in the a**.

My suggestion is to use just:

----

tData = tData.replace(/\\\\/g,'\\\\\\\\');

tData = tData.replace(/\\"/g,'\\\\"');

ruleGroup += "\\"data\\":\\"" + tData + "\\"}";

----

with this you should not need to make any server side changes.

08/12/2009
17:28

tony

Sofia, Bulgaria

Moderator

Members

Moderators

Forum Posts: 7721

Member Since:
30/10/2007

Offline

Hello,

You are right. Will implement it.Simple things are much better

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.

All RSS

Forum Timezone: Europe/Sofia

Most Users Ever Online: 715

Currently Online:
74 Guest(s)

Currently Browsing this Page:
1 Guest(s)