From ae8dcd22c1c8155462059b6aeea46dbd2c96b3e6 Mon Sep 17 00:00:00 2001
From: Tom Laermans <tom@localhost>
Date: Sat, 20 Nov 2010 23:21:38 +0000
Subject: [PATCH] Only show port in red if Operational Down and Admin Up, as
 suggested by mziel (http://www.observium.org/forum/index.php/topic,82.0.html)

git-svn-id: http://www.observium.org/svn/observer/trunk@1729 61d68cd4-352d-0410-923a-c4978735b2b8
---
 html/pages/device/edit/ports.inc.php | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/html/pages/device/edit/ports.inc.php b/html/pages/device/edit/ports.inc.php
index 03eddabb9..aebfe6da4 100644
--- a/html/pages/device/edit/ports.inc.php
+++ b/html/pages/device/edit/ports.inc.php
@@ -29,10 +29,10 @@ while($device = mysql_fetch_array($query)) {
   echo("<td align=left>".$device['ifDescr'] . "</td>");
   echo("<td align=right>". $device['ifAdminStatus']."</td>");
 
-  # Mark interfaces which are down yet not ignored, or up - yet ignored - as to draw the attention 
+  # Mark interfaces which are OperDown (but not AdminDown) yet not ignored, or up - yet ignored - as to draw the attention 
   # to a possible problem.
   #
-  $outofsync =  ($device['ignore'] == ($device['ifOperStatus'] == 'down' ? 1 : 0))  ? "" : "class=red";
+  $outofsync =  ($device['ignore'] == ($device['ifOperStatus'] == 'down' && $device['ifAdminStatus'] != 'down' ? 1 : 0))  ? "" : "class=red";
 
   echo("<td align=right><span ".$outofsync.">". $device['ifOperStatus']."</span></td>");