5 files changed, 58 insertions, 52 deletions

diff --git a/src/latex_templates/devis.php b/src/latex_templates/devis.php
index b686d22..34c0dd3 100644
--- a/src/latex_templates/devis.php
+++ b/src/latex_templates/devis.php
@@ -22,6 +22,10 @@ if($data['total_HT'] != 0)
         $data['total_HT'] = ((string) $data['total_HT']) . '€';
 }
 
+if($data['prix_devis'] === 0){ // un float est fait juste avant
+        $data['prix_devis'] = "gratuit";
+}
+
 ob_start();
 ?>
 \documentclass{report}
@@ -90,7 +94,7 @@ ob_start();
         
         Déplacement : \textcolor{vert}{<?= $data['deplacement'] ?>}
         
-        Devis Payant : \textcolor{vert}{<?= $data['prix_devis'] ?>}
+        Prix Devis : \textcolor{vert}{<?= $data['prix_devis'] ?>}
         
         Total HT : \textcolor{vert}{<?= $data['total_HT'] ?>}
         
diff --git a/src/model/entities/Client.php b/src/model/entities/Client.php
index 97951be..a0bc405 100644
--- a/src/model/entities/Client.php
+++ b/src/model/entities/Client.php
@@ -173,31 +173,27 @@ class Client
         return str_replace($search, '', $data);
     }
 
-    // une partie du code vient de Model::hydrateFromForm()
-    // à mettre dans une classe ClientManager?
-    public function enterCustomer(ZenityForms $Form): bool
+    public static function controlForm(string $input): string|false
     {
-        $input = exec($Form->get());
-        if($input == ''){
-            echo "debug: annulation lors de l'enregistrement d'un nouveau client\n";
-            return false;
-        }
-
         $data_array = explode('|', $input); // array
-        if($data_array[0] == ''){
-            echo "debug: données insuffisantes, le nom du client doit au minimum être renseigné\n";
+
+        if($data_array[0] == '' || $data_array[1] == ''){
+            echo "debug: données insuffisantes, renseignez au moins un nom et un code client\n";
             return false;
         }
-        else{
-            $this->setAll($data_array);
-        }
 
+        return $input;
+    }
+
+    // une partie du code vient de Model::hydrateFromForm()
+    // à mettre dans une classe ClientManager?
+    public function enterCustomer(string $input): void
+    {
+        $this->setAll(explode('|', $input));
         self::$entityManager->persist($this);
         self::$entityManager->flush();
         
         //$this->id = $this->db->lastInsertId(); // synchro automatique avec doctrine
-        
-        return true;
     }
 
     public static function find(ZenityEntry $SearchCustomerForm, ZenityList $SearchCustomerResults): ?Client
diff --git a/src/model/entities/Devis.php b/src/model/entities/Devis.php
index 8518f8e..ce2a043 100644
--- a/src/model/entities/Devis.php
+++ b/src/model/entities/Devis.php
@@ -43,10 +43,8 @@ class Devis
 

     public static EntityManager $entityManager;

 

-    public function __construct(Prestation $presta = null)

-    {

-        if($presta != null)

-        {

+    public function __construct(?Prestation $presta = null){

+        if($presta != null){

             $this->setPresta($presta);

         }

     }

@@ -63,7 +61,7 @@ class Devis
     }

     

     public function getAllWithWindowFields(): array

-    {
+    {

         //~ $taches = ["Tâches:" => $this->taches];

         //~ $champs_communs = [

             //~ "Total Main d'oeuvre:" => $this->total_main_d_oeuvre,

@@ -86,7 +84,7 @@ class Devis
             "Total HT:" => $this->total_HT,

             "Delai de livraison:" => $this->delai_livraison,

             "Durée de validité:" => $this->validite_devis,

-            "Devis signé:" => $this->signature_devis];
+            "Devis signé:" => $this->signature_devis];

     }

 

     // setters

@@ -165,7 +163,7 @@ class Devis
         $this->total_HT = (float) $value;

         return($this);

     }

