c2c-oerpscenario team mailing list archive

["Fabien \(Open ERP\)" <fp@xxxxxxxxxxx>]

** Attachment added: "invoice.png"
   https://bugs.launchpad.net/openobject-addons/+bug/667920/+attachment/1779003/+files/invoice.png

** Changed in: openobject-addons
       Status: Confirmed => Invalid

-- 
You received this bug notification because you are a member of C2C
OERPScenario, which is subscribed to the OpenERP Project Group.
https://bugs.launchpad.net/bugs/667920

Title:
  [6.0RC1] Children tax calculation "included in base amount"

Status in OpenObject Addons Modules:
  Invalid

Bug description:
  We have a parent tax of 5% and another tax 7.5% which is applied on the original value AND the 5% tax.  So we set the following:

tax 5%
   in the section children taxes we check "included in base amount", then add the 7.5% tax

in the Sales order, the taxes are calcualted correctly, but when the invoice is generated the parent/child relationship gets reversed.
----------------------------------------------------------------
We believe the problem is that the "included in base amount" code is not in the correct place in the file addons/account/accout.py

class account_tax

function _unit_compute

the 2 lines of code:
           #if tax.include_base_amount:
            #    cur_price_unit+=amount2
need to be INSIDE the children tax calculation, not outside of it, so the could would look like:

 if tax.child_ids:
                if tax.child_depend:
                    latest = res.pop()
                if tax.include_base_amount:
                    amount = cur_price_unit + amount2
                else:
                    amount = amount2

(notice the placement of include_base_amount).

You will need to look at the original code to understand this, but I believe this is correct and WORKS when we test it.