Move apply filters popover ⇒ NP (elastic#51566)

* Move ApplyFiltersPopoverContent and applyFiltersPopover to NP * code review
lizozom · Nov 26, 2019 · 8f36b5c · 8f36b5c
1 parent e32fb65
commit 8f36b5c
Show file tree

Hide file tree

Showing 8 changed files with 1,474 additions and 87 deletions.
diff --git a/src/core/MIGRATION.md b/src/core/MIGRATION.md
diff --git a/src/legacy/core_plugins/data/public/filter/action/apply_filter_action.ts b/src/legacy/core_plugins/data/public/filter/action/apply_filter_action.ts
@@ -29,10 +29,10 @@ import {
   esFilters,
   FilterManager,
   TimefilterContract,
+  applyFiltersPopover,
   changeTimeFilter,
   extractTimeFilter,
 } from '../../../../../../plugins/data/public';
-import { applyFiltersPopover } from '../apply_filters/apply_filters_popover';
 import { IndexPatternsStart } from '../../index_patterns';
 export const GLOBAL_APPLY_FILTER_ACTION = 'GLOBAL_APPLY_FILTER_ACTION';
 

diff --git a/src/legacy/core_plugins/data/public/filter/apply_filters/apply_filters_popover.tsx b/src/legacy/core_plugins/data/public/filter/apply_filters/apply_filters_popover.tsx
diff --git a/src/legacy/core_plugins/data/public/index.ts b/src/legacy/core_plugins/data/public/index.ts
@@ -29,7 +29,6 @@ export function plugin() {
 /** @public types */
 export { DataSetup, DataStart };
 
-export { ApplyFiltersPopover } from './filter';
 export {
   Field,
   FieldType,

diff --git a/..._filters/apply_filter_popover_content.tsx → ..._filters/apply_filter_popover_content.tsx b/..._filters/apply_filter_popover_content.tsx → ..._filters/apply_filter_popover_content.tsx
@@ -30,17 +30,12 @@ import {
 } from '@elastic/eui';
 import { FormattedMessage } from '@kbn/i18n/react';
 import React, { Component } from 'react';
-import { IndexPattern } from '../../index_patterns';
-import {
-  mapAndFlattenFilters,
-  esFilters,
-  utils,
-  FilterLabel,
-} from '../../../../../../plugins/data/public';
+import { mapAndFlattenFilters, esFilters, utils, IIndexPattern } from '../..';
+import { FilterLabel } from '../filter_bar';
 
 interface Props {
   filters: esFilters.Filter[];
-  indexPatterns: IndexPattern[];
+  indexPatterns: IIndexPattern[];
   onCancel: () => void;
   onSubmit: (filters: esFilters.Filter[]) => void;
 }

diff --git a/...core_plugins/data/public/filter/index.tsx → ...i/apply_filters/apply_filters_popover.tsx b/...core_plugins/data/public/filter/index.tsx → ...i/apply_filters/apply_filters_popover.tsx
@@ -17,4 +17,25 @@
  * under the License.
  */
 
-export { ApplyFiltersPopover } from './apply_filters';
+import React from 'react';
+import { ApplyFiltersPopoverContent } from './apply_filter_popover_content';
+import { IIndexPattern, esFilters } from '../..';
+
+type CancelFnType = () => void;
+type SubmitFnType = (filters: esFilters.Filter[]) => void;
+
+export const applyFiltersPopover = (
+  filters: esFilters.Filter[],
+  indexPatterns: IIndexPattern[],
+  onCancel: CancelFnType,
+  onSubmit: SubmitFnType
+) => {
+  return (
+    <ApplyFiltersPopoverContent
+      indexPatterns={indexPatterns}
+      filters={filters}
+      onCancel={onCancel}
+      onSubmit={onSubmit}
+    />
+  );
+};
diff --git a/...data/public/filter/apply_filters/index.ts → ...ins/data/public/ui/apply_filters/index.ts b/...data/public/filter/apply_filters/index.ts → ...ins/data/public/ui/apply_filters/index.ts
@@ -17,4 +17,4 @@
  * under the License.
  */
 
-export { ApplyFiltersPopover } from './apply_filters_popover';
+export { applyFiltersPopover } from './apply_filters_popover';
diff --git a/src/plugins/data/public/ui/index.ts b/src/plugins/data/public/ui/index.ts
@@ -17,4 +17,5 @@
  * under the License.
  */
 
-export * from './filter_bar';
+export { FilterBar } from './filter_bar';
+export { applyFiltersPopover } from './apply_filters';