fixes

elastic · Jun 12, 2019 · 5ce25c5 · 5ce25c5
1 parent 46e3d1a
commit 5ce25c5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/legacy/ui/public/vis/vis_filters/vis_filters.js b/src/legacy/ui/public/vis/vis_filters/vis_filters.js
@@ -103,7 +103,7 @@ const createFiltersFromEvent = (event) => {
   return filters;
 };
 
-const VisFiltersProvider = (getAppState) => {
+const VisFiltersProvider = (getAppState, $timeout) => {
 
   const pushFilters = (filters, simulate) => {
     const appState = getAppState();
@@ -113,6 +113,8 @@ const VisFiltersProvider = (getAppState) => {
         return i === flatFilters.findIndex(f => _.isEqual(v, f));
       });
       pushFilterBarFilters(appState, deduplicatedFilters);
+      // to trigger angular digest cycle, we can get rid of this once we have either new filterManager or actions API
+      $timeout(_.noop, 0);
     }
   };
 

diff --git a/src/legacy/ui/public/visualize/loader/embedded_visualize_handler.ts b/src/legacy/ui/public/visualize/loader/embedded_visualize_handler.ts
@@ -203,7 +203,7 @@ export class EmbeddedVisualizeHandler {
     this.events$.subscribe(event => {
       if (this.actions[event.name]) {
         event.data.aggConfigs = getTableAggs(this.vis);
-        const newFilters = this.actions[event.name](event.data);
+        const newFilters = this.actions[event.name](event.data) || [];
         visFilters.pushFilters(newFilters);
       }
     });