menu et fil d'ariane gèrent le champ reachable dans page

2 files changed, 31 insertions, 30 deletions

diff --git a/src/view/BreadcrumbBuilder.php b/src/view/BreadcrumbBuilder.php
index f1fdddf..77f4c4c 100644
--- a/src/view/BreadcrumbBuilder.php
+++ b/src/view/BreadcrumbBuilder.php
@@ -7,10 +7,10 @@ class BreadcrumbBuilder extends AbstractBuilder
 {
     public function __construct(Node $node)
     {
-        $this->html = $this->breadcrumbHTML(false);
+        $this->html = $this->breadcrumbHTML();
     }
 
-    private function breadcrumbHTML(bool $links = false): string
+    private function breadcrumbHTML(): string
     {
         $asset = 'assets/home.svg'; // => BDD?
         $breadcrumb_array = Director::$page_path->getArray(); // tableau de Page
@@ -23,21 +23,16 @@ class BreadcrumbBuilder extends AbstractBuilder
             $html .= '<nav class="breadcrumb" aria-label="Breadcrumb">' . "\n";
             $html .= '<a href="' . new URL . '"><img src="' . $asset . '"></a><span class="arrow"> →</span>' . "\n";
 
-            // partie intermédiaire
+            // partie intermédiaire (pas de lien sur le dernier élément)
             for($i = 0; $i < ($nb_of_entries - 1); $i++)
             {
                 // liens optionnels
-                if($links)
+                if($breadcrumb_array[$i]->isReachable())
                 {
-                    $html .= '<a href="';
-                    for($j = 1; $j < $i; $j++) // chemin sans la fin
-                    {
-                        $html .= new URL(['page' => $breadcrumb_array[$i]->getPagePath()]); 
-                    }
-                    $html .= '">';
+                    $html .= '<a href="' . new URL(['page' => $breadcrumb_array[$i]->getPagePath()]) . '">';
                 }
                 $html .= '<span>' . $breadcrumb_array[$i]->getPageName() . '</span>';
-                if($links)
+                if($breadcrumb_array[$i]->isReachable())
                 {
                     $html .= '</a>';
                 }
diff --git a/src/view/NavBuilder.php b/src/view/NavBuilder.php
index e7254b1..2ef6bc8 100644
--- a/src/view/NavBuilder.php
+++ b/src/view/NavBuilder.php
@@ -22,20 +22,12 @@ class NavBuilder extends AbstractBuilder
 
         foreach($nav_data->getChildren() as $data)
         {
-            $class = '';
+            $li_class = '';
             if(isset($current[$level]) && $data->getEndOfPath() === $current[$level]->getEndOfPath()){
-                $class = ' current';
+                $li_class = 'current';
             }
-            
-            if(count($data->getChildren()) > 0) // titre de catégorie
-            {
-                $nav_html .= '<li class="drop-down'. $class . '"><p>' . $data->getPageName() . '</p><ul class="sub-menu">' . "\n";
-                $level++;
-                $nav_html .= $this->navMainHTML($data, $current);
-                $level--;
-                $nav_html .= '</ul></li>' . "\n";
-            }
-            else
+
+            if($data->isReachable())
             {
                 $target = '';
                 if(str_starts_with($data->getEndOfPath(), 'http')) // lien vers autre site
@@ -47,14 +39,28 @@ class NavBuilder extends AbstractBuilder
                 {
                     $link = new URL(['page' => $data->getPagePath()]); // $link = objet
                 }
-                /*else
-                {
-                    echo "else page d'accueil" . '<br>';
-                    $link = new URL; // page d'accueil
-                }*/
-
-                $nav_html .= '<a href="' . $link . '"' . $target . '><li class="'. $class . '"><p>' . $data->getPageName() . '</p></li></a>' . "\n";
+                $nav_html .= '<a href="' . $link . '"' . $target . '>';
+            }
+            else{
+                $nav_html .= '<a>';
             }
+            
+            if(count($data->getChildren()) > 0) // titre de catégorie
+            {
+                $li_class = $data->getParent() == null ? 'drop-down' : 'drop-right';
+                
+                $nav_html .= '<li class="'. $li_class . '"><p>' . $data->getPageName() . '</p><ul class="sub-menu">' . "\n";
+                $level++;
+                $nav_html .= $this->navMainHTML($data, $current);
+                $level--;
+                $nav_html .= '</ul></li>' . "\n";
+            }
+            else
+            {
+                $nav_html .= '<li class="'. $li_class . '"><p>' . $data->getPageName() . '</p></li>' . "\n";
+            }
+            
+            $nav_html .= "</a>\n";
         }
         return $nav_html;
     }