visibilité bouton Modifier la page, sécurité si &mode=page_modif est utilisé de manière non attendue

5 files changed, 18 insertions, 14 deletions

diff --git a/public/index.php b/public/index.php
index 6149f7a..9caee0a 100644
--- a/public/index.php
+++ b/public/index.php
@@ -12,9 +12,8 @@ B -
     1/ passer à des chemins modernes "ciblant des ressources" genre /chemin/de/la/page
         le mode modification de page doit thérioquement être appelé comme ça: /chemin/de/la/page/modif_page
         apparemment, le from=nom_page pour les formulaires ne se fait pas...
-    2/ utiliser le routeur de symfony: nécéssite que tous les contrôleurs soient des classes avec un namespace */
-
-// http-foundation possède aussi une classe Session. intéressant!
+    2/ utiliser le routeur de symfony: nécéssite que tous les contrôleurs soient des classes avec un namespace
+    3/ http-foundation possède aussi une classe Session. intéressant! */
 
 declare(strict_types=1);
 
@@ -53,7 +52,8 @@ ini_set('session.cookie_httponly', 'On');
 ini_set('session.use_strict_mode', 'On');
 ini_set('session.cookie_secure', 'On');
 session_start();
-$_SESSION['admin'] = !isset($_SESSION['admin']) ? false : $_SESSION['admin']; // intialisation sur faux
+
+$_SESSION['admin'] = $_SESSION['admin'] ?? false;
 if($_SESSION['admin'] === false || empty($_SESSION['user'])){ // OUT !!
     $_SESSION['user'] = '';
     $_SESSION['admin'] = false;
diff --git a/src/controller/ViewController.php b/src/controller/ViewController.php
index f66baff..9117f0f 100644
--- a/src/controller/ViewController.php
+++ b/src/controller/ViewController.php
@@ -5,7 +5,6 @@
 
 declare(strict_types=1);
 
-use App\Entity\Article;
 use App\Entity\Node;
 use Doctrine\ORM\EntityManager;
 use Symfony\Component\HttpFoundation\Request;
@@ -19,21 +18,24 @@ class ViewController extends AbstractBuilder // ViewController est aussi le prem
 
     public function buildView(EntityManager $entityManager, Request $request): Response
     {
-        // accès au modèle
+        /* 1/ accès au modèle */
         $director = new Director($entityManager, true);
         $director->makeRootNode(htmlspecialchars($request->query->get('id') ?? ''));
         self::$root_node = $director->getNode();
 
+
+        /* 2/ traitement de quelques paramètres */
+
         // mode modification d'une page activé
-        if($_SESSION['admin'] && $request->query->has('page')
+        if($_SESSION['admin']
             && $request->query->has('mode') && $request->query->get('mode') === 'page_modif'
-            && $request->query->get('page') !== 'connexion' && $request->query->get('page') !== 'article' && $request->query->get('page') !== 'nouvelle_page' && $request->query->get('page') !== 'menu_chemins'){
-            // les contrôles de la 2è ligne devraient utiliser un tableau
+            && !in_array(CURRENT_PAGE, ['article', 'nouvelle_page', 'menu_chemins', 'user_edit', 'connection']))
+        {
             MainBuilder::$modif_mode = true;
         }
 
         // page article: mode création et erreurs d'id
-        if($request->query->has('page') && $request->query->get('page') === 'article'){
+        if(CURRENT_PAGE === 'article'){
             if($_SESSION['admin']){
                 if(!$request->query->has('id')){
                     return new Response($this->html, 302);
@@ -54,7 +56,8 @@ class ViewController extends AbstractBuilder // ViewController est aussi le prem
             }
         }
 
-        //début de la construction de la page
+
+        /* 3/ construction de la page avec builders et vues */
         $this->useChildrenBuilder(self::$root_node);
 
         return new Response($this->html, 200);
diff --git a/src/model/entities/Node.php b/src/model/entities/Node.php
index 08961e0..f7af6de 100644
--- a/src/model/entities/Node.php
+++ b/src/model/entities/Node.php
@@ -32,7 +32,7 @@ class Node
     private int $position;
 
     #[ORM\ManyToOne(targetEntity: self::class)]
-    //#[ORM\ManyToOne(targetEntity: self::class, fetch: 'EAGER')] // À TESTER
+    //#[ORM\ManyToOne(targetEntity: self::class, fetch: 'EAGER')] // fetch: 'eager' désactive le mode fainéant de doctrine
     #[ORM\JoinColumn(name: "parent_id", referencedColumnName: "id_node", onDelete: "SET NULL", nullable: true)]
     private ?self $parent = null;
 
diff --git a/src/view/AbstractBuilder.php b/src/view/AbstractBuilder.php
index 7b53010..dd83a42 100644
--- a/src/view/AbstractBuilder.php
+++ b/src/view/AbstractBuilder.php
@@ -7,7 +7,7 @@ use App\Entity\Node;
 
 abstract class AbstractBuilder
 {
-        protected const VIEWS_PATH = '../src/view/templates/';
+        protected const string VIEWS_PATH = '../src/view/templates/';
         protected string $html = '';
     protected int $id_node;
 
diff --git a/src/view/FooterBuilder.php b/src/view/FooterBuilder.php
index cf88476..f20dd2e 100644
--- a/src/view/FooterBuilder.php
+++ b/src/view/FooterBuilder.php
@@ -72,7 +72,8 @@ class FooterBuilder extends AbstractBuilder
     private function makePageModifModeButton(): string
     {
         $link_edit_page = new URL(['page' => CURRENT_PAGE]);
-        if(CURRENT_PAGE !== 'article'){
+        if(!in_array(CURRENT_PAGE, ['article', 'nouvelle_page', 'menu_chemins'])) // ajouter 'user_edit' et 'connection' le jour où ces pages auront un footer
+        {
             if(MainBuilder::$modif_mode){
                 $link_edit_label = 'Sortir du mode modification';
             }