Merge branch 'auxesisgh-92'

jessereynolds · Mar 25, 2012 · b73e9ee · b73e9ee
2 parents 3fc2858 + e863feb
commit b73e9ee
Showing 1 changed file with 18 additions and 4 deletions.
diff --git a/lib/visage-app/public/javascripts/builder.js b/lib/visage-app/public/javascripts/builder.js
@@ -523,11 +523,25 @@ var ChartBuilder = new Class({
         show.addEvent('click', function(e) {
             e.stop();
 
-            this.showGraphs();
 
-            // Fade the save button once graphs have been rendered.
-            this.save.fade.delay(1500, this.save, 'in')
-            this.profile_name.fade.delay(1500, this.profile_name, 'in')
+            var hosts   = $(this.searchers.host).getElements("div.token.finalized"),
+                metrics = $(this.searchers.metric).getElements("div.token.finalized");
+
+            if (hosts.length > 0 && metrics.length > 0) {
+                this.showGraphs();
+                // Fade the save button once graphs have been rendered.
+                this.save.fade.delay(1500, this.save, 'in')
+                this.profile_name.fade.delay(1500, this.profile_name, 'in')
+            } else {
+                new Message({
+                    title:    'Oops!',
+                    message:  'You need to specify some hosts and metrics before you can show graphs.',
+                    top:      true,
+                    iconPath: '/images/',
+                    icon:     'caution.png',
+                }).say();
+            }
+
         }.bind(this));
 
     },