This is a migrated thread and some comments may be shown as answers.

[Solved] Bug removing folumn filter with apostrophe

8 Answers 74 Views
Grid
This is a migrated thread and some comments may be shown as answers.
This question is locked. New answers and comments are not allowed.
Omar Piani
Top achievements
Rank 1
Omar Piani asked on 15 Apr 2010, 11:23 AM
Hi great telerik,

i've found a little bug:

Firebug returns :

uncaught exception: Syntax error, unrecognized expression: id$=d'alessandro]
Line 0

after filtering a column with a string with apostrophes (d'alessandro is a common italian surname)
filter work correcly, but i cant remove the filter, if i click filter icon, nothing appends, and i get the error.
A normal problem of not escaped apostrophes on javascript?

regards.

8 Answers, 1 is accepted

Sort by
0
Omar Piani
Top achievements
Rank 1
answered on 15 Apr 2010, 11:24 AM
folumn?? lol

forget:
I have MVC 1, asp.net 3.5, Telerik dll version 2010.1.309.135
0
Omar Piani
Top achievements
Rank 1
answered on 19 Apr 2010, 08:30 AM
up :(
0
Accepted
Atanas Korchev
Telerik team
answered on 19 Apr 2010, 10:15 AM
Hello Omar Piani,

I was able to reproduce this exception. To fix it you need to patch the provided source code. Replace the following code in telerik.grid.filtering.js:

$.each(column.filters || [], function(i) {
                $filterMenu.find('.t-filter-operator:eq(' + i + ')')
                           .val(this.operator)
                           .end()
                           .find(':text:eq(' + i + '),select:not(.t-filter-operator):eq(' + i + ')')
                           .val(value(column, this.value))
                           .end()
                           .find(':radio[id$=' + this.value + ']')
                           .attr('checked', true);
            });

with this
$.each(column.filters || [], function(i) {
                $filterMenu.find('.t-filter-operator:eq(' + i + ')')
                           .val(this.operator)
                           .end()
                           .find(':text:eq(' + i + '),select:not(.t-filter-operator):eq(' + i + ')')
                           .val(value(column, this.value));
                
                if (column.type == 'Boolean')
                    $filterMenu.find(':radio[id$=' + this.value + ']')
                               .attr('checked', true);
            });
Regards,
Atanas Korchev
the Telerik team

Do you want to have your say when we set our development plans? Do you want to know when a feature you care about is added or when a bug fixed? Explore the Telerik Public Issue Tracking system and vote to affect the priority of the items.
0
Omar Piani
Top achievements
Rank 1
answered on 19 Apr 2010, 10:22 AM
thanks!
0
James
Top achievements
Rank 1
answered on 25 Jun 2010, 04:30 PM
We have the same issue and have been using the dll as opposed to a specific js file. How do we include the source code in our solution? We have looked through our download and cannot find this particular js file (telerik.grid.filtering.js).

Thanks in advance,
James Hollyhomes.
0
Atanas Korchev
Telerik team
answered on 25 Jun 2010, 04:39 PM
Hello James,

This problem has been fixed in the latest official release (2010.1.518). You can download it and try upgrading.

telerik.grid.filtering.js is included in the source code. It is located in the Scripts folder within Telerik.Web.Mvc.

Regards,
Atanas Korchev
the Telerik team
Do you want to have your say when we set our development plans? Do you want to know when a feature you care about is added or when a bug fixed? Explore the Telerik Public Issue Tracking system and vote to affect the priority of the items
0
James
Top achievements
Rank 1
answered on 25 Jun 2010, 04:46 PM
Thanks very much for you quick response.

My apologies but we are actually using the AJAX asp.net controls not the MVC.  Do we need to post this question within the other forum ?  We have tried filtering on an apostrophe using the latest version in .NET 4 and this still does not work.

Thanks James Hollyhomes
0
Atanas Korchev
Telerik team
answered on 25 Jun 2010, 04:57 PM
Hi James,

Yes you should open a new forum thread in the Grid for ASP.NET Ajax forum (or open a support ticket) .

Regards,
Atanas Korchev
the Telerik team
Do you want to have your say when we set our development plans? Do you want to know when a feature you care about is added or when a bug fixed? Explore the Telerik Public Issue Tracking system and vote to affect the priority of the items
Tags
Grid
Asked by
Omar Piani
Top achievements
Rank 1
Answers by
Omar Piani
Top achievements
Rank 1
Atanas Korchev
Telerik team
James
Top achievements
Rank 1
Share this question
or