Merge pull request #15 from Timshel/ignore-undefined

Ignore undefined parameters in serialize
gre · Feb 11, 2015 · 79b244e · 79b244e
2 parents 5c9f29b + 0f6c15d
commit 79b244e
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/build/qajax.min.js b/build/qajax.min.js
diff --git a/src/qajax.js b/src/qajax.js
@@ -26,7 +26,7 @@
   function serializeQuery(paramsObj) {
     var k, params = [];
     for (k in paramsObj) {
-      if (paramsObj.hasOwnProperty(k)) {
+      if (paramsObj.hasOwnProperty(k) && paramsObj[k] !== undefined) {
         params.push(encodeURIComponent(k) + "=" + encodeURIComponent(paramsObj[k]));
       }
     }

diff --git a/test/qajax.js b/test/qajax.js
@@ -33,7 +33,7 @@ test("check the API needed for the test engine", function() {
   ok(typeof Qajax.filterStatus=="function", "filterStatus exists");
   ok(typeof Qajax.filterSuccess=="function", "filterSuccess exists");
   ok(typeof Qajax.getJSON=="function", "getJSON exists");
-  equal(Qajax.serialize({ foo: 123, bar: "toto" }), "foo=123&bar=toto", "serialize works");
+  equal(Qajax.serialize({ foo: 123, bar: "toto", toto: undefined }), "foo=123&bar=toto", "serialize works (ignore undefined)");
 });
 
 asyncTest("simple Qajax.getJSON() successful", 1, function() {