Changeset 1909

Timestamp:

11/18/06 10:35:02 (2 years ago)

Author:

crschmidt

Message:

Parameters for URLs should be URI encoded. Add test for params that need
to be URI encoded in test_Util, and add Tim Schaub's patch from #381 to make
test pass.

Files:

• trunk/openlayers/lib/OpenLayers/Util.js (modified) (1 diff)
• trunk/openlayers/tests/test_Util.html (modified) (1 diff)

trunk/openlayers/lib/OpenLayers/Util.js

@@ -381 +381 @@
-        var value = params[key];
-        if ((value != null) && (typeof value != 'function')) {
-
+
+
-        }
-    }

trunk/openlayers/tests/test_Util.html

@@ -182 +182 @@
-
-        t.eq( OpenLayers.Util.getParameterString(params), "foo=bar&chicken=1.5", "getParameterString returns correctly");    
+        t.eq( OpenLayers.Util.getParameterString({'a:':'b='}), "a%3A=b%3D", "getParameterString returns correctly with non-ascii keys/values");    
-    }
-