fix: add locale parameter for isAlpha and isAlphanumeric validat… (#406)

* added locale for isAlpha and isAlphanumeric with tests
* changed the parameter pattern on both functions
* removed default en-US lacale
* Added numberOptions on IsNumberString

Author: rubiin

src/decorator/decorators.ts

@@ -504,12 +504,13 @@ export function IsBooleanString(validationOptions?: ValidationOptions) {
 /**
  * Checks if the string is a number.
  */
-export function IsNumberString(validationOptions?: ValidationOptions) {
+export function IsNumberString(validationOptions?: ValidationOptions, NumberOptions?: IsNumberOptions) {
     return function (object: Object, propertyName: string) {
         const args: ValidationMetadataArgs = {
             type: ValidationTypes.IS_NUMBER_STRING,
             target: object.constructor,
             propertyName: propertyName,
+            constraints: [NumberOptions],
             validationOptions: validationOptions
         };
         getFromContainer(MetadataStorage).addValidationMetadata(new ValidationMetadata(args));
@@ -555,12 +556,13 @@ export function NotContains(seed: string, validationOptions?: ValidationOptions)
 /**
  * Checks if the string contains only letters (a-zA-Z).
  */
-export function IsAlpha(validationOptions?: ValidationOptions) {
+export function IsAlpha(locale?: string, validationOptions?: ValidationOptions) {
     return function (object: Object, propertyName: string) {
         const args: ValidationMetadataArgs = {
             type: ValidationTypes.IS_ALPHA,
             target: object.constructor,
             propertyName: propertyName,
+            constraints: [locale], 
             validationOptions: validationOptions
         };
         getFromContainer(MetadataStorage).addValidationMetadata(new ValidationMetadata(args));
@@ -570,12 +572,13 @@ export function IsAlpha(validationOptions?: ValidationOptions) {
 /**
  * Checks if the string contains only letters and numbers.
  */
-export function IsAlphanumeric(validationOptions?: ValidationOptions) {
+export function IsAlphanumeric(locale?: string, validationOptions?: ValidationOptions) {
     return function (object: Object, propertyName: string) {
         const args: ValidationMetadataArgs = {
             type: ValidationTypes.IS_ALPHANUMERIC,
             target: object.constructor,
             propertyName: propertyName,
+            constraints: [locale], 
             validationOptions: validationOptions
         };
         getFromContainer(MetadataStorage).addValidationMetadata(new ValidationMetadata(args));

src/validation/Validator.ts

@@ -174,9 +174,9 @@ export class Validator {
             case ValidationTypes.NOT_CONTAINS:
                 return this.notContains(value, metadata.constraints[0]);
             case ValidationTypes.IS_ALPHA:
-                return this.isAlpha(value);
+                return this.isAlpha(value, metadata.constraints[0]);
             case ValidationTypes.IS_ALPHANUMERIC:
-                return this.isAlphanumeric(value);
+                return this.isAlphanumeric(value, metadata.constraints[0]);
             case ValidationTypes.IS_DECIMAL:
                 return this.isDecimal(value, metadata.constraints[0]);
             case ValidationTypes.IS_ASCII:
@@ -495,16 +495,16 @@ export class Validator {
      * Checks if the string contains only letters (a-zA-Z).
      * If given value is not a string, then it returns false.
      */
-    isAlpha(value: string): boolean {
-        return typeof value === "string" && this.validatorJs.isAlpha(value);
+    isAlpha(value: string, locale?: ValidatorJS.AlphaLocale): boolean {
+        return typeof value === "string" && this.validatorJs.isAlpha(value, locale);
     }
 
     /**
      * Checks if the string contains only letters and numbers.
      * If given value is not a string, then it returns false.
      */
-    isAlphanumeric(value: string): boolean {
-        return typeof value === "string" && this.validatorJs.isAlphanumeric(value);
+    isAlphanumeric(value: string, locale?: ValidatorJS.AlphanumericLocale): boolean {
+        return typeof value === "string" && this.validatorJs.isAlphanumeric(value, locale);
     }
 
     /**

test/functional/validation-functions-and-decorators.spec.ts

@@ -1261,7 +1261,7 @@ describe("NotContains", function() {
 
 describe("IsAlpha", function() {
 
-    const constraint = "";
+    const constraint = "en-GB";
     const validValues = ["hellomynameisalex"];
     const invalidValues = [null, undefined, "hello1mynameisalex"];
 
@@ -1279,11 +1279,11 @@ describe("IsAlpha", function() {
     });
 
     it("should not fail if method in validator said that its valid", function() {
-        validValues.forEach(value => validator.isAlpha(value).should.be.true);
+        validValues.forEach(value => validator.isAlpha(value, constraint).should.be.true);
     });
 
     it("should fail if method in validator said that its invalid", function() {
-        invalidValues.forEach(value => validator.isAlpha(value).should.be.false);
+        invalidValues.forEach(value => validator.isAlpha(value, constraint).should.be.false);
     });
 
     it("should return error object with proper data", function(done) {