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

Proposed by Stefan Rijnhart (Opener)
Status: Merged
Merged at revision: 181
Proposed branch: lp:~therp-nl/banking-addons/ba61-move_ref
Merge into: lp:banking-addons/6.1
Diff against target: 12 lines (+1/-1)
1 file modified
account_direct_debit/model/account_payment.py (+1/-1)
To merge this branch: bzr merge lp:~therp-nl/banking-addons/ba61-move_ref
Reviewer Review Type Date Requested Status
Holger Brunn (Therp) code review Approve
Maxime Chambreuil (http://www.savoirfairelinux.com) code review Approve
Pedro Manuel Baeza code review, no test Approve
Stéphane Bidoul (Acsone) (community) code review, no test Approve
Review via email: mp+187034@code.launchpad.net

Description of the change

To post a comment you must log in.
Revision history for this message
Stéphane Bidoul (Acsone) (sbi) wrote :

LGTM

review: Approve (code review, no test)
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) :
review: Approve (code review, no test)
Revision history for this message
Maxime Chambreuil (http://www.savoirfairelinux.com) (max3903) :
review: Approve (code review)
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
=== modified file 'account_direct_debit/model/account_payment.py'
--- account_direct_debit/model/account_payment.py 2013-01-21 11:30:46 +0000
+++ account_direct_debit/model/account_payment.py 2013-09-23 14:00:07 +0000
@@ -153,7 +153,7 @@
153 move_id = account_move_obj.create(cr, uid, {153 move_id = account_move_obj.create(cr, uid, {
154 'journal_id': order.mode.transfer_journal_id.id,154 'journal_id': order.mode.transfer_journal_id.id,
155 'name': 'Debit order %s' % line.move_line_id.move_id.name,155 'name': 'Debit order %s' % line.move_line_id.move_id.name,
156 'reference': 'DEB%s' % line.move_line_id.move_id.name,156 'ref': 'DEB%s' % line.move_line_id.move_id.name,
157 }, context=context)157 }, context=context)
158158
159 # TODO: take multicurrency into account159 # TODO: take multicurrency into account

Subscribers

People subscribed via source and target branches