// remove $noTabLink from compatibility tabs

admin-dev/functions.php

@@ -453,7 +453,6 @@ function runAdminTab($tab, $ajaxMode = false)
 	// $tab = $_REQUEST['tab'];
 	if ($adminObj = checkingTab($tab))
 	{
-		$noTabLink = $adminObj->noTabLink;
 		// init is different for new tabs (AdminController) and old tabs (AdminTab)
 		if ($adminObj instanceof AdminController)
 		{

admin-dev/header.inc.php

@@ -311,10 +311,6 @@ foreach ($tabs AS $t)
 			<ul class="submenu">';
 		$subTabs = Tab::getTabs(Context::getContext()->language->id, (int)$t['id_tab']);
 
-		// @todo need a better way than using noTabLink property, keeping the fact to avoid db modification
-		if (!in_array($t['class_name'], $noTabLink))
-			array_unshift($subTabs, $t);
-
 		foreach ($subTabs AS $t2)
 			if (checkTabRights($t2['id_tab']) === true AND (bool)$t2['active'])
 				echo '<li><a href="index.php?controller='.$t2['class_name'].'&token='.Tools::getAdminTokenLite($t2['class_name']).'">'.$t2['name'].'</a></li>';

classes/AdminTab.php

@@ -190,8 +190,6 @@ abstract class AdminTabCore
 	public $ignore_sleep = false;
 
 	public static $tabParenting = array(
-//		'AdminProducts' => 'AdminCatalog',
-//		'AdminCategories' => 'AdminCatalog',
 		'AdminCms' => 'AdminCmsContent',
 		'AdminCmsCategories' => 'AdminCmsContent',
 		'AdminOrdersStates' => 'AdminStatuses',
@@ -202,9 +200,6 @@ abstract class AdminTabCore
 		'AdminStatsTab' => 'AdminStats'
 	);
 
-	/** @var array noTabLink array of admintabs with no content */
-	public $noTabLink = array('AdminCatalog', 'AdminPriceRule', 'AdminTools', 'AdminStock', 'AdminAdmin');
-
 	public function __construct()
 	{
 		$this->context = Context::getContext();