Serialize: use jquery type and clean up formatting

DanielNill · DanielNill · commit 784bc0159599 · 2015-07-02T22:20:16.000-07: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" && v !== null ? i : "" ) + "]",
+					prefix + "[" + ( jQuery.type( v ) === "object" ? i : "" ) + "]",
 					v,
 					traditional,
 					add
diff --git a/test/unit/serialize.js b/test/unit/serialize.js
@@ -77,8 +77,8 @@ 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");
+  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;
