additional visualizations plugin cleanup before moving to NP

src/core/MIGRATION.md

@@ -1178,10 +1178,10 @@ import { setup, start } from '../core_plugins/visualizations/public/legacy';
 | `ui/index_patterns`                               | `data.indexPatterns`                                         | still in progress                                                                                                                    |
 | `ui/registry/field_formats`                       | `data.fieldFormats`                                          |                                                                                                                                      |
 | `ui/registry/feature_catalogue`                   | `home.featureCatalogue.register`                             | Must add `home` as a dependency in your kibana.json.                                                                                 |
-| `ui/registry/vis_types`                           | `visualizations.types`                                       | --                                                                                                                                   |
-| `ui/vis`                                          | `visualizations.types`                                       | --                                                                                                                                   |
+| `ui/registry/vis_types`                           | `visualizations`                                       | --                                                                                                                                   |
+| `ui/vis`                                          | `visualizations`                                       | --                                                                                                                                   |
 | `ui/share`                                        | `share`                                                      | `showShareContextMenu` is now called `toggleShareContextMenu`, `ShareContextMenuExtensionsRegistryProvider` is now called `register` |
-| `ui/vis/vis_factory`                              | `visualizations.types`                                       | --                                                                                                                                   |
+| `ui/vis/vis_factory`                              | `visualizations`                                       | --                                                                                                                                   |
 | `ui/vis/vis_filters`                              | `visualizations.filters`                                     | --                                                                                                                                   |
 | `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 |
 

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

@@ -38,7 +38,7 @@ import {
 } from '../../../../../../plugins/data/public';
 
 import { buildTabularInspectorData } from './build_tabular_inspector_data';
-import { calculateObjectHash } from '../../../../visualizations/public';
+import { calculateObjectHash } from '../../../../../../plugins/kibana_utils/common/calculate_object_hash';
 import { tabifyAggResponse } from '../../../../../core_plugins/data/public';
 import { PersistedState } from '../../../../../../plugins/visualizations/public';
 import { Adapters } from '../../../../../../plugins/inspector/public';

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

@@ -22,8 +22,9 @@ import { i18n } from '@kbn/i18n';
 import { createInputControlVisController } from './vis_controller';
 import { getControlsTab } from './components/editor/controls_tab';
 import { OptionsTab } from './components/editor/options_tab';
-import { Status, defaultFeedbackMessage } from '../../visualizations/public';
+import { Status } from '../../visualizations/public';
 import { InputControlVisDependencies } from './plugin';
+import { defaultFeedbackMessage } from '../../../../plugins/kibana_utils/common/default_feedback_message';
 
 export function createInputControlVisTypeDefinition(deps: InputControlVisDependencies) {
   const InputControlVisController = createInputControlVisController(deps);

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

@@ -59,7 +59,7 @@ export class InputControlVisPlugin implements Plugin<Promise<void>, void> {
     };
 
     expressions.registerFunction(createInputControlVisFn);
-    visualizations.types.createBaseVisualization(
+    visualizations.createBaseVisualization(
       createInputControlVisTypeDefinition(visualizationDependencies)
     );
   }

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

@@ -50,7 +50,6 @@ export function setServices(newServices: any) {
 // EXPORT legacy static dependencies, should be migrated when available in a new version;
 export { angular };
 export { wrapInI18nContext } from 'ui/i18n';
-export { buildVislibDimensions } from '../../../visualizations/public';
 export { getRequestInspectorStats, getResponseInspectorStats } from '../../../data/public';
 // @ts-ignore
 export { intervalOptions } from 'ui/agg_types';

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

@@ -45,7 +45,6 @@ import { getPainlessError } from './get_painless_error';
 import { discoverResponseHandler } from './response_handler';
 import {
   angular,
-  buildVislibDimensions,
   getRequestInspectorStats,
   getResponseInspectorStats,
   getServices,
@@ -818,14 +817,28 @@ function discoverController(
     if ($scope.opts.timefield) {
       const tabifiedData = tabifyAggResponse($scope.vis.aggs, resp);
       $scope.searchSource.rawResponse = resp;
-      Promise.resolve(
-        buildVislibDimensions($scope.vis, {
-          timefilter,
-          timeRange: $scope.timeRange,
-          searchSource: $scope.searchSource,
-        })
-      ).then(resp => {
-        $scope.histogramData = discoverResponseHandler(tabifiedData, resp);
+      const metric = $scope.vis.aggs.aggs[1];
+      const agg = $scope.vis.aggs.aggs[0];
+      agg.params.timeRange = $scope.timeRange;
+      agg.buckets.setBounds(agg);
+      const { esUnit, esValue } = agg.buckets.getInterval();
+      $scope.histogramData = discoverResponseHandler(tabifiedData, {
+        x: {
+          accessor: 1,
+          label: agg.makeLabel(),
+          params: {
+            date: true,
+            interval: moment.duration(esValue, esUnit),
+            intervalESValue: esValue,
+            intervalESUnit: esUnit,
+            format: agg.buckets.getScaledDateFormat(),
+            bounds: agg.buckets.getBounds(),
+          },
+        },
+        y: {
+          accessor: 0,
+          label: metric.makeLabel(),
+        },
       });
     }
 
@@ -993,7 +1006,7 @@ function discoverController(
       },
     };
 
-    $scope.vis = new visualizations.Vis(
+    $scope.vis = visualizations.createVis(
       $scope.searchSource.getField('index'),
       visSavedObject.visState
     );

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

@@ -29,7 +29,7 @@ import { getServices } from '../../../../kibana_services';
 
 function getMapsAppBaseUrl() {
   const mapsAppVisAlias = getServices()
-    .visualizations.types.getAliases()
+    .visualizations.getAliases()
     .find(({ name }) => {
       return name === 'maps';
     });
@@ -38,7 +38,7 @@ function getMapsAppBaseUrl() {
 
 export function isMapsAppRegistered() {
   return getServices()
-    .visualizations.types.getAliases()
+    .visualizations.getAliases()
     .some(({ name }) => {
       return name === 'maps';
     });

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

@@ -22,7 +22,7 @@ import { i18n } from '@kbn/i18n';
 import { npStart } from 'ui/new_platform';
 import { SavedObjectLoader } from '../../../../../plugins/saved_objects/public';
 import { createSavedDashboardLoader } from '../dashboard';
-import { TypesService, createSavedVisLoader } from '../../../visualizations/public';
+import { start as visualizations } from '../../../visualizations/public/np_ready/public/legacy';
 import { createSavedSearchesLoader } from '../../../../../plugins/discover/public';
 
 /**
@@ -58,10 +58,7 @@ const services = {
 
 savedObjectManagementRegistry.register({
   id: 'savedVisualizations',
-  service: createSavedVisLoader({
-    ...services,
-    ...{ visualizationTypes: new TypesService().start() },
-  }),
+  service: visualizations.savedVisualizationsLoader,
   title: 'visualizations',
 });
 

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

@@ -107,7 +107,7 @@ export function initVisualizeApp(app, deps) {
         resolve: {
           savedVis: function(redirectWhenMissing, $route, $rootScope, kbnUrl) {
             const { core, data, savedVisualizations, visualizations } = deps;
-            const visTypes = visualizations.types.all();
+            const visTypes = visualizations.all();
             const visType = find(visTypes, { name: $route.current.params.type });
             const shouldHaveIndex = visType.requiresSearch && visType.options.showIndexSelection;
             const hasIndex =

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

@@ -111,9 +111,7 @@ describe('RegionMapsVisualizationTests', function() {
 
       if (!visRegComplete) {
         visRegComplete = true;
-        visualizationsSetup.types.createBaseVisualization(
-          createRegionMapTypeDefinition(dependencies)
-        );
+        visualizationsSetup.createBaseVisualization(createRegionMapTypeDefinition(dependencies));
       }
 
       RegionMapsVisualization = createRegionMapVisualization(dependencies);

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

@@ -70,7 +70,7 @@ export class RegionMapPlugin implements Plugin<Promise<void>, void> {
 
     expressions.registerFunction(createRegionMapFn);
 
-    visualizations.types.createBaseVisualization(
+    visualizations.createBaseVisualization(
       createRegionMapTypeDefinition(visualizationDependencies)
     );
   }

src/legacy/core_plugins/tile_map/public/__tests__/coordinate_maps_visualization.js

@@ -88,9 +88,7 @@ describe('CoordinateMapsVisualizationTest', function() {
 
       if (!visRegComplete) {
         visRegComplete = true;
-        visualizationsSetup.types.createBaseVisualization(
-          createTileMapTypeDefinition(dependencies)
-        );
+        visualizationsSetup.createBaseVisualization(createTileMapTypeDefinition(dependencies));
       }
 
       CoordinateMapsVisualization = createTileMapVisualization(dependencies);

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

@@ -64,9 +64,7 @@ export class TileMapPlugin implements Plugin<Promise<void>, void> {
 
     expressions.registerFunction(() => createTileMapFn(visualizationDependencies));
 
-    visualizations.types.createBaseVisualization(
-      createTileMapTypeDefinition(visualizationDependencies)
-    );
+    visualizations.createBaseVisualization(createTileMapTypeDefinition(visualizationDependencies));
   }
 
   public start(core: CoreStart) {

src/legacy/core_plugins/timelion/public/app.js

@@ -42,7 +42,7 @@ import '../../data/public/legacy';
 import './services/saved_sheet_register';
 
 import rootTemplate from 'plugins/timelion/index.html';
-import { createSavedVisLoader, TypesService } from '../../visualizations/public';
+import { start as visualizations } from '../../visualizations/public/np_ready/public/legacy';
 
 import { loadKbnTopNavDirectives } from '../../../../plugins/kibana_legacy/public';
 loadKbnTopNavDirectives(npStart.plugins.navigation.ui);
@@ -127,13 +127,7 @@ app.controller('timelion', function(
   timefilter.enableAutoRefreshSelector();
   timefilter.enableTimeRangeSelector();
 
-  const savedVisualizations = createSavedVisLoader({
-    savedObjectsClient: npStart.core.savedObjects.client,
-    indexPatterns: npStart.plugins.data.indexPatterns,
-    chrome: npStart.core.chrome,
-    overlays: npStart.core.overlays,
-    visualizationTypes: new TypesService().start(),
-  });
+  const savedVisualizations = visualizations.savedVisualizationsLoader;
   const timezone = Private(timezoneProvider)();
 
   const defaultExpression = '.es(*)';

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

@@ -39,7 +39,7 @@ export class MarkdownPlugin implements Plugin<void, void> {
   }
 
   public setup(core: CoreSetup, { expressions, visualizations }: MarkdownPluginSetupDependencies) {
-    visualizations.types.createReactVisualization(markdownVisDefinition);
+    visualizations.createReactVisualization(markdownVisDefinition);
     expressions.registerFunction(createMarkdownVisFn);
   }
 

src/legacy/core_plugins/vis_type_metric/public/metric_vis_type.test.ts

@@ -40,7 +40,7 @@ describe('metric_vis - createMetricVisTypeDefinition', () => {
   let vis: Vis;
 
   beforeAll(() => {
-    visualizationsSetup.types.createReactVisualization(createMetricVisTypeDefinition());
+    visualizationsSetup.createReactVisualization(createMetricVisTypeDefinition());
     (npStart.plugins.data.fieldFormats.getType as jest.Mock).mockImplementation(() => {
       return fieldFormats.UrlFormat;
     });
@@ -59,7 +59,7 @@ describe('metric_vis - createMetricVisTypeDefinition', () => {
 
     // TODO: remove when Vis is converted to typescript. Only importing Vis as type
     // @ts-ignore
-    vis = new visualizationsStart.Vis(stubIndexPattern, {
+    vis = visualizationsStart.createVis(stubIndexPattern, {
       type: 'metric',
       aggs: [{ id: '1', type: 'top_hits', schema: 'metric', params: { field: 'ip' } }],
     });
@@ -80,7 +80,7 @@ describe('metric_vis - createMetricVisTypeDefinition', () => {
     };
 
     const el = document.createElement('div');
-    const metricVisType = visualizationsStart.types.get('metric');
+    const metricVisType = visualizationsStart.get('metric');
     const Controller = metricVisType.visualization;
     const controller = new Controller(el, vis);
     const render = (esResponse: any) => {

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

@@ -45,7 +45,7 @@ export class MetricVisPlugin implements Plugin<void, void> {
     { expressions, visualizations, charts }: MetricVisPluginSetupDependencies
   ) {
     expressions.registerFunction(createMetricVisFn);
-    visualizations.types.createReactVisualization(createMetricVisTypeDefinition());
+    visualizations.createReactVisualization(createMetricVisTypeDefinition());
   }
 
   public start(core: CoreStart) {

src/legacy/core_plugins/vis_type_table/public/agg_table/__tests__/agg_table.js

@@ -110,7 +110,7 @@ describe('Table Vis - AggTable Directive', function() {
   beforeEach(initLocalAngular);
 
   ngMock.inject(function() {
-    visualizationsSetup.types.createBaseVisualization(tableVisTypeDefinition);
+    visualizationsSetup.createBaseVisualization(tableVisTypeDefinition);
   });
 
   beforeEach(ngMock.module('kibana/table_vis'));

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

@@ -44,7 +44,7 @@ export class TableVisPlugin implements Plugin<Promise<void>, void> {
   ) {
     expressions.registerFunction(createTableVisFn);
 
-    visualizations.types.createBaseVisualization(tableVisTypeDefinition);
+    visualizations.createBaseVisualization(tableVisTypeDefinition);
   }
 
   public start(core: CoreStart) {

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

@@ -53,7 +53,7 @@ export class TagCloudPlugin implements Plugin<void, void> {
       colors: charts.colors,
     };
     expressions.registerFunction(createTagCloudFn);
-    visualizations.types.createBaseVisualization(
+    visualizations.createBaseVisualization(
       createTagCloudVisTypeDefinition(visualizationDependencies)
     );
   }

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

@@ -66,7 +66,7 @@ export class TimelionVisPlugin implements Plugin<void, void> {
     };
 
     expressions.registerFunction(() => getTimelionVisualizationConfig(dependencies));
-    visualizations.types.createReactVisualization(getTimelionVisDefinition(dependencies));
+    visualizations.createReactVisualization(getTimelionVisDefinition(dependencies));
   }
 
   public start(core: CoreStart, plugins: PluginsStart) {

src/legacy/core_plugins/vis_type_timeseries/public/metrics_type.ts

@@ -25,7 +25,7 @@ import { metricsRequestHandler } from './request_handler';
 import { EditorController } from './editor_controller';
 // @ts-ignore
 import { PANEL_TYPES } from '../../../../plugins/vis_type_timeseries/common/panel_types';
-import { defaultFeedbackMessage } from '../../visualizations/public';
+import { defaultFeedbackMessage } from '../../../../plugins/kibana_utils/common/default_feedback_message';
 
 export const metricsVisDefinition = {
   name: 'metrics',

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

@@ -57,7 +57,7 @@ export class MetricsPlugin implements Plugin<Promise<void>, void> {
   ) {
     expressions.registerFunction(createMetricsFn);
     setUISettings(core.uiSettings);
-    visualizations.types.createReactVisualization(metricsVisDefinition);
+    visualizations.createReactVisualization(metricsVisDefinition);
   }
 
   public start(core: CoreStart, { data }: MetricsPluginStartDependencies) {

src/legacy/core_plugins/vis_type_vega/public/__tests__/vega_visualization.js

@@ -93,7 +93,7 @@ describe('VegaVisualizations', () => {
 
       if (!visRegComplete) {
         visRegComplete = true;
-        visualizationsSetup.types.createBaseVisualization(
+        visualizationsSetup.createBaseVisualization(
           createVegaTypeDefinition(vegaVisualizationDependencies)
         );
       }

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

@@ -84,9 +84,7 @@ export class VegaPlugin implements Plugin<Promise<void>, void> {
 
     expressions.registerFunction(() => createVegaFn(visualizationDependencies));
 
-    visualizations.types.createBaseVisualization(
-      createVegaTypeDefinition(visualizationDependencies)
-    );
+    visualizations.createBaseVisualization(createVegaTypeDefinition(visualizationDependencies));
   }
 
   public start(core: CoreStart, { data }: VegaPluginStartDependencies) {

src/legacy/core_plugins/vis_type_vega/public/vega_type.ts

@@ -19,10 +19,11 @@
 
 import { i18n } from '@kbn/i18n';
 // @ts-ignore
-import { Status, defaultFeedbackMessage } from '../../visualizations/public';
+import { Status } from '../../visualizations/public';
 import { DefaultEditorSize } from '../../vis_default_editor/public';
 import { VegaVisualizationDependencies } from './plugin';
 import { VegaVisEditor } from './components';
+import { defaultFeedbackMessage } from '../../../../plugins/kibana_utils/common/default_feedback_message';
 
 import { createVegaRequestHandler } from './vega_request_handler';
 // @ts-ignore

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

@@ -97,14 +97,14 @@ export class VisTypeVislibPlugin implements Plugin<Promise<void>, void> {
       // Register legacy vislib types that have been converted
       convertedFns.forEach(expressions.registerFunction);
       convertedTypes.forEach(vis =>
-        visualizations.types.createBaseVisualization(vis(visualizationDependencies))
+        visualizations.createBaseVisualization(vis(visualizationDependencies))
       );
     }
 
     // Register non-converted types
     vislibFns.forEach(expressions.registerFunction);
     vislibTypes.forEach(vis =>
-      visualizations.types.createBaseVisualization(vis(visualizationDependencies))
+      visualizations.createBaseVisualization(vis(visualizationDependencies))
     );
   }
 

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

@@ -72,7 +72,7 @@ export class VisTypeXyPlugin implements Plugin<Promise<void>, void> {
 
     visFunctions.forEach((fn: any) => expressions.registerFunction(fn));
     visTypeDefinitions.forEach((vis: any) =>
-      visualizations.types.createBaseVisualization(vis(visualizationDependencies))
+      visualizations.createBaseVisualization(vis(visualizationDependencies))
     );
   }