changing vis.API.events (elastic#25280) (elastic#25802)

src/core_plugins/kbn_vislib_vis_types/public/gauge.js

@@ -80,6 +80,9 @@ export default function GaugeVisType(Private, i18n) {
         }
       },
     },
+    events: {
+      brush: { disabled: true },
+    },
     editorConfig: {
       collections: {
         gaugeTypes: ['Arc', 'Circle'],

src/core_plugins/kbn_vislib_vis_types/public/goal.js

@@ -76,6 +76,9 @@ export default function GoalVisType(Private, i18n) {
         }
       },
     },
+    events: {
+      brush: { disabled: true },
+    },
     editorConfig: {
       collections: {
         gaugeTypes: ['Arc', 'Circle'],

src/core_plugins/kbn_vislib_vis_types/public/pie.js

@@ -46,6 +46,9 @@ export default function HistogramVisType(Private, i18n) {
         }
       },
     },
+    events: {
+      brush: { disabled: true },
+    },
     editorConfig: {
       collections: {
         legendPositions: [{

src/core_plugins/metric_vis/public/metric_vis_controller.js

@@ -159,7 +159,7 @@ export class MetricVisComponent extends Component {
       return;
     }
     const table = this.props.visData;
-    this.props.vis.API.events.addFilter(table, metric.columnIndex, metric.rowIndex);
+    this.props.vis.API.events.filter({ table, column: metric.columnIndex, row: metric.rowIndex });
   };
 
   _renderMetric = (metric, index) => {

src/core_plugins/region_map/public/region_map_visualization.js

@@ -154,7 +154,7 @@ export function RegionMapsVisualizationProvider(Private, config, i18n) {
         }
 
         const rowIndex = this._chartData.rows.findIndex(row => row[0] === event);
-        this._vis.API.events.addFilter(this._chartData, 0, rowIndex, event);
+        this._vis.API.events.filter({ table: this._chartData, column: 0, row: rowIndex, value: event });
       });
 
       this._choroplethLayer.on('styleChanged', (event) => {

src/core_plugins/tagcloud/public/tag_cloud_visualization.js

@@ -47,9 +47,9 @@ export class TagCloudVisualization {
       if (!this._bucketAgg) {
         return;
       }
-      this._vis.API.events.addFilter(
-        event.meta.data, 0, event.meta.rowIndex
-      );
+      this._vis.API.events.filter({
+        table: event.meta.data, column: 0, row: event.meta.rowIndex
+      });
     });
     this._renderComplete$ = Rx.fromEvent(this._tagCloud, 'renderComplete');
 

src/ui/public/vis/__tests__/_vis.js

@@ -19,7 +19,6 @@
 
 import _ from 'lodash';
 import ngMock from 'ng_mock';
-import sinon from 'sinon';
 import expect from 'expect.js';
 import { VisProvider } from '..';
 import FixturesStubbedLogstashIndexPatternProvider from 'fixtures/stubbed_logstash_index_pattern';
@@ -42,15 +41,6 @@ describe('Vis Class', function () {
     listeners: { click: _.noop }
   };
 
-  // Wrap the given vis type definition in a state, that can be passed to vis
-  const state = (type) => ({
-    type: {
-      visConfig: { defaults: {} },
-      schemas: {},
-      ...type,
-    }
-  });
-
   beforeEach(ngMock.module('kibana'));
   beforeEach(ngMock.inject(function (Private) {
     Vis = Private(VisProvider);
@@ -116,41 +106,4 @@ describe('Vis Class', function () {
     });
   });
 
-  describe('vis addFilter method', () => {
-    let aggConfig;
-    let data;
-
-    beforeEach(() => {
-      aggConfig = {
-        type: { name: 'terms' },
-        params: {},
-        createFilter: sinon.stub()
-      };
-
-      data = {
-        columns: [{
-          id: 'col-0',
-          title: 'test',
-          aggConfig
-        }],
-        rows: [{ 'col-0': 'US' }]
-      };
-    });
-
-
-    it('adds a simple filter', () => {
-      const vis = new Vis(indexPattern, state({ requestHandler: 'none' }));
-      vis.API.events.addFilter(data, 0, 0);
-      expect(aggConfig.createFilter.callCount).to.be(1);
-      expect(aggConfig.createFilter.getCall(0).args[0]).to.be('US');
-    });
-
-    it('adds a filter if value is provided instead of row index', () => {
-      const vis = new Vis(indexPattern, state({ requestHandler: 'none' }));
-      vis.API.events.addFilter(data, 0, -1, 'UK');
-      expect(aggConfig.createFilter.callCount).to.be(1);
-      expect(aggConfig.createFilter.getCall(0).args[0]).to.be('UK');
-    });
-  });
-
 });

src/ui/public/vis/__tests__/vis_types/base_vis_type.js

@@ -18,9 +18,16 @@
  */
 
 import expect from 'expect.js';
-import { BaseVisType } from '../../vis_types/base_vis_type';
+import ngMock from 'ng_mock';
+import { BaseVisTypeProvider } from '../../vis_types/base_vis_type';
 
 describe('Base Vis Type', function () {
+  let BaseVisType;
+
+  beforeEach(ngMock.module('kibana'));
+  beforeEach(ngMock.inject(function (Private) {
+    BaseVisType = Private(BaseVisTypeProvider);
+  }));
 
   describe('initialization', () => {
     it('should throw if mandatory properties are missing', () => {

src/ui/public/vis/__tests__/vis_types/react_vis_type.js

@@ -18,10 +18,13 @@
  */
 
 import expect from 'expect.js';
-import { ReactVisType } from '../../vis_types/react_vis_type';
+import ngMock from 'ng_mock';
+import { ReactVisTypeProvider } from '../../vis_types/react_vis_type';
 
 describe('React Vis Type', function () {
 
+  let ReactVisType;
+
   const visConfig = {
     name: 'test',
     title: 'test',
@@ -31,6 +34,11 @@ describe('React Vis Type', function () {
     type: { visConfig: { component: 'test' } }
   };
 
+  beforeEach(ngMock.module('kibana'));
+  beforeEach(ngMock.inject(function (Private) {
+    ReactVisType = Private(ReactVisTypeProvider);
+  }));
+
   describe('initialization', () => {
     it('should throw if component is not set', () => {
       expect(() => {

src/ui/public/vis/vis.js

@@ -32,20 +32,17 @@ import _ from 'lodash';
 import { VisTypesRegistryProvider } from '../registry/vis_types';
 import { AggConfigs } from './agg_configs';
 import { PersistedState } from '../persisted_state';
-import { onBrushEvent } from '../utils/brush_event';
 import { FilterBarQueryFilterProvider } from '../filter_bar/query_filter';
 import { updateVisualizationConfig } from './vis_update';
 import { SearchSourceProvider } from '../courier/search_source';
 import { SavedObjectsClientProvider } from '../saved_objects';
 import { timefilter } from 'ui/timefilter';
-import { VisFiltersProvider } from './vis_filters';
 
 export function VisProvider(Private, indexPatterns, getAppState) {
   const visTypes = Private(VisTypesRegistryProvider);
   const queryFilter = Private(FilterBarQueryFilterProvider);
   const SearchSource = Private(SearchSourceProvider);
   const savedObjectsClient = Private(SavedObjectsClientProvider);
-  const visFilter = Private(VisFiltersProvider);
 
   class Vis extends EventEmitter {
     constructor(indexPattern, visState) {
@@ -73,14 +70,8 @@ export function VisProvider(Private, indexPatterns, getAppState) {
         timeFilter: timefilter,
         queryFilter: queryFilter,
         events: {
-          // the filter method will be removed in the near feature
-          // you should rather use addFilter method below
-          filter: visFilter.filter,
-          createFilter: visFilter.createFilter,
-          addFilter: visFilter.addFilter,
-          brush: (event) => {
-            onBrushEvent(event, getAppState());
-          }
+          filter: data => this.eventsSubject.next({ name: 'filterBucket', data }),
+          brush: data => this.eventsSubject.next({ name: 'brush', data }),
         },
         getAppState,
       };

src/ui/public/vis/vis_factory.js

@@ -17,11 +17,13 @@
  * under the License.
  */
 
-import { BaseVisType, AngularVisTypeProvider, ReactVisType, VislibVisTypeProvider } from './vis_types';
+import { BaseVisTypeProvider, AngularVisTypeProvider, ReactVisTypeProvider, VislibVisTypeProvider } from './vis_types';
 
 export const VisFactoryProvider = (Private) => {
   const AngularVisType = Private(AngularVisTypeProvider);
   const VislibVisType = Private(VislibVisTypeProvider);
+  const BaseVisType = Private(BaseVisTypeProvider);
+  const ReactVisType = Private(ReactVisTypeProvider);
 
   return {
     createBaseVisualization: (config) => {

src/ui/public/vis/vis_filters.js

@@ -20,6 +20,7 @@
 import _ from 'lodash';
 import { FilterBarPushFiltersProvider } from '../filter_bar/push_filters';
 import { FilterBarQueryFilterProvider } from '../filter_bar/query_filter';
+import { onBrushEvent } from '../utils/brush_event';
 
 const getTerms = (table, columnIndex, rowIndex) => {
   if (rowIndex === -1) {
@@ -41,26 +42,26 @@ const getTerms = (table, columnIndex, rowIndex) => {
   }))];
 };
 
-export function VisFiltersProvider(Private, getAppState) {
+const createFilter = (data, columnIndex, rowIndex, cellValue) => {
+  const { aggConfig, id: columnId } = data.columns[columnIndex];
+  let filter = [];
+  const value = rowIndex > -1 ? data.rows[rowIndex][columnId] : cellValue;
+  if (value === null || value === undefined) {
+    return;
+  }
+  if (aggConfig.type.name === 'terms' && aggConfig.params.otherBucket) {
+    const terms = getTerms(data, columnIndex, rowIndex);
+    filter = aggConfig.createFilter(value, { terms });
+  } else {
+    filter = aggConfig.createFilter(value);
+  }
+  return filter;
+};
+
+const VisFiltersProvider = (Private, getAppState) => {
   const filterBarPushFilters = Private(FilterBarPushFiltersProvider);
   const queryFilter = Private(FilterBarQueryFilterProvider);
 
-  const  createFilter = (data, columnIndex, rowIndex, cellValue) => {
-    const { aggConfig, id: columnId } = data.columns[columnIndex];
-    let filter = [];
-    const value = rowIndex > -1 ? data.rows[rowIndex][columnId] : cellValue;
-    if (value === null || value === undefined) {
-      return;
-    }
-    if (aggConfig.type.name === 'terms' && aggConfig.params.otherBucket) {
-      const terms = getTerms(data, columnIndex, rowIndex);
-      filter = aggConfig.createFilter(value, { terms });
-    } else {
-      filter = aggConfig.createFilter(value);
-    }
-    return filter;
-  };
-
   const filter = (event, { simulate } = {}) => {
     let data = event.datum.aggConfigResult;
     const filters = [];
@@ -87,14 +88,18 @@ export function VisFiltersProvider(Private, getAppState) {
     return filters;
   };
 
-  const  addFilter = (data, columnIndex, rowIndex, cellValue) => {
-    const filter = createFilter(data, columnIndex, rowIndex, cellValue);
+  const addFilter = (event) => {
+    const filter = createFilter(event.table, event.column, event.row, event.value);
     queryFilter.addFilters(filter);
   };
 
   return {
-    createFilter,
     addFilter,
-    filter
+    filter,
+    brush: (event) => {
+      onBrushEvent(event, getAppState());
+    },
   };
-}
+};
+
+export { VisFiltersProvider, createFilter };

src/ui/public/vis/vis_types/angular_vis_type.js

@@ -17,11 +17,12 @@
  * under the License.
  */
 
-import { BaseVisType } from './base_vis_type';
+import { BaseVisTypeProvider } from './base_vis_type';
 import $ from 'jquery';
 
 
-export function AngularVisTypeProvider($compile, $rootScope) {
+export function AngularVisTypeProvider(Private, $compile, $rootScope) {
+  const BaseVisType = Private(BaseVisTypeProvider);
 
   class AngularVisController {
     constructor(domeElement, vis) {