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

Fix variable context

Browse Source 
Avoid attribution to the variable $newProducts inside the if. It caused the code to potentially misbehave throwing errors of:
"variable $newProducts not defined".
This commit is contained in:
dlage
2013-09-01 22:47:41 +01:00
parent f7fefe789d
commit 5f1e08e7e2
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
modules/blocknewproducts/blocknewproducts.php
+5 -2
View File
@@ -111,7 +111,10 @@ class BlockNewProducts extends Module
{ {
if (!$this->isCached('blocknewproducts.tpl', $this->getCacheId())) if (!$this->isCached('blocknewproducts.tpl', $this->getCacheId()))
{ {
if (!Configuration::get('PS_BLOCK_NEWPRODUCTS_DISPLAY') && !($newProducts = Product::getNewProducts((int)$params['cookie']->id_lang, 0, (int)Configuration::get('NEW_PRODUCTS_NBR')))) if (!Configuration::get('PS_BLOCK_NEWPRODUCTS_DISPLAY'))
return;
$newProducts = Product::getNewProducts((int) $params['cookie']->id_lang, 0, (int) Configuration::get('NEW_PRODUCTS_NBR'));
if (!$newProducts)
return; return;
$this->smarty->assign(array( $this->smarty->assign(array(
@@ -151,4 +154,4 @@ class BlockNewProducts extends Module
{ {
$this->_clearCache('blocknewproducts.tpl'); $this->_clearCache('blocknewproducts.tpl');
} }
} }