← Back to team overview

c2c-oerpscenario team mailing list archive

[Bug 510726] Re: [sale_tax_include][account_tax_include] rounding error on taxed amount

 

Won't fix: we rewrite everything in the trunk branch so that the
sale_ta_include and account_tax_include modules are not usefull anymore.
Everything is tax include/excluded by default in trunk according to the
configuration of the taxes.

** Changed in: openobject-addons
       Status: Confirmed => Won't Fix

-- 
[sale_tax_include][account_tax_include] rounding error on taxed amount
https://bugs.launchpad.net/bugs/510726
You received this bug notification because you are a member of C2C
OERPScenario, which is subscribed to the OpenERP Project Group.

Status in OpenObject Addons Modules: Won't Fix

Bug description:
I have a bug with account_tax_includ and sale_tax_include (purchase_tax_include don't work on my installation)

Create a new product with 9€ for the price and a include VAT tax of 19,6% (ex : sale French VAT)
Create a new sale order (or a new invoice, it's the same bug)
And add the product
Compute taxe

And... the result is
untaxed amount : 7,53€
taxed amount : 1,48€
total amount : 9,01€  !!!

The problem is simply, OpenERP have made a round on temporary untaxed amount :
untaxed amount = 9/1,196 = 7,525083612 ==> 7,53€
taxed amount = 7,53x1,196 = 9,00588 ==> 9,01€

So 9,00 = 9,01 ;)

The solution is to not round the tempory value of untaxed amount : 
untaxed amount = 9/1,196 = 7,525083612
taxed amount = 7,525083612x1,196 = 9€

After calculating the taxe untaxe amount can be rounded without problem.
Moreover before returning value of untaxe amount and total amount are rounded (line 89-90)

I join a patch, if you disagree with the idea of deleting the round option in compute_inv, maybe we can add a optional parameter to compute_inv method to choose if we want a rounding value of tax amount or not; in the case of sale_tax_include and account_tax_include the optional parameter will be use to disable the rounding funtion.