lancés et captures d'exceptions page maintenance, page menu et chemins: interdiction pour une page d'avoir pour parent une adresse d'un site, regression contructeur de MenuBuilder, MAJ de paquets dans composer.json

10 files changed, 72 insertions, 26 deletions

diff --git a/composer.json b/composer.json
index 8d97d87..b7ad2e5 100644
--- a/composer.json
+++ b/composer.json
@@ -2,17 +2,17 @@
     "name": "ordipolo/cms",
     "description": "CMS maison avec PHP, doctrine et tinyMCE",
     "require": {
+        "composer": "*",
         "doctrine/dbal": "^4.3",
         "doctrine/orm": "^3.5",
-        "symfony/cache": "^7.3",
-        "composer": "*",
+        "symfony/cache": "^7.0",
         "htmlawed/htmlawed": "^1.2",
-        "tinymce/tinymce": "^8.2",
-        "phpmailer/phpmailer": "^7.0",
-        "symfony/http-foundation": "^7.3",
+        "tinymce/tinymce": "^8.5",
+        "mklkj/tinymce-i18n": "^26.5",
+        "phpmailer/phpmailer": "^7.1",
         "twbs/bootstrap-icons": "^1.13",
+        "symfony/http-foundation": "^7.0",
         "symfony/var-exporter": "^7.0",
-        "mklkj/tinymce-i18n": "^25.11",
         "symfony/process": "^7.0"
     },
     "scripts": {
diff --git a/public/js/maintenance.js b/public/js/maintenance.js
index 3d3d5e8..078c223 100644
--- a/public/js/maintenance.js
+++ b/public/js/maintenance.js
@@ -43,4 +43,24 @@ function cleanLogs(){
 }
 
 // notification après restauration
-// et éventuellement récupérer le message de l'exception d'une autre manière
+document.addEventListener('DOMContentLoaded', function(){
+        const params = new URLSearchParams(window.location.search);
+        // ça pourrait être bien de récupérer le message d'erreur de l'exception d'une autre manière (message dans la variable globale window? c'est faisable??)
+
+        if(typeof window.error_message !== "undefined"){
+                toastNotify(window.error_message);
+        }
+
+        if(params.has('read_backups_dir')){
+                toastNotify("Une erreur s'est produite:<br>" + params.get('read_backups_dir'));
+        }
+
+        if(params.has('database_restauration')){
+                if(params.get('database_restauration') === 'successful'){
+                        toastNotify("La base de données a été restaurée avec succès !!");
+                }
+                else{
+                        toastNotify("Une erreur s'est produite:<br>" + params.get('database_restauration'));
+                }
+        }
+});
diff --git a/public/js/menu.js b/public/js/menu.js
index c0ec623..41c8709 100644
--- a/public/js/menu.js
+++ b/public/js/menu.js
@@ -45,15 +45,18 @@ function moveOneLevelDown(page_id)
     })
     .then(response => response.json())
     .then(data => {
-        if(data.success)
-        {
+        if(data.success){
                         // affichage
                         nav_zone.innerHTML = '';
                 nav_zone.insertAdjacentHTML('afterbegin', data.nav);
                 menu_edit_buttons.innerHTML = '';
                         menu_edit_buttons.insertAdjacentHTML('afterbegin', data.menu_buttons);
         }
-        else {
+        else if(!data.success && data.error == 'new_parent_is_a_link'){
+            toastNotify("Action interdite, une page ne peut avoir pour parent une adresse vers un site web.");
+            console.log("Action interdite, une page ne peut avoir pour parent une adresse vers un site web.");
+        }
+        else{
             console.error('Échec du déplacement');
         }
     })
diff --git a/src/controller/MaintenanceController.php b/src/controller/MaintenanceController.php
index f3352d8..3b58fe6 100644
--- a/src/controller/MaintenanceController.php
+++ b/src/controller/MaintenanceController.php
@@ -5,7 +5,6 @@ declare(strict_types=1);
 
 use Doctrine\ORM\EntityManager;
 use App\Entity\log;
-use Symfony\Component\Process\Exception\ProcessFailedException;
 use Symfony\Component\HttpFoundation\File\UploadedFile;
 
 class MaintenanceController
@@ -61,9 +60,8 @@ class MaintenanceController
                         die;
                 }
                 // exeptions lancées dans Backup::mySQLdump
-                catch(ProcessFailedException $e){ // pas d'info $e pour le client
-                        header('Location: ' . new URL(['page' => 'maintenance', 'error' => '500']));
-                        die;
+                catch(RuntimeException $e){ // pas d'info $e pour le client7
+                        header('Location: ' . new URL(['page' => 'maintenance', 'get_last_dump' => $e->getMessage()]));
                 }
                 die;
         }
diff --git a/src/controller/MenuAndPathsController.php b/src/controller/MenuAndPathsController.php
index 4d37f3d..799f14b 100644
--- a/src/controller/MenuAndPathsController.php
+++ b/src/controller/MenuAndPathsController.php
@@ -141,7 +141,7 @@ class MenuAndPathsController
                 $page = Model::$menu->findPageById((int)$id);
 
         $parent = $page->getParent(); // peut être null
-        if($parent == null){
+        if($parent === null){
             $parent = Model::$menu;
         }
 
@@ -150,6 +150,12 @@ class MenuAndPathsController
         if($page->getPosition() > 1){
             foreach($parent->getChildren() as $child){
                 if($child->getPosition() === $page->getPosition() - 1){
+                    // refus si $parent est une adresse, ça va casser le lien, exemple: index.php?page=chemin/http://un_site_web.fr/vers/ici
+                    if(str_starts_with($child->getEndOfPath(), 'http')){
+                        echo json_encode(['success' => false, 'error' => 'new_parent_is_a_link']);
+                        die;
+                    }
+
                     $page->setParent($child);
                     break;
                 }
diff --git a/src/controller/UserController.php b/src/controller/UserController.php
index 9746a47..f911d2d 100644
--- a/src/controller/UserController.php
+++ b/src/controller/UserController.php
@@ -92,7 +92,13 @@ class UserController
                                 $_SESSION['user']['role'] = $user->getRole();
 
                                 EmailService::cleanEmails($entityManager);
-                                Backup::mySQLdump($entityManager, 'auto'); // créer un nouveau backup
+                                
+                                try{
+                                        Backup::mySQLdump($entityManager, 'auto'); // créer un nouveau backup
+                                }
+                                catch(RuntimeException $e){
+                                        echo '<script>window.error_message = "' . $e->getMessage() . '";</script>';
+                                }
 
                                 $url = new URL(isset($_GET['from']) ? ['page' => $_GET['from']] : []);
                                 isset($_GET['id']) ? $url->addParams(['id' => $_GET['id']]) : '';
diff --git a/src/service/Backup.php b/src/service/Backup.php
index 8a3030f..c886617 100644
--- a/src/service/Backup.php
+++ b/src/service/Backup.php
@@ -74,12 +74,17 @@ class Backup
 
         static public function getBackupList(): array
         {
+                $files = scandir(Backup::$backup_dir); // affiche un warning si échoue (à cacher en prod)
+                if(!$files){
+                        throw new RuntimeException("Le serveur a rencontré une erreur:<br>Accès aux backups impossible faute de permissions.");
+                }
+
                 $backup_array = [];
-                foreach(scandir(Backup::$backup_dir) as $file){
-                    if($file[0] === '.'){
-                        continue;
-                    }
-                    $backup_array[] = $file;
+                foreach($files as $file){
+                        if($file[0] === '.'){
+                                continue;
+                        }
+                        $backup_array[] = $file;
                 }
                 return $backup_array;
         }
@@ -185,6 +190,7 @@ class Backup
                         password=" . Config::$password . "\n
                         host=" . Config::$db_host . "\n");
 
+                //$file_name = self::gzipExtract($file_name); // '.gz' ajouté à la fin
 
                 $command = new Process([
                     $engine, // mariadb ou mysql
diff --git a/src/view/MaintenanceBuilder.php b/src/view/MaintenanceBuilder.php
index d9c52b1..c5f53ab 100644
--- a/src/view/MaintenanceBuilder.php
+++ b/src/view/MaintenanceBuilder.php
@@ -11,7 +11,14 @@ class MaintenanceBuilder extends AbstractBuilder
                 $viewFile = self::VIEWS_PATH . $node->getName() . '.php';
 
                 // noter qu'un backup vient d'être créé depuis ViewDirector
-                $backup_array = Backup::getBackupList();
+                try{
+                        $backup_array = Backup::getBackupList();
+                }
+                // exeptions lancées dans Backup::mySQLdump
+                catch(RuntimeException $e){ // pas d'info $e pour le client
+                        $backup_array = [];
+                        echo '<script>window.error_message = "' . $e->getMessage() . '";</script>';
+                }
                 $backup_options = '';
                 for($i = count($backup_array) - 1; $i >= 0; $i--){
                         $backup_options .= '<option value="' . $backup_array[$i] . '">' . $backup_array[$i] . '</option>';
diff --git a/src/view/MenuBuilder.php b/src/view/MenuBuilder.php
index 85335df..0395dfa 100644
--- a/src/view/MenuBuilder.php
+++ b/src/view/MenuBuilder.php
@@ -13,10 +13,10 @@ class MenuBuilder extends AbstractBuilder
     //private int $margin_left_multiplier = 29;
     private string $options = '';
 
-    public function __construct(Node $node, bool $template = true)
+    public function __construct(?Node $node, bool $template = true)
     {
-        // dans une ancienne version $node pouvait être null mais je ne sais plus pourquoi
-        $viewFile = self::VIEWS_PATH . $node->getName() . '.php';
+        // $node peut $etre null parce qu'on ne construit pas de page dans MenuAndPathsController, on ne lit pas la table "node", etc
+        $viewFile = $node === null ? self::VIEWS_PATH . 'menu.php' : self::VIEWS_PATH . $node->getName() . '.php';
         
         if(file_exists($viewFile))
         {
diff --git a/src/view/templates/menu.php b/src/view/templates/menu.php
index 1159455..ccfb518 100644
--- a/src/view/templates/menu.php
+++ b/src/view/templates/menu.php
@@ -31,7 +31,7 @@
                     <p><img src="assets/arrow-right.svg"> devenir une branche de l'élément précédent</p>
                         <p><img src="assets/arrow-up.svg"><img src="assets/arrow-down.svg"> déplacer la branche parmi celles de même niveau</p>
                     <p><input type="checkbox" checked> afficher/cacher</p>
-                    <p><img src="assets/edit.svg"> modifier un lien</p>
+                    <p><img src="assets/save.svg"> enregistrer une modification</p>
                     <p><img src="assets/delete-bin.svg"> supprimer un lien</p>
                 </div>
         </aside>