Fix: Le choix du destinataire affichait un message d'erreur alors que tout est ok

htdocs/comm/action/fiche.php

@@ -232,7 +232,7 @@ if ($_GET["action"] == 'create')
       if ($_GET["socid"])
 	{
 	  print '<tr><td>'.$langs->trans("ActionOnContact").'</td><td width="40%">';
-          print $html->select_contacts($_GET["socid"],'','contactid',1,1);
+          $html->select_contacts($_GET["socid"],'','contactid',1,1);
     	  print '</td></tr>';
 	}
 
@@ -324,7 +324,7 @@ if ($_GET["action"] == 'create')
       if ($_GET["socid"])
 	{
 	  print '<tr><td nowrap>'.$langs->trans("ActionOnContact").'</td><td>';
-          print $html->select_contacts($_GET["socid"],'','contactid',1,1);
+          $html->select_contacts($_GET["socid"],'','contactid',1,1);
     	  print '</td></tr>';
 	}
       

htdocs/comm/propal.php

@@ -538,7 +538,7 @@ if ($_GET['propalid'] > 0)
 
 	        print '<tr><td>'.$langs->trans('Ref').'</td><td colspan="5">'.$propal->ref_url.'</td></tr>';
 
-            $rowspan=6;
+            $rowspan=9;
             
             // Société
             print '<tr><td>'.$langs->trans('Company').'</td><td colspan="5">';
@@ -558,13 +558,21 @@ if ($_GET['propalid'] > 0)
             print dolibarr_print_date($propal->date,'%a %d %B %Y');
             print '</td>';
 
+            if ($conf->projet->enabled) $rowspan++;
+    
+            // Notes
+            print '<td valign="top" colspan="2" width="50%" rowspan="'.$rowspan.'">'.$langs->trans('Note').' :<br>'. nl2br($propal->note).'</td>';
+    		print '</tr>';
+    
+    		// Date fin propal
+    		print '<tr>';
 			print '<td>';
 			print '<table class="nobordernopadding" width="100%"><tr><td>';
 			print $langs->trans('DateEndPropal');
 			print '</td>';
 			if ($_GET['action'] != 'editecheance' && $propal->brouillon) print '<td align="right"><a href="'.$_SERVER["PHP_SELF"].'?action=editecheance&amp;propalid='.$propal->id.'">'.img_edit($langs->trans('SetConditions'),1).'</a></td>';
 			print '</tr></table>';
-            print '<td>';
+            print '<td colspan="3">';
             if ($propal->brouillon && $_GET['action'] == 'editecheance')
             {
                 print '<form name="editecheance" action="'.$_SERVER["PHP_SELF"].'?propalid='.$propal->id.'" method="post">';
@@ -589,7 +597,10 @@ if ($_GET['propalid'] > 0)
             print '</tr>';
     		
 			
-			// date de livraison
+			// date de livraison (conditonné sur PROPAL_ADD_SHIPPING_DATE car carac à
+			// gérer par les commandes et non les propal
+			if ($conf->global->PROPAL_ADD_SHIPPING_DATE)
+			{
 				print '<tr><td>';
 				print '<table class="nobordernopadding" width="100%"><tr><td>';
 				print $langs->trans('DateDelivery');
@@ -610,6 +621,8 @@ if ($_GET['propalid'] > 0)
 					print dolibarr_print_date($propal->date_livraison,'%a %d %B %Y');
 				}
 				print '</td>';
+				print '</tr>';
+			}
 						
 			// Conditions et modes de réglement
 			print '<tr><td>';
@@ -628,14 +641,17 @@ if ($_GET['propalid'] > 0)
 				$html->form_conditions_reglement($_SERVER['PHP_SELF'].'?propalid='.$propal->id,$propal->cond_reglement_id,'none');
 			}
 			print '</td>';
+			print '</tr>';
 			
+			// Mode paiement
+			print '<tr>';
 			print '<td width="25%">';
 			print '<table class="nobordernopadding" width="100%"><tr><td>';
 			print $langs->trans('PaymentMode');
 			print '</td>';
 			if ($_GET['action'] != 'editmode' && $propal->brouillon) print '<td align="right"><a href="'.$_SERVER["PHP_SELF"].'?action=editmode&amp;propalid='.$propal->id.'">'.img_edit($langs->trans('SetMode'),1).'</a></td>';
 			print '</tr></table>';
-			print '</td><td width="25%">';
+			print '</td><td colspan="3">';
 			if ($_GET['action'] == 'editmode')
 			{
 				$html->form_modes_reglement($_SERVER['PHP_SELF'].'?propalid='.$propal->id,$propal->mode_reglement_id,'mode_reglement_id');
@@ -664,10 +680,7 @@ if ($_GET['propalid'] > 0)
 				$html->form_contacts($_SERVER['PHP_SELF'].'?propalid='.$propal->id,$societe,$propal->contactid,'none');
 			}
 			print '</td>';
-				    
+			print '</tr>';
-            if ($conf->projet->enabled) $rowspan++;
-    
-            print '<td valign="top" colspan="2" width="50%" rowspan="'.$rowspan.'">'.$langs->trans('Note').' :<br>'. nl2br($propal->note).'</td></tr>';
 				    
 			// Projet
             if ($conf->projet->enabled)
@@ -716,6 +729,7 @@ if ($_GET['propalid'] > 0)
             }
     
             // Remise globale
+/*
             print '<tr><td>';
 			print '<table class="nobordernopadding" width="100%"><tr><td>';
             print $langs->trans('GlobalDiscount');
@@ -737,6 +751,7 @@ if ($_GET['propalid'] > 0)
 				print $propal->remise_percent.'%';
 			}
 			print '</td></tr>';
+*/
 
 			// Amount   
             print '<tr><td height="10">'.$langs->trans('AmountHT').'</td>';

htdocs/html.form.class.php

@@ -498,7 +498,7 @@ class Form
 	                }
 	            }
 	            if ($htmlname != 'none') print '</select>';
-	            // return 1;  // affiche 1 à l'écran !
+	            return 1;
 	        }
 	        else
 	        {