partie client de la restauration de BDD, script bin/mysqldump.php, changements permissions dossiers, renommage ViewDirector

11 files changed, 113 insertions, 89 deletions

diff --git a/bin/mode.php b/bin/mode.php
deleted file mode 100755
index 552ce4e..0000000
--- a/bin/mode.php
+++ /dev/null
@@ -1,65 +0,0 @@
-#!/usr/bin/env php
-<?php
-// bin/mode.php
-
-declare(strict_types=1);
-
-use App\Entity\AppMetadata;
-use Doctrine\ORM\EntityManager;
-
-
-chdir(dirname(__FILE__)); // pour être au même niveau que l'appli dans /public
-require "../vendor/autoload.php";
-Config::load('../config/config.ini'); // pour TABLE_PREFIX
-require '../src/model/doctrine-bootstrap.php';
-
-const ALLOWED_MODES = ['run', 'maintenance'];
-
-// aide
-$aide = "Usage : php bin/mode.php <mode>\nModes disponibles : " . implode(', ', ALLOWED_MODES) . "\n";
-// version avec deux paramètres proposée par claude
-/*$aide = "Usage : php bin/mode.php <mode> [--by <auteur>]
-    Modes disponibles : " . implode(', ', ALLOWED_MODES) . "
-    Exemple : php bin/mode.php maintenance --by alice\n";*/
-if($argc < 2 || in_array($argv[1], ['--help', '-h'])){
-    echo $aide;
-    exit(0);
-}
-
-// validation du mode
-if(!in_array($argv[1], ALLOWED_MODES)){
-    echo "Erreur : mode '$argv[1]' invalide.\n";
-    echo "Modes disponibles : " . implode(', ', ALLOWED_MODES) . "\n";
-    exit(1);
-}
-
-// paramètre --by
-/*$by = 'cli';
-for ($i = 2; $i < $argc; $i++) {
-    if ($argv[$i] === '--by' && isset($argv[$i + 1])) {
-        $by = $argv[$i + 1];
-        break;
-    }
-}*/
-
-// changement BDD
-try{
-    AppMode::load($entityManager);
-    $current = AppMode::get();
-
-    if($current === $argv[1]){
-        echo "Le mode est déjà '$argv[1]', aucun changement.\n";
-        exit(0);
-    }
-
-    AppMode::set($entityManager, $argv[1]);
-    echo "Mode changé : '$current' => '$argv[1]'\n";
-
-    // le mode deux paramètres permettra d'indiquer son nom et automatiquement de d'enregistrer la date du changement
-    //AppMode::set($entityManager, $argv[1], 'system');
-    //echo "Mode changé : '$current' => '$argv[1]' (par $by)\n";
-}
-catch(LogicException $e){
-    echo "Erreur : " . $e->getMessage() . "\n";
-    exit(1);
-}
\ No newline at end of file
diff --git a/bin/mysqldump.php b/bin/mysqldump.php
new file mode 100644
index 0000000..4356a81
--- /dev/null
+++ b/bin/mysqldump.php
@@ -0,0 +1,13 @@
+#!/usr/bin/env php
+<?php
+// bin/mysqldump.php
+
+declare(strict_types=1);
+
+chdir(dirname(__FILE__));
+require "../vendor/autoload.php";
+Config::load('../config/config.ini');
+require '../src/model/doctrine-bootstrap.php';
+
+$file_name = Backup::mySQLdump($entityManager); // créer un nouveau backup
+echo realpath($file_name) . "\n";
\ No newline at end of file
diff --git a/public/css/maintenance.css b/public/css/maintenance.css
index 887f3a7..8fd0601 100644
--- a/public/css/maintenance.css
+++ b/public/css/maintenance.css
@@ -19,4 +19,12 @@
 .maintenance table th, .maintenance table td
 {
         border: 1px black solid;
+}
+.maintenance select, .maintenance input[type="file"]::file-selector-button
+{
+        color: #ff1d04;
+        font-size: medium;
+        border-radius: 4px;
+        background-color: white;
+        border: lightgrey 2px outset;
 }
\ No newline at end of file
diff --git a/public/index.php b/public/index.php
index bd6725e..591fff7 100644
--- a/public/index.php
+++ b/public/index.php
@@ -4,7 +4,7 @@
 /* plan d'action pour "symfonyfier" le site
 A - 
     1/ de vrais contrôleurs: classes et méthodes prenant une requête en entrée et retournant une réponse
-    (début de séparation contrôleurs et classes métier, exemple: ViewController/Model)
+    (début de séparation contrôleurs et classes métier)
     2/ routeur structuré: méthodes GET et POST, content-type, admin
     3/ routeur amélioré: pré-routage avec méthodes HTTP: GET, HEAD, POST, PUT, PATCH, DELETE, etc
     4/ réécriture avec les classes Request et Response sans toucher les liens
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>