Merge pull request #32609 from Expensify/revert-31685-fix/31385

Revert "Only allow alphabetic and latin characters for some fields in `CompanyStep`"
Expensify · Dec 7, 2023 · 20ef4d6 · 20ef4d6
2 parents a2cf591 + f4a3bcf
commit 20ef4d6
Show file tree

Hide file tree

Showing 2 changed files with 47 additions and 55 deletions.
diff --git a/src/libs/ValidationUtils.ts b/src/libs/ValidationUtils.ts
@@ -30,6 +30,17 @@ function validateCardNumber(value: string): boolean {
     return sum % 10 === 0;
 }
 
+/**
+ * Validating that this is a valid address (PO boxes are not allowed)
+ */
+function isValidAddress(value: string): boolean {
+    if (!CONST.REGEX.ANY_VALUE.test(value)) {
+        return false;
+    }
+
+    return !CONST.REGEX.PO_BOX.test(value);
+}
+
 /**
  * Validate date fields
  */
@@ -193,6 +204,40 @@ function isValidWebsite(url: string): boolean {
     return new RegExp(`^${URL_REGEX_WITH_REQUIRED_PROTOCOL}$`, 'i').test(url) && isLowerCase;
 }
 
+function validateIdentity(identity: Record<string, string>): Record<string, boolean> {
+    const requiredFields = ['firstName', 'lastName', 'street', 'city', 'zipCode', 'state', 'ssnLast4', 'dob'];
+    const errors: Record<string, boolean> = {};
+
+    // Check that all required fields are filled
+    requiredFields.forEach((fieldName) => {
+        if (isRequiredFulfilled(identity[fieldName])) {
+            return;
+        }
+        errors[fieldName] = true;
+    });
+
+    if (!isValidAddress(identity.street)) {
+        errors.street = true;
+    }
+
+    if (!isValidZipCode(identity.zipCode)) {
+        errors.zipCode = true;
+    }
+
+    // dob field has multiple validations/errors, we are handling it temporarily like this.
+    if (!isValidDate(identity.dob) || !meetsMaximumAgeRequirement(identity.dob)) {
+        errors.dob = true;
+    } else if (!meetsMinimumAgeRequirement(identity.dob)) {
+        errors.dobAge = true;
+    }
+
+    if (!isValidSSNLastFour(identity.ssnLast4)) {
+        errors.ssnLast4 = true;
+    }
+
+    return errors;
+}
+
 function isValidUSPhone(phoneNumber = '', isCountryCodeOptional?: boolean): boolean {
     const phone = phoneNumber || '';
     const regionCode = isCountryCodeOptional ? CONST.COUNTRY.US : undefined;
@@ -259,51 +304,6 @@ function isValidPersonName(value: string) {
     return /^[^\d^!#$%*=<>;{}"]+$/.test(value);
 }
 
-/**
- * Validating that this is a valid address (PO boxes are not allowed)
- */
-function isValidAddress(value: string): boolean {
-    if (!isValidLegalName(value)) {
-        return false;
-    }
-
-    return !CONST.REGEX.PO_BOX.test(value);
-}
-
-function validateIdentity(identity: Record<string, string>): Record<string, boolean> {
-    const requiredFields = ['firstName', 'lastName', 'street', 'city', 'zipCode', 'state', 'ssnLast4', 'dob'];
-    const errors: Record<string, boolean> = {};
-
-    // Check that all required fields are filled
-    requiredFields.forEach((fieldName) => {
-        if (isRequiredFulfilled(identity[fieldName])) {
-            return;
-        }
-        errors[fieldName] = true;
-    });
-
-    if (!isValidAddress(identity.street)) {
-        errors.street = true;
-    }
-
-    if (!isValidZipCode(identity.zipCode)) {
-        errors.zipCode = true;
-    }
-
-    // dob field has multiple validations/errors, we are handling it temporarily like this.
-    if (!isValidDate(identity.dob) || !meetsMaximumAgeRequirement(identity.dob)) {
-        errors.dob = true;
-    } else if (!meetsMinimumAgeRequirement(identity.dob)) {
-        errors.dobAge = true;
-    }
-
-    if (!isValidSSNLastFour(identity.ssnLast4)) {
-        errors.ssnLast4 = true;
-    }
-
-    return errors;
-}
-
 /**
  * Checks if the provided string includes any of the provided reserved words
  */
@@ -384,6 +384,7 @@ export {
     meetsMinimumAgeRequirement,
     meetsMaximumAgeRequirement,
     getAgeRequirementError,
+    isValidAddress,
     isValidDate,
     isValidPastDate,
     isValidSecurityCode,
@@ -395,6 +396,7 @@ export {
     getFieldRequiredErrors,
     isValidUSPhone,
     isValidWebsite,
+    validateIdentity,
     isValidTwoFactorCode,
     isNumericWithSpecialChars,
     isValidRoutingNumber,
@@ -407,8 +409,6 @@ export {
     isValidValidateCode,
     isValidDisplayName,
     isValidLegalName,
-    isValidAddress,
-    validateIdentity,
     doesContainReservedWord,
     isNumeric,
     isValidAccountRoute,

diff --git a/src/pages/ReimbursementAccount/CompanyStep.js b/src/pages/ReimbursementAccount/CompanyStep.js
@@ -88,10 +88,6 @@ function CompanyStep({reimbursementAccount, reimbursementAccountDraft, getDefaul
         ];
         const errors = ValidationUtils.getFieldRequiredErrors(values, requiredFields);
 
-        if (values.companyName && !ValidationUtils.isValidLegalName(values.companyName)) {
-            errors.companyName = 'bankAccount.error.companyName';
-        }
-
         if (values.addressStreet && !ValidationUtils.isValidAddress(values.addressStreet)) {
             errors.addressStreet = 'bankAccount.error.addressStreet';
         }
@@ -100,10 +96,6 @@ function CompanyStep({reimbursementAccount, reimbursementAccountDraft, getDefaul
             errors.addressZipCode = 'bankAccount.error.zipCode';
         }
 
-        if (values.addressCity && !ValidationUtils.isValidLegalName(values.addressCity)) {
-            errors.addressCity = 'bankAccount.error.addressCity';
-        }
-
         if (values.companyPhone && !ValidationUtils.isValidUSPhone(values.companyPhone, true)) {
             errors.companyPhone = 'bankAccount.error.phoneNumber';
         }