Merge lp:~akretion-team/openobject-addons/trunk-addons-fix-analytic into lp:openobject-addons

Proposed by Alexis de Lattre
Status: Merged
Merged at revision: 7028
Proposed branch: lp:~akretion-team/openobject-addons/trunk-addons-fix-analytic
Merge into: lp:openobject-addons
Diff against target: 20 lines (+2/-2)
1 file modified
account/account_invoice.py (+2/-2)
To merge this branch: bzr merge lp:~akretion-team/openobject-addons/trunk-addons-fix-analytic
Reviewer Review Type Date Requested Status
qdp (OpenERP) Pending
Review via email: mp+114737@code.launchpad.net

Description of the change

This merge proposal fixes a regression that I introduced in my branch https://code.launchpad.net/~akretion-team/openobject-addons/trunk-addons-analytic-on-tax-v2/+merge/114287 that was merged yesterday.

My error comes from the confusion in field names :
- on account.invoice.line and account.invoice.tax, the "Analytic account" is named "account_analytic_id"
- on account.move.line, the same field is named "analytic_account_id"

To post a comment you must log in.
Revision history for this message
Alexis de Lattre (alexis-via) wrote :

The runbot is green on this branch.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'account/account_invoice.py'
2--- account/account_invoice.py 2012-07-10 21:21:15 +0000
3+++ account/account_invoice.py 2012-07-12 21:17:20 +0000
4@@ -1002,7 +1002,7 @@
5 'quantity': x.get('quantity',1.00),
6 'product_id': x.get('product_id', False),
7 'product_uom_id': x.get('uos_id', False),
8- 'analytic_account_id': x.get('analytic_account_id', False),
9+ 'analytic_account_id': x.get('account_analytic_id', False),
10 }
11
12 def action_number(self, cr, uid, ids, context=None):
13@@ -1691,7 +1691,7 @@
14 'account_id': t['account_id'],
15 'tax_code_id': t['tax_code_id'],
16 'tax_amount': t['tax_amount'],
17- 'analytic_account_id': t['account_analytic_id'],
18+ 'account_analytic_id': t['account_analytic_id'],
19 })
20 return res
21

Subscribers

People subscribed via source and target branches

to all changes: