From 84a9365922819399525c56e541ac6ce3055fa358 Mon Sep 17 00:00:00 2001
From: Tom Laermans <tom@localhost>
Date: Mon, 19 Sep 2011 13:59:08 +0000
Subject: [PATCH] fix links and menu bar for health sensors

git-svn-id: http://www.observium.org/svn/observer/trunk@2513 61d68cd4-352d-0410-923a-c4978735b2b8
---
 html/includes/print-menubar.php | 30 +++++++++++++++++++++++-------
 html/pages/health.inc.php       |  4 ++--
 2 files changed, 25 insertions(+), 9 deletions(-)

diff --git a/html/includes/print-menubar.php b/html/includes/print-menubar.php
index 67c427c666..517da4506e 100644
--- a/html/includes/print-menubar.php
+++ b/html/includes/print-menubar.php
@@ -353,33 +353,49 @@ $menu_sensors = $used_sensors;
             <div class="col_1">
 
 <ul>
-        <li><a href="health/mempool/"><img src="images/icons/memory.png" border="0" align="absmiddle" /> Memory</a></li>
-        <li><a href="health/processor/"><img src="images/icons/processor.png" border="0" align="absmiddle" /> Processor</a></li>
-        <li><a href="health/storage/"><img src="images/icons/storage.png" border="0" align="absmiddle" /> Storage</a></li>
+        <li><a href="health/metric=mempool/"><img src="images/icons/memory.png" border="0" align="absmiddle" /> Memory</a></li>
+        <li><a href="health/metric=processor/"><img src="images/icons/processor.png" border="0" align="absmiddle" /> Processor</a></li>
+        <li><a href="health/metric=storage/"><img src="images/icons/storage.png" border="0" align="absmiddle" /> Storage</a></li>
 <?php
 if ($menu_sensors)
 {
   $sep = 0;
-  echo('<li><hr width="140" /></li>');
+  echo('<li><hr width="97%" /></li>');
 }
 
 foreach (array('fanspeed','humidity','temperature') as $item)
 {
   if ($menu_sensors[$item])
   {
-    echo ('<li><a href="health/'.$item.'/"><img src="images/icons/'.$item.'.png" border="0" align="absmiddle" /> '.ucfirst($item).'</a></li>');
+    echo ('<li><a href="health/metric='.$item.'/"><img src="images/icons/'.$item.'.png" border="0" align="absmiddle" /> '.ucfirst($item).'</a></li>');
     unset($menu_sensors[$item]);$sep++;
   }
 }
 
 if ($sep)
 {
-  echo('<li><hr width="140" /></li>');
+  echo('<li><hr width="97%" /></li>');
   $sep = 0;
 }
+
+foreach (array('current','frequency','power','voltage') as $item)
+{
+  if ($menu_sensors[$item])
+  {
+    echo ('<li><a href="health/metric='.$item.'/"><img src="images/icons/'.$item.'.png" border="0" align="absmiddle" /> '.ucfirst($item).'</a></li>');
+    unset($menu_sensors[$item]);$sep++;
+  }
+}
+
+if ($sep && array_keys($menu_sensors))
+{
+  echo('<li><hr width="97%" /></li>');
+  $sep = 0;
+}
+
 foreach (array_keys($menu_sensors) as $item)
 {
-  echo ('<li><a href="health/'.$item.'/"><img src="images/icons/'.$item.'.png" border="0" align="absmiddle" /> '.ucfirst($item).'</a></li>');
+  echo ('<li><a href="health/metric='.$item.'/"><img src="images/icons/'.$item.'.png" border="0" align="absmiddle" /> '.ucfirst($item).'</a></li>');
   unset($menu_sensors[$item]);$sep++;
 }
 
diff --git a/html/pages/health.inc.php b/html/pages/health.inc.php
index bb63f06abb..5baa083cce 100644
--- a/html/pages/health.inc.php
+++ b/html/pages/health.inc.php
@@ -65,14 +65,14 @@ if ($vars['view'] == "graphs")
 
 echo(' | ');
 
-if ($vars['view'] == "nographs")
+if ($vars['view'] != "graphs")
 {
   echo('<span class="pagemenu-selected">');
 }
 
 echo(generate_link("No Graphs",$link_array,array('metric'=> $vars['metric'], 'view' => "detail")));
 
-if ($vars['view'] == "nographs") 
+if ($vars['view'] != "graphs") 
 { 
   echo('</span>'); 
 }