Changeset 3755

Timestamp:

11/24/07 19:55:52 (8 months ago)

Author:

carndt

Message:

Fix ticket #1456 (url allow key with multiple values)

Files:

• branches/1.1/CHANGELOG.txt (modified) (1 diff)
• branches/1.1/turbogears/controllers.py (modified) (1 diff)
• branches/1.1/turbogears/tests/test_controllers.py (modified) (1 diff)

branches/1.1/CHANGELOG.txt

@@ -68 +68 @@
-  Every string starting with a dash (``-``) indicates that the column will 
-  have its default ordering reversed (#1618).
+* turbogears.url() allows to to create an url with multiple values for the 
+  same key (#1456).
-
-Fixes

branches/1.1/turbogears/controllers.py

@@ -514 +514 @@
-            pairs = [(key, value)]
-        for (k,v) in pairs:
-
-
+
+
+
+
-            args.append("%s=%s" % (k, urllib.quote(str(v))))
-    if args:

branches/1.1/turbogears/tests/test_controllers.py

@@ -585 +585 @@
-                ["/foo?bar=qwer&bar=asdf", "/foo?bar=asdf&bar=qwer"]
-
+    def test_unicode(self):
+        """url() can handle unicode parameters"""
+        testutil.create_request("/")
+        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):
+        """url can handle list parameters, with unicode too"""
+        testutil.create_request("/")
+        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()
+
-
-def test_index_trailing_slash():