Skip to content

Commit c942b44

Browse files
elasticmachineelasticmachine
authored
Merge branch 'master' into fixing-bucket-span-estimator-loading-max_buckets
2 parents e5f48e2 + a6489aa commit c942b44

File tree

53 files changed

+201
-247
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

53 files changed

+201
-247
lines changed

‎src/core/MIGRATION.md‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1178,10 +1178,10 @@ import { setup, start } from '../core_plugins/visualizations/public/legacy';
11781178
| `ui/index_patterns` | `data.indexPatterns` | still in progress |
11791179
| `ui/registry/field_formats` | `data.fieldFormats` | |
11801180
| `ui/registry/feature_catalogue` | `home.featureCatalogue.register` | Must add `home` as a dependency in your kibana.json. |
1181-
| `ui/registry/vis_types` | `visualizations.types` | -- |
1182-
| `ui/vis` | `visualizations.types` | -- |
1181+
| `ui/registry/vis_types` | `visualizations` | -- |
1182+
| `ui/vis` | `visualizations` | -- |
11831183
| `ui/share` | `share` | `showShareContextMenu` is now called `toggleShareContextMenu`, `ShareContextMenuExtensionsRegistryProvider` is now called `register` |
1184-
| `ui/vis/vis_factory` | `visualizations.types` | -- |
1184+
| `ui/vis/vis_factory` | `visualizations` | -- |
11851185
| `ui/vis/vis_filters` | `visualizations.filters` | -- |
11861186
| `ui/utils/parse_es_interval` | `import { parseEsInterval } from '../data/public'` | `parseEsInterval`, `ParsedInterval`, `InvalidEsCalendarIntervalError`, `InvalidEsIntervalFormatError` items were moved to the `Data Plugin` as a static code |
11871187

‎src/legacy/core_plugins/data/public/search/expressions/esaggs.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ import {
3838
} from '../../../../../../plugins/data/public';
3939

4040
import { buildTabularInspectorData } from './build_tabular_inspector_data';
41-
import { calculateObjectHash } from '../../../../visualizations/public';
41+
import { calculateObjectHash } from '../../../../../../plugins/kibana_utils/common';
4242
import { tabifyAggResponse } from '../../../../../core_plugins/data/public';
4343
import { PersistedState } from '../../../../../../plugins/visualizations/public';
4444
import { Adapters } from '../../../../../../plugins/inspector/public';

‎src/legacy/core_plugins/input_control_vis/public/input_control_vis_type.ts‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,9 @@ import { i18n } from '@kbn/i18n';
2222
import { createInputControlVisController } from './vis_controller';
2323
import { getControlsTab } from './components/editor/controls_tab';
2424
import { OptionsTab } from './components/editor/options_tab';
25-
import { Status, defaultFeedbackMessage } from '../../visualizations/public';
25+
import { Status } from '../../visualizations/public';
2626
import { InputControlVisDependencies } from './plugin';
27+
import { defaultFeedbackMessage } from '../../../../plugins/kibana_utils/common';
2728

