FIO-8632: Fixes an issue where required validation is not triggered f…

…or multiple value components like Select if it has no values added
formio · Aug 6, 2024 · 3a877d1 · 3a877d1
1 parent dbd8da3
commit 3a877d1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 10 deletions.
diff --git a/src/process/validation/index.ts b/src/process/validation/index.ts
@@ -48,14 +48,6 @@ export function validationRules(
     }
     const validationRules: ValidationRuleInfo[] = [];
     return rules.reduce((acc, rule: ValidationRuleInfo) => {
-        if (context.component.multiple &&
-            Array.isArray(context.value) &&
-            context.value?.length === 0 &&
-            !rule.fullValue
-        ) {
-            return acc;
-        }
-
         if (rule.shouldProcess && rule.shouldProcess(context)) {
             acc.push(rule);
         }

diff --git a/src/process/validation/rules/validateRegexPattern.ts b/src/process/validation/rules/validateRegexPattern.ts
@@ -1,4 +1,4 @@
-import { FieldError } from 'error';
+import { FieldError } from '../../../error';
 import { TextAreaComponent, TextFieldComponent, RuleFn, RuleFnSync, ValidationContext } from 'types';
 import { ProcessorInfo } from 'types/process/ProcessorInfo';
 
@@ -15,7 +15,7 @@ export const shouldValidate = (context: ValidationContext) => {
     }
 
     const pattern = component.validate?.pattern;
-    if (!pattern) {
+    if (!pattern || !value || typeof value !== 'string') {
         return false;
     }
     return true;