Return true for null values in constraints validator

If the value is required, make sure to specify that using the required validator.

Author: elpete

models/validators/NestedConstraintsValidator.cfc

@@ -35,7 +35,12 @@ component
 		struct rules
 	){
 		// return true if no data to check, type needs a data element to be checked.
-		if ( isNull( arguments.targetValue ) || !isStruct( arguments.targetValue ) ) {
+		if ( isNull( arguments.targetValue ) ) {
+			return true;
+		}
+
+		// return false if the value is not a struct.
+		if ( !isStruct( arguments.targetValue ) ) {
 			var args = {
 				message : "The '#arguments.field#' value '#(
 					isSimpleValue( arguments.targetValue ) ? arguments.targetValue : serializeJSON(

test-harness/tests/specs/validators/NestedConstraintsValidatorTest.cfc

@@ -89,6 +89,35 @@ component extends="coldbox.system.testing.BaseTestCase" appMapping="/root" {
 				expect( vResult.getAllErrors() ).toBeEmpty();
 			} );
 
+			it( "does no validation when a null value is passed", function(){
+				var vResult = createMock( "cbvalidation.models.result.ValidationResult" ).init();
+				expect(
+					validator.validate(
+						validationResult: vResult,
+						target          : this,
+						field           : "address",
+						targetValue     : javacast( "null", "" ),
+						validationData  : {
+							"streetOne" : { required : true, type : "string" },
+							"streetTwo" : { required : false, type : "string" },
+							"city"      : { required : true, type : "string" },
+							"state"     : {
+								required : true,
+								type     : "string",
+								size     : 2
+							},
+							"zip" : {
+								required : true,
+								type     : "numeric",
+								size     : 5
+							}
+						},
+						rules: {}
+					)
+				).toBeTrue();
+				expect( vResult.getAllErrors() ).toBeEmpty();
+			} );
+
 			it( "shows the nested field name when a nested constraint fails", function(){
 				var vResult = createMock( "cbvalidation.models.result.ValidationResult" ).init();
 				expect(