Realign _.toQuery with jQuery 3 $.param (#218)

Author: jgonggrijp

test/util.strings.js

@@ -41,7 +41,7 @@ $(document).ready(function() {
 
   QUnit.test('toQuery', function(assert) {
     var obj = {'foo&bar': 'baz', 'test': 'total success', 'nested': {'works': 'too'}, 'isn\'t': ['that', 'cool?']};
-    assert.equal(_.toQuery(obj), 'foo%26bar=baz&test=total+success&nested%5Bworks%5D=too&isn\'t%5B%5D=that&isn\'t%5B%5D=cool%3F', 'can convert a hash to a query string');
+    assert.equal(_.toQuery(obj), 'foo%26bar=baz&test=total%20success&nested%5Bworks%5D=too&isn\'t%5B%5D=that&isn\'t%5B%5D=cool%3F', 'can convert a hash to a query string');
     assert.equal(_.toQuery(obj), jQuery.param(obj), 'query serialization matchs jQuery.param()');
   });
 

underscore.util.strings.js

@@ -17,14 +17,13 @@
 
   // No reason to create regex more than once
   var plusRegex = /\+/g;
-  var spaceRegex = /\%20/g;
   var bracketRegex = /(?:([^\[]+))|(?:\[(.*?)\])/g;
 
   var urlDecode = function(s) {
     return decodeURIComponent(s.replace(plusRegex, '%20'));
   };
   var urlEncode = function(s) {
-    return encodeURIComponent(s).replace(spaceRegex, '+');
+    return encodeURIComponent(s);
   };
 
   var buildParams = function(prefix, val, top) {