dinislage/PrestaShop
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

// Fixed misuse of count() (mexique1)

Browse Source 
git-svn-id: http://dev.prestashop.com/svn/v1/branches/1.5.x@14302 b9a71923-0436-4b27-9f14-aed3839534dd
This commit is contained in:
bMancone
2012-03-27 15:15:56 +00:00
parent 427d139207
commit f45ef0ca7f
1 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
modules/blocktopmenu/blocktopmenu.php
+6 -6
View File
@@ -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;