fix menu when child > 0

core/lib/Thelia/Core/Template/Loop/CategoryTree.php

@@ -84,10 +84,14 @@ class CategoryTree extends BaseI18nLoop
             $loopResultRow = new LoopResultRow();
 
             $loopResultRow
-                ->set("ID", $result->getId())->set("TITLE", $result->getVirtualColumn('i18n_TITLE'))
+                ->set("ID", $result->getId())
-                ->set("PARENT", $result->getParent())->set("URL", $result->getUrl($locale))
+                ->set("TITLE", $result->getVirtualColumn('i18n_TITLE'))
-                ->set("VISIBLE", $result->getVisible() ? "1" : "0")->set("LEVEL", $level)
+                ->set("PARENT", $result->getParent())
-                ->set('CHILD_COUNT', $result->countChild())->set('PREV_LEVEL', $previousLevel)
+                ->set("URL", $result->getUrl($locale))
+                ->set("VISIBLE", $result->getVisible() ? "1" : "0")
+                ->set("LEVEL", $level)
+                ->set('CHILD_COUNT', $result->countChild())
+                ->set('PREV_LEVEL', $previousLevel)
             ;
 
             $loopResult->addRow($loopResultRow);

templates/default/category.html

@@ -46,6 +46,4 @@
         </aside>
 
     </div>
-
-    </div><!-- /.container -->
 {/block}

templates/default/includes/menu.html

@@ -3,25 +3,28 @@
     <div class="block-content">
         <nav class="nav-categories">
             <ul id="category" class="accordion">
-                {assign "previousLevel" 0}
+            {assign "previousLevel" 0}
-                {loop name="cat-parent" type="category-tree" category="0"}
+            {loop name="cat-parent" type="category-tree" category="0"}
 
-                {for $foo=1 to $previousLevel-$LEVEL}
+            {for $foo=1 to $previousLevel-$LEVEL}
-            </ul>
+                    </ul>
-            </li>
+                </li>
             {/for}
 
             {if $CHILD_COUNT > 0 }
-            <li><a class="accordion-toggle" data-toggle="collapse" data-parent="#category" href="{$URL}#collapse{$ID}">{$TITLE} <span class="amount">({$CHILD_COUNT})</span></a>
+            <li>
-
+                <a class="accordion-toggle" data-toggle="collapse" data-parent="#category" href="{$URL}#collapse{$ID}">{$TITLE} <span class="amount">({$CHILD_COUNT})</span></a>
                 <ul id="collapse{$ID}" class="collapse">
-                    {else}
+            {else}
-
+                <li><a href="{$URL}">{$TITLE} <span class="amount">(0)</span></a></li>
-                    <li><a href="{$URL}">{$TITLE} <span class="amount">(0)</span></a></li>
+            {/if}
-                    {/if}
+                {assign "previousLevel" $LEVEL}
-                    {assign "previousLevel" $LEVEL}
+            {/loop}
-                    {/loop}
+            {for $i=$previousLevel to 1 step -1}
                 </ul>
+            </li>
+            {/for}
+            </ul>
         </nav>
     </div>
-</section>
+</section>