Skip to content

Commit

Permalink
Reuse the parent class code + indentation fix
Browse files Browse the repository at this point in the history
  • Loading branch information
JoryHogeveen authored and NKoonen committed Sep 29, 2020
1 parent 2698f0e commit 11e0e47
Showing 1 changed file with 24 additions and 105 deletions.
129 changes: 24 additions & 105 deletions override/classes/tax/TaxRulesTaxManager.php
Original file line number Diff line number Diff line change
Expand Up @@ -30,109 +30,28 @@
*/
class TaxRulesTaxManager extends TaxRulesTaxManagerCore
{
public $address;
public $type;
public $tax_calculator;

/**
* @var \PrestaShop\PrestaShop\Core\ConfigurationInterface
*/
private $configurationManager;

/**
* @param Address $address
* @param mixed $type An additional parameter for the tax manager (ex: tax rules id for TaxRuleTaxManager)
*/
public function __construct(Address $address, $type, PrestaShop\PrestaShop\Core\ConfigurationInterface $configurationManager = null)
{
if ($configurationManager === null) {
$this->configurationManager = ServiceLocator::get('\\PrestaShop\\PrestaShop\\Core\\ConfigurationInterface');
} else {
$this->configurationManager = $configurationManager;
}

$this->address = $address;
$this->type = $type;
}

/**
* Returns true if this tax manager is available for this address.
*
* @return bool
*/
public static function isAvailableForThisAddress(Address $address)
{
return true; // default manager, available for all addresses
}

/**
* Return the tax calculator associated to this address.
*
* @return TaxCalculator
*/
public function getTaxCalculator()
{
static $tax_enabled = null;

if (isset($this->tax_calculator)) {
return $this->tax_calculator;
}

if ($tax_enabled === null) {
#The check if customer is in the Tax free group
$tax_enabled = $this->configurationManager->get('PS_TAX') && !in_array(Configuration::get('VATCHECKER_NO_TAX_GROUP'), Customer::getGroupsStatic($this->address->id_customer));
}

if (!$tax_enabled) {
return new TaxCalculator(array());
}

$taxes = array();
$postcode = 0;

if (!empty($this->address->postcode)) {
$postcode = $this->address->postcode;
}

$cache_id = (int) $this->address->id_country . '-' . (int) $this->address->id_state . '-' . $postcode . '-' . (int) $this->type;

if (!Cache::isStored($cache_id)) {
$rows = Db::getInstance()->executeS('
SELECT tr.*
FROM `' . _DB_PREFIX_ . 'tax_rule` tr
JOIN `' . _DB_PREFIX_ . 'tax_rules_group` trg ON (tr.`id_tax_rules_group` = trg.`id_tax_rules_group`)
WHERE trg.`active` = 1
AND tr.`id_country` = ' . (int) $this->address->id_country . '
AND tr.`id_tax_rules_group` = ' . (int) $this->type . '
AND tr.`id_state` IN (0, ' . (int) $this->address->id_state . ')
AND (\'' . pSQL($postcode) . '\' BETWEEN tr.`zipcode_from` AND tr.`zipcode_to`
OR (tr.`zipcode_to` = 0 AND tr.`zipcode_from` IN(0, \'' . pSQL($postcode) . '\')))
ORDER BY tr.`zipcode_from` DESC, tr.`zipcode_to` DESC, tr.`id_state` DESC, tr.`id_country` DESC');

$behavior = 0;
$first_row = true;

foreach ($rows as $row) {
$tax = new Tax((int) $row['id_tax']);

$taxes[] = $tax;

// the applied behavior correspond to the most specific rules
if ($first_row) {
$behavior = $row['behavior'];
$first_row = false;
}

if ($row['behavior'] == 0) {
break;
}
}
$result = new TaxCalculator($taxes, $behavior);
Cache::store($cache_id, $result);

return $result;
}

return Cache::retrieve($cache_id);
}
/**
* Return the tax calculator associated to this address.
*
* @return TaxCalculator
*/
public function getTaxCalculator()
{
static $tax_enabled = null;

if (isset($this->tax_calculator)) {
return $this->tax_calculator;
}

if ($tax_enabled === null) {
#The check if customer is in the Tax free group
$tax_enabled = Configuration::get('PS_TAX') && !in_array(Configuration::get('VATCHECKER_NO_TAX_GROUP'), Customer::getGroupsStatic($this->address->id_customer));
}

if (!$tax_enabled) {
return new TaxCalculator(array());
}

return parent::getTaxCalculator();
}
}

0 comments on commit 11e0e47

Please sign in to comment.