Account for null or missing values

Author: elpete

models/validators/EmptyValidator.cfc

@@ -40,6 +40,11 @@ component extends="BaseValidator" accessors="true" singleton {
 			);
 		}
 
+        // return true if no data to check, empty needs a data element to be checked.
+		if ( isNull( arguments.targetValue ) ) {
+			return true;
+		}
+
 		var isFilled = hasValue( arguments.targetValue );
 		if ( arguments.validationData && !isFilled ) {
 			return true;

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

@@ -16,6 +16,10 @@ component extends="coldbox.system.testing.BaseModelTest" model="cbvalidation.mod
 				variables.result = createMock( "cbvalidation.models.result.ValidationResult" ).init();
 			} );
 
+            it( "skips over null values", function() {
+                expect( model.validate( variables.result, this, "test", javacast( "null", "" ), true ) ).toBeTrue();
+            } );
+
 			describe( "empty: true", function(){
 				it( "passes when given an empty string", function(){
 					expect( model.validate( variables.result, this, "test", "", true ) ).toBeTrue();