From 7db8ef441ecb575c563f36e23d32b1fd101bb9e6 Mon Sep 17 00:00:00 2001 From: fBrignoli Date: Wed, 21 Sep 2011 15:43:12 +0000 Subject: [PATCH] // Tax::initializeAddress =>Address::initialize --- admin-dev/tabs/AdminProducts.php | 86 ++++++++++++++++---------------- classes/PaymentModule.php | 8 +-- classes/Product.php | 3 +- classes/tax/Tax.php | 33 ++---------- 4 files changed, 52 insertions(+), 78 deletions(-) diff --git a/admin-dev/tabs/AdminProducts.php b/admin-dev/tabs/AdminProducts.php index 0df55c2d1..6233abbf5 100644 --- a/admin-dev/tabs/AdminProducts.php +++ b/admin-dev/tabs/AdminProducts.php @@ -34,7 +34,7 @@ class AdminProducts extends AdminTab public function __construct() { - + $this->table = 'product'; $this->className = 'Product'; $this->lang = true; @@ -502,7 +502,7 @@ class AdminProducts extends AdminTab Tools::getValue('attribute_upc'), Tools::getValue('minimal_quantity'), Tools::getValue('available_date')); - + if ($id_reason = (int)Tools::getValue('id_mvt_reason') AND (int)Tools::getValue('attribute_mvt_quantity') > 0 AND $id_reason > 0) { if (!$product->addStockMvt(Tools::getValue('attribute_mvt_quantity'), $id_reason, $id_product_attribute, NULL, $this->context->employee->id)) @@ -859,17 +859,17 @@ class AdminProducts extends AdminTab $shops = false; if (Shop::isMultiShopActivated()) $shops = Shop::getShops(); - $obj = new Product((int)Tools::getValue('id_product')); + $obj = new Product((int)Tools::getValue('id_product')); $countImages = (int)Db::getInstance()->getValue('SELECT COUNT(*) FROM '._DB_PREFIX_.'image WHERE id_product = '.(int)$obj->id); $images = Image::getImages($this->context->language->id, $obj->id); $imagesTotal = Image::getImagesTotal($obj->id); $html = $this->getLineTableImage($result['success'], $imagesTotal + 1, $this->token, $shops); - die(Tools::jsonEncode(array("success" => $html))); + die(Tools::jsonEncode(array("success" => $html))); } else die(Tools::jsonEncode($result)); } - + if (Tools::getValue('updateProductImageShopAsso')) { if ($id_image = (int)Tools::getValue('id_image') AND $id_shop = (int)Tools::getValue('id_shop')) @@ -878,7 +878,7 @@ class AdminProducts extends AdminTab else die(Db::getInstance()->Execute('DELETE FROM '._DB_PREFIX_.'image_shop WHERE `id_image`='.(int)$id_image.' AND `id_shop`='.(int)$id_shop)); } - + if (Tools::getValue('deleteImage')) { $image = new Image((int)Tools::getValue('id_image')); @@ -1491,7 +1491,7 @@ class AdminProducts extends AdminTab /* Tabs */ $this->displayFormInformations($obj, $currency); $this->displayFormImages($obj, $this->token); - if (Combination::isFeatureActive()) + if (Combination::isFeatureActive()) $countAttributes = (int)Db::getInstance()->getValue('SELECT COUNT(*) FROM '._DB_PREFIX_.'product_attribute WHERE id_product = '.(int)$obj->id); $countAttachments = (int)Db::getInstance()->getValue('SELECT COUNT(*) FROM '._DB_PREFIX_.'product_attachment WHERE id_product = '.(int)$obj->id); if ($obj->id) @@ -1593,7 +1593,7 @@ class AdminProducts extends AdminTab $specificPrices = SpecificPrice::getByProductId((int)($obj->id)); $specificPricePriorities = SpecificPrice::getPriority((int)($obj->id)); - $taxRate = $obj->getTaxesRate(Tax::initializeAddress()); + $taxRate = $obj->getTaxesRate(Address::initialize()); $tmp = array(); foreach ($shops as $shop) @@ -2643,7 +2643,7 @@ class AdminProducts extends AdminTab '; // Check if product has combination, to display the available date only for the product or for each combination - if (Combination::isFeatureActive()) + if (Combination::isFeatureActive()) $countAttributes = (int)Db::getInstance()->getValue('SELECT COUNT(*) FROM '._DB_PREFIX_.'product_attribute WHERE id_product = '.(int)$obj->id); if (isset($countAttributes) && $countAttributes == 0) @@ -2880,14 +2880,14 @@ class AdminProducts extends AdminTab

'; - echo ' + echo ' - +

'.$this->l('Format:').' JPG, GIF, PNG. '.$this->l('Filesize:').' '.($this->maxImageSize / 1000).''.$this->l('Kb max.').'

