From be0fcbee47008e09cc13feecdef09b9fb20579bf Mon Sep 17 00:00:00 2001
From: Blakko <fabio.tacchelli@gmail.com>
Date: Wed, 21 Mar 2018 14:32:47 +0100
Subject: [PATCH] Do not fire cardinality queries on edit mode

---
 .../public/siren_auto_join_vis_controller.js   | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/src/kibi_plugins/siren_auto_join_vis/public/siren_auto_join_vis_controller.js b/src/kibi_plugins/siren_auto_join_vis/public/siren_auto_join_vis_controller.js
index 9153be5be..23456f9a8 100644
--- a/src/kibi_plugins/siren_auto_join_vis/public/siren_auto_join_vis_controller.js
+++ b/src/kibi_plugins/siren_auto_join_vis/public/siren_auto_join_vis_controller.js
@@ -344,11 +344,12 @@ function controller($scope, $rootScope, Private, kbnIndex, config, kibiState, ge
     // here grab visible buttons and request count update
 
     const buttons = sirenAutoJoinHelper.getVisibleVirtualEntitySubButtons($scope.tree, $scope.vis.params.layout);
-
-    _addButtonQuery(buttons, currentDashboardId)
-    .then(results => {
-      updateCounts(results, $scope);
-    });
+    if (!edit) {
+      _addButtonQuery(buttons, currentDashboardId)
+      .then(results => {
+        updateCounts(results, $scope);
+      });
+    }
   };
 
   // As buttons are shown on UI side as a tree
@@ -590,7 +591,12 @@ function controller($scope, $rootScope, Private, kbnIndex, config, kibiState, ge
       }
 
       return promise
-      .then(tree => sirenAutoJoinHelper.updateTreeCardinalityCounts(tree))
+      .then(tree => {
+        if (edit) {
+          return tree;
+        }
+        return sirenAutoJoinHelper.updateTreeCardinalityCounts(tree);
+      })
       .then(tree => {
         if (!tree || !tree.nodes.length) {
           return Promise.resolve({});