factorisation 1_new_service.php vers functions.php

23 files changed, 217 insertions, 211 deletions

diff --git a/README.txt b/README.txt
index d454b27..172bbb0 100644
--- a/README.txt
+++ b/README.txt
@@ -3,7 +3,7 @@ Ce programme est destiné à un usage privé et hors-ligne,
 il ne nécessite pas de déclaration à la CNIL.
 Il ne comporte aucune fonctionnalité réseau, ni aucune sécurité.
 L'utilisateur est responsable des informations enregistrées,
-lesquelles ne sont destinées sortir de son ordinteur.
+lesquelles ne sont pas destinées sortir de son ordinteur.
 
 
 OS compatible:
diff --git a/data/dev.sqlite b/data/dev.sqlite
index dce37bd..c933839 100644
--- a/data/dev.sqlite
+++ b/data/dev.sqlite
diff --git a/src/Config.php b/src/Config.php
index 3e2b204..b4f8e5b 100644
--- a/src/Config.php
+++ b/src/Config.php
@@ -49,8 +49,8 @@ class Config
                     {

                         if(self::checkPath($field, $value))

                         {

-                            $value = self::slashAtEndOfPath($value);

-                            self::$$field = $value;

+                            self::$$field = self::slashAtEndOfPath($value);

+                            //self::$$field = $value;

                         }

                     }

                     else // tester le reste?

diff --git a/src/Dates.php b/src/Dates.php
index ff6e318..eb57ca8 100644
--- a/src/Dates.php
+++ b/src/Dates.php
@@ -1,5 +1,5 @@
 <?php
-// php/DateTimestamp.php
+// src/DateTimestamp.php
 //
 // attention à strtotime(), php "devine" le format de la date selon qu'elle comporte des slashs / ou des tirets -
 // pour php une date avec des slashs / est américaine, mais si on fait comme ça aussi par ici
diff --git a/src/Latex.php b/src/Latex.php
index c3d56af..9ae9435 100644
--- a/src/Latex.php
+++ b/src/Latex.php
@@ -1,5 +1,5 @@
 <?php
-// php/Latex.php
+// src/Latex.php
 //
 // génération du code LaTeX
 
diff --git a/src/dependances.php b/src/dependances.php
index 66ce414..a3dd320 100644
--- a/src/dependances.php
+++ b/src/dependances.php
@@ -1,5 +1,5 @@
 <?php
-// php/dependances.php
+// src/dependances.php
 // Vérification des dépendances nécessaires au fonctionnement de ce programmes.
 
 // Environnement dans lequel ce programmes a été créé:
diff --git a/src/functions.php b/src/functions.php
index bd91ddf..0b075b5 100644
--- a/src/functions.php
+++ b/src/functions.php
@@ -22,6 +22,72 @@ function windowAppCommand(string $app, string $path = ''): string
     return $command;
 }
 
+function makeObjectCLient()
+{
+    // fenêtres
+    $QuestionNouveauClient = new ZenityQuestion(ZenitySetup::$question_nouveau_client_text);
+    $RechercheClient = new zenityEntry(ZenitySetup::$recherche_client_text);
+    $ResultatsRechercheClient = new ZenityList(ZenitySetup::$resultats_recherche_client_text, []);
+    $NouveauClient = new ZenityForms(ZenitySetup::$nouveau_client_text, ZenitySetup::$nouveau_client_entrees);
+    
+    $Client = new Clients;
+    
+    // est ce que le client est déjà dans la base?
+    $client_inconnu = true;
+    if(exec($QuestionNouveauClient->get()) == '0') // $? = 0 signifie oui, double égal == pour le transtypage
+    {
+        echo "choix: recherche d'une client\n";
+        $input = exec($RechercheClient->get());
+        if($input == '')
+        {
+            echo "debug: recherche annulée ou saisie vide\n";
+            return 0;
+        }
+        
+        echo "debug: recherche effectuée\n";
+        $ResultatsRechercheClient->setListRows(rechercheClient($input, $Client), $Client->getTable()); // recherche silencieuse
+        
+        // sélection parmi les résultats
+        $input = exec($ResultatsRechercheClient->get()); // renvoie l'ID de la table 'clients'
+        $ResultatsRechercheClient->cleanCommand();
+        
+        if($input == '')
+        {
+            echo "debug: client pas trouvé ou pas sélectionné\n";
+            return 0;
+        }
+        
+        echo "debug: client sélectionné\n";
+        $Client->hydrate($Client->findById($input));
+        $client_inconnu = false;
+    }
+    else
+    {
+        echo "choix: nouveau client\n";
+    }
+    
+    // on n'a pas cherché OU on n'a pas trouvé
+    if($client_inconnu)
+    {
+        $input = exec($NouveauClient->get());
+        if($input == '')
+        {
+            echo "debug: annulation lors de l'enregistrement d'un nouveau client\n";
+            return 0;
+        }
+        if(!$Client->hydrateFromForm($input))
+        {
+            // messages d'erreur dans hydrateFromForm()
+            return 0;
+        }
+        unset($input);
+        $Client->create();
+        $Client->setID(); // sans paramètre, exécute un $this->db->lastInsertId()
+    }
+    
+    return $Client;
+}
+
 // NOTE 1: en PHP les objets sont passés aux fonctions par référence par défaut, toutefois ce n'est pas entièrement vrai
 // NOTE 2: PHP n'a pas de pointeur mais des références, une référence est un alias qui ne contient pas l'objet lui-même
 // NOTE 3: la variable créée lors d'un "new" est elle-même une référence contenant un identifiant (= le pointeur?)
