i have exposed filters on a site where views remembers the user´s input. you get the results below in a table. one of the columns is sortable and if you press it then all german ü ä ö changes in the textfield of the exposed filter to strange signs. i think the javascript has a wrong coding, it should have utf8 i think

CommentFileSizeAuthor
#6 decodeURI.patch969 bytesmerlinofchaos
#4 sshot-1.png10.3 KBmp
#4 sshot-2.png10.74 KBmp
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented

i disabled my js and tryed out his site
-> sort worked perfect

=> the problem should be in js or sort related php

ajax_view.js

mp’s picture

mp CreditAttribution: mp commented

if you turn of js then the focus of the website jumps to the top and doesn´t stay at the place where you press the sort link. so you always have to scroll down.

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented

There isn't really enough info here for me to recreate this and debug this. I don't instantly know what's wrong and you talk about exposed filters and clickable sorts as though they're the same thing.

mp’s picture

mp CreditAttribution: mp commented
FileSize
sshot-2.png10.74 KB
sshot-1.png10.3 KB

i made 2 screenshots ... i hope you see now what the problem is
if not contact me please on IRC, i´m online every day, nick J-O-K-E-R

thanks + best wishes
markus

mp’s picture

mp CreditAttribution: mp commented

merlinofchaos - you see now what´s the problem is? is it possible to solve it?

best wishes
markus

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Component: table style » User interface
Status: Active » Needs review
FileSize
decodeURI.patch969 bytes

This wasn't code I wrote so it took me a bit to track this down.

I'm not 100% sure this is the right patch, so I want to get Nedjo to say yes or no, and you to let me know if this patch fixes it:

The summary is that it appears we should be using decodeURI instead of unescape when parsing query parameters.

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Status: Needs review » Fixed

Ok, got the go ahead from Nedjo, tho it uses decodeURIComponent instead of decodeURI. Committed.

mp’s picture

mp CreditAttribution: mp commented

works perfect :)
thanks a lot

best wishes
markus

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.