Fixing truncate multiple spaces so it does not mutate the data in the…

… validation system.
formio · Apr 17, 2024 · 5033358 · 5033358
1 parent b9ebc0a
commit 5033358
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/src/process/normalize/index.ts b/src/process/normalize/index.ts
@@ -252,6 +252,10 @@ const normalizeTextFieldComponentValue = (
     value: any,
     path: string
 ) => {
+    // If the component has truncate multiple spaces enabled, then normalize the value to remove extra spaces.
+    if (component.truncateMultipleSpaces && typeof value === 'string') {
+        value = value.trim().replace(/\s{2,}/g, ' ');
+    }
     if (component.allowMultipleMasks && component.inputMasks && component.inputMasks.length > 0) {
         if (Array.isArray(value)) {
             return value.map((val) => normalizeMaskValue(component, defaultValues, val, path));

diff --git a/src/process/validation/index.ts b/src/process/validation/index.ts
@@ -165,7 +165,8 @@ function handleError(error: FieldError | null, context: ValidationContext) {
 }
 
 export const validateProcess: ValidationProcessorFn = async (context) => {
-    const { component, data, row, path, instance, scope, rules, skipValidation, value } = context;
+    const { component, data, row, path, instance, scope, rules, skipValidation } = context;
+    let { value } = context;
     if (!scope.validated) scope.validated = [];
     if (!scope.errors) scope.errors = [];
     if (!rules || !rules.length) {
@@ -216,7 +217,7 @@ export const validateProcess: ValidationProcessorFn = async (context) => {
         return;
     }
     if (component.truncateMultipleSpaces && value && typeof value === 'string') {
-        set(data, path, value.trim().replace(/\s{2,}/g, ' '));
+        value = value.trim().replace(/\s{2,}/g, ' ');
     }
     for (const rule of rulesToExecute) {
         try {
@@ -279,7 +280,7 @@ export const validateProcessSync: ValidationProcessorFnSync = (context) => {
         return;
     }
     if (component.truncateMultipleSpaces && value && typeof value === 'string') {
-        set(data, path, value.trim().replace(/\s{2,}/g, ' '));
+        value = value.trim().replace(/\s{2,}/g, ' ');
     }
     for (const rule of rulesToExecute) {
         try {