// Fixed misuse of count() (mexique1)
This commit is contained in:
1 parent
82389a10b9
commit
69834966a5
1 file changed
+6
-6
@@ -379,37 +379,37 @@ class Blocktopmenu extends Module
|
|||||||
{
|
{
|
||||||
case 'CAT':
|
case 'CAT':
|
||||||
$category = new Category($id, $id_lang);
|
$category = new Category($id, $id_lang);
|
||||||
if (!is_null($category->id))
|
if (Validate::isLoadedObject($category))
|
||||||
$this->_html .= '<option value="CAT'.$id.'">'.$category->name.'</option>'.PHP_EOL;
|
$this->_html .= '<option value="CAT'.$id.'">'.$category->name.'</option>'.PHP_EOL;
|
||||||
break;
|
break;
|
||||||
|
|
||||||
case 'PRD':
|
case 'PRD':
|
||||||
$product = new Product($id, true, $id_lang);
|
$product = new Product($id, true, $id_lang);
|
||||||
if (!is_null($product->id))
|
if (Validate::isLoadedObject($product))
|
||||||
$this->_html .= '<option value="PRD'.$id.'">'.$product->name.'</option>'.PHP_EOL;
|
$this->_html .= '<option value="PRD'.$id.'">'.$product->name.'</option>'.PHP_EOL;
|
||||||
break;
|
break;
|
||||||
|
|
||||||
case 'CMS':
|
case 'CMS':
|
||||||
$cms = new CMS($id, $id_lang);
|
$cms = new CMS($id, $id_lang);
|
||||||
if (count($cms))
|
if (Validate::isLoadedObject($cms))
|
||||||
$this->_html .= '<option value="CMS'.$id.'">'.$cms->meta_title.'</option>'.PHP_EOL;
|
$this->_html .= '<option value="CMS'.$id.'">'.$cms->meta_title.'</option>'.PHP_EOL;
|
||||||
break;
|
break;
|
||||||
|
|
||||||
case 'CMS_CAT':
|
case 'CMS_CAT':
|
||||||
$category = new CMSCategory($id, $id_lang);
|
$category = new CMSCategory($id, $id_lang);
|
||||||
if (count($category))
|
if (Validate::isLoadedObject($category))
|
||||||
$this->_html .= '<option value="CMS_CAT'.$id.'">'.$category->name.'</option>'.PHP_EOL;
|
$this->_html .= '<option value="CMS_CAT'.$id.'">'.$category->name.'</option>'.PHP_EOL;
|
||||||
break;
|
break;
|
||||||
|
|
||||||
case 'MAN':
|
case 'MAN':
|
||||||
$manufacturer = new Manufacturer($id, $id_lang);
|
$manufacturer = new Manufacturer($id, $id_lang);
|
||||||
if (!is_null($manufacturer->id))
|
if (Validate::isLoadedObject($manufacturer))
|
||||||
$this->_html .= '<option value="MAN'.$id.'">'.$manufacturer->name.'</option>'.PHP_EOL;
|
$this->_html .= '<option value="MAN'.$id.'">'.$manufacturer->name.'</option>'.PHP_EOL;
|
||||||
break;
|
break;
|
||||||
|
|
||||||
case 'SUP':
|
case 'SUP':
|
||||||
$supplier = new Supplier($id, $id_lang);
|
$supplier = new Supplier($id, $id_lang);
|
||||||
if (!is_null($supplier->id))
|
if (Validate::isLoadedObject($supplier))
|
||||||
$this->_html .= '<option value="SUP'.$id.'">'.$supplier->name.'</option>'.PHP_EOL;
|
$this->_html .= '<option value="SUP'.$id.'">'.$supplier->name.'</option>'.PHP_EOL;
|
||||||
break;
|
break;
|
||||||
|
|
||||||
|
|||||||
Reference in new issue
Block a user