From 79ea4cbe1ad35d65c21758e4a3944c08ed710a71 Mon Sep 17 00:00:00 2001
From: Dane Everitt <dane@daneeveritt.com>
Date: Sun, 15 Jul 2018 18:11:29 -0700
Subject: [PATCH] Correct N+1 utilization checking

---
 .../components/dashboard/Dashboard.vue        | 36 ++++++++++++-------
 1 file changed, 24 insertions(+), 12 deletions(-)

diff --git a/resources/assets/scripts/components/dashboard/Dashboard.vue b/resources/assets/scripts/components/dashboard/Dashboard.vue
index de2d91c52..79a16fa72 100644
--- a/resources/assets/scripts/components/dashboard/Dashboard.vue
+++ b/resources/assets/scripts/components/dashboard/Dashboard.vue
@@ -64,11 +64,13 @@
         },
 
         /**
-         * Once the page is mounted set a function to run every 5 seconds that will
+         * Once the page is mounted set a function to run every 10 seconds that will
          * iterate through the visible servers and fetch their resource usage.
          */
         mounted: function () {
-            this._iterateServerResourceUse();
+            window.setTimeout(() => {
+                this._iterateServerResourceUse();
+            }, 5000);
         },
 
         computed: {
@@ -133,9 +135,6 @@
                         }
 
                         window.events.$emit(`server:${server.uuid}::resources`, response.data.attributes);
-                    })
-                    .catch(err => {
-                        console.error(err);
                     });
             },
 
@@ -144,12 +143,23 @@
              *
              * @private
              */
-            _iterateServerResourceUse: function (initialTimeout = 5000) {
-                window.setTimeout(() => {
-                    if (this.documentVisible) {
-                        return window.setTimeout(this._iterateServerResourceUse(), 5000);
-                    }
-                }, initialTimeout);
+            _iterateServerResourceUse: function (loop = true) {
+                // Try again in 10 seconds, window is not in the foreground.
+                if (!this.documentVisible && loop) {
+                    window.setTimeout(() => {
+                        this._iterateServerResourceUse();
+                    }, 10000);
+                }
+
+                this.servers.forEach(server => {
+                    this.getResourceUse(server);
+                });
+
+                if (loop) {
+                    window.setTimeout(() => {
+                        this._iterateServerResourceUse();
+                    }, 10000);
+                }
             },
 
             /**
@@ -167,7 +177,9 @@
                 // If it has been more than 30 seconds since this window was put into the background
                 // lets go ahead and refresh all of the listed servers so that they have fresh stats.
                 const diff = differenceInSeconds(new Date(), this.backgroundedAt);
-                this._iterateServerResourceUse(diff > 30 ? 1 : 5000);
+                if (diff > 30) {
+                    this._iterateServerResourceUse(false);
+                }
             },
         }
     };