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

3 files changed, 25 insertions, 18 deletions

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