JVM tab will now only appear when there is a JVM running

cloudius-systems · Sep 3, 2014 · 238cd19 · 238cd19
1 parent f46598f
commit 238cd19
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 3 deletions.
diff --git a/osv/App.js b/osv/App.js
@@ -26,9 +26,16 @@ OSv.App = Davis(function() {
     profilerHandler.handler();
   });
 
-  this.get("/dashboard/jvm", function() {
-    jvmHandler.handler();
-  });
+  var self = this;
+  OSv.API.JVM.version()
+    .then(function () {
+      self.get("/dashboard/jvm", function() {
+          jvmHandler.handler();
+      });
+    })
+   .fail(function () {
+      baseHandler.removeJVMTab();
+   });
 
   this.bind('runRoute', function (data) {
     runRoute.initCustomEvent('runRoute', true, true, data);

diff --git a/osv/PageHandlers/BaseHandler.js b/osv/PageHandlers/BaseHandler.js
@@ -24,6 +24,10 @@ OSv.PageHandlers.BaseHandler = (function() {
     $("[data-swagger-href]").attr("href", OSv.Settings.BasePath);
   };
 
+  BaseHandler.prototype.removeJVMTab = function() {
+    $("a[href='/dashboard/jvm']").parent("li").remove();
+  };
+
   BaseHandler.prototype.subscribe = function() {
     var self = this;