modification d'une URL page Menu et chemin, htmlspecialchars sur les URL du menu à l'affichage

5 files changed, 56 insertions, 22 deletions

diff --git a/src/controller/MenuAndPathsController.php b/src/controller/MenuAndPathsController.php
index d429287..5779b39 100644
--- a/src/controller/MenuAndPathsController.php
+++ b/src/controller/MenuAndPathsController.php
@@ -14,9 +14,15 @@ class MenuAndPathsController
         $previous_page = Director::$menu_data->findPageById((int)$_POST["location"]); // (int) à cause de declare(strict_types=1);
         $parent = $previous_page->getParent();
 
+        $url_input = trim($_POST["url_input"]); // faire htmlspecialchars à l'affichage
+        if(!filter_var($url_input, FILTER_VALIDATE_URL) || !str_starts_with($url_input, 'http')){
+            header("Location: " . new URL(['page' => $_GET['from'], 'error' => 'invalide_url']));
+            die;
+        }
+
         $page = new Page(
             trim(htmlspecialchars($_POST["label_input"])),
-            filter_var($_POST["url_input"], FILTER_VALIDATE_URL),
+            $url_input,
             true, true, false,
             $previous_page->getPosition(),
             $parent); // peut et DOIT être null si on est au 1er niveau
@@ -24,7 +30,7 @@ class MenuAndPathsController
         // on a donné à la nouvelle entrée la même position qu'à la précédente,
         // addChild l'ajoute à la fin du tableau "children" puis on trie
         // exemple avec 2 comme position demandée: 1 2 3 4 2 devient 1 2 3 4 5 et la nouvelle entrée sera en 3è position
-        if($parent == null){
+        if(!$parent){
             $parent = Director::$menu_data;
         }
         $parent->addChild($page); // true pour réindexer les positions en BDD
@@ -36,6 +42,25 @@ class MenuAndPathsController
         die;
     }
 
+    static public function editUrlEntry(EntityManager $entityManager, array $json): void
+    {
+        $url_input = trim($json['url_input']); // faire htmlspecialchars à l'affichage
+        $page = $entityManager->find('App\Entity\Page', $json['id']);
+
+        if(!$page){
+            echo json_encode(['success' => false, 'message' => "id invalide"]);
+        }
+        elseif(!filter_var($url_input, FILTER_VALIDATE_URL) || !str_starts_with($url_input, 'http')){
+            echo json_encode(['success' => false, 'message' => "la chaîne envoyée n'est pas une URL valide"]);
+        }
+        else{
+            $page->setEndOfPath($url_input);
+            $entityManager->flush();
+            echo json_encode(['success' => true, 'url_input' => $url_input]);
+        }
+        die;
+    }
+
     static public function deleteUrlMenuEntry(EntityManager $entityManager): void
     {
         Director::$menu_data = new Menu($entityManager);
@@ -163,7 +188,6 @@ class MenuAndPathsController
         else{
                 echo json_encode(['success' => false]);
         }
-            
             die;
         }
 
