Serialize: handle arrays with null values

DanielNill · DanielNill · commit 783ab70ecc8b · 2015-06-29T16:24:25.000-04:00
diff --git a/src/serialize.js b/src/serialize.js
@@ -25,7 +25,7 @@ function buildParams( prefix, obj, traditional, add ) {
 			} else {
 				// Item is non-scalar (array or object), encode its numeric index.
 				buildParams(
-					prefix + "[" + ( typeof v === "object" ? i : "" ) + "]",
+					prefix + "[" + ( typeof v === "object" && v !== null ? i : "" ) + "]",
 					v,
 					traditional,
 					add
diff --git a/test/unit/serialize.js b/test/unit/serialize.js
@@ -1,7 +1,7 @@
 module("serialize", { teardown: moduleTeardown });
 
 test("jQuery.param()", function() {
-	expect(22);
+	expect(23);
 
 	var params, settings;
 
@@ -77,6 +77,9 @@ test("jQuery.param()", function() {
 	params = {"test": {"length": 3, "foo": "bar"} };
 	equal( jQuery.param( params, false ), "test%5Blength%5D=3&test%5Bfoo%5D=bar", "Sub-object with a length property" );
 
+  params = {"test": [1,2,null]};
+  equal(jQuery.param(params, false), "test%5B%5D=1&test%5B%5D=2&test%5B%5D=", "object with array property with null value");
+
 	if ( jQuery.ajaxSettings === settings ) {
 		delete jQuery.ajaxSettings;
 	} else {
