From ac1c1d80fb3a5beffc4a351fd45aa984f92af2ea Mon Sep 17 00:00:00 2001
From: Thibaud Chauviere <thibaud.chauviere@prestashop.com>
Date: Fri, 26 Jun 2015 14:11:07 +0200
Subject: [PATCH] [*] MO : Blockopmenu can now manage hidden categories without
 displaying them in the front office

---
 blocktopmenu.php | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/blocktopmenu.php b/blocktopmenu.php
index 128066d..10c58a4 100644
--- a/blocktopmenu.php
+++ b/blocktopmenu.php
@@ -468,6 +468,8 @@ class Blocktopmenu extends Module
 		$id_lang = (int)$this->context->language->id;
 		$id_shop = (int)Shop::getContextShopID();
 
+		//var_dump($menu_items);die('end');
+
 		foreach ($menu_items as $item)
 		{
 			if (!$item)
@@ -479,7 +481,7 @@ class Blocktopmenu extends Module
 			switch (substr($item, 0, strlen($value[1])))
 			{
 				case 'CAT':
-					$this->_menu .= $this->generateCategoriesMenu(Category::getNestedCategories($id, $id_lang, true, $this->user_groups));
+					$this->_menu .= $this->generateCategoriesMenu(Category::getNestedCategories($id, $id_lang, false, $this->user_groups));
 					break;
 
 				case 'PRD':
@@ -609,6 +611,11 @@ class Blocktopmenu extends Module
 			else
 				$link = $this->context->link->getPageLink('index');
 
+			/* Whenever a category is not active we shouldnt display it to customer */
+			if ((bool)$cat->active === false) {
+				break;
+			}
+
 			$html .= '<li'.(($this->page_name == 'category'
 				&& (int)Tools::getValue('id_category') == (int)$category['id_category']) ? ' class="sfHoverForce"' : '').'>';
 			$html .= '<a href="'.$link.'" title="'.$category['name'].'">'.$category['name'].'</a>';
@@ -1161,7 +1168,7 @@ class Blocktopmenu extends Module
 		$shops_to_get = Shop::getContextListShopID();
 
 		foreach ($shops_to_get as $shop_id)
-			$html .= $this->generateCategoriesOption($this->customGetNestedCategories($shop_id, null, (int)$this->context->language->id, true), $items);
+			$html .= $this->generateCategoriesOption($this->customGetNestedCategories($shop_id, null, (int)$this->context->language->id, false), $items);
 		$html .= '</optgroup>';
 
 		// BEGIN Shops
@@ -1206,7 +1213,7 @@ class Blocktopmenu extends Module
 	}
 
 
-	public function customGetNestedCategories($shop_id, $root_category = null, $id_lang = false, $active = true, $groups = null, $use_shop_restriction = true, $sql_filter = '', $sql_sort = '', $sql_limit = '')
+	public function customGetNestedCategories($shop_id, $root_category = null, $id_lang = false, $active = false, $groups = null, $use_shop_restriction = true, $sql_filter = '', $sql_sort = '', $sql_limit = '')
 	{
 		if (isset($root_category) && !Validate::isInt($root_category))
 			die(Tools::displayError());