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

9 files changed, 188 insertions, 33 deletions

diff --git a/public/assets/save-nb.svg b/public/assets/save-nb.svg
new file mode 100644
index 0000000..f7fe755
--- /dev/null
+++ b/public/assets/save-nb.svg
@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24" fill="currentColor"><path d="M7 19V13H17V19H19V7.82843L16.1716 5H5V19H7ZM4 3H17L21 7V20C21 20.5523 20.5523 21 20 21H4C3.44772 21 3 20.5523 3 20V4C3 3.44772 3.44772 3 4 3ZM9 15V19H15V15H9Z"></path></svg>
\ No newline at end of file
diff --git a/public/assets/save.svg b/public/assets/save.svg
new file mode 100644
index 0000000..f2405a5
--- /dev/null
+++ b/public/assets/save.svg
@@ -0,0 +1,37 @@
+<?xml version="1.0" encoding="UTF-8" standalone="no"?>
+<svg
+   viewBox="0 0 24 24"
+   fill="currentColor"
+   version="1.1"
+   id="svg1"
+   sodipodi:docname="save.svg"
+   inkscape:version="1.4.2 (ebf0e940d0, 2025-05-08)"
+   xmlns:inkscape="http://www.inkscape.org/namespaces/inkscape"
+   xmlns:sodipodi="http://sodipodi.sourceforge.net/DTD/sodipodi-0.dtd"
+   xmlns="http://www.w3.org/2000/svg"
+   xmlns:svg="http://www.w3.org/2000/svg">
+  <defs
+     id="defs1" />
+  <sodipodi:namedview
+     id="namedview1"
+     pagecolor="#ffffff"
+     bordercolor="#000000"
+     borderopacity="0.25"
+     inkscape:showpageshadow="2"
+     inkscape:pageopacity="0.0"
+     inkscape:pagecheckerboard="0"
+     inkscape:deskcolor="#d1d1d1"
+     inkscape:zoom="5.5"
+     inkscape:cx="16.090909"
+     inkscape:cy="19.545455"
+     inkscape:window-width="1330"
+     inkscape:window-height="1032"
+     inkscape:window-x="586"
+     inkscape:window-y="0"
+     inkscape:window-maximized="1"
+     inkscape:current-layer="svg1" />
+  <path
+     style="fill:#dd3333;stroke:#dd3333;stroke-width:0.687187"
+     d="M 3.7700827,20.814384 C 3.5436283,20.721841 3.2752699,20.427615 3.1737307,20.160547 3.0560503,19.851024 3.007568,16.786203 3.0400116,11.70747 3.0870587,4.3427109 3.1154561,3.7154214 3.415392,3.4154406 3.712924,3.1178639 4.2867777,3.0867003 10.324483,3.0402361 l 6.584608,-0.050673 2.052117,2.0601995 2.052117,2.0601994 -0.05212,6.575086 c -0.04777,6.027435 -0.07915,6.602112 -0.37665,6.899564 -0.300533,0.300481 -0.91908,0.327199 -8.363636,0.361255 C 7.6163,20.966927 4.0059136,20.910757 3.7700807,20.814384 Z M 7,16 v -3 h 5 5 v 3 3 h 1 1 V 13.322457 7.6449149 L 17.686369,6.3224575 16.372737,5 H 10.686369 5 v 7 7 h 1 1 z m 8,1 V 15 H 12 9 v 2 2 h 3 3 z"
+     id="path2" />
+</svg>
diff --git a/public/css/menu.css b/public/css/menu.css
index 62e6623..57d842a 100644
--- a/public/css/menu.css
+++ b/public/css/menu.css
@@ -70,7 +70,7 @@
         vertical-align: middle;
         border: 2px transparent solid;
 }