diff --git a/src/router.php b/src/router.php
index 04441a9..3c3c773 100644
--- a/src/router.php
+++ b/src/router.php
@@ -156,9 +156,12 @@ elseif($_SERVER['REQUEST_METHOD'] === 'POST'){
                 elseif($_GET['menu_edit'] === 'switch_positions' && isset($json['id1']) && isset($json['id2'])){
                     MenuAndPathsController::switchPositions($entityManager, $json);
                 }
-                elseif($_GET['menu_edit'] === 'displayInMenu' && isset($json['id']) && isset($json['checked'])){
+                elseif($_GET['menu_edit'] === 'display_in_menu' && isset($json['id']) && isset($json['checked'])){
                     MenuAndPathsController::displayInMenu($entityManager, $json);
                 }
+                elseif($_GET['menu_edit'] === 'edit_url_entry' && isset($json['id']) && isset($json['url_input'])){
+                    MenuAndPathsController::editUrlEntry($entityManager, $json);
+                }
             }
 
             /* -- mode Modification d'une page -- */
diff --git a/src/view/MenuBuilder.php b/src/view/MenuBuilder.php
index 9d4dda1..bc64e30 100644
--- a/src/view/MenuBuilder.php
+++ b/src/view/MenuBuilder.php
@@ -13,18 +13,13 @@ class MenuBuilder extends AbstractBuilder
     //private int $margin_left_multiplier = 29;
     private string $options = '';
 
-    public function __construct(Node $node = null, bool $template = true)
+    public function __construct(Node $node, bool $template = true)
     {
-        //parent::__construct($node);
+        // impossible de me rappeler pourquoi j'ai écrit ce test sur $node, pourquoi $node serait null?
         $viewFile = $node === null ? self::VIEWS_PATH . 'menu.php' : self::VIEWS_PATH . $node->getName() . '.php';
         
         if(file_exists($viewFile))
         {
-            /*if(!empty($node->getNodeData()->getData()))
-            {
-                extract($node->getNodeData()->getData());
-            }*/
-
             if($_SESSION['admin']){
                 $this->unfoldMenu(Director::$menu_data);
                 
@@ -63,13 +58,26 @@ class MenuBuilder extends AbstractBuilder
                 </span>
                 <button>' . $entry->getPageName() . '</button>';
 
+            // seul la modification des URL est possible pour l'instant, les noms des entrées de menu attendront
             if(str_starts_with($entry->getEndOfPath(), 'http')){
-                $this->html .= '<span id="edit-i' . $entry->getId() . '"><img class="move_entry_icon" src="assets/edit.svg" onclick="editUrlEntry(' . $entry->getId() . ')"></span>
-                    <i class="url">' . $entry->getEndOfPath() . '</i>
-                    <form style="display: inline;" id="delete-i' . $entry->getId() . '" method="post" action="' . new URL(['from' => 'menu_chemins']) . '">
+                $this->html .= '<form style="display: inline;" id="delete-i' . $entry->getId() . '" method="post" action="' . new URL(['from' => 'menu_chemins']) . '">
                         <input type="hidden" name="delete" value="' . $entry->getId() . '">
                         <input type="image" class="move_entry_icon" src="assets/delete-bin.svg" alt="delete link button" onclick="return confirm(\'Voulez-vous vraiment supprimer cette entrée?\');">
-                    </form>';
+                    </form>
+                    <span class="url">
+                        <input type="url" value="' . htmlspecialchars($entry->getEndOfPath()) . '">
+                        <img class="move_entry_icon" src="assets/save.svg" onclick="editUrlEntry(' . $entry->getId() . ')">
+                    </span>';
+
+                // code à recycler pour pouvoir modifier le nom de l'entrée de menu correspondant aux liens
+                /*$this->html .= '<span id="cancel-i' . $entry->getId() . '">
+                        <input type="hidden" name="cancel" value="' . $entry->getId() . '">
+                        <button class="hidden" onclick="cancelUrlEntry(' . $entry->getId() . ')">Annuler</button>
+                    </span>
+                    <span id="submit-i' . $entry->getId() . '">
+                        <input type="hidden" name="submit" value="' . $entry->getId() . '">
+                        <input type="submit" class="hidden" onclick="submitUrlEntry(' . $entry->getId() . ')">
+                    </span>';*/
             }
             else{
                 $this->html .= '<i class="path">' . $entry->getPagePath() . '</i>';
diff --git a/src/view/NavBuilder.php b/src/view/NavBuilder.php
index a9cf49c..2cbdef9 100644
--- a/src/view/NavBuilder.php
+++ b/src/view/NavBuilder.php
@@ -38,7 +38,7 @@ class NavBuilder extends AbstractBuilder
                 {
                     if(str_starts_with($data->getEndOfPath(), 'http')) // lien vers autre site
                     {
-                        $link .= '<a href="' . $data->getEndOfPath() . '" target="_blank">';
+                        $link .= '<a href="' . htmlspecialchars($data->getEndOfPath()) . '" target="_blank">';
                     }
                     elseif($data->getEndOfPath() != '') // lien relatif
                     {
diff --git a/src/view/templates/menu.php b/src/view/templates/menu.php
index 55c9ff9..d78c665 100644
--- a/src/view/templates/menu.php
+++ b/src/view/templates/menu.php
@@ -2,18 +2,18 @@
 <section class="menu">
         <h3>Menu et chemins</h3>
         <div class="new_page_button">
-                <p >Créer une <a href="<?= new URL(['page' => 'nouvelle_page']) ?>"><button style="color: #ff1d04;">Nouvelle page</button></a>.</p>
+                <p>Créer une <a href="<?= new URL(['page' => 'nouvelle_page']) ?>"><button style="color: #ff1d04;">Nouvelle page</button></a>.</p>
         </div>
         <div class="url_form_zone">
-                <p>Ajouter au menu un lien vers un site web quelconque avec le formulaire ci-dessous:</p>
+                <p>Créer une entrée dans le menu avec une adresse vers un site quelconque:</p>
                 <form method="post" action="<?= new URL(['from' => 'menu_chemins']) ?>">
                         <p>
-                                <label for="label_input">Nom:</label>
+                                <label for="label_input">Nom dans le menu:</label>
                                 <input id="label_input" type="text" name="label_input">
                         </p>
                         <p>
-                                <label for="url_input">Adresse URL:</label>
-                                <input id="url_input" type="url" name="url_input">
+                                <label for="url_input">Adresse (collez votre lien):</label>
+                                <input id="url_input" type="url" name="url_input" placeholder="http://">
                         </p>
                         <p>
                                 <label>Placer le lien juste après cette entrée:</label>
@@ -38,5 +38,4 @@
         <div id="menu_edit_buttons">
 <?= $this->html ?>
         </div>
-        
 </section>
\ No newline at end of file