Merge pull request #34 from coldbox-modules/elpete-patch-2

Ignore null values to inList validator

Author: lmajano

models/validators/InListValidator.cfc

@@ -23,13 +23,25 @@ component accessors="true" implements="cbvalidation.models.validators.IValidator
 	* @validationDataThe validation data the validator was created with
 	*/
 	boolean function validate(required cbvalidation.models.result.IValidationResult validationResult, required any target, required string field, any targetValue, any validationData){
+		if ( isNull( arguments.targetValue ) ) {
+		    return true;
+		}
 
-		if( !isNull(arguments.targetValue) AND listFindNoCase(arguments.validationData, arguments.targetValue)){
+		if ( listFindNoCase( arguments.validationData, arguments.targetValue ) ) {
 			return true;
 		}
 
-		var args = {message="The '#arguments.field#' value is not in the constraint list: #arguments.validationData#",field=arguments.field,validationType=getName(),validationData=arguments.validationData};
-		var error = validationResult.newError(argumentCollection=args).setErrorMetadata({inlist=arguments.validationData});
+		var args = {
+		    "message" = "The '#arguments.field#' value is not in the constraint list: #arguments.validationData#",
+			"field" = arguments.field,
+			"validationType" = getName(),
+			"validationData" = arguments.validationData
+		};
+		var error = validationResult
+		    .newError( argumentCollection = args )
+			.setErrorMetadata( {
+			    "inlist" = arguments.validationData
+			} );
 		validationResult.addError( error );
 		return false;
 	}
@@ -41,4 +53,4 @@ component accessors="true" implements="cbvalidation.models.validators.IValidator
 		return name;
 	}
 
-}
+}