[trunk/6.1]account_payment Cannot import move lines without an invoice

Bug #986088 reported by Stefan Rijnhart (Opener)
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Banking Addons
Fix Released
Low
Stefan Rijnhart (Opener)
Odoo Addons (MOVED TO GITHUB)
Fix Released
Medium
OpenERP R&D Addons Team 3

Bug Description

Hi,

account_payment allows the user to import move lines using 'Select Invoices to Pay' (this misnoner may be part of the problem, but currently this allows the user to select *move lines*, not invoices and I think that is correct).

There is a field 'currency' on the payment.order.line which is required. This field is taken from the invoice of the move line, in payment_order_create.create_payment(). If the move line does not have an invoice, the currency field is left empty and the system raises an exception about the invalid value for this field.

I would like to see that the currency in that case is taken from the journal with a fallback on the company currency.

Cheers,
Stefan.

Related branches

summary: - [account_payment] Cannot import move lines without an invoice
+ [trunk/6.1]account_payment Cannot import move lines without an invoice
Changed in openobject-addons:
status: New → Confirmed
importance: Undecided → Low
assignee: nobody → OpenERP R&D Addons Team 3 (openerp-dev-addons3)
Changed in openobject-addons:
status: Confirmed → In Progress
Revision history for this message
Amit Bhavsar (Open ERP) (amb-openerp) wrote :

Hello Stefan,

Thanks for reporting!

It has been fixed on lp:~openerp-dev/openobject-addons/trunk-bug-986088-amb branch with following revision no and id.

rev no : 6756
rev id : 6756 <email address hidden>

Thanks again!

Changed in openobject-addons:
status: In Progress → Fix Committed
Amit Parik (amit-parik)
Changed in openobject-addons:
importance: Low → Medium
Revision history for this message
Fabien (Open ERP) (fp-tinyerp) wrote :

fix released, thanks for reporting.

Changed in openobject-addons:
status: Fix Committed → Fix Released
Changed in banking-addons:
status: New → Fix Committed
assignee: nobody → Stefan Rijnhart (Therp) (stefan-therp)
importance: Undecided → Low
Changed in banking-addons:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.