put all the filter into the query object

elastic · Oct 8, 2024 · 9c8b5a3 · 9c8b5a3
1 parent 0041cf7
commit 9c8b5a3
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 37 deletions.
diff --git a/packages/kbn-alerts-ui-shared/src/rule_form/rule_actions/rule_actions_alerts_filter.tsx b/packages/kbn-alerts-ui-shared/src/rule_form/rule_actions/rule_actions_alerts_filter.tsx
@@ -85,16 +85,7 @@ export const RuleActionsAlertsFilter = ({
 
   const onFiltersUpdated = useCallback(
     (filters: Filter[]) => {
-      const updatedFilters = filters.map((filter) => {
-        const { $state, meta, ...rest } = filter;
-        return {
-          $state,
-          meta,
-          query: filter?.query ? { ...filter.query } : { ...rest },
-        };
-      });
-
-      updateQuery({ filters: updatedFilters });
+      updateQuery({ filters });
     },
     [updateQuery]
   );

diff --git a/packages/kbn-es-query/src/filters/build_filters/custom_filter.ts b/packages/kbn-es-query/src/filters/build_filters/custom_filter.ts
@@ -8,7 +8,7 @@
  */
 
 import * as estypes from '@elastic/elasticsearch/lib/api/typesWithBodyKey';
-import { Filter, FilterMeta, FILTERS, FilterStateStore } from './types';
+import { Filter, FILTERS, FilterStateStore } from './types';
 
 /** @public */
 export type CustomFilter = Filter;
@@ -27,20 +27,21 @@ export type CustomFilter = Filter;
  */
 export function buildCustomFilter(
   indexPatternString: string,
-  queryDsl: estypes.QueryDslQueryContainer,
+  query: estypes.QueryDslQueryContainer,
   disabled: boolean,
   negate: boolean,
   alias: string | null,
   store: FilterStateStore
 ): Filter {
-  const meta: FilterMeta = {
-    index: indexPatternString,
-    type: FILTERS.CUSTOM,
-    disabled,
-    negate,
-    alias,
+  return {
+    query,
+    meta: {
+      index: indexPatternString,
+      type: FILTERS.CUSTOM,
+      disabled,
+      negate,
+      alias,
+    },
+    $state: { store },
   };
-  const filter: Filter = { ...queryDsl, meta };
-  filter.$state = { store };
-  return filter;
 }
diff --git a/.../alerting/public/pages/maintenance_windows/components/create_maintenance_windows_form.tsx b/.../alerting/public/pages/maintenance_windows/components/create_maintenance_windows_form.tsx
@@ -117,30 +117,15 @@ export const CreateMaintenanceWindowForm = React.memo<CreateMaintenanceWindowFor
 
   const { data: ruleTypes, isLoading: isLoadingRuleTypes } = useGetRuleTypes();
 
-  const transformQueryFilters = (filtersToTransform: Filter[]): Filter[] => {
-    return filtersToTransform.map((filter) => {
-      const { $state, meta, ...rest } = filter;
-      return {
-        $state,
-        meta,
-        query: filter?.query ? { ...filter.query } : { ...rest },
-      };
-    });
-  };
-
   const scopedQueryPayload = useMemo(() => {
     if (!isScopedQueryEnabled) {
       return null;
     }
     if (!query && !filters.length) {
       return null;
     }
-    const transformedFilters = transformQueryFilters(filters);
 
-    return {
-      kql: query,
-      filters: transformedFilters,
-    };
+    return { kql: query, filters };
   }, [isScopedQueryEnabled, query, filters]);
 
   const submitMaintenanceWindow = useCallback<FormSubmitHandler<FormProps>>(