Taxes amount should be rounded to 2 digits as static?

Bug #541409 reported by Leigh Willard
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Opinion
Medium
Unassigned

Bug Description

In file account/account.py, in the class account_tax, the routine
compute (line 1443), the value 'price_accuracy' is being used to round
the taxes. I believe that instead this should be the currency "rounding
factor". The 'price_accuracy' affects the display of all prices, but
that is not the problem, and we cannot change that for our clients because final prices
must be displayed with 2 digits only.

''rounding_factor" seems the appropriate setting.

Thanks,
Leigh Willard
epsilon technologies / tinyatwork (Quebec partners)

Revision history for this message
Raphaël Valyi - http://www.akretion.com (rvalyi) wrote :

Hello Leigh,

could you please confirm this is a duplicate of that critical bug https://bugs.launchpad.net/bugs/452854
and it seems what you propose is exactly the patch we suggested (we are still waiting OpenERP S.A. to take a stance on our patch or improve it).
Can you please confirm that?

Thank you very much for reporting.

Changed in openobject-addons:
assignee: nobody → aag(OpenERP) (aag-openerp)
Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote :

Hello Experts,

This is an issue where big amounts(millions) may take an effect.

We have followed price_accuracy parameter to concern taxes amounts.

Should it be 2 as static or variably any?

Thanks.

Changed in openobject-addons:
importance: Undecided → Medium
summary: - taxes rounding
+ Taxes amount should be rounded to 2 digits as static?
Changed in openobject-addons:
status: New → Opinion
Changed in openobject-addons:
assignee: Atik Agewan(OpenERP) (aag-openerp) → nobody
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.