-.menu #location:hover, .menu input[type=submit]:hover, .move_entry_icon:hover
+.menu #location:hover, .menu input[type=submit]:hover, .menu .move_entry_icon:hover
 {
         background-color: yellow;
         border-radius: 4px;
@@ -106,8 +106,17 @@
 {
         margin: 5px;
         display: flex;
+        flex-wrap: wrap;
         align-items: center;
 }
+.menu .url
+{
+        text-wrap: nowrap;
+}
+.menu .url input[type=url]
+{
+        width: 50%;
+}
 
 @media screen and (min-width: 80rem) {
         i{}
diff --git a/public/js/menu.js b/public/js/menu.js
index 8f1f326..ef58c42 100644
--- a/public/js/menu.js
+++ b/public/js/menu.js
@@ -122,7 +122,7 @@ function checkMenuEntry(page_id){
         const checkbox = clicked_menu_entry.querySelector("input");
         let color;
 
-        fetch('index.php?menu_edit=displayInMenu', {
+        fetch('index.php?menu_edit=display_in_menu', {
         method: 'POST',
         headers: {
         'Content-Type': 'application/json'
@@ -131,16 +131,14 @@ function checkMenuEntry(page_id){
     })
     .then(response => response.json())
     .then(data => {
-        if(data.success)
-        {
+        if(data.success){
                 color = checkbox.checked ? "#ff1d04" : "grey";
                 clicked_menu_entry.querySelector("button").style.color = color;
 
                 nav_zone.innerHTML = '';
                 nav_zone.insertAdjacentHTML('afterbegin', data.nav);
         }
-        else {
-
+        else{
             console.error('Échec de l\'inversion');
         }
     })
@@ -149,13 +147,89 @@ function checkMenuEntry(page_id){
     });
 }
 
-
+// seul la modification des URL est possible pour l'instant, les noms des entrées de menu attendront
 function editUrlEntry(page_id){
-        const selected_div = document.getElementById(page_id);
-        console.log(selected_div.id);
+    const parent_div = document.getElementById(page_id);
+    const url_input = parent_div.querySelector('.url').querySelector('input').value;
+    
+    fetch('index.php?menu_edit=edit_url_entry', {
+        method: 'POST',
+        headers: {
+        'Content-Type': 'application/json'
+        },
+        body: JSON.stringify({ id: page_id, url_input: url_input })
+    })
+    .then(response => response.json())
+    .then(data => {
+        if(data.success){
+            findParentByTagName(document.getElementById('m_' + page_id), 'a').href = data.url_input; // MAJ menu
+            toastNotify("Nouvelle adresse enregistrée avec succès")
+        }
+        else{
+            toastNotify("Erreur rencontrée par le serveur, changements non pris en compte");
+            console.error("Erreur rencontrée par le serveur, changements non pris en compte");
+        }
+    })
+    .catch(error => {
+        console.error('Erreur:', error);
+    });
 }
 
-function deleteUrlEntry(page_id){
+
+
+// code à recycler pour pouvoir modifier le nom de l'entrée de menu correspondant aux liens
+/*function editUrlEntry(page_id){
+        const parent_div = document.getElementById(page_id);
+    parent_div.querySelector('i').classList.add('hidden');
+    parent_div.querySelector('.url').querySelector('input').classList.remove('hidden');
+    parent_div.querySelector('#edit-i' + page_id).classList.add('hidden');
+    parent_div.querySelector('#delete-i' + page_id).querySelector('input[type=image]').classList.add('hidden');
+    parent_div.querySelector('#cancel-i' + page_id).querySelector('button').classList.remove('hidden');
+    parent_div.querySelector('#submit-i' + page_id).querySelector('input[type=submit]').classList.remove('hidden');
+}
+function cancelUrlEntry(page_id){
+    const parent_div = document.getElementById(page_id);
+    parent_div.querySelector('.url').querySelector('input').value = parent_div.querySelector('i').textContent; // textContent (contrairement à innerHTML) ne transforme pas les & en entités HTML
+    closeUrlEntry(page_id, parent_div);
+}
+function submitUrlEntry(page_id){
+    const parent_div = document.getElementById(page_id);
+    const url_input = parent_div.querySelector('.url').querySelector('input').value;
+
+    fetch('index.php?menu_edit=edit_url_entry', {
+        method: 'POST',
+        headers: {
+        'Content-Type': 'application/json'
+        },
+        body: JSON.stringify({ id: page_id, url_input: url_input })
+    })
+    .then(response => response.json())
+    .then(data => {
+        if(data.success){
+            parent_div.querySelector('i').innerHTML = data.url_input; // MAJ <i>
+            findParentByTagName(document.getElementById('m_' + page_id), 'a').href = data.url_input; // MAJ menu
+            closeUrlEntry(page_id, parent_div);
+        }
+        else{
+            toastNotify("Erreur rencontrée par le serveur, changements non pris en compte");
+            console.error("Erreur rencontrée par le serveur, changements non pris en compte");
+        }
+    })
+    .catch(error => {
+        console.error('Erreur:', error);
+    });
+}
+function closeUrlEntry(page_id, parent_div){
+    parent_div.querySelector('i').classList.remove('hidden');
+    parent_div.querySelector('.url').querySelector('input').classList.add('hidden');
+    parent_div.querySelector('#edit-i' + page_id).classList.remove('hidden');
+    parent_div.querySelector('#delete-i' + page_id).querySelector('input[type=image]').classList.remove('hidden');
+    parent_div.querySelector('#cancel-i' + page_id).querySelector('button').classList.add('hidden');
+    parent_div.querySelector('#submit-i' + page_id).querySelector('input[type=submit]').classList.add('hidden');
+}*/
+
+/*function deleteUrlEntry(page_id){
         const selected_div = document.getElementById(page_id);
         console.log(selected_div.id);
-}
\ No newline at end of file
+}*/
+
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