From 2a193a5f5995a76d9e0c42a0190808251f3b27a0 Mon Sep 17 00:00:00 2001
From: Daniel STAN <daniel.stan@crans.org>
Date: Wed, 8 Oct 2014 18:08:34 +0200
Subject: [PATCH] =?UTF-8?q?autostatus:=20cosm=C3=A9tique=20de=20l'info=20a?=
 =?UTF-8?q?dditionnelle?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 gestion/gen_confs/autostatus.py | 21 +++++++++------------
 wiki/macro/HostStatus.py        |  4 +++-
 2 files changed, 12 insertions(+), 13 deletions(-)

diff --git a/gestion/gen_confs/autostatus.py b/gestion/gen_confs/autostatus.py
index 1ea4244a..7117aa21 100755
--- a/gestion/gen_confs/autostatus.py
+++ b/gestion/gen_confs/autostatus.py
@@ -220,15 +220,16 @@ class autostatus(gen_config) :
             if m.nom() in self.pasimportant :
                 continue
 
-            info = filter(lambda x: not x.startswith("<"), m.info())
-            if len(info) != len(m.info()):
+
+            info = m.info()
+            if any(i.startswith('<') for i in info):
                 continue
+            info = filter(lambda i: not i.startswith(';'), info) + ['']
 
             # les bornes
             if isinstance(m, BorneWifi):
                 # C'est une borne
-                info = filter(lambda x: not x.startswith("<"), m.info())
-                if info and len(info) == len(m.info()):
+                if info:
                     # on regarde si c'est une borne de batiment
                     if ( info[0][0:3] == "Au " ) and ( len(info[0]) == 5 ) :
                         bornes[ "1-" + info[0][4]+info[0][3] + "-" + m.nom() ] = [ m.nom().split(".")[0] , m.nom() , 'none' , self.mail_contact(m.nom()) , info[0] ]
@@ -241,8 +242,7 @@ class autostatus(gen_config) :
                     pass
 
             # les switchs
-            elif m.info() and 'switch' in m.info()[0].lower() :
-
+            elif any('switch' in i.lower() for i in info):
                 # est ce que c'est un batiment
                 if 'bat' == m.nom()[0:3] :
                     index = '1-' + m.nom().split(".")[0]
@@ -250,7 +250,7 @@ class autostatus(gen_config) :
                     index = '0-' + m.nom().split(".")[0]
 
                 # on ajoute au dictionnaire
-                switchs[ index ] = [ m.nom().split(".")[0] , m.nom() , 'none' , self.mail_contact(m.nom()) , m.info()[0] ]
+                switchs[ index ] = [ m.nom().split(".")[0] , m.nom() , 'none' , self.mail_contact(m.nom()) , info[0] ]
 
             # la ferme
             elif '.ferme.crans.org' in m.nom() :
@@ -262,14 +262,11 @@ class autostatus(gen_config) :
 
             # les serveurs
             else :
-                if ".wifi.crans.org" in m.nom() :
+                if ".wifi.crans.org" in m.nom():
                     nom = m.nom().split(".")[0] + ".wifi"
                 else :
                     nom = m.nom().split(".")[0]
-                if m.info() :
-                    serveurs[ nom ] = [ nom , m.nom() , 'none' , self.mail_contact(m.nom()) , m.info()[0] ]
-                else :
-                    serveurs[ nom ] = [ nom , m.nom() , 'none' , self.mail_contact(m.nom()) , 'Infos non disponibles' ]
+                serveurs[ nom ] = [ nom , m.nom() , 'none' , self.mail_contact(m.nom()) , info[0] ]
 
         # route vers l'extérieur
         ########################
diff --git a/wiki/macro/HostStatus.py b/wiki/macro/HostStatus.py
index dda609c8..ae7b1958 100644
--- a/wiki/macro/HostStatus.py
+++ b/wiki/macro/HostStatus.py
@@ -16,7 +16,9 @@ def execute(macro, text) :
             except IndexError:
                 dec = u"N/A"
             dec = dec.strip()
-            descriptions[host] = u'%s (%s)' % (host, dec)
+            descriptions[host] = u'%s' % host
+            if dec:
+                descriptions[host] += u' (%s)' % dec
 
     lines = open('/usr/scripts/var/local.status').readlines()