2829
export function createInputControlVisTypeDefinition(deps: InputControlVisDependencies) {
2930
const InputControlVisController = createInputControlVisController(deps);

‎src/legacy/core_plugins/input_control_vis/public/plugin.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ export class InputControlVisPlugin implements Plugin<Promise<void>, void> {
5959
};
6060

6161
expressions.registerFunction(createInputControlVisFn);
62-
visualizations.types.createBaseVisualization(
62+
visualizations.createBaseVisualization(
6363
createInputControlVisTypeDefinition(visualizationDependencies)
6464
);
6565
}

‎src/legacy/core_plugins/kibana/public/discover/kibana_services.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@ export function setServices(newServices: any) {
5050
// EXPORT legacy static dependencies, should be migrated when available in a new version;
5151
export { angular };
5252
export { wrapInI18nContext } from 'ui/i18n';
53-
export { buildVislibDimensions } from '../../../visualizations/public';
5453
export { getRequestInspectorStats, getResponseInspectorStats } from '../../../data/public';
5554
// @ts-ignore
5655
export { intervalOptions } from 'ui/agg_types';

‎src/legacy/core_plugins/kibana/public/discover/np_ready/angular/discover.js‎

Lines changed: 35 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,6 @@ import { getPainlessError } from './get_painless_error';
4545
import { discoverResponseHandler } from './response_handler';
4646
import {
4747
angular,
48-
buildVislibDimensions,
4948
getRequestInspectorStats,
5049
getResponseInspectorStats,
5150
getServices,
@@ -76,6 +75,7 @@ const {
7675
import { getRootBreadcrumbs, getSavedSearchBreadcrumbs } from '../helpers/breadcrumbs';
7776
import {
7877
esFilters,
78+
fieldFormats,
7979
indexPatterns as indexPatternsUtils,
8080
} from '../../../../../../../plugins/data/public';
8181
import { getIndexPatternId } from '../helpers/get_index_pattern_id';
@@ -812,21 +812,45 @@ function discoverController(
812812
$fetchObservable.next();
813813
};
814814

815+
function getDimensions(aggs, timeRange) {
816+
const [metric, agg] = aggs;
817+
agg.params.timeRange = timeRange;
818+
const bounds = agg.params.timeRange ? timefilter.calculateBounds(agg.params.timeRange) : null;
819+
agg.buckets.setBounds(bounds);
820+
821+
const { esUnit, esValue } = agg.buckets.getInterval();
822+
return {
823+
x: {
824+
accessor: 0,
825+
label: agg.makeLabel(),
826+
format: fieldFormats.serialize(agg),
827+
params: {
828+
date: true,
829+
interval: moment.duration(esValue, esUnit),
830+
intervalESValue: esValue,
831+
intervalESUnit: esUnit,
832+
format: agg.buckets.getScaledDateFormat(),
833+
bounds: agg.buckets.getBounds(),
834+
},
835+
},
836+
y: {
837+
accessor: 1,
838+
format: fieldFormats.serialize(metric),
839+
label: metric.makeLabel(),
840+
},
841+
};
842+
}
843+
815844
function onResults(resp) {
816845
logInspectorResponse(resp);
817846

818847
if ($scope.opts.timefield) {
819848
const tabifiedData = tabifyAggResponse($scope.vis.aggs, resp);
820849
$scope.searchSource.rawResponse = resp;
821-
Promise.resolve(
822-
buildVislibDimensions($scope.vis, {
823-
timefilter,
824-
timeRange: $scope.timeRange,
825-
searchSource: $scope.searchSource,
826-
})
827-
).then(resp => {
828-
$scope.histogramData = discoverResponseHandler(tabifiedData, resp);
829-
});
850+
$scope.histogramData = discoverResponseHandler(
851+
tabifiedData,
852+
getDimensions($scope.vis.aggs.aggs, $scope.timeRange)
853+
);
830854
}
831855

832856
$scope.hits = resp.hits.total;
@@ -993,7 +1017,7 @@ function discoverController(
9931017
},
9941018
};
9951019

996-
$scope.vis = new visualizations.Vis(
1020+
$scope.vis = visualizations.createVis(
9971021
$scope.searchSource.getField('index'),
9981022
visSavedObject.visState
9991023
);

‎src/legacy/core_plugins/kibana/public/discover/np_ready/components/field_chooser/lib/visualize_url_utils.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ import { getServices } from '../../../../kibana_services';
2929

3030
function getMapsAppBaseUrl() {
3131
const mapsAppVisAlias = getServices()
32-
.visualizations.types.getAliases()
32+
.visualizations.getAliases()
3333
.find(({ name }) => {
3434
return name === 'maps';
3535
});
@@ -38,7 +38,7 @@ function getMapsAppBaseUrl() {
3838

3939
export function isMapsAppRegistered() {
4040
return getServices()
41-
.visualizations.types.getAliases()
41+
.visualizations.getAliases()
4242
.some(({ name }) => {
4343
return name === 'maps';
4444
});

‎src/legacy/core_plugins/kibana/public/management/saved_object_registry.ts‎

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ import { i18n } from '@kbn/i18n';
2222
import { npStart } from 'ui/new_platform';
2323
import { SavedObjectLoader } from '../../../../../plugins/saved_objects/public';
2424
import { createSavedDashboardLoader } from '../dashboard';
25-
import { TypesService, createSavedVisLoader } from '../../../visualizations/public';
25+
import { start as visualizations } from '../../../visualizations/public/np_ready/public/legacy';
2626
import { createSavedSearchesLoader } from '../../../../../plugins/discover/public';
2727

2828
/**
@@ -58,10 +58,7 @@ const services = {
5858

5959
savedObjectManagementRegistry.register({
6060
id: 'savedVisualizations',
61-
service: createSavedVisLoader({
62-
...services,
63-
...{ visualizationTypes: new TypesService().start() },
64-
}),
61+
service: visualizations.savedVisualizationsLoader,
6562
title: 'visualizations',
6663
});
6764

‎src/legacy/core_plugins/kibana/public/visualize/np_ready/legacy_app.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ export function initVisualizeApp(app, deps) {
102102
resolve: {
103103
savedVis: function(redirectWhenMissing, $route, $rootScope, kbnUrl) {
104104
const { core, data, savedVisualizations, visualizations } = deps;
105-
const visTypes = visualizations.types.all();
105+
const visTypes = visualizations.all();
106106
const visType = find(visTypes, { name: $route.current.params.type });
107107
const shouldHaveIndex = visType.requiresSearch && visType.options.showIndexSelection;
108108
const hasIndex =

‎src/legacy/core_plugins/region_map/public/__tests__/region_map_visualization.js‎

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -111,9 +111,7 @@ describe('RegionMapsVisualizationTests', function() {
111111

112112
if (!visRegComplete) {
113113
visRegComplete = true;
114-
visualizationsSetup.types.createBaseVisualization(
115-
createRegionMapTypeDefinition(dependencies)
116-
);
114+
visualizationsSetup.createBaseVisualization(createRegionMapTypeDefinition(dependencies));
117115
}
118116

119117
RegionMapsVisualization = createRegionMapVisualization(dependencies);
@@ -160,7 +158,7 @@ describe('RegionMapsVisualizationTests', function() {
160158

161159
imageComparator = new ImageComparator();
162160

163-
vis = new visualizationsStart.Vis(indexPattern, {
161+
vis = visualizationsStart.createVis(indexPattern, {
164162
type: 'region_map',
165163
});
166164

0 commit comments

Comments
 (0)