Ticket #734: Util.getArgs.patch

tests/test_Util.html

@@ -571 +571 @@
-        t.eq(OpenLayers.Util.createDiv().id, "OpenLayersDiv3", "Div created is sequential, starting at lastSeqID in Util.");
-    }
-
+    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>
-</head>

lib/OpenLayers/Util.js

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