From 22390eded5a2f23a93d9c549b03a2c2270a93c11 Mon Sep 17 00:00:00 2001 From: Carlos Dauden Date: Tue, 10 Sep 2019 18:30:19 +0200 Subject: [PATCH] [IMP] account_payment_order: Select first partner bank --- account_payment_order/__manifest__.py | 2 +- account_payment_order/models/account_move_line.py | 12 ++++-------- 2 files changed, 5 insertions(+), 9 deletions(-) diff --git a/account_payment_order/__manifest__.py b/account_payment_order/__manifest__.py index 06a17f6c6..3e5f67795 100644 --- a/account_payment_order/__manifest__.py +++ b/account_payment_order/__manifest__.py @@ -8,7 +8,7 @@ { 'name': 'Account Payment Order', - 'version': '12.0.1.0.1', + 'version': '12.0.1.0.2', 'license': 'AGPL-3', 'author': "ACSONE SA/NV, " "Therp BV, " diff --git a/account_payment_order/models/account_move_line.py b/account_payment_order/models/account_move_line.py index 6138387e0..f5c251624 100644 --- a/account_payment_order/models/account_move_line.py +++ b/account_payment_order/models/account_move_line.py @@ -3,7 +3,8 @@ # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html). from lxml import etree -from odoo import models, fields, api +from odoo import api, fields, models +from odoo.fields import first from odoo.osv import orm @@ -58,13 +59,8 @@ class AccountMoveLine(models.Model): # in this case if payment_order.payment_type == 'outbound': amount_currency *= -1 - partner_bank_id = False - if not self.partner_bank_id: - # Select partner bank account automatically if there is only one - if len(self.partner_id.bank_ids) == 1: - partner_bank_id = self.partner_id.bank_ids[0].id - else: - partner_bank_id = self.partner_bank_id.id + partner_bank_id = self.partner_bank_id.id or first( + self.partner_id.bank_ids).id vals = { 'order_id': payment_order.id, 'partner_bank_id': partner_bank_id,