From d2483ad4db866550b33c07fb82dd77e93c15de89 Mon Sep 17 00:00:00 2001
From: CJ Cenizal <cj@cenizal.com>
Date: Wed, 15 Feb 2017 13:19:30 -0800
Subject: [PATCH] Add isFetchingItems flag to disable the PromptForItems when
 fetching items in Visualize and Dashboard listing tables.

---
 .../dashboard/listing/dashboard_listing.html   |  2 +-
 .../dashboard/listing/dashboard_listing.js     | 10 +++++++---
 .../visualize/listing/visualize_listing.html   |  2 +-
 .../visualize/listing/visualize_listing.js     | 18 +++++++++++-------
 4 files changed, 20 insertions(+), 12 deletions(-)

diff --git a/src/core_plugins/kibana/public/dashboard/listing/dashboard_listing.html b/src/core_plugins/kibana/public/dashboard/listing/dashboard_listing.html
index c4f930bda5a71..47ba74333372b 100644
--- a/src/core_plugins/kibana/public/dashboard/listing/dashboard_listing.html
+++ b/src/core_plugins/kibana/public/dashboard/listing/dashboard_listing.html
@@ -87,7 +87,7 @@
     <!-- PromptForItems -->
     <div
       class="kuiPanel kuiPanel--centered kuiPanel--withHeader"
-      ng-if="!listingController.items.length && !listingController.filter"
+      ng-if="!listingController.isFetchingItems && !listingController.items.length && !listingController.filter"
     >
       <div class="kuiPromptForItems">
         <div class="kuiPromptForItems__message">
diff --git a/src/core_plugins/kibana/public/dashboard/listing/dashboard_listing.js b/src/core_plugins/kibana/public/dashboard/listing/dashboard_listing.js
index 1ae98b0637589..1f4526865699a 100644
--- a/src/core_plugins/kibana/public/dashboard/listing/dashboard_listing.js
+++ b/src/core_plugins/kibana/public/dashboard/listing/dashboard_listing.js
@@ -41,9 +41,12 @@ export function DashboardListingController($injector, $scope) {
     this.pageOfItems = limitTo(this.items, this.pager.pageSize, this.pager.startIndex);
   };
 
-  const fetchObjects = () => {
+  const fetchItems = () => {
+    this.isFetchingItems = true;
+
     dashboardService.find(this.filter)
       .then(result => {
+        this.isFetchingItems = false;
         this.items = result.hits;
         calculateItemsOnPage();
       });
@@ -57,6 +60,7 @@ export function DashboardListingController($injector, $scope) {
     selectedItems = this.pageOfItems.slice(0);
   };
 
+  this.isFetchingItems = false;
   this.items = [];
   this.pageOfItems = [];
   this.filter = '';
@@ -65,7 +69,7 @@ export function DashboardListingController($injector, $scope) {
 
   $scope.$watch(() => this.filter, () => {
     deselectAll();
-    fetchObjects();
+    fetchItems();
   });
 
   /**
@@ -115,7 +119,7 @@ export function DashboardListingController($injector, $scope) {
       const selectedIds = selectedItems.map(item => item.id);
 
       dashboardService.delete(selectedIds)
-        .then(fetchObjects)
+        .then(fetchItems)
         .then(() => {
           deselectAll();
         })
diff --git a/src/core_plugins/kibana/public/visualize/listing/visualize_listing.html b/src/core_plugins/kibana/public/visualize/listing/visualize_listing.html
index f4eb808c75d20..70dda2a951266 100644
--- a/src/core_plugins/kibana/public/visualize/listing/visualize_listing.html
+++ b/src/core_plugins/kibana/public/visualize/listing/visualize_listing.html
@@ -85,7 +85,7 @@
     <!-- PromptForItems -->
     <div
       class="kuiPanel kuiPanel--centered kuiPanel--withHeader"
-      ng-if="!listingController.items.length && !listingController.filter"
+      ng-if="!listingController.isFetchingItems && !listingController.items.length && !listingController.filter"
     >
       <div class="kuiPromptForItems">
         <div class="kuiPromptForItems__message">
diff --git a/src/core_plugins/kibana/public/visualize/listing/visualize_listing.js b/src/core_plugins/kibana/public/visualize/listing/visualize_listing.js
index 0cd96bee764d9..dbfdc447d6dd4 100644
--- a/src/core_plugins/kibana/public/visualize/listing/visualize_listing.js
+++ b/src/core_plugins/kibana/public/visualize/listing/visualize_listing.js
@@ -42,12 +42,15 @@ export function VisualizeListingController($injector, $scope) {
     this.pageOfItems = limitTo(this.items, this.pager.pageSize, this.pager.startIndex);
   };
 
-  const fetchObjects = () => {
+  const fetchItems = () => {
+    this.isFetchingItems = true;
+
     visualizationService.find(this.filter)
-    .then(result => {
-      this.items = result.hits;
-      calculateItemsOnPage();
-    });
+      .then(result => {
+        this.isFetchingItems = false;
+        this.items = result.hits;
+        calculateItemsOnPage();
+      });
   };
 
   const deselectAll = () => {
@@ -58,6 +61,7 @@ export function VisualizeListingController($injector, $scope) {
     selectedItems = this.pageOfItems.slice(0);
   };
 
+  this.isFetchingItems = false;
   this.items = [];
   this.pageOfItems = [];
   this.filter = '';
@@ -66,7 +70,7 @@ export function VisualizeListingController($injector, $scope) {
 
   $scope.$watch(() => this.filter, () => {
     deselectAll();
-    fetchObjects();
+    fetchItems();
   });
 
   /**
@@ -145,7 +149,7 @@ export function VisualizeListingController($injector, $scope) {
       const selectedIds = selectedItems.map(item => item.id);
 
       visualizationService.delete(selectedIds)
-        .then(fetchObjects)
+        .then(fetchItems)
         .then(() => {
           deselectAll();
         })