Le test sur saisi des param�tres (compte, montant, type de paiement) et fait avant les ecritures en base.

htdocs/compta/paiement.php

@@ -50,15 +50,13 @@ if ($_POST["action"] == 'add_paiement')
 {
     $error = 0;
 
-    if ($_POST["paiementid"] > 0)
-    {
     $datepaye = $db->idate(mktime(12, 0 , 0,
                         $_POST["remonth"],
                         $_POST["reday"],
                         $_POST["reyear"]));
-
     $paiement_id = 0;
     $total = 0;
+    // Génère tableau des montants amounts
     $amounts = array();
     foreach ($_POST as $key => $value)
     {
@@ -71,9 +69,32 @@ if ($_POST["action"] == 'add_paiement')
         }
     }
 
-        if ($total > 0)
+    // Effectue les vérifications des parametres
+    if ($_POST["paiementid"] <= 0)
     {
+        $fiche_erreur_message = '<div class="error">'.$langs->trans("ErrorFieldRequired",$langs->trans("PaymentMode")).'</div>';
+        $error++;
+    }
 
+    if ($conf->banque->enabled)
+    {
+        // Si module bank actif, un compte est obligatoire lors de la saisie
+        // d'un paiement
+        if (! $_POST["accountid"])
+        {
+            $fiche_erreur_message = '<div class="error">'.$langs->trans("ErrorFieldRequired",$langs->trans("AccountToCredit")).'</div>';
+            $error++;   
+        }
+    }
+
+    if ($total <= 0)
+    {
+        $fiche_erreur_message = '<div class="error">'.$langs->trans("ErrorFieldRequired",$langs->trans("Amount")).'</div>';
+        $error++;
+    }
+
+    if (! $error)
+    {
         $db->begin();
 
         // Creation de la ligne paiement
@@ -102,15 +123,6 @@ if ($_POST["action"] == 'add_paiement')
             }
 
             if ($conf->banque->enabled)
-                {
-                    // Si module bank actif, un compte est obligatoire lors de la saisie
-                    // d'un paiement
-                    if (! $_POST["accountid"])
-                    {
-                        $fiche_erreur_message = '<div class="error">'.$langs->trans("ErrorFieldRequired",$langs->trans("AccountToCredit")).'</div>';
-                        $error++;   
-                    }
-                    else
             {
                 // Insertion dans llx_bank
                 $label = "Règlement facture";
@@ -124,7 +136,6 @@ if ($_POST["action"] == 'add_paiement')
                 '',
                 $user);
 
-    
                 // Mise a jour fk_bank dans llx_paiement.
                 // On connait ainsi le paiement qui a généré l'écriture bancaire
                 if ($bank_line_id > 0)
@@ -156,11 +167,11 @@ if ($_POST["action"] == 'add_paiement')
                     $error++;
                 }
             }
-                }
 
         }
         else
         {
+            $this->error=$paiement->error;
             $error++;
         }
 
@@ -176,15 +187,6 @@ if ($_POST["action"] == 'add_paiement')
             $db->rollback();
         }
     }
-        else
-        {
-            $fiche_erreur_message = '<div class="error">'.$langs->trans("ErrorFieldRequired",$langs->trans("Amount")).'</div>';
-        }
-    }
-    else
-    {
-        $fiche_erreur_message = '<div class="error">'.$langs->trans("ErrorFieldRequired",$langs->trans("PaymentMode")).'</div>';
-    }
 }
 
 // Sécurité accés client