From 78e8d3004d10e06406b28f57f5dc577009395a00 Mon Sep 17 00:00:00 2001
From: Laurent Destailleur <eldy@users.sourceforge.net>
Date: Sat, 25 Dec 2004 19:03:22 +0000
Subject: [PATCH] Qual: Simplification du code.

---
 htdocs/html.form.class.php | 80 ++++++++++++++++----------------------
 1 file changed, 33 insertions(+), 47 deletions(-)

diff --git a/htdocs/html.form.class.php b/htdocs/html.form.class.php
index 98156297b31..2bc1de7ccf3 100644
--- a/htdocs/html.form.class.php
+++ b/htdocs/html.form.class.php
@@ -764,60 +764,46 @@ class Form
     		\param	name            nom de la zone select
     		\param	array           tableau de key+valeur
     		\param	id              key présélectionnée
-    		\param	empty           1 si il faut un valeur "-" dans la liste, 0 sinon
+    		\param	empty           1 si il faut un valeur " " dans la liste, 0 sinon
     		\param	key_libelle     1 pour afficher la key dans la valeur "[key] value"
     */
 		
   function select_array($name, $array, $id='', $empty=0, $key_libelle=0)
     {
-      print '<select name="'.$name.'">';
-      
-      $i = 0;
+        print '<select name="'.$name.'">';
+        
+        $i = 0;
+        
+        if (strlen($id)) {
+            if ($empty == 1)
+            {
+                $array[0] = "&nbsp;";
+            }
+            reset($array);
+        }
+        
+        while (list($key, $value) = each ($array))
+        {
+            print "<option value=\"$key\" ";
 
-      if (strlen($id))
-	{
-	  if ($empty == 1)
-	    {
-	      $array[0] = "-";
-	    }
-	  reset($array);
+            // Si il faut présélectionner une valeur
+            if ($id && $id == $key)
+            {
+                print "selected";
+            }
 
-	  while (list($key, $value) = each ($array))
-	    {
-	      print "<option value=\"$key\" ";
-	      if ($id == $key)
-		{
-		  print "selected";
-		}
-	      if ($key_libelle)
-		{
-		  print ">[$key] $value</option>\n";  
-		}
-	      else
-		{
-		  print ">$value</option>\n";
-		}
-	    }
-	}
-      else
-	{
-	  while (list($key, $value) = each ($array) )
-	    {
-	      print "<option value=\"$key\" ";
-	      if ($key_libelle)
-		{
-		  print ">[$key] $value</option>\n";  
-		}
-	      else
-		{
-		  print ">$value</option>\n";
-		}
-	    }
-	
-	}
-
-      print "</select>";
-    
+            if ($key_libelle)
+            {
+                print ">[$key] $value</option>\n";  
+            }
+            else
+            {
+                if ($value=="-") { $value="&nbsp;"; }
+                print ">$value</option>\n";
+            }
+        }
+        
+        print "</select>";
     }
   /*
    *    \brief  Renvoie la chaîne de caractère décrivant l'erreur