[*] FO : added smarty cache on productscategory

modules/crossselling/crossselling.php

@@ -88,6 +88,7 @@ class CrossSelling extends Module
 			{			
 				Configuration::updateValue('CROSSSELLING_DISPLAY_PRICE', (int)Tools::getValue('displayPrice'));
 				Configuration::updateValue('CROSSSELLING_NBR', (int)Tools::getValue('productNbr'));
+				$this->_clearCache('crossselling.tpl');
 				$this->_html .= $this->displayConfirmation($this->l('Settings updated successfully'));
 			}
 		}

modules/productscategory/config.xml

@@ -2,7 +2,7 @@
 <module>
 	<name>productscategory</name>
 	<displayName><![CDATA[Products Category]]></displayName>
-            <version><![CDATA[1.3]]></version>
+	<version><![CDATA[1.4]]></version>
 	<description><![CDATA[Display products of the same category on the product page.]]></description>
 	<author><![CDATA[PrestaShop]]></author>
 	<tab><![CDATA[front_office_features]]></tab>

modules/productscategory/productscategory.php

@@ -34,7 +34,7 @@ class productsCategory extends Module
 	public function __construct()
  	{
  	 	$this->name = 'productscategory';
- 	 	$this->version = '1.3';
+ 	 	$this->version = '1.4';
 		$this->author = 'PrestaShop';
  	 	$this->tab = 'front_office_features';
 		$this->need_instance = 0;
@@ -50,16 +50,22 @@ class productsCategory extends Module
 
 	public function install()
 	{
-	 	if (!parent::install() OR !$this->registerHook('productfooter') OR !$this->registerHook('header') OR !Configuration::updateValue('PRODUCTSCATEGORY_DISPLAY_PRICE', 0))
-	 		return false;
-	 	return true;
+		Configuration::updateValue('PRODUCTSCATEGORY_DISPLAY_PRICE', 0);
+		$this->_clearCache('productscategory.tpl');
+	 	return (parent::install()
+			&& $this->registerHook('productfooter')
+			&& $this->registerHook('header')
+			&& $this->registerHook('addproduct')
+			&& $this->registerHook('updateproduct')
+			&& $this->registerHook('deleteproduct')
+		);
 	}
 	
 	public function uninstall()
 	{
-	 	if (!parent::uninstall() OR !Configuration::deleteByName('PRODUCTSCATEGORY_DISPLAY_PRICE'))
-	 		return false;
-	 	return true;
+		Configuration::deleteByName('PRODUCTSCATEGORY_DISPLAY_PRICE');
+		$this->_clearCache('productscategory.tpl');
+	 	return parent::uninstall();
 	}
 	
 	public function getContent()
@@ -70,6 +76,7 @@ class productsCategory extends Module
 		elseif (Tools::isSubmit('submitCross'))
 		{
 			Configuration::updateValue('PRODUCTSCATEGORY_DISPLAY_PRICE', Tools::getValue('displayPrice'));
+			$this->_clearCache('productscategory.tpl');
 			$this->_html .= $this->displayConfirmation($this->l('Settings updated successfully'));
 		}
 		$this->_html .= '
@@ -100,9 +107,13 @@ class productsCategory extends Module
 	
 	public function hookProductFooter($params)
 	{
-		$idProduct = (int)(Tools::getValue('id_product'));
-		$product = new Product((int)($idProduct));
+		$id_product = (int)$params['product']->id;
+		$product = $params['product'];
 		
+		$cache_id = 'productscategory|'.$id_product.'|'.(isset($params['category']->id_category) ? (int)$params['category']->id_category : $product->id_category_default);
+
+		if (!$this->isCached('productscategory.tpl', $this->getCacheId($cache_id)))
+		{
 			/* If the visitor has came to this product by a category, use this one */
 			if (isset($params['category']->id_category))
 				$category = $params['category'];
@@ -110,7 +121,7 @@ class productsCategory extends Module
 			else
 			{
 				if (isset($product->id_category_default) AND $product->id_category_default > 1)
-				$category = New Category((int)($product->id_category_default));
+					$category = new Category((int)$product->id_category_default);
 			}
 			
 			if (!Validate::isLoadedObject($category) OR !$category->active) 
@@ -125,7 +136,7 @@ class productsCategory extends Module
 			if (is_array($categoryProducts) AND sizeof($categoryProducts))
 			{
 				foreach ($categoryProducts AS $key => $categoryProduct)
-				if ($categoryProduct['id_product'] == $idProduct)
+					if ($categoryProduct['id_product'] == $id_product)
 					{
 						unset($categoryProducts[$key]);
 						break;
@@ -134,7 +145,7 @@ class productsCategory extends Module
 				$taxes = Product::getTaxCalculationMethod();
 				if (Configuration::get('PRODUCTSCATEGORY_DISPLAY_PRICE'))
 					foreach ($categoryProducts AS $key => $categoryProduct)
-					if ($categoryProduct['id_product'] != $idProduct)
+						if ($categoryProduct['id_product'] != $id_product)
 						{
 							if ($taxes == 0 OR $taxes == 2)
 								$categoryProducts[$key]['displayed_price'] = Product::getPriceStatic((int)$categoryProduct['id_product'], true, NULL, 2);
@@ -144,7 +155,7 @@ class productsCategory extends Module
 			
 				// Get positions
 				$middlePosition = round($sizeOfCategoryProducts / 2, 0);
-			$productPosition = $this->getCurrentProduct($categoryProducts, (int)$idProduct);
+				$productPosition = $this->getCurrentProduct($categoryProducts, (int)$id_product);
 			
 				// Flip middle product with current product
 				if ($productPosition)
@@ -167,8 +178,8 @@ class productsCategory extends Module
 				'categoryProducts' => $categoryProducts,
 				'middlePosition' => (int)$middlePosition,
 				'ProdDisplayPrice' => Configuration::get('PRODUCTSCATEGORY_DISPLAY_PRICE')));
-
-		return $this->display(__FILE__, 'productscategory.tpl');
+		}
+		return $this->display(__FILE__, 'productscategory.tpl', $this->getCacheId($cache_id));
 	}
 	
 	public function hookHeader($params)
@@ -177,4 +188,19 @@ class productsCategory extends Module
 		$this->context->controller->addJS($this->_path.'productscategory.js');
 		$this->context->controller->addJqueryPlugin('serialScroll');
 	}
+
+	public function hookAddProduct($params)
+	{
+		$this->_clearCache('productscategory.tpl');
+	}
+
+	public function hookUpdateProduct($params)
+	{
+		$this->_clearCache('productscategory.tpl');
+	}
+
+	public function hookDeleteProduct($params)
+	{
+		$this->_clearCache('productscategory.tpl');
+	}
 }

modules/productscategory/upgrade/install-1.4.php

@@ -0,0 +1,9 @@
+<?php
+
+if (!defined('_PS_VERSION_'))
+	exit;
+
+function upgrade_module_1_4($object)
+{
+	return ($object->registerHook('addproduct') && $object->registerHook('updateproduct') && $object->registerHook('deleteproduct'));
+}