diff --git a/src/core.js b/src/core.js index 7eec1dc37..11c57a027 100644 --- a/src/core.js +++ b/src/core.js @@ -1349,7 +1349,7 @@ $.extend( $.validator, { if ( this.checkable( element ) ) { return this.getLength( value, element ) > 0; } - return value.length > 0; + return value !== undefined && value !== null && value.length > 0; }, // https://jqueryvalidation.org/email-method/ diff --git a/test/rules.js b/test/rules.js index cfd9f8577..1be57bf49 100644 --- a/test/rules.js +++ b/test/rules.js @@ -402,3 +402,23 @@ QUnit.test( "rules() - on unexpected input", function( assert ) { result = nonFormElement.rules( "add", "whatever" ); assert.deepEqual( result, undefined, "can work on a non-form element" ); } ); + +QUnit.test( "required method should return false for undefined & null values", function( assert ) { + var username = $( "#usernamec" ), + urlc = $( "#urlc" ), + v = $( "#testForm1clean" ).validate( { + rules: { + username: { + required: true, + normalizer: function() { return null; } + }, + urlc: { + required: true, + normalizer: function() { return undefined; } + } + } + } ); + + assert.notOk( v.element( username ), "The username element should be invalid" ); + assert.notOk( v.element( urlc ), "The urlc element should be invalid" ); +} );
Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.
Alternative Proxies: