COMPASS-385 - Check data exists before modifying overlays for RTSS ta…

…bles (#700)
mongodb-js · Dec 20, 2016 · 08e274a · 08e274a
1 parent 138512f
commit 08e274a
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/src/internal-packages/server-stats/lib/store/current-op-store.js b/src/internal-packages/server-stats/lib/store/current-op-store.js
@@ -50,15 +50,17 @@ const CurrentOpStore = Reflux.createStore({
     }
     this.overlayIndex = index;
     this.inOverlay = true;
-    this.trigger(visErrors[this.overlayIndex], visOps[this.overlayIndex]);
+    const data = visOps.length === 0 ? [] : visOps[this.overlayIndex];
+    this.trigger(visErrors[this.overlayIndex], data);
   },
 
   mouseOut: function() {
     this.inOverlay = false;
     const startPause = Math.max(this.endPause - this.xLength, 0);
     const visOps = this.allOps.slice(startPause, this.endPause);
     const visErrors = this.errored.slice(startPause, this.endPause);
-    this.trigger(visErrors[this.overlayIndex], visOps[visOps.length - 1]);
+    const data = visOps.length === 0 ? [] : visOps[visOps.length - 1];
+    this.trigger(visErrors[this.overlayIndex], data);
   },
 
   currentOp: function() {

diff --git a/src/internal-packages/server-stats/lib/store/top-store.js b/src/internal-packages/server-stats/lib/store/top-store.js
@@ -51,15 +51,17 @@ const TopStore = Reflux.createStore({
     }
     this.overlayIndex = index;
     this.inOverlay = true;
-    this.trigger(visErrors[this.overlayIndex], visOps[this.overlayIndex]);
+    const data = visOps.length === 0 ? [] : visOps[this.overlayIndex];
+    this.trigger(visErrors[this.overlayIndex], data);
   },
 
   mouseOut: function() {
     this.inOverlay = false;
     const startPause = Math.max(this.endPause - this.xLength, 0);
     const visOps = this.allOps.slice(startPause, this.endPause);
     const visErrors = this.errored.slice(startPause, this.endPause);
-    this.trigger(visErrors[this.overlayIndex], visOps[visOps.length - 1]);
+    const data = visOps.length === 0 ? [] : visOps[visOps.length - 1];
+    this.trigger(visErrors[this.overlayIndex], data);
   },
 
   // Calculate list as current hottest collection (like Cloud and system top)