Merge lp:~jviares/openobject-addons/extra-6.0 into lp:openobject-addons/extra-6.0

Proposed by Vicente
Status: Merged
Merged at revision: 5703
Proposed branch: lp:~jviares/openobject-addons/extra-6.0
Merge into: lp:openobject-addons/extra-6.0
Diff against target: 13 lines (+3/-0)
1 file modified
account_payment_extension/wizard/wizard_payment_order.py (+3/-0)
To merge this branch: bzr merge lp:~jviares/openobject-addons/extra-6.0
Reviewer Review Type Date Requested Status
Serpent Consulting Services Approve
Review via email: mp+93537@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Vicente (jviares) wrote :

Bug in account_payment_extension/wizard/wizard_payment_order.py
  function search_entries:
  If the amount field is empty, the function doesn't return line ids

Revision history for this message
Serpent Consulting Services (serpent-consulting-services) wrote :

Vicente,

The wizard should be converted now to osv_memory.

The change of code is fine.

Thanks.

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/wizard/wizard_payment_order.py'
2--- account_payment_extension/wizard/wizard_payment_order.py 2011-08-29 17:42:17 +0000
3+++ account_payment_extension/wizard/wizard_payment_order.py 2012-02-17 07:33:19 +0000
4@@ -110,6 +110,9 @@
5 if abs(line.amount_to_pay) <= amount:
6 amount -= abs(line.amount_to_pay)
7 selected_ids.append( line.id )
8+ else:
9+ selected_ids = line_ids
10+
11 return {
12 'entries': selected_ids,
13 }