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.


git-svn-id: http://svn.openlayers.org/trunk/openlayers@1909 dc9f47b5-9b13-0410-9fdd-eb0c1a62fdaf

lib/OpenLayers/Util.js

@@ -380,7 +380,8 @@ OpenLayers.Util.getParameterString = function(params) {
     for (var key in params) {
         var value = params[key];
         if ((value != null) && (typeof value != 'function')) {
-            paramsArray.push(key + "=" + value);
+            paramsArray.push(encodeURIComponent(key) + "=" +
+                             encodeURIComponent(value));
         }
     }
     

tests/test_Util.html

@@ -181,6 +181,7 @@
                      }
 
         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");    
     }
 
     function test_08_Util_createAlphaImageDiv(t) {