Skip to content

Commit ab81287

Browse files
author
Liza Katz
authored
Move saved queries service + language switcher ⇒ NP (#51812) (#51863)
* Move saved queries service + language switcher to NP * test fixes * test fix * fix ts * mock search service
1 parent ba001d3 commit ab81287

File tree

39 files changed

+131
-120
lines changed

39 files changed

+131
-120
lines changed

src/legacy/core_plugins/data/index.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
import { resolve } from 'path';
2121
import { Legacy } from '../../../../kibana';
2222
import { mappings } from './mappings';
23-
import { SavedQuery } from './public';
23+
import { SavedQuery } from '../../../plugins/data/public';
2424

2525
// eslint-disable-next-line import/no-default-export
2626
export default function DataPlugin(kibana: any) {

src/legacy/core_plugins/data/public/index.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,12 @@ export {
3838
StaticIndexPattern,
3939
} from './index_patterns';
4040
export { QueryStringInput } from './query';
41-
export { SearchBar, SearchBarProps, SavedQueryAttributes, SavedQuery } from './search';
41+
export { SearchBar, SearchBarProps } from './search';
42+
export {
43+
SavedQueryAttributes,
44+
SavedQuery,
45+
SavedQueryTimeFilter,
46+
} from '../../../../plugins/data/public';
4247

4348
/** @public static code */
4449
export * from '../common';

src/legacy/core_plugins/data/public/plugin.ts

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
*/
1919

2020
import { CoreSetup, CoreStart, Plugin } from 'kibana/public';
21-
import { SearchService, SearchStart, createSearchBar, StatetfulSearchBarProps } from './search';
21+
import { createSearchBar, StatetfulSearchBarProps } from './search';
2222
import { IndexPatternsService, IndexPatternsSetup, IndexPatternsStart } from './index_patterns';
2323
import { Storage, IStorageWrapper } from '../../../../../src/plugins/kibana_utils/public';
2424
import { DataPublicPluginStart } from '../../../../plugins/data/public';
@@ -51,7 +51,6 @@ export interface DataSetup {
5151
*/
5252
export interface DataStart {
5353
indexPatterns: IndexPatternsStart;
54-
search: SearchStart;
5554
ui: {
5655
SearchBar: React.ComponentType<StatetfulSearchBarProps>;
5756
};
@@ -71,7 +70,6 @@ export interface DataStart {
7170

7271
export class DataPlugin implements Plugin<DataSetup, DataStart, {}, DataPluginStartDependencies> {
7372
private readonly indexPatterns: IndexPatternsService = new IndexPatternsService();
74-
private readonly search: SearchService = new SearchService();
7573

7674
private setupApi!: DataSetup;
7775
private storage!: IStorageWrapper;
@@ -119,7 +117,6 @@ export class DataPlugin implements Plugin<DataSetup, DataStart, {}, DataPluginSt
119117
return {
120118
...this.setupApi!,
121119
indexPatterns: indexPatternsService,
122-
search: this.search.start(savedObjects.client),
123120
ui: {
124121
SearchBar,
125122
},
@@ -128,6 +125,5 @@ export class DataPlugin implements Plugin<DataSetup, DataStart, {}, DataPluginSt
128125

129126
public stop() {
130127
this.indexPatterns.stop();
131-
this.search.stop();
132128
}
133129
}

src/legacy/core_plugins/data/public/query/query_bar/components/query_string_input.test.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ import {
2525

2626
import { EuiFieldText } from '@elastic/eui';
2727
import React from 'react';
28-
import { QueryLanguageSwitcher } from './language_switcher';
28+
import { QueryLanguageSwitcher } from '../../../../../../../plugins/data/public';
2929
import { QueryStringInput, QueryStringInputUI } from './query_string_input';
3030
import { coreMock } from '../../../../../../../core/public/mocks';
3131
const startMock = coreMock.createStart();

src/legacy/core_plugins/data/public/query/query_bar/components/query_string_input.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,13 +46,13 @@ import {
4646
matchPairs,
4747
getQueryLog,
4848
Query,
49+
QueryLanguageSwitcher,
4950
} from '../../../../../../../plugins/data/public';
5051
import {
5152
withKibana,
5253
KibanaReactContextValue,
5354
toMountPoint,
5455
} from '../../../../../../../plugins/kibana_react/public';
55-
import { QueryLanguageSwitcher } from './language_switcher';
5656
import { fetchIndexPatterns } from './fetch_index_patterns';
5757

5858
interface Props {

src/legacy/core_plugins/data/public/search/index.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,5 +17,4 @@
1717
* under the License.
1818
*/
1919

20-
export * from './search_service';
2120
export * from './search_bar';

src/legacy/core_plugins/data/public/search/search_bar/components/saved_query_management/save_query_form.tsx

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,11 @@ import {
3535
} from '@elastic/eui';
3636
import { i18n } from '@kbn/i18n';
3737
import { sortBy, isEqual } from 'lodash';
38-
import { SavedQuery, SavedQueryAttributes } from '../../index';
39-
import { SavedQueryService } from '../../lib/saved_query_service';
38+
import {
39+
SavedQuery,
40+
SavedQueryAttributes,
41+
SavedQueryService,
42+
} from '../../../../../../../../plugins/data/public';
4043

4144
interface Props {
4245
savedQuery?: SavedQueryAttributes;

src/legacy/core_plugins/data/public/search/search_bar/components/saved_query_management/saved_query_list_item.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ import { EuiListGroupItem, EuiConfirmModal, EuiOverlayMask, EuiIconTip } from '@
2222
import React, { Fragment, useState } from 'react';
2323
import classNames from 'classnames';
2424
import { i18n } from '@kbn/i18n';
25-
import { SavedQuery } from '../../index';
25+
import { SavedQuery } from '../../../../../../../../plugins/data/public';
2626

2727
interface Props {
2828
savedQuery: SavedQuery;

src/legacy/core_plugins/data/public/search/search_bar/components/saved_query_management/saved_query_management_component.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,7 @@ import {
3535
import { i18n } from '@kbn/i18n';
3636
import React, { FunctionComponent, useEffect, useState, Fragment } from 'react';
3737
import { sortBy } from 'lodash';
38-
import { SavedQuery } from '../../index';
39-
import { SavedQueryService } from '../../lib/saved_query_service';
38+
import { SavedQuery, SavedQueryService } from '../../../../../../../../plugins/data/public';
4039
import { SavedQueryListItem } from './saved_query_list_item';
4140

4241
const perPage = 50;

src/legacy/core_plugins/data/public/search/search_bar/components/search_bar.test.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ const mockTimeHistory = {
3838
jest.mock('../../../../../../../plugins/data/public', () => {
3939
return {
4040
FilterBar: () => <div className="filterBar" />,
41+
createSavedQueryService: () => {},
4142
};
4243
});
4344

0 commit comments

Comments
 (0)