Merge lp:~pedro.baeza/ocb-addons/fix-lp_1254631 into lp:ocb-addons

Proposed by Pedro Manuel Baeza
Status: Merged
Approved by: Holger Brunn (Therp)
Approved revision: no longer in the source branch.
Merged at revision: 10164
Proposed branch: lp:~pedro.baeza/ocb-addons/fix-lp_1254631
Merge into: lp:ocb-addons
Diff against target: 14 lines (+2/-2)
1 file modified
account/account_invoice.py (+2/-2)
To merge this branch: bzr merge lp:~pedro.baeza/ocb-addons/fix-lp_1254631
Reviewer Review Type Date Requested Status
Holger Brunn (Therp) code review Approve
Stefan Rijnhart (Opener) Approve
Yannick Vaucher @ Camptocamp code review, no test Approve
Review via email: mp+216728@code.launchpad.net

Description of the change

Fix for the bug #1254631. Details on the bug report.

To post a comment you must log in.
Revision history for this message
Yannick Vaucher @ Camptocamp (yvaucher-c2c) wrote :

LGTM

review: Approve (code review, no test)
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Tnx!

review: Approve
Revision history for this message
Holger Brunn (Therp) (hbrunn) :
review: Approve (code review)

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 2014-03-24 09:12:54 +0000
3+++ account/account_invoice.py 2014-04-22 13:40:24 +0000
4@@ -519,8 +519,8 @@
5 acc_id = p.property_account_payable.id
6 partner_payment_term = p.property_supplier_payment_term and p.property_supplier_payment_term.id or False
7 fiscal_position = p.property_account_position and p.property_account_position.id or False
8- if p.bank_ids:
9- bank_id = p.bank_ids[0].id
10+ if p.commercial_partner_id.bank_ids:
11+ bank_id = p.commercial_partner_id.bank_ids[0].id
12
13 result = {'value': {
14 'account_id': acc_id,