@@ -3162,7 +3162,7 @@ class AdminProducts extends AdminTab '.(Tools::getValue('id_image') ? '' : '').' -
+
@@ -3171,7 +3171,7 @@ class AdminProducts extends AdminTab var come_from = \''.$this->table.'\'; var alternate = \''.($this->_orderWay == 'DESC' ? '1' : '0' ).'\'; - + @@ -3185,12 +3185,12 @@ class AdminProducts extends AdminTab $(\'.image_shop\').change(function() { $.post("ajax-tab.php", { - updateProductImageShopAsso: 1, - id_image:$(this).attr("name"), - id_shop: $(this).val(), + updateProductImageShopAsso: 1, + id_image:$(this).attr("name"), + id_shop: $(this).val(), active:$(this).attr("checked"), - id_product : "'.(int)Tools::getValue('id_product').'", - id_category : "'.(int)$this->_category->id.'", + id_product : "'.(int)Tools::getValue('id_product').'", + id_category : "'.(int)$this->_category->id.'", token : "'.Tools::getAdminTokenLite('AdminCatalog').'", tab : "AdminCatalog", updateproduct : 1, @@ -3208,7 +3208,7 @@ class AdminProducts extends AdminTab foreach ($images AS $k => $image) echo $this->getLineTableImage($image, $imagesTotal, $token, $shops); - + echo '
@@ -3240,7 +3240,7 @@ class AdminProducts extends AdminTab echo ' '; } - + public function getLineTableImage($image, $imagesTotal, $token, $shops) { if (Shop::isMultiShopActivated()) @@ -3273,7 +3273,7 @@ class AdminProducts extends AdminTab '; return $html; } - + public function initCombinationImagesJS() { if (!($obj = $this->loadObject(true))) @@ -3299,7 +3299,7 @@ class AdminProducts extends AdminTab $this->displayWarning($this->l('This feature has been disabled, you can active this feature at this page:').' '.$this->l('Performances').''); return; } - + $attributeJs = array(); $attributes = Attribute::getAttributes($this->context->language->id, true); foreach ($attributes AS $k => $attribute) @@ -3624,7 +3624,7 @@ class AdminProducts extends AdminTab $this->displayWarning($this->l('This feature has been disabled, you can active this feature at this page:').' '.$this->l('Performances').''); return; } - + parent::displayForm(); if ($obj->id) diff --git a/classes/PaymentModule.php b/classes/PaymentModule.php index 233dd1812..435fd2743 100644 --- a/classes/PaymentModule.php +++ b/classes/PaymentModule.php @@ -216,11 +216,11 @@ abstract class PaymentModuleCore extends Module $tax_calculator = new TaxCalculator(); if (!Tax::excludeTaxeOption()) { - $address = Tax::initializeAddress($vat_address->id); + $address = Address::initialize($vat_address->id); $id_tax_rules = (int)Product::getIdTaxRulesGroupByIdProduct((int)$product['id_product']); - + $tax_manager = TaxManagerFactory::getManager($vat_address, $id_tax_rules); - $tax_calculator = $tax_manager->getTaxCalculator(); + $tax_calculator = $tax_manager->getTaxCalculator(); } $ecotaxTaxRate = 0; @@ -298,7 +298,7 @@ abstract class PaymentModuleCore extends Module OrderDetail::saveTaxCalculatorStatic($db->Insert_ID(), $tax_calculator); unset($tax_calculator); - + // Insert discounts from cart into order_discount table $discounts = $cart->getDiscounts(); $discountsList = ''; diff --git a/classes/Product.php b/classes/Product.php index cc1933369..e9c9eda2a 100644 --- a/classes/Product.php +++ b/classes/Product.php @@ -3308,7 +3308,7 @@ class ProductCore extends ObjectModel public function getTaxesRate(Address $address = NULL) { if (!$address OR !$address->id_country) - $address = Tax::initializeAddress(); + $address = Address::initialize(); $tax_manager = TaxManagerFactory::getManager($address, $this->id_tax_rules_group); $tax_calculator = $tax_manager->getTaxCalculator(); @@ -3596,3 +3596,4 @@ class ProductCore extends ObjectModel return isset($row['reference']); } } + diff --git a/classes/tax/Tax.php b/classes/tax/Tax.php index e2e40b1ff..09e6fd860 100644 --- a/classes/tax/Tax.php +++ b/classes/tax/Tax.php @@ -199,7 +199,7 @@ class TaxCore extends ObjectModel */ public static function getProductEcotaxRate($id_address = NULL) { - $address = Tax::initializeAddress($id_address); + $address = Address::initialize($id_address); $tax_manager = TaxManagerFactory::getManager($address, (int)Configuration::get('PS_ECOTAX_TAX_RULES_GROUP_ID')); $tax_calculator = $tax_manager->getTaxCalculator(); @@ -215,7 +215,7 @@ class TaxCore extends ObjectModel */ public static function getCarrierTaxRate($id_carrier, $id_address = NULL) { - $address = Tax::initializeAddress($id_address); + $address = Address::initialize($id_address); $id_tax_rules = (int)Carrier::getIdTaxRulesGroupByIdCarrier((int)$id_carrier); $tax_manager = TaxManagerFactory::getManager($address, $id_tax_rules); @@ -224,33 +224,6 @@ class TaxCore extends ObjectModel return $tax_calculator->getTotalRate(); } - /** - * Initiliaze an address corresponding to the id address if any or to the - * default shop configuration - * - * @param int $id_address - * @return Address address - */ - public static function initializeAddress($id_address = NULL) - { - // set the default address - $address = new Address(); - $address->id_country = (int)Context::getContext()->country->id; - $address->id_state = 0; - $address->postcode = 0; - - // if an id_address has been specified retrieve the address - if ($id_address) - { - $address = new Address((int)$id_address); - - if (!Validate::isLoadedObject($address)) - throw new Exception('Invalid address'); - } - - return $address; - } - /** * Return the product tax rate using the tax rules system * @@ -282,7 +255,7 @@ class TaxCore extends ObjectModel */ public static function getProductTaxRate($id_product, $id_address = NULL) { - $address = Tax::initializeAddress($id_address); + $address = Address::initialize($id_address); $id_tax_rules = (int)Product::getIdTaxRulesGroupByIdProduct($id_product); $tax_manager = TaxManagerFactory::getManager($address, $id_tax_rules);