Changeset 3328

Timestamp:

06/14/07 15:11:45 (1 year ago)

Author:

euzuro

Message:

2nd attempt fix for #734 -- tests pass in all browsers

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

@@ -610 +610 @@
-        url = window.location.href;
-    }
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-    var args = new Object();
-    pairs = query.split(/[&;]/);
-    for(var i = 0; i < pairs.length; ++i) {
-
-
-
-
+
+
+
+
+
+
+
-        }
-    }

trunk/openlayers/tests/test_Util.html

@@ -572 +572 @@
-    }
-
+    function test_Util_getArgs(t) {
+        t.plan(5);
+        t.eq(OpenLayers.Util.getArgs('http://www.example.com'), {}, "getArgs works when args = ''");
+        t.eq(OpenLayers.Util.getArgs('http://www.example.com?'), {}, "getArgs works when args = '?'");
+        t.eq(OpenLayers.Util.getArgs('http://www.example.com?hello=world&foo=bar'),
+                                     {'hello' : 'world', 'foo': 'bar'}, "getArgs works when args = '?hello=world&foo=bar'");
+        t.eq(OpenLayers.Util.getArgs('http://www.example.com?hello=&foo=bar'),
+                                     {'hello' : '', 'foo': 'bar'}, "getArgs works when args = '?hello=&foo=bar'");
+        t.eq(OpenLayers.Util.getArgs('http://www.example.com?foo=bar#bugssucks'),
+                                     {'foo': 'bar'}, "getArgs works when using a fragment identifier");
+    }
-   // -->
-  </script>