Additional: Fixed validation for specific case for Spanish NIFs. (jquery-validation#1914)

jarey · staabm · commit 7457cc640103 · 2017-01-28T12:48:57.000+01:00
diff --git a/src/additional/nifES.js b/src/additional/nifES.js
@@ -18,7 +18,7 @@ $.validator.addMethod( "nifES", function( value ) {
 
 	// Test specials NIF (starts with K, L or M)
 	if ( /^[KLM]{1}/.test( value ) ) {
-		return ( value[ 8 ] === String.fromCharCode( 64 ) );
+		return ( value[ 8 ] === "TRWAGMYFPDXBNJZSQVHLCKE".charAt( value.substring( 8, 1 ) % 23 ) );
 	}
 
 	return false;
diff --git a/test/methods.js b/test/methods.js
@@ -1316,6 +1316,7 @@ QUnit.test( "nifES", function( assert ) {
 	assert.ok( method( "15762034L" ), "NIF valid" );
 	assert.ok( method( "05122654W" ), "NIF valid" );
 	assert.ok( method( "05122654w" ), "NIF valid: lower case" );
+	assert.ok( method( "M1503708Z" ), "NIF valid. Temporary foreign nif" );
 	assert.ok( !method( "1144105R" ), "NIF invalid: less than 8 digits without zero" );
 	assert.ok( !method( "11441059 R" ), "NIF invalid: white space" );
 	assert.ok( !method( "11441059" ), "NIF invalid: no letter" );

Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@ $.validator.addMethod( "nifES", function( value ) {`
`18`	`18`
`19`	`19`	`// Test specials NIF (starts with K, L or M)`
`20`	`20`	`if ( /^[KLM]{1}/.test( value ) ) {`
`21`		`- return ( value[ 8 ] === String.fromCharCode( 64 ) );`
	`21`	`+ return ( value[ 8 ] === "TRWAGMYFPDXBNJZSQVHLCKE".charAt( value.substring( 8, 1 ) % 23 ) );`
`22`	`22`	`}`
`23`	`23`
`24`	`24`	`return false;`