@@ -30,78 +96,65 @@ function windowAppCommand(string $app, string $path = ''): string
 function rechercheClient(string $input, Clients $Client): array
 {
     $input_array = explode(' ', $input); // si plusieurs mot, on les recherche tous l'un après l'autre
-    
     $result = $Client->findByKeywords($input_array, 'prenom_nom'); // on obtient un tableau à deux dimensions avec les entrées trouvées
     return($result);
 }
 
-function makeFolder(string $path)
-{
-    if(!file_exists($path))
-    {
-        mkdir($path);
-        chmod($path, 0755); // droits en octal
-    }
-}
-
-function makeLatexAndPdfDocument($documentType, Clients $Client = null, Prestations $Presta = null, $Details = null) // $Details peut être de type DevisFactures ou Locations
+function makeLatexAndPdfDocument(Clients $Client = null, Prestations $Presta = null, $PrestaDetails = null)
 {
     $latex = '';
     $year = '';
     $data = [];
-    $file_name = $documentType . '.tex';
     $latex_path = Config::$latex_path;
     $pdf_path = Config::$pdf_path;
     
-    // tableau associatif avec des données des différents objets
-    // note: la propriété 'commentaires' de $Presta écrase celle de $Client (pas grave, les clients n'ont pas besoin d'en avoir connaissance)
-    // encore une fois, ce programme est destiné à un usage privé et hors ligne, sans déclaration CNIL
+    // verso d'une enveloppe
+    $latex = makeLatex('enveloppe_verso'); // pas de données transmises, elles sont dans la classe Config
+    $file_name = 'enveloppe_verso.tex';
+    makeFiles($latex_path, $pdf_path, $file_name, $latex);
     
-    if($Client == null) // verso d'une enveloppe
-    {
-        $latex = makeLatex($documentType); // pas de données transmises, elles sont dans la classe Config
-        $file_name = 'enveloppe_verso';
-    }
-    else
+    if($Client !== null)
     {
         $data = $Client->getAll();
         
-        if($Presta == null) // recto d'une enveloppe
-        {
-            $latex_path .= 'enveloppes_recto/';
-            $pdf_path .= 'enveloppes_recto/';
-            $data['code_postal'] = implode(' \ ', str_split($data['code_postal'])); // code postal avec 2 espaces entre chaque chiffre: 2 \ 9 \ 0 \ 0 \ 0
-            $latex = makeLatex($documentType, $data); // injection des variables
-            $file_name = $Client->getCodeClient() . '.tex';
-        }
-        else // facture, devis, location
+        // recto d'une enveloppe
+        $latex_recto_path = $latex_path . 'enveloppes_recto/';
+        $pdf_verso_path = $pdf_path . 'enveloppes_recto/';
+        $data['code_postal'] = implode(' \ ', str_split($data['code_postal'])); // code postal avec 2 espaces entre chaque chiffre: 2 \ 9 \ 0 \ 0 \ 0
+        $latex = makeLatex('enveloppe_recto', $data); // injection des variables
+        $file_name = $Client->getCodeClient() . '.tex';
+        makeFiles($latex_recto_path, $pdf_verso_path, $file_name, $latex);
+        
+        // facture, devis, location
+        if($Presta !== null && $PrestaDetails !== null)
         {
-            $data = array_merge($data, $Presta->getAll());
-            $Date = new Dates($Presta->getDate()); // entrée = timestamp (doit être un "int"!!)
-            $year = $Date->getYear();
-            
-            // créer un sous-dossier pour la nouvelle année
-            $latex_path .= $year . '/';
-            $pdf_path .= $year . '/';
-            
-            if($Details != null)
+            $type = $Presta->getTypePresta();
+            $file_name = $type . '.tex';
+            if($type === 'facture' || $type === 'devis' || $type === 'location')
             {
-                $data = array_merge($data, $Details->getAll());
+                $data = array_merge($data, $Presta->getAll());
+                $Date = new Dates($Presta->getDate()); // entrée = timestamp (doit être un "int"!!)
+                $year = $Date->getYear();
+                $latex_year_path = $latex_path . $year . '/'; // un sous-dossier par année
+                $pdf_year_path = $pdf_path . $year . '/';
+                $data = array_merge($data, $PrestaDetails->getAll());
+                $latex = makeLatex($type, $data, $Date); // injection des variables
+                $file_name = $Presta->getCodePresta() . '.tex';
+                makeFiles($latex_year_path, $pdf_year_path, $file_name, $latex);
+            }
+            elseif($type === 'cesu' || $type === 'non_vendue')
+            {} // pas de document
+            else
+            {
+                echo "debug: erreur génération latex, type de prestation \n";
+                return 0;
             }
-            
-            $latex = makeLatex($documentType, $data, $Date); // injection des variables
-            $file_name = $Presta->getCodePresta() . '.tex';
         }
     }
-    
-    // création des fichiers
-    makeFolder($latex_path);
-    makeFolder($pdf_path);
-    file_put_contents($latex_path. $file_name, $latex); // écriture du fichier
-    latexToPdf($latex_path, $file_name, $pdf_path);
 }
 
-function makeLatex(string $documentType, array $data = [], Dates $Date = null)
+
+function makeLatex(string $type, array $data = [], Dates $Date = null)
 {
     $date = '';
     if($Date != null)
@@ -113,13 +166,30 @@ function makeLatex(string $documentType, array $data = [], Dates $Date = null)
     // on obtient la variable $latex avec ob_get_clean()
     // le include() ici, c'est du génie ou c'est moche ou les deux?
     // un nouveau fichier php est inséré à chaque itération
-    include('latex_templates/' . $documentType . '.php');
+    include('latex_templates/' . $type . '.php');
     
     // on retourne le buffer
     // normallement le code PHP inséré avec include est nettoyé en quittant la fonction
     return($latex);
 }
 
+function makeFiles($latex_path, $pdf_path, $file_name, $latex)
+{
+    makeFolder($latex_path);
+    makeFolder($pdf_path);
+    file_put_contents($latex_path. $file_name, $latex); // écriture du fichier
+    latexToPdf($latex_path, $file_name, $pdf_path);
+}
+
+function makeFolder(string $path)
+{
+    if(!file_exists($path))
+    {
+        mkdir($path);
+        chmod($path, 0755); // droits en octal
+    }
+}
+
 // compilation à partir d'un fichier .tex
 function latexToPdf(string $latexPath, string $fileName, string $pdfPath)
 {
diff --git a/src/main.php b/src/main.php
index 2b2da84..13d77c3 100755
--- a/src/main.php
+++ b/src/main.php
@@ -1,13 +1,16 @@
 #!/bin/php
 <?php
-// php/main.php
+// src/main.php
+
+require('functions.php');
+require('model/File.php');
 
 // configuration du programme par l'utilisateur
-//~ require('../config.php'); // fichier de l'utilisateur: à déplacer peut-être dans un dossier config
 require('Config.php'); // classe structure de données, lit config.php et préviens les erreurs (par exemple les / aux chemins manquants)
 Config::readFile('../config.ini');
+makeFolder(Config::$latex_path);
+makeFolder(Config::$pdf_path);
 Config::hydrate();
-
 // et à coder peut-être plus tard
 //require('config_window.php'); // configuration user-friendly et assistée avec des messages d'erreur et des indications
 
@@ -20,9 +23,7 @@ Config::$db_path .= Config::$db_name . '.' . Config::$sgbd;
 $sqlitebrowser_enable = false;
 $sqlite_cli = ''; // commande sqlite ou sqlite3
 $x_term_command = ''; // commande terminal en mode graphique
-
 require('dependances.php'); // vérification des dépendances
-require('functions.php');
 require('Dates.php'); // gère dates et timestamps
 Dates::$date_format = Config::$date_format; // dates européennes ou américaines
 
@@ -43,10 +44,13 @@ require('model/Locations.php');
 
 require('view/Zenity.php'); // commande système zenity
 require('view/ZenitySetup.php'); // texte dans les fenêtres ET instanciation (un objet = une commande)
-require('Latex.php'); // générer le code LaTeX
+
+// factoriser et créer une classe
+//require('Latex.php'); // générer le code LaTeX
 
 require('sections/1_new_service.php');
-require('sections/2_modify_service.php');
+require('sections/2_quotations.php');
+require('sections/3_modify_service.php');
 
 // boucle principale
 $boucle = true;
@@ -73,22 +77,28 @@ while($boucle)
                 echo "debug: main.php: hello, code retour " . $code_retour . "\n";
         }
     
-    // -- SECTION 2: Modifier un enregistrement --
+    // -- SECTION 2: Devis --
     elseif($choix_niv1 === ZenitySetup::$menu_principal_entrees[1] || $code_retour === 2)
         {
                 echo("choix: $choix_niv1\n");
                 $code_retour = modifyService();
         }
+    // -- SECTION 3: Modifier un enregistrement --
+    elseif($choix_niv1 === ZenitySetup::$menu_principal_entrees[2] || $code_retour === 3)
+        {
+                echo("choix: $choix_niv1\n");
+                $code_retour = modifyService();
+        }
     
-    // -- SECTION 3: Consulter/analyser les données --
-        elseif($choix_niv1 === ZenitySetup::$menu_principal_entrees[2] || $code_retour === 3)
+    // -- SECTION 4: Consulter/analyser les données --
+        elseif($choix_niv1 === ZenitySetup::$menu_principal_entrees[3] || $code_retour === 4)
         {
                 echo("choix: $choix_niv1\n");
                 // quel affichage? des tableaux avec zenity? LaTeX? une page web? un autre outil servant à faire des tableaux et graphiques
         }
         
-        // -- SECTION 4: Imprimer --
-        elseif($choix_niv1 === ZenitySetup::$menu_principal_entrees[3] || $code_retour === 4) // = Imprimer un document
+        // -- SECTION 5: Imprimer --
+        elseif($choix_niv1 === ZenitySetup::$menu_principal_entrees[4] || $code_retour === 5) // = Imprimer un document
         {
                 echo("choix: $choix_niv1\n");
                 $MenuDocuments = new ZenityList(ZenitySetup::$menu_documents_text, ZenitySetup::$menu_documents_entrees);
@@ -123,8 +133,8 @@ while($boucle)
                 }
         }
 
-        // -- SECTION 5: Supports de communication --
-        elseif($choix_niv1 === ZenitySetup::$menu_principal_entrees[4] || $code_retour === 5) // = Communication
+        // -- SECTION 6: Supports de communication --
+        elseif($choix_niv1 === ZenitySetup::$menu_principal_entrees[5] || $code_retour === 6) // = Communication
         {
                 echo("choix: $choix_niv1\n");
                 $MenuCommunication = new ZenityList(ZenitySetup::$menu_communication_text, ZenitySetup::$menu_communication_entrees);
@@ -147,8 +157,8 @@ while($boucle)
                 }
         }
 
