Merge lp:~therp-nl/banking-addons/ba61-lp986088-fix_import_move_lines_without_invoice into lp:banking-addons/6.1

Proposed by Stefan Rijnhart (Opener)
Status: Merged
Merged at revision: 153
Proposed branch: lp:~therp-nl/banking-addons/ba61-lp986088-fix_import_move_lines_without_invoice
Merge into: lp:banking-addons/6.1
Diff against target: 11 lines (+1/-1)
1 file modified
account_banking/wizard/account_payment_order.py (+1/-1)
To merge this branch: bzr merge lp:~therp-nl/banking-addons/ba61-lp986088-fix_import_move_lines_without_invoice
Reviewer Review Type Date Requested Status
Joël Grand-Guillaume @ camptocamp Approve
Review via email: mp+142691@code.launchpad.net

Description of the change

In account_banking, a method from account.payment is replaced by a slightly modified version. This branch applies a bugfix that was applied to the original method. To keep the changes compared to the original method as small as possible, I did not fix the line length of the orignal patch.

To post a comment you must log in.
Revision history for this message
Joël Grand-Guillaume @ camptocamp (jgrandguillaume-c2c) wrote :

LGTM as we want to be able to track the same line in case of trouble !

Revision history for this message
Joël Grand-Guillaume @ camptocamp (jgrandguillaume-c2c) wrote :

Forget the approve...

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'account_banking/wizard/account_payment_order.py'
2--- account_banking/wizard/account_payment_order.py 2011-12-08 21:20:05 +0000
3+++ account_banking/wizard/account_payment_order.py 2013-01-10 13:43:20 +0000
4@@ -112,7 +112,7 @@
5 'state': state,
6 ### end account banking
7 'date': date_to_pay,
8- 'currency': line.invoice and line.invoice.currency_id.id or False,
9+ 'currency': line.invoice and line.invoice.currency_id.id or line.journal_id.currency.id or line.journal_id.company_id.currency_id.id,
10 }, context=context)
11 return {'type': 'ir.actions.act_window_close'}
12

Subscribers

People subscribed via source and target branches