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/ (+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:

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
=== modified file 'account_banking/wizard/'
--- account_banking/wizard/ 2011-12-08 21:20:05 +0000
+++ account_banking/wizard/ 2013-01-10 13:43:20 +0000
@@ -112,7 +112,7 @@
112 'state': state,112 'state': state,
113 ### end account banking113 ### end account banking
114 'date': date_to_pay,114 'date': date_to_pay,
115 'currency': line.invoice and or False,115 'currency': line.invoice and or or,
116 }, context=context)116 }, context=context)
117 return {'type': 'ir.actions.act_window_close'}117 return {'type': 'ir.actions.act_window_close'}


People subscribed via source and target branches