Tax rounding and catalog price rules

Question

On my store we have a product that costs £2.95 including tax (at current UK rate of 20%). The tax seems to calculate fine here.

However, I then have a Customer Group that gets 30% off, which should make the price £2.07. When I first load both the Category and Product page, it correctly shows £2.07 for this configurable product and logged in with the Customer Group.

But then when I choose one of the Simple Product options, the price displays as £2.06 - 1p difference. However, when the product is added to the cart, it goes in at the correct amount of £2.07 so the issue appears to be the rounding on the display price.

I'm using Magento 1.9.0.1 and wondered if other people have the same issue with the combination of tax and customer group discount?

Regards, Neil

Answer 1

Try overridding Mage_Core_Model_Store::roundPrice().
Change

public function roundPrice($price)
    {
        return round($price, 2);
    }

to

public function roundPrice($price)
    {
        return round($price, 4);
    }

Answer 2

There are some settings that could cause this.

Under System -> Configuration -> Taxes -> Calculation, change the "Apply after discount" setting.

Under System -> Configuration -> Sales -> Checkout Totals Sort Order, change them.

Also: do you input the prices excluding tax, and show them including tax (or the other way round?).

Can you confirm that your taxes have been set up as shown in the EU tax guide? GB is still in the EU, you know ;)

http://merch.docs.magento.com/ce/user_guide/Magento_Community_Edition_User_Guide.html#tax/configuration-eu.html

=== UPDATE ===

I am not sure what causes this problem, but maybe one of the following is a solution.

1. What happens if you change the product price from 2.95 to 2.9520 in the backend?

2. spConfig is filled by $this->getJsonConfig(). If you are not using a 3rd party extension for configurable products, this method can be found in Mage_Catalog_Block_Product_View_Type_Configurable. You could override this block in a new module and modify the function:

In your new module's config.xml file, add this section:

<blocks>
    <catalog>
        <rewrite>               
            <product_view_type_configurable>Namespace_Modulename_Block_Catalog_Product_View_Type_Configurable</product_view_type_configurable>
        </rewrite>
    </catalog>
</blocks>

Then create the file Namespace/Modulename/Block/Catalog/Product/View/Type/Configurable.php with these contents:

<?php
class Namespace_Modulename_Block_Catalog_Product_View_Type_Configurable extends Mage_Catalog_Block_Product_View_Type_Configurable {

    public function getJsonConfig() {
        // rest of method

        // this can be found pretty much near the end
        $config = array(
            'attributes' => $attributes,
            'template' => str_replace('%s', '#{price}', $store->getCurrentCurrency()->getOutputFormat()),
            // basePrice has been changed: added round()
            'basePrice' => $this->_registerJsPrice(round($this->_convertPrice($currentProduct->getFinalPrice(), 2))),
            'oldPrice' => $this->_registerJsPrice($this->_convertPrice($currentProduct->getPrice())),
            'productId' => $currentProduct->getId(),
            'chooseText' => Mage::helper('catalog')->__('Choose an Option...'),
            'taxConfig' => $taxConfig
        );


        // rest of method
    }
}

It would be even better to use an observer for the event catalog_product_type_configurable_price which is dispatched in the same method a few lines earlier, but I am not sure about the overall impact for the page if you change the price of the configurable product there.

Another solution would be to override the javascript class Product.Config and do the rounding manually. A pretty good explanation of how to override JS core classes can be found here:

https://stackoverflow.com/a/11946804/1130898

Answer 3

Download Magento 1.9.2 and try create the same situation.

If you have the same rounding error again, go to the admin page and look for a message under the menu that will say:

Warning tax configuration can result in rounding errors. 
Please see source for more details.
Click here to go to Tax configuration and change your settings.

If not, give the 'guys' from Magento a ping, you just found a error. And there happy to create a new test-cases for it. So it will never happen again. Thrust me.... ;-)

Answer 4

We were getting the same issue recently, 1 cent difference, and it was because of Bluejalappeno Tieredproducts extension (are you using this?) which was extending core getFinalPrice method. The extension was returning product price 3 decimal places while in javascript it was getting incorrectly modified to 2 decimal places, resulting in price inconsistencies when "displayed" and when "adding to cart".

Check from where your product final price is coming from (extension or your local changes), and make sure it returns in 2 decimal places format before it displays. You can convert the price into 2 decimal places like this if it's coming in more than 2 decimal places: $finalPrice = number_format($finalPrice, '2', '.', ',');

Answer 5

thank you for your help. We actually found that the issue was with the /js/varien/product.js. When choosing a Simple product on a Configurable, the product.js was not rounding the same way. The product was still adding to the cart at the correct price but the JS that changed the visible price was incorrect.

So we modified this line:

return formatCurrency(a, this.priceFormat)

with:

var b = (Math.round((a * 100).toFixed(4))/100).toString();return formatCurrency(b, this.priceFormat);