-    public function setPrixDevis($value)

+    public function setPrixDevis(string $value): Devis

     {

         $value = str_replace(',', '.', $value);

         $this->prix_devis = (float) $value;

@@ -219,8 +217,8 @@ class Devis
         }

         $data_array = explode('|', $answers); // array

 

-        $check = false;

-        if(count($data_array) === 9) // facture normale

+        //$check = false;

+        if(count($data_array) === 9) // facture normale, "=== 9" = merdique

         {

             $this->getPresta()->setModePaiement(''); // sécurité, doctrine plante si "null"

             $this->setAll($data_array);

diff --git a/src/sections/1_customer.php b/src/sections/1_customer.php
index d282af2..d0d49ff 100644
--- a/src/sections/1_customer.php
+++ b/src/sections/1_customer.php
@@ -3,8 +3,6 @@
 //
 // -- SECTION 1: Fichier clients et propects --
 
-use Doctrine\ORM\EntityManager;
-
 function newCustomer(): array
 {
     // fenêtres
@@ -13,29 +11,38 @@ function newCustomer(): array
     $FinSection1 = new ZenityList(ZenitySetup::$fin_section_1['text'], ZenitySetup::$fin_section_1['entrees']);
 
     $Customer = new Client;
+
     
     // -- partie 2: saisie des infos --
-    if($Customer->enterCustomer($NouveauClient))
-    {
-        // -- partie 3: on fait quoi maintenant --
-        $choix_niv3 = exec($FinSection1->get());
-        if($choix_niv3 === ZenitySetup::$fin_section_1['entrees'][0])
-        {
-            return [2, $Customer]; // section 2: newService()
-        }
-        elseif($choix_niv3 === ZenitySetup::$fin_section_1['entrees'][1])
-        {
-            return [3, $Customer]; // section 3: modifyData()
-        }
-        elseif($choix_niv3 === ZenitySetup::$fin_section_1['entrees'][2])
-        {
-            return [1, null]; // relancer section 1: newCustomer()
-        }
-        else
-        {
-            return [0, null]; // menu principal
-        }
+    $input = exec($NouveauClient->get());
+
+    // annulation
+    if($input == ''){
+        echo "debug: annulation lors de l'enregistrement d'un nouveau client\n";
+        return [0, null]; // menu principal
+    }
+
+    $input = Client::controlForm($input);
+
+    // vide
+    if(!$input){
+        return [1, null]; // relancer section 1: newCustomer()
+    }
+    $Customer->enterCustomer($input);
+
+
+    // -- partie 3: on fait quoi maintenant --
+    $choix_niv3 = exec($FinSection1->get());
+    if($choix_niv3 === ZenitySetup::$fin_section_1['entrees'][0]){
+        return [2, $Customer]; // section 2: newService()
+    }
+    elseif($choix_niv3 === ZenitySetup::$fin_section_1['entrees'][1]){
+        return [3, $Customer]; // section 3: modifyData()
+    }
+    elseif($choix_niv3 === ZenitySetup::$fin_section_1['entrees'][2]){
+        return [1, null]; // relancer section 1: newCustomer()
+    }
+    else{
+        return [0, null]; // menu principal
     }
-    
-    return [0, null]; // menu principal
 }
diff --git a/à faire.txt b/à faire.txt
index 013e1e8..3009ed9 100644
--- a/à faire.txt
+++ b/à faire.txt
@@ -1,9 +1,10 @@
 À FAIRE:
 
-empêcher création d'un client sans code client
-
 champ booléen "payé" dans la table prestations
 
+les prix non-entiers écrits avec une virgule perdent leurs décimales avec (float)
+ça se passe dans les entités aux les lignes comme $this->total_HT = (float)$input[6];
+
 filtre saisie caractères spéciaux: #, saut de ligne (inutile en latex)
 
 une fenêtre au lancement pour prévenir si la BDD ou le mapping ne sont pas créés