Merge lp:~openerp-canada/openobject-addons/lp667920 into lp:openobject-addons

Status: Rejected
Rejected by: Fabien (Open ERP)
Proposed branch: lp:~openerp-canada/openobject-addons/lp667920
Merge into: lp:openobject-addons
Diff against target: 26 lines (+4/-3)
1 file modified
account/account.py (+4/-3)
To merge this branch: bzr merge lp:~openerp-canada/openobject-addons/lp667920
Reviewer Review Type Date Requested Status
Fabien (Open ERP) Disapprove
Review via email: mp+42123@code.launchpad.net

Description of the change

Fix bug lp667920

To post a comment you must log in.
Revision history for this message
Fabien (Open ERP) (fp-tinyerp) wrote :

See how to do in the related bug.

No need to patch openerp for this.

review: Disapprove

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'account/account.py'
2--- account/account.py 2010-11-26 16:13:59 +0000
3+++ account/account.py 2010-11-29 15:31:35 +0000
4@@ -1848,7 +1848,10 @@
5 if tax.child_ids:
6 if tax.child_depend:
7 latest = res.pop()
8- amount = amount2
9+ if tax.include_base_amount:
10+ amount = cur_price_unit + amount2
11+ else:
12+ amount = amount2
13 child_tax = self._unit_compute(cr, uid, tax.child_ids, amount, address_id, product, partner, quantity)
14 res.extend(child_tax)
15 if tax.child_depend:
16@@ -1865,8 +1868,6 @@
17 r[name+'_sign'] = latest[name+'_sign']
18 r['amount'] = data['amount']
19 latest[name+'_code_id'] = False
20- if tax.include_base_amount:
21- cur_price_unit+=amount2
22 return res
23
24 def compute_all(self, cr, uid, taxes, price_unit, quantity, address_id=None, product=None, partner=None):
25
26=== modified file 'account/installer.py'

Subscribers

People subscribed via source and target branches

to all changes: