7 files changed, 91 insertions, 23 deletions

diff --git a/src/controller/MaintenanceController.php b/src/controller/MaintenanceController.php
index fca45f1..c60ca1c 100644
--- a/src/controller/MaintenanceController.php
+++ b/src/controller/MaintenanceController.php
@@ -52,7 +52,7 @@ class MaintenanceController
         static public function getLastDump(EntityManager $entityManager): void
         {
                 try{
-                        $file_path = Backup::mySQLdump($entityManager);
+                        $file_path = Backup::getLastBackupName();
                         header('Content-Type: application/octet-stream'); // signifie fichier quelconque, du binaire quoi!
                         header('Content-Disposition: attachment; filename="' . basename($file_path) . '"'); // pour provoquer un téléchargement et non pour afficher
                         header('Content-Length: ' . filesize($file_path)); // peut servir côté client (barre de progression...)
diff --git a/src/controller/ViewController.php b/src/controller/ViewDirector.php
index cf3477c..6883b73 100644
--- a/src/controller/ViewController.php
+++ b/src/controller/ViewDirector.php
@@ -1,5 +1,5 @@
 <?php
-// src/view/ViewController.php
+// src/view/ViewDirector.php
 //
 // génère le HTML avec des Builder
 
@@ -10,7 +10,7 @@ use Doctrine\ORM\EntityManager;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
 
-class ViewController extends AbstractBuilder // ViewController est aussi le premier Builder
+class ViewDirector extends AbstractBuilder // ViewDirector est aussi le premier Builder
 {
     static public Node $root_node;
 
@@ -61,6 +61,9 @@ class ViewController extends AbstractBuilder // ViewController est aussi le prem
         if(CURRENT_PAGE === 'article' && !IS_ADMIN && self::$root_node->getNodeByName('main')->getAdoptedChild() === null){
             return new Response($this->html, 302);
         }
+        elseif(CURRENT_PAGE === 'maintenance'){
+            Backup::mySQLdump($entityManager); // créer un nouveau backup
+        }
 
 
         /* 4/ construction de la page avec builders et vues */
diff --git a/src/service/Backup.php b/src/service/Backup.php
index d628c27..313d1f6 100644
--- a/src/service/Backup.php
+++ b/src/service/Backup.php
@@ -37,7 +37,12 @@ class Backup
                 ]);
 
                 try{
+                        // unlink et chmod permettent que le serveur et l'utilisateur appelant bin/mysqldump.php réussissent
+                        if(file_exists($file_path)){
+                                unlink($file_path);
+                        }
                         $command->mustRun(); // comme run() mais lance une ProcessFailedException
+                        chmod($file_path, 0666);
                         return $file_path;
                 }
                 finally{
@@ -59,6 +64,23 @@ class Backup
                 }*/
         }
 
+        static public function getBackupList(): array
+        {
+                $backup_array = [];
+                foreach(scandir(Backup::$backup_dir) as $file){
+                    if($file[0] === '.'){
+                        continue;
+                    }
+                    $backup_array[] = $file;
+                }
+                return $backup_array;
+        }
+        static public function getLastBackupName(): string
+        {
+                $backup_list = self::getBackupList();
+                return $backup_list[count($backup_list) - 1];
+        }
+
         static public function cleanBackups(): void {
                 $files = glob(self::$backup_dir . '/*.sql');
                 usort($files, fn($a, $b) => filemtime($b) <=> filemtime($a)); // filemtime = date de dernière modification
diff --git a/src/service/Installation.php b/src/service/Installation.php
index eb4b6db..995ed4a 100644
--- a/src/service/Installation.php
+++ b/src/service/Installation.php
@@ -12,10 +12,14 @@ class Installation
                 // ajouter plus tard zlib pour la compression des backups
                 foreach($extensions as $extension){
                 if(!extension_loaded($extension)){
-                    echo("<p>l'extension <b>" . $extension . '</b> est manquante</p>');
+                    echo("<p>l'extension <b>" . $extension . "</b> est manquante.</p>");
                     $flag = true;
                 }
             }
+            if(!class_exists(DOMDocument::class)){ // théoriquement plus fiable que extension_loaded()
+                echo("<p>l'extension <b>dom</b> est manquante.</p>");
+                $flag = true;
+            }
 
             /*if(!extension_loaded('imagick') && !extension_loaded('gd')){
                 echo("<p>il manque une de ces extensions au choix pour le traitement des images: <b>imagick</b> (de préférence) ou <b>gd</b>.</p>");
@@ -33,27 +37,40 @@ class Installation
         static public function checkFilesAndFoldersRights(): void
         {
                 // -- droits des fichiers et dossiers --
-            $droits_dossiers = 0755;
-            $droits_fichiers = 0644;
+                $droits_dossiers = 0777;
 
+                $flag = false;
             if(!file_exists('user_data')){
-                // créer le dossier user_data
-                mkdir('user_data/');
-                chmod('user_data/', $droits_dossiers);
-                echo '<p style="color: red;">Le dossier public/user_data introuvable et le serveur n\'a pas la permission de le créer.<br>
-                Pour faire ça bien:<br>sudo -u "serveur web" mkdir /chemin/du/site/public/user_data</p>
-                <p>Aide: "serveur web" se nomme "www-data" sur debian et ubuntu, il s\'appelera "http" sur d\'autres distributions.</p>';
-                die;
+                try{
+                        mkdir('user_data/');
+                        chmod('user_data/', $droits_dossiers);
+                    }
+                    catch(Exception $e){
+                        echo '<p style="color: red;">Le dossier public/user_data introuvable et le serveur n\'a pas la permission de le créer.<br>
+                        Pour faire ça bien:<br>sudo -u "serveur web" mkdir /chemin/du/site/public/user_data</p>';
+                        echo $e;
+                        $flag = true;
+                    }
             }
             if(!file_exists('../var')){
-                mkdir('../var');
-                chmod('../var', $droits_dossiers);
-                //
+                try{
+                        mkdir('../var');
+                        chmod('../var', $droits_dossiers);
+                    }
+                    catch(Exception $e){
+                        echo $e;
+                        $flag = true;
+                    }
             }
             if(!file_exists('../var/backups')){
-                mkdir('../var/backups');
-                chmod('../var/backups', $droits_dossiers);
-                //
+                try{
+                        mkdir('../var/backups');
+                        chmod('../var/backups', $droits_dossiers); // autoriser à la fois le serveur et les scripts dans bin/
+                    }
+                    catch(Exception $e){
+                        echo $e;
+                        $flag = true;
+                    }
             }
 
             // droits 600 pour celui-ci
@@ -63,7 +80,7 @@ class Installation
                 echo '<p>Il doit obligatoirement contenir les codes de la base de données, le protocole http ou https (et éventuellement le port) utilisé pour créer les liens internes.<br>
                         Un modèle est disponible, il s\'agit du fichier config/config-template.ini</p>
                         <p>Ce fichier a une importance critique. Si vous le pouvez faites en sorte que le serveur en soit le propriétaire et donner lui des droits 600.</p>';
-                die;
+                $flag = true;
             }
             /*else{
                 // propriétaire du fichier
@@ -78,6 +95,9 @@ class Installation
                                 }
                 }
             }*/
+            if($flag){
+                die;
+            }
 
             // tester les liens internes
         //
diff --git a/src/service/router.php b/src/service/router.php
index 6973656..98f8fd6 100644
--- a/src/service/router.php
+++ b/src/service/router.php
@@ -52,7 +52,7 @@ if($request->getMethod() === 'GET'){
     }
 
     // construction d'une page
-    $response = (new ViewController)->buildView($entityManager, $request); // utilise Model
+    $response = (new ViewDirector)->buildView($entityManager, $request); // utilise Model
     // parenthèses nécéssaires autour de l'instanciation pour PHP < 8.4
 }
 
diff --git a/src/view/MaintenanceBuilder.php b/src/view/MaintenanceBuilder.php
index f5c60ed..d9c52b1 100644
--- a/src/view/MaintenanceBuilder.php
+++ b/src/view/MaintenanceBuilder.php
@@ -10,6 +10,13 @@ class MaintenanceBuilder extends AbstractBuilder
         public function __construct(Node $node){
                 $viewFile = self::VIEWS_PATH . $node->getName() . '.php';
 
+                // noter qu'un backup vient d'être créé depuis ViewDirector
+                $backup_array = Backup::getBackupList();
+                $backup_options = '';
+                for($i = count($backup_array) - 1; $i >= 0; $i--){
+                        $backup_options .= '<option value="' . $backup_array[$i] . '">' . $backup_array[$i] . '</option>';
+                }
+
                 if(file_exists($viewFile)){
                         ob_start();
                     require $viewFile;
diff --git a/src/view/templates/maintenance.php b/src/view/templates/maintenance.php
index 3501fa4..58e04c0 100644
--- a/src/view/templates/maintenance.php
+++ b/src/view/templates/maintenance.php
@@ -19,9 +19,25 @@
         <div class="basic_div">
                 <p>
                         <a href="<?= new URL(['action' => 'get_mysqldump']) ?>">
-                                <button id="get_mysqldump">Télécharger la base de données</button>
+                                <button id="get_mysqldump">Télécharger une sauvegarde de la base de données</button>
                         </a><br>
-                        <i>Réalise un "mysqldump", vous obtiendrez un unique fichier contenant toute la BDD.</i>
+                        <i>Obtenir un fichier SQL à conserver sur votre ordinateur. Une sauvegarde est réalisée à chaque visite de cette page.</i>
+                </p>
+        </div>
+        <div class="basic_div">
+                <p>Restaurer la base de données à partir d'un fichier SQL.<br>
+                        <i>Attention l'actuelle BDD sera écrasée!</i>
+                </p>
+                <p>
+                        <label for="">Utiliser une sauvegarde conservée sur le serveur</label>
+                        <select>
+                                <?= $backup_options ?>
+                        </select>
+                </p>
+                <p>
+                        <label for="restore_sql_dump">Utiliser un fichier sur votre ordinateur:</label>
+                        <input id="restore_sql_dump" type="file" accept=".sql" name="restore_sql_dump">
+                        
                 </p>
         </div>