http_build_query: adjust to current browser (and apparently Node) behavior of first iterating numeric indexes regardless of placement within the literal)

brettz9 · brettz9 · commit e8c6d28b80b3 · 2014-01-15T08:47:54.000+08:00
diff --git a/functions/url/http_build_query.js b/functions/url/http_build_query.js
@@ -9,12 +9,12 @@ function http_build_query(formdata, numeric_prefix, arg_separator) {
   // +   input by: Dreamer
   // +   bugfixed by: Brett Zamir (http://brett-zamir.me)
   // +   bugfixed by: MIO_KODUKI (http://mio-koduki.blogspot.com/)
-  // %        note 1: If the value is null, key and value is skipped in http_build_query of PHP. But, phpjs is not.
+  // %        note 1: If the value is null, key and value are skipped in the http_build_query of PHP while in phpjs they are not.
   // -    depends on: urlencode
   // *     example 1: http_build_query({foo: 'bar', php: 'hypertext processor', baz: 'boom', cow: 'milk'}, '', '&amp;');
   // *     returns 1: 'foo=bar&amp;php=hypertext+processor&amp;baz=boom&amp;cow=milk'
   // *     example 2: http_build_query({'php': 'hypertext processor', 0: 'foo', 1: 'bar', 2: 'baz', 3: 'boom', 'cow': 'milk'}, 'myvar_');
-  // *     returns 2: 'php=hypertext+processor&myvar_0=foo&myvar_1=bar&myvar_2=baz&myvar_3=boom&cow=milk'
+  // *     returns 2: 'myvar_0=foo&myvar_1=bar&myvar_2=baz&myvar_3=boom&php=hypertext+processor&cow=milk'
   var value, key, tmp = [],
       that = this;
 
