From 675a8fe95baa1c3dda6873aa7f0ab87ea2aa5822 Mon Sep 17 00:00:00 2001 From: mDeflotte Date: Thu, 29 Sep 2011 07:07:06 +0000 Subject: [PATCH] //Use protected instead of private function --- controllers/front/OrderOpcController.php | 2 +- controllers/front/ProductController.php | 8 ++++---- controllers/front/StoresController.php | 8 ++++---- controllers/front/SupplierController.php | 4 ++-- 4 files changed, 11 insertions(+), 11 deletions(-) diff --git a/controllers/front/OrderOpcController.php b/controllers/front/OrderOpcController.php index 9a77612e5..c2898092d 100644 --- a/controllers/front/OrderOpcController.php +++ b/controllers/front/OrderOpcController.php @@ -51,7 +51,7 @@ class OrderOpcControllerCore extends ParentOrderController { $txtMessage = urldecode(Tools::getValue('message')); $this->_updateMessage($txtMessage); - if (sizeof($this->errors)) + if (sizeof($this->errors)) die('{"hasError" : true, "errors" : ["'.implode('\',\'', $this->errors).'"]}'); die(true); } diff --git a/controllers/front/ProductController.php b/controllers/front/ProductController.php index 7e0316931..9e6eb75b3 100644 --- a/controllers/front/ProductController.php +++ b/controllers/front/ProductController.php @@ -211,7 +211,7 @@ class ProductControllerCore extends FrontController /** * Assign price and tax to the template */ - private function assignPriceAndTax() + protected function assignPriceAndTax() { $id_customer = (isset($this->context->customer) ? (int)($this->context->customer->id) : 0); $group_reduction = (100 - Group::getReduction($id_customer)) / 100; @@ -252,7 +252,7 @@ class ProductControllerCore extends FrontController /** * Assign template vars related to images */ - private function assignImages() + protected function assignImages() { $images = $this->product->getImages((int)$this->context->cookie->id_lang); $productImages = array(); @@ -285,7 +285,7 @@ class ProductControllerCore extends FrontController /** * Assign template vars related to attribute groups and colors */ - private function assignAttributesGroups() + protected function assignAttributesGroups() { $colors = array(); $attributesGroups = $this->product->getAttributesGroups($this->context->language->id); // @todo (RM) should only get groups and not all declination ? @@ -447,7 +447,7 @@ class ProductControllerCore extends FrontController /** * Assign template vars related to category */ - private function assignCategory() + protected function assignCategory() { $category = false; if (isset($_SERVER['HTTP_REFERER']) diff --git a/controllers/front/StoresController.php b/controllers/front/StoresController.php index 7a21a1ee3..9a8fe0020 100644 --- a/controllers/front/StoresController.php +++ b/controllers/front/StoresController.php @@ -47,7 +47,7 @@ class StoresControllerCore extends FrontController /** * get formatted string address */ - private function processStoreAddress($store) + protected function processStoreAddress($store) { $ignore_field = array( 'firstname', @@ -87,7 +87,7 @@ class StoresControllerCore extends FrontController /** * Assign template vars for simplified stores */ - private function assignStoresSimplified() + protected function assignStoresSimplified() { $stores = Db::getInstance()->ExecuteS(' SELECT s.*, cl.name country, st.iso_code state @@ -110,7 +110,7 @@ class StoresControllerCore extends FrontController /** * Assign template vars for classical stores */ - private function assignStores() + protected function assignStores() { $distanceUnit = Configuration::get('PS_DISTANCE_UNIT'); if (!in_array($distanceUnit, array('km', 'mi'))) @@ -169,7 +169,7 @@ class StoresControllerCore extends FrontController /** * Display the Xml for showing the nodes in the google map */ - private function displayXmlMaps($stores) + protected function displayXmlMaps($stores) { $dom = new DOMDocument('1.0'); $node = $dom->createElement('markers'); diff --git a/controllers/front/SupplierController.php b/controllers/front/SupplierController.php index 5ec8cdda4..02c7a1b22 100644 --- a/controllers/front/SupplierController.php +++ b/controllers/front/SupplierController.php @@ -89,7 +89,7 @@ class SupplierControllerCore extends FrontController /** * Assign template vars if displaying one supplier */ - private function assignOne() + protected function assignOne() { $nbProducts = $this->supplier->getProducts($this->supplier->id, null, null, null, $this->orderBy, $this->orderWay, true); $this->pagination((int)$nbProducts); @@ -104,7 +104,7 @@ class SupplierControllerCore extends FrontController /** * Assign template vars if displaying the supplier list */ - private function assignAll() + protected function assignAll() { if (Configuration::get('PS_DISPLAY_SUPPLIERS')) {