Revert "change filter processor to be more verbose and have compModel…

…Type in scope" This reverts commit 6d6c278.
formio · Apr 12, 2024 · be37bee · be37bee
1 parent d1763b9
commit be37bee
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 30 deletions.
diff --git a/src/process/filter/__tests__/filter.test.ts b/src/process/filter/__tests__/filter.test.ts
@@ -23,7 +23,7 @@ it('Should not filter empty array value for dataGrid component', async () => {
     };
     const context: any = generateProcessorContext(dataGridComp, data);
     filterProcessSync(context);
-    expect(context.scope.filter).to.deep.equal({'dataGrid': {'compModelType': 'array', 'include': true}});
+    expect(context.scope.filter).to.deep.equal({'dataGrid': true});
 });
 
 it('Should not filter empty array value for editGrid component', async () => {
@@ -46,7 +46,7 @@ it('Should not filter empty array value for editGrid component', async () => {
     };
     const context: any = generateProcessorContext(editGridComp, data);
     filterProcessSync(context);
-    expect(context.scope.filter).to.deep.equal({'editGrid': {'compModelType': 'array', 'include': true}});
+    expect(context.scope.filter).to.deep.equal({'editGrid': true});
 });
 
 it('Should not filter empty array value for datTable component', async () => {
@@ -69,5 +69,5 @@ it('Should not filter empty array value for datTable component', async () => {
     };
     const context: any = generateProcessorContext(dataTableComp, data);
     filterProcessSync(context);
-    expect(context.scope.filter).to.deep.equal({'dataTable': {'compModelType': 'array', 'include': true}});
+    expect(context.scope.filter).to.deep.equal({'dataTable': true});
 });
diff --git a/src/process/filter/index.ts b/src/process/filter/index.ts
@@ -12,31 +12,18 @@ export const filterProcessSync: ProcessorFnSync<FilterScope> = (context: FilterC
         const modelType = Utils.getModelType(component);
         switch (modelType) {
             case 'dataObject':
-                scope.filter[absolutePath] = {
-                    compModelType: modelType,
-                    include: true,
-                    value: {data: {}}
-                };
+                scope.filter[absolutePath] = {data: {}};
                 break;
             case 'array':
-                scope.filter[absolutePath] = {
-                    compModelType: modelType,
-                    include: true,
-                };
+                scope.filter[absolutePath] = true;
                 break;
             case 'object':
                 if (component.type !== 'container') {
-                    scope.filter[absolutePath] = {
-                        compModelType: modelType,
-                        include: true,
-                    };
+                    scope.filter[absolutePath] = true;
                 }
                 break;
             default:
-                scope.filter[absolutePath] = {
-                    compModelType: modelType,
-                    include: true,
-                };
+                scope.filter[absolutePath] = true;
                 break;
         }
     }
@@ -50,13 +37,13 @@ export const filterPostProcess: ProcessorFnSync<FilterScope> = (context: FilterC
     const { scope, submission } = context;
     const filtered = {};
     for (const path in scope.filter) {
-        if (scope.filter[path].include) {
+        if (scope.filter[path]) {
             let value = get(submission?.data, path);
             if (isObject(value) && isObject(scope.filter[path])) {
-                if (scope.filter[path].compModelType === 'dataObject') {
-                    value = {...value, ...scope.filter[path].value, data: (value as any)?.data}
+                if ((value as any).data) {
+                    value = {...value, ...scope.filter[path], data: (value as any)?.data}
                 } else {
-                    value = {...value, ...scope.filter[path].value}
+                    value = {...value, ...scope.filter[path]}
                 }
             }
             set(filtered, path, value);

diff --git a/src/types/process/filter/FilterScope.ts b/src/types/process/filter/FilterScope.ts
@@ -1,8 +1,4 @@
 import { ProcessorScope } from "..";
 export type FilterScope = {
-    filter: Record<string, {
-        compModelType: string;
-        include: boolean;
-        value?: any;
-    }>;
-} & ProcessorScope;
+    filter: Record<string, any>;
+} & ProcessorScope;