Skip to content

Commit ec8c414

Browse files
committed
handle source column differences in embeddable as well
1 parent 8d9ac00 commit ec8c414

File tree

3 files changed

+14
-9
lines changed

3 files changed

+14
-9
lines changed

src/plugins/discover/public/application/components/discover.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@ import { SkipBottomButton } from './skip_bottom_button';
3434
import { esFilters, IndexPatternField, search } from '../../../../data/public';
3535
import { DiscoverSidebarResponsive } from './sidebar';
3636
import { DiscoverProps } from './types';
37-
import { getDisplayedColumns } from '../helpers/columns';
3837
import { SortPairArr } from '../angular/doc_table/lib/get_sort';
3938
import { SEARCH_FIELDS_FROM_SOURCE } from '../../../common';
4039
import { popularizeField } from '../helpers/popularize_field';
@@ -390,7 +389,7 @@ export function Discover({
390389
<div className="dscDiscoverGrid">
391390
<DataGridMemoized
392391
ariaLabelledBy="documentsAriaLabel"
393-
columns={getDisplayedColumns(state.columns, indexPattern)}
392+
columns={columns}
394393
expandedDoc={expandedDoc}
395394
indexPattern={indexPattern}
396395
rows={rows}

src/plugins/discover/public/application/components/discover_grid/discover_grid.tsx

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ import {
3535
} from './discover_grid_columns';
3636
import { defaultPageSize, gridStyle, pageSizeArr, toolbarVisibility } from './constants';
3737
import { DiscoverServices } from '../../../build_services';
38+
import { getDisplayedColumns } from '../../helpers/columns';
3839

3940
interface SortObj {
4041
id: string;
@@ -146,7 +147,8 @@ export const DiscoverGrid = ({
146147
showTimeCol,
147148
sort,
148149
}: DiscoverGridProps) => {
149-
const defaultColumns = columns.includes('_source');
150+
const displayedColumns = getDisplayedColumns(columns, indexPattern);
151+
const defaultColumns = displayedColumns.includes('_source');
150152

151153
/**
152154
* Pagination
@@ -207,19 +209,19 @@ export const DiscoverGrid = ({
207209
const randomId = useMemo(() => htmlIdGenerator()(), []);
208210

209211
const euiGridColumns = useMemo(
210-
() => getEuiGridColumns(columns, settings, indexPattern, showTimeCol, defaultColumns),
211-
[columns, indexPattern, showTimeCol, settings, defaultColumns]
212+
() => getEuiGridColumns(displayedColumns, settings, indexPattern, showTimeCol, defaultColumns),
213+
[displayedColumns, indexPattern, showTimeCol, settings, defaultColumns]
212214
);
213215
const schemaDetectors = useMemo(() => getSchemaDetectors(), []);
214216
const popoverContents = useMemo(() => getPopoverContents(), []);
215217
const columnsVisibility = useMemo(
216218
() => ({
217-
visibleColumns: getVisibleColumns(columns, indexPattern, showTimeCol) as string[],
219+
visibleColumns: getVisibleColumns(displayedColumns, indexPattern, showTimeCol) as string[],
218220
setVisibleColumns: (newColumns: string[]) => {
219221
onSetColumns(newColumns);
220222
},
221223
}),
222-
[columns, indexPattern, showTimeCol, onSetColumns]
224+
[displayedColumns, indexPattern, showTimeCol, onSetColumns]
223225
);
224226
const sorting = useMemo(() => ({ columns: sortingColumns, onSort: onTableSort }), [
225227
sortingColumns,
@@ -316,7 +318,7 @@ export const DiscoverGrid = ({
316318
indexPattern={indexPattern}
317319
hit={expandedDoc}
318320
// if default columns are used, dont make them part of the URL - the context state handling will take care to restore them
319-
columns={defaultColumns ? [] : columns}
321+
columns={defaultColumns ? [] : displayedColumns}
320322
onFilter={onFilter}
321323
onRemoveColumn={onRemoveColumn}
322324
onAddColumn={onAddColumn}

src/plugins/discover/public/application/embeddable/search_embeddable.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ import { DiscoverGridSettings } from '../components/discover_grid/types';
4747
import { DiscoverServices } from '../../build_services';
4848
import { ElasticSearchHit } from '../doc_views/doc_views_types';
4949
import { getDefaultSort } from '../angular/doc_table/lib/get_default_sort';
50+
import { handleSourceColumnState } from '../angular/helpers';
5051

5152
interface SearchScope extends ng.IScope {
5253
columns?: string[];
@@ -374,7 +375,10 @@ export class SearchEmbeddable
374375

375376
// If there is column or sort data on the panel, that means the original columns or sort settings have
376377
// been overridden in a dashboard.
377-
searchScope.columns = this.input.columns || this.savedSearch.columns;
378+
searchScope.columns = handleSourceColumnState(
379+
{ columns: this.input.columns || this.savedSearch.columns },
380+
this.services.core.uiSettings
381+
).columns;
378382
const savedSearchSort =
379383
this.savedSearch.sort && this.savedSearch.sort.length
380384
? this.savedSearch.sort

0 commit comments

Comments
 (0)