Code review comment for lp:~cprov/launchpad/bug-404144-unicode-ppa-searches

Revision history for this message
Celso Providelo (cprov) wrote :

Tom,

I've discussed a simpler approach to fix this problem with Michael.

It consists of changing query_string_parameters() to already decode variables, and on the Distribution:+search view we can simply use the original form QUERY_STRING, instead of rebuilding it with urlencode.

Can you please re-review it ? https://pastebin.canonical.com/20373/

« Back to merge proposal