TheCoreDev/sterivein
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #545 from Yochima/yochima-branch

Browse Source 
Use now the dispatcher to delete feature_product when we change the temp...
fixed : elseloop executed even if loop is empty
This commit is contained in:
Julien
2014-07-28 11:42:53 +02:00
parent efd1d871fd 86b3d12e99
commit 6963754d62
2 changed files with 15 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
core/lib/Thelia/Action/Product.php
View File

@@ -269,7 +269,14 @@ class Product extends BaseAction implements EventSubscriberInterface
$product = $event->getProduct();
// Delete all product feature relations
FeatureProductQuery::create()->filterByProduct($product)->delete($con);
if(null !== $featureProducts = FeatureProductQuery::create()->findByProductId($product->getId())){
/** @var \Thelia\Model\FeatureProduct $featureProduct */
foreach($featureProducts as $featureProduct){
$eventDelete = new FeatureProductDeleteEvent($product->getId(), $featureProduct->getFeatureId());
$event->getDispatcher()->dispatch(TheliaEvents::PRODUCT_FEATURE_DELETE_VALUE, $eventDelete);
}
}
// Delete all product attributes sale elements
ProductSaleElementsQuery::create()->filterByProduct($product)->delete($con);

11
core/lib/Thelia/Core/Template/Smarty/Plugins/TheliaLoop.php
View File

@@ -215,12 +215,15 @@ class TheliaLoop extends AbstractSmartyPlugin
*/
public function theliaElseloop($params, $content, /** @noinspection PhpUnusedParameterInspection */ $template, &$repeat)
{
// When encountering close tag, check if loop has results.
if ($repeat === false) {
return $this->checkEmptyLoop($params) ? $content : '';
//Block the smarty interpretation in the elseloop
if ($content === null) {
if ( ! $this->checkEmptyLoop($params)){
$repeat = false;
return '';
}
}
return '';
return $content;
}
/**