Merge lp:~pedro.baeza/account-payment/6.1-payment_extension-bug into lp:~account-payment-team/account-payment/6.1

Proposed by Pedro Manuel Baeza
Status: Merged
Merged at revision: 106
Proposed branch: lp:~pedro.baeza/account-payment/6.1-payment_extension-bug
Merge into: lp:~account-payment-team/account-payment/6.1
Diff against target: 12 lines (+1/-1)
1 file modified
account_payment_extension/account_move_line.py (+1/-1)
To merge this branch: bzr merge lp:~pedro.baeza/account-payment/6.1-payment_extension-bug
Reviewer Review Type Date Requested Status
mikel Approve
Review via email: mp+183468@code.launchpad.net

Description of the change

In the last revision a bug was introduced in the new code, so please review this urgently to avoid problems in the branch.

Thank you.

Regards.

To post a comment you must log in.
Revision history for this message
mikel (mikel-martin) wrote :

Sorry for that.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'account_payment_extension/account_move_line.py'
2--- account_payment_extension/account_move_line.py 2013-07-15 09:44:51 +0000
3+++ account_payment_extension/account_move_line.py 2013-09-02 13:45:45 +0000
4@@ -48,7 +48,7 @@
5
6 for line_id, invoice_id in cursor.fetchall():
7 name = invoice_obj.name_get(cursor, user, [invoice_id], context=context)
8- res[line_id] = (invoice_id, len(name)>0 and name[0])
9+ res[line_id] = name and name[0] or False
10 return res
11
12 #===========================================================================

Subscribers

People subscribed via source and target branches