← Back to team overview

openerp-expert-accounting team mailing list archive

Re: [Bug 707923] Re: [6.0 and 5] invoice tax - rounding issue

 

2011/11/4 Bogdan Stanciu <bogdanovidiu.stanciu@xxxxxxxxx>:
>
> If you allow me, I would like to suggest to create your addon as a
> switch (something like account_cancel) which integrates smoothly in the
> standard version. Dumping it in localisations would create the need to
> replicate it for countries affected, and it would loose visibility...
>

I understand your argument, indeed the generic module I suggest to use
is 'c2c_account_tax_rounding'. From today,
'account_invoice_tax_by_column' doesn't contain anything useful to
generic taxes, except of italian VAT.

I would rely on the c2c_account_tax_rounding way, or similar, just
because the computation 'by column' is much more close to computation
without roundings than the 'by line' one.
I'm not totally sure whether c2c_account_tax_rounding correctly
implements this or not, but it works for our tests.

If OpenERP S.A. took into account this issue, we could discuss with
them the best way to patch the account module.

Regards


-- 
Lorenzo Battistini

-- 
You received this bug notification because you are a member of OpenERP
Accounting Experts, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/707923

Title:
  [6.0 and 5] invoice tax - rounding issue

Status in OpenERP Addons (modules):
  Won't Fix

Bug description:
  see attachment

  due to rounding of the calculated tax for each position we get totals
  which are wrong.

  In Austria we usually group the basis per tax rate and calculate the
  tax from this basis to avoid such errors.

  in account/invoice.py
  class account_invoice_tax  - 
  def compute must be modified 

  actually only some lines of code to group the invoice lines before
  calculating the tax

  probably this has been discussed (but not solved) already

To manage notifications about this bug go to:
https://bugs.launchpad.net/openobject-addons/+bug/707923/+subscriptions


References