-        // -- SECTION 6: BDD --
-        elseif($choix_niv1 === ZenitySetup::$menu_principal_entrees[5] || $code_retour === 6) // = Base de données
+        // -- SECTION 7: BDD --
+        elseif($choix_niv1 === ZenitySetup::$menu_principal_entrees[6] || $code_retour === 7) // = Base de données
         {
                 echo("choix: $choix_niv1\n");
                 if($sqlitebrowser_enable)
diff --git a/src/model/CESU.php b/src/model/CESU.php
index 8771607..dbb4023 100644
--- a/src/model/CESU.php
+++ b/src/model/CESU.php
@@ -1,5 +1,5 @@
 <?php
-// model/CESU.php
+// src/model/CESU.php
 
 class CESU extends Model
 {
diff --git a/src/model/Clients.php b/src/model/Clients.php
index d841451..8f460c6 100644
--- a/src/model/Clients.php
+++ b/src/model/Clients.php
@@ -1,5 +1,5 @@
 <?php
-// model/Clients.php
+// src/model/Clients.php
 
 class Clients extends Model
 {
diff --git a/src/model/DB.php b/src/model/DB.php
index 52d6479..4e54015 100644
--- a/src/model/DB.php
+++ b/src/model/DB.php
@@ -1,5 +1,5 @@
 <?php
-// php/DB.php
+// src/model/DB.php
 
 // cette classe applique le pattern "singleton"
 // but: ne permettre qu'une seule instance de la classe (laquelle sera éventuellement globale)
diff --git a/src/model/DevisFactures.php b/src/model/DevisFactures.php
index 92a5443..c9b7d18 100644
--- a/src/model/DevisFactures.php
+++ b/src/model/DevisFactures.php
@@ -1,5 +1,5 @@
 <?php
-// model/DevisFactures.php
+// src/model/DevisFactures.php
 
 class DevisFactures extends Model
 {
diff --git a/src/model/File.php b/src/model/File.php
new file mode 100644
index 0000000..8bcdead
--- /dev/null
+++ b/src/model/File.php
@@ -0,0 +1,7 @@
+<?php
+// src/model/File.php
+//
+// manipulations des fichiers
+
+class File
+{}
diff --git a/src/model/Locations.php b/src/model/Locations.php
index 149c1f1..b669e7c 100644
--- a/src/model/Locations.php
+++ b/src/model/Locations.php
@@ -1,5 +1,5 @@
 <?php
-// model/Locations.php
+// src/model/Locations.php
 
 class Locations extends Model
 {
diff --git a/src/model/Model.php b/src/model/Model.php
index 0240a9e..40a9fcb 100644
--- a/src/model/Model.php
+++ b/src/model/Model.php
@@ -1,5 +1,5 @@
 <?php
-// php/Model.php
+// src/model/Model.php
 
 abstract class Model extends DB
 {
diff --git a/src/model/Prestations.php b/src/model/Prestations.php
index d246a3d..94dd805 100644
--- a/src/model/Prestations.php
+++ b/src/model/Prestations.php
@@ -1,5 +1,5 @@
 <?php
-// model/Prestations.php
+// src/model/Prestations.php
 
 class Prestations extends Model
 {
diff --git a/src/model/StructTablesDB.php b/src/model/StructTablesDB.php
index b1e3c7a..75a4383 100644
--- a/src/model/StructTablesDB.php
+++ b/src/model/StructTablesDB.php
@@ -1,5 +1,5 @@
 <?php
-// model/StructTablesDB.php
+// src/model/StructTablesDB.php
 
 // structure de données façon C
 // créer les requêtes avec implode(', ', $DbStructure)
diff --git a/src/model/traits.php b/src/model/traits.php
index a00032e..03121be 100644
--- a/src/model/traits.php
+++ b/src/model/traits.php
@@ -1,5 +1,5 @@
 <?php
-// model/traits.php
+// src/model/traits.php
 //
 // fonctions à utiliser par les enfants de Model
 
diff --git a/src/sections/1_new_service.php b/src/sections/1_new_service.php
index ae2d2ef..f9105b7 100644
--- a/src/sections/1_new_service.php
+++ b/src/sections/1_new_service.php
@@ -1,15 +1,11 @@
 <?php
-// sections/1_new_service.php
+// src/sections/1_new_service.php
 //
 // -- SECTION 1: Nouvelle prestation --
 
 function newService(): int // code de retour, si 0 retour menu principal, si 2 aller à la section 2, etc
 {
     // fenêtres
-    $QuestionNouveauClient = new ZenityQuestion(ZenitySetup::$question_nouveau_client_text);
-    $RechercheClient = new zenityEntry(ZenitySetup::$recherche_client_text);
-    $ResultatsRechercheClient = new ZenityList(ZenitySetup::$resultats_recherche_client_text, []);
-    $NouveauClient = new ZenityForms(ZenitySetup::$nouveau_client_text, ZenitySetup::$nouveau_client_entrees);
     $MenuEnregistrement = new ZenityList(ZenitySetup::$menu_enregistrement_text, ZenitySetup::$menu_enregistrement_entrees);
     $Calendrier = new ZenityCalendar(ZenitySetup::$calendar_text);
     $FormulaireDevis = new ZenityForms(ZenitySetup::$formulaire_text, ZenitySetup::$formulaire_devis_entrees);
@@ -20,63 +16,12 @@ function newService(): int // code de retour, si 0 retour menu principal, si 2 a
     $Recapitulatif = new ZenityList(ZenitySetup::$recapitulatif_text, ZenitySetup::$recapitulatif_entrees); // tableau à multiples colonnes
     $QuestionModifierPrestation = new ZenityQuestion(ZenitySetup::$question_modification_text);
     
-    $Client = new Clients;
-    
     
     // -- partie 1: le client --
-    
-    // est ce que le client est déjà dans la base?
-    $client_inconnu = true;
-    if(exec($QuestionNouveauClient->get()) == '0') // $? = 0 signifie oui, double égal == pour le transtypage
+    $Client = makeObjectCLient();
+    if($Client === 0)
     {
-        echo "choix: recherche d'une client\n";
-        $input = exec($RechercheClient->get());
-        if($input == '')
-        {
-            echo "debug: recherche annulée ou saisie vide\n";
-            return(0);
-        }
-        
-        echo "debug: recherche effectuée\n";
-        $ResultatsRechercheClient->setListRows(rechercheClient($input, $Client), $Client->getTable()); // recherche silencieuse
-        unset($input);
-        
-        // niveau 4: sélection parmi les résultats
-        $choix_niv4 = exec($ResultatsRechercheClient->get()); // renvoie l'ID de la table 'clients'
-        $ResultatsRechercheClient->cleanCommand();
-        
-        if($choix_niv4 == '')
-        {
-            echo "debug: client pas trouvé ou pas sélectionné\n";
-            return(0);
-        }
-        
-        echo "debug: client sélectionné\n";
-        $Client->hydrate($Client->findById($choix_niv4));
-        $client_inconnu = false;
-    }
-    else
-    {
-        echo "choix: nouveau client\n";
-    }
-    
-    // on n'a pas cherché OU on n'a pas trouvé
-    if($client_inconnu)
-    {
-        $input = exec($NouveauClient->get());
-        if($input == '')
-        {
-            echo "debug: annulation lors de l'enregistrement d'un nouveau client\n";
-            return(0);
-        }
-        if(!$Client->hydrateFromForm($input))
-        {
-            // messages d'erreur dans hydrateFromForm()
-            return(0);
-        }
-        unset($input);
-        $Client->create();
-        $Client->setID(); // sans paramètre, exécute un $this->db->lastInsertId()
+        return 0; // menu principal
     }
     
     
@@ -87,7 +32,7 @@ function newService(): int // code de retour, si 0 retour menu principal, si 2 a
     if($choix_niv2 === '')
     {
         echo "debug: annulation menu enregistrement\n";
-        return(0); // menu principal
+        return 0; // menu principal
     }
     
     $Presta = new Prestations($Client->getID());
@@ -98,7 +43,7 @@ function newService(): int // code de retour, si 0 retour menu principal, si 2 a
     if($Date->getDate() == '') // on n'a pas cliqué sur "annuler"
     {
         echo "debug: annulation à la saisie d'une date\n";
-        return(0); // menu principal
+        return 0; // menu principal
     }
     
     $Presta->setDate($Date->getTimestamp()); // un entier pour la BDD
@@ -106,47 +51,47 @@ function newService(): int // code de retour, si 0 retour menu principal, si 2 a
     switch($choix_niv2)
     {
         // formulaire - étape 2/3
-        case ZenitySetup::$menu_enregistrement_entrees[0]: // "Devis"
-            $Details = new DevisFactures('devis');
-            $Presta->setTypePresta('devis');
-            $input = exec($FormulaireDevis->get());
-            break;
-        case ZenitySetup::$menu_enregistrement_entrees[1]: // "Facture"
-            $Details = new DevisFactures('factures'); // 'factures' est le nom de la table, pas le type de presta
+        //~ case ZenitySetup::$menu_enregistrement_entrees[0]: // "Devis"
+            //~ $PrestaDetails = new DevisFactures('devis');
+            //~ $Presta->setTypePresta('devis');
+            //~ $input = exec($FormulaireDevis->get());
+            //~ break;
+        case ZenitySetup::$menu_enregistrement_entrees[0]: // "Facture"
+            $PrestaDetails = new DevisFactures('factures'); // 'factures' est le nom de la table, pas le type de presta
             $Presta->setTypePresta('facture');
             $input =exec($FormulaireFacture->get());
             break;
-        case ZenitySetup::$menu_enregistrement_entrees[2]: // "CESU"
-            $Details = new CESU();
+        case ZenitySetup::$menu_enregistrement_entrees[1]: // "CESU"
+            $PrestaDetails = new CESU();
             $Presta->setTypePresta('cesu');
             $input = exec($FormulaireCesu->get());
             break;
-        case ZenitySetup::$menu_enregistrement_entrees[3]: // "Location"
-            $Details = new Locations();
+        case ZenitySetup::$menu_enregistrement_entrees[2]: // "Location"
+            $PrestaDetails = new Locations();
             $Presta->setTypePresta('location');
             $input = exec($FormulaireLocation->get());
             break;
-        case ZenitySetup::$menu_enregistrement_entrees[4]: // "Prestation non vendue"
+        case ZenitySetup::$menu_enregistrement_entrees[3]: // "Prestation non vendue"
             $Presta->setTypePresta('non_vendue');
             break;
         default: // inutile normallement, cas déjà géré avant
             echo "debug: sortie du menu enregistrement incorrect\n";
-            return(0);
+            return 0;
     }
     
     $Presta->makeCodePresta($Date, $Client->getCodeClient()); // d'un objet à l'autre
     
-    if($choix_niv2 != ZenitySetup::$menu_enregistrement_entrees[4]) // si presta non vendue, saut étape 3/3
+    if($choix_niv2 != ZenitySetup::$menu_enregistrement_entrees[3]) // si presta non vendue, saut étape 3/3
     {
         if($input == '') // annulation
         {
             echo "debug: annulation lors de l'enregistrement des détails de la prestation\n";
-            return(0);
+            return 0;
         }
-        elseif(isset($Details) && !$Details->hydrateFromForm($input, $Presta)) // echec de l'hydratation
+        elseif(isset($PrestaDetails) && !$PrestaDetails->hydrateFromForm($input, $Presta)) // echec de l'hydratation
         {
             echo "debug: erreur de hydrateFromForm()\n"; // messages d'erreur à mettre ici ou dans hydrateFromForm()?
-            return(0);
+            return 0;
         }
         unset($input);
     }
@@ -167,7 +112,7 @@ function newService(): int // code de retour, si 0 retour menu principal, si 2 a
     //~ if(exec('echo $?') != 1)
     //~ {
         //~ echo "debug: annulation à la saisie d'un commentaire\n";
-        //~ return(0);
+        //~ return 0;
     //~ }
     $Presta->setCommentaires($input);
     unset($input);
@@ -175,53 +120,17 @@ function newService(): int // code de retour, si 0 retour menu principal, si 2 a
     $Presta->create();
     $Presta->setID(); // sans paramètre, exécute un $this->db->lastInsertId()
     
-    if(isset($Details)) // presta non vendue
+    if(isset($PrestaDetails)) // presta non vendue
     {
-        $Details->setIDPresta($Presta->getID()); // d'un objet à l'autre
-        $Details->create();
-        $Details->setID(); // sans paramètre, exécute un $this->db->lastInsertId()
+        $PrestaDetails->setIDPresta($Presta->getID()); // d'un objet à l'autre
+        $PrestaDetails->create();
+        $PrestaDetails->setID(); // sans paramètre, exécute un $this->db->lastInsertId()
     }
     
     
     // -- partie 3: LaTeX --
     
-    // document de la prestation
-    switch($choix_niv2)
-    {
-        case ZenitySetup::$menu_enregistrement_entrees[0]: // "Devis"
-            $type = 'devis';
-            makeLatexAndPdfDocument($type, $Client, $Presta, $Details);
-            break;
-        case ZenitySetup::$menu_enregistrement_entrees[1]: // "Facture"
-            $type = 'facture';
-            makeLatexAndPdfDocument($type, $Client, $Presta, $Details);
-            break;
-        case ZenitySetup::$menu_enregistrement_entrees[2]: // "CESU"
-            // pas de facture
-            break;
-        case ZenitySetup::$menu_enregistrement_entrees[3]: // "Location"
-            $type = 'location';
-            makeLatexAndPdfDocument($type, $Client, $Presta, $Details);
-            break;
-        case ZenitySetup::$menu_enregistrement_entrees[4]: // "Prestation non vendue"
-            // pas de facture
-            break;
-        default: // inutile normallement, cas déjà géré avant
-            echo "debug: sortie du menu enregistrement incorrect\n";
-            return(0);
-    }
-    
-    // enveloppe recto
-    $type = 'enveloppe_recto';
-    makeLatexAndPdfDocument($type, $Client);
-    
-    // le verso ne dépend pas du client mais de l'entreprise, dans la classe Config
-    if(!file_exists(Config::$pdf_path . 'enveloppe_verso.pdf'))
-    {
-        $type = 'enveloppe_verso';
-        makeLatexAndPdfDocument($type);
-    }
-    
+    makeLatexAndPdfDocument($Client, $Presta, $PrestaDetails);
     
     // -- partie 4: récapitulatif --
     
@@ -237,10 +146,10 @@ function newService(): int // code de retour, si 0 retour menu principal, si 2 a
     // zenityQuestion
     if(exec($QuestionModifierPrestation->get()) == '0')
     {
-        return(2); // section 2
+        return 3; // section "Modifier un enregistrement"
     }
     else
     {
-        return(0); // menu principal
+        return 0; // menu principal
     }
 }
diff --git a/src/sections/2_quotations.php b/src/sections/2_quotations.php
new file mode 100644
index 0000000..76b66fd
--- /dev/null
+++ b/src/sections/2_quotations.php
@@ -0,0 +1,9 @@
+<?php

+// src/sections/2_quotations.php

+//

+// -- SECTION 2: Devis --

+

+function quotation(): int // code de retour, si 2 aller à la section 2, etc

+{

+    return(0); // menu principal

+}

diff --git a/src/sections/2_modify_service.php b/src/sections/3_modify_service.php
index 72e2605..3147969 100644
--- a/src/sections/2_modify_service.php
+++ b/src/sections/3_modify_service.php
@@ -1,7 +1,7 @@
 <?php
-// sections/2_modify_service.php
+// src/sections/3_modify_service.php
 //
-// -- SECTION 2: Modifier une prestation --
+// -- SECTION 3: Modifier une prestation --
 
 function modifyService(): int // code de retour, si 2 aller à la section 2, etc
 {
diff --git a/src/view/Zenity.php b/src/view/Zenity.php
index 091a2c0..73c8c3e 100644
--- a/src/view/Zenity.php
+++ b/src/view/Zenity.php
@@ -1,5 +1,5 @@
 <?php
-// php/Zenity.php
+// src/view/Zenity.php
 //
 // générer les commandes qui ouvrent les fenêtres zenity, un objet = une commande
 // s'utilisent comme ceci: exec($Objet->get());
diff --git a/src/view/ZenitySetup.php b/src/view/ZenitySetup.php
index cd7e364..b6d83f6 100644
--- a/src/view/ZenitySetup.php
+++ b/src/view/ZenitySetup.php
@@ -1,5 +1,5 @@
 <?php
-// view/ZenitySetup.php
+// src/view/ZenitySetup.php
 //
 // ces variables sont des paramètres de la commande zenity (--text, --add-entry, --column)
 // les utiliser à l'instanciation des objets Zenity à la manière de l'exemple suivant:
@@ -8,11 +8,12 @@
 class ZenitySetup
 {
     static public $menu_principal_text = "Gestion d'une micro-entreprise";
-    static public $menu_principal_entrees = ["Nouvelle prestation", "Modifier enregistrement", "Consulter/analyser les données", "Imprimer un document", "Communication", "Base de données"];
+    static public $menu_principal_entrees = ["Nouvelle prestation", "Devis", "Modifier un enregistrement", "Consulter/analyser les données", "Imprimer un document", "Communication", "Base de données"];
     static public $question_nouveau_client_text = "Ce client figure t\'il déjà dans le fichier clients?";
     static public $menu_enregistrement_text = "Type d\'enregistrement?";
-    static public $menu_enregistrement_entrees = ["Devis", "Facture", "CESU", "Location", "Prestation non vendue"];
-    static public $menu_documents_text = "Création de documents LaTeX";
+    //static public $menu_enregistrement_entrees = ["Devis", "Facture", "CESU", "Location", "Prestation non vendue"];
+    static public $menu_enregistrement_entrees = ["Facture", "CESU", "Location", "Prestation non vendue"];
+    static public $menu_documents_text = "Imprimer un document LaTeX";
     static public $menu_documents_entrees = ["Devis", "Facture", "Lettre avec adresse", "Livre des recettes", "Registre des achats", "Bilan annuel"];
     static public $menu_communication_text = "Imprimer un support de communication";
     static public $menu_communication_entrees = ["Flyer (nécessite gimp)", "Carte de visite (nécessite scribus)", "Explorateur de fichiers"];