Timelion graph is not refreshing content after searching or filtering (#67023)

DianaDerevyankina · DianaDerevyankina · commit 650505bfe1fc · 2020-05-20T15:26:31.000+03:00
* Timelion graph is not refreshing content after searching or filtering in Kibana Closes #66560 * Change test beforeEach configuration to match request body * Remove unused TimelionRequestQuery
diff --git a/src/plugins/vis_type_timelion/server/routes/run.ts b/src/plugins/vis_type_timelion/server/routes/run.ts
@@ -31,30 +31,6 @@ import { ConfigManager } from '../lib/config_manager';
 
 const timelionDefaults = getNamespacesSettings();
 
-export interface TimelionRequestQuery {
-  payload: {
-    sheet: string[];
-    extended?: {
-      es: {
-        filter: {
-          bool: {
-            filter: string[] | object;
-            must: string[];
-            should: string[];
-            must_not: string[];
-          };
-        };
-      };
-    };
-  };
-  time?: {
-    from?: string;
-    interval: string;
-    timezone: string;
-    to?: string;
-  };
-}
-
 export function runRoute(
   router: IRouter,
   {
diff --git a/src/plugins/vis_type_timelion/server/series_functions/es/es.test.js b/src/plugins/vis_type_timelion/server/series_functions/es/es.test.js
@@ -239,7 +239,7 @@ describe('es', () => {
             to: 5,
           },
           request: {
-            payload: {
+            body: {
               extended: {
                 es: {
                   filter: {
@@ -258,7 +258,7 @@ describe('es', () => {
         });
       });
 
-      it('adds the contents of payload.extended.es.filter to a filter clause of the bool', () => {
+      it('adds the contents of body.extended.es.filter to a filter clause of the bool', () => {
         config.kibana = true;
         const request = fn(config, tlConfig, emptyScriptedFields);
         const filter = request.body.query.bool.filter.bool;
diff --git a/src/plugins/vis_type_timelion/server/series_functions/es/lib/build_request.js b/src/plugins/vis_type_timelion/server/series_functions/es/lib/build_request.js
@@ -38,7 +38,7 @@ export default function buildRequest(config, tlConfig, scriptedFields, timeout)
 
   // Use the kibana filter bar filters
   if (config.kibana) {
-    bool.filter = _.get(tlConfig, 'request.payload.extended.es.filter');
+    bool.filter = _.get(tlConfig, 'request.body.extended.es.filter');
   }
 
   const aggs = {
diff --git a/src/plugins/vis_type_timelion/server/types.ts b/src/plugins/vis_type_timelion/server/types.ts
@@ -18,4 +18,3 @@
  */
 
 export { TimelionFunctionInterface, TimelionFunctionConfig } from './lib/classes/timelion_function';
-export { TimelionRequestQuery } from './routes/run';

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ export default function buildRequest(config, tlConfig, scriptedFields, timeout)`
`38`	`38`
`39`	`39`	`// Use the kibana filter bar filters`
`40`	`40`	`if (config.kibana) {`
`41`		`- bool.filter = _.get(tlConfig, 'request.payload.extended.es.filter');`
	`41`	`+ bool.filter = _.get(tlConfig, 'request.body.extended.es.filter');`
`42`	`42`	`}`
`43`	`43`
`44`	`44`	`const aggs = {`
Original file line number	Diff line number	Diff line change
`@@ -18,4 +18,3 @@`
`18`	`18`	`*/`
`19`	`19`
`20`	`20`	`export { TimelionFunctionInterface, TimelionFunctionConfig } from './lib/classes/timelion_function';`
`21`		`-export { TimelionRequestQuery } from './routes/run';`