Ticket #1456: url_multi_value_fixed.patch

turbogears/controllers.py

@@ -512 +512 @@
         else:
             pairs = [(key, value)]
         for (k,v) in pairs:
-            if isinstance(value, unicode):
-                value = value.encode("utf8")
+            if isinstance(v, unicode):
+                v = v.encode("utf8")
             args.append("%s=%s" % (k, urllib.quote(str(v))))
     if args:
         result += "?" + "&".join(args)

turbogears/tests/test_controllers.py

@@ -572 +572 @@
         assert url("/foo", bar=("asdf","qwer")) in \
                 ["/foo?bar=qwer&bar=asdf", "/foo?bar=asdf&bar=qwer"]
 
+    def test_unicode(self):
+        "tg.url can handle unicode parameters"
+        assert url('/',x=
+            u'\N{LATIN SMALL LETTER A WITH GRAVE}'+
+            u'\N{LATIN SMALL LETTER E WITH GRAVE}'+
+            u'\N{LATIN SMALL LETTER I WITH GRAVE}'+
+            u'\N{LATIN SMALL LETTER O WITH GRAVE}'+
+            u'\N{LATIN SMALL LETTER U WITH GRAVE}') == '/?x=%C3%A0%C3%A8%C3%AC%C3%B2%C3%B9'
+
+    def test_list(self):
+        "tg.url can handle list parameters, with unicode too"
+        assert url('/',foo=['bar', u'\N{LATIN SMALL LETTER A WITH GRAVE}']) == '/?foo=bar&foo=%C3%A0'
+
+
     def tearDown(self):
         turbogears.config.update({"server.webpath" : ""})
         turbogears.startup.startTurboGears()