From 1a99d22f19949cf8eb689a6e2d6eda859c963cf9 Mon Sep 17 00:00:00 2001 From: qdp-odoo Date: Fri, 9 Mar 2018 15:48:57 +0100 Subject: [PATCH] [IMP] account: filter res.partner.bank on account.journal In order to avoid people selecting a bank account not belonging to the partner's company, we now filter using a domain. Original idea from PR #23480. Courtesy of Miquel Raich (Eficent) --- addons/account/models/account.py | 3 ++- addons/account/views/account_view.xml | 1 + 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/addons/account/models/account.py b/addons/account/models/account.py index b7d8d52f096..4f6685f7fc0 100644 --- a/addons/account/models/account.py +++ b/addons/account/models/account.py @@ -410,7 +410,8 @@ class AccountJournal(models.Model): belongs_to_company = fields.Boolean('Belong to the user\'s current company', compute="_belong_to_company", search="_search_company_journals",) # Bank journals fields - bank_account_id = fields.Many2one('res.partner.bank', string="Bank Account", ondelete='restrict', copy=False, domain="[('partner_id','=', company_id)]") + company_partner_id = fields.Many2one('res.partner', related='company_id.partner_id', string='Account Holder', readonly=True, store=False) + bank_account_id = fields.Many2one('res.partner.bank', string="Bank Account", ondelete='restrict', copy=False, domain="[('partner_id','=', company_partner_id)]") bank_statements_source = fields.Selection([('undefined', 'Undefined'),('manual', 'Record Manually')], string='Bank Feeds', default='undefined') bank_acc_number = fields.Char(related='bank_account_id.acc_number') bank_id = fields.Many2one('res.bank', related='bank_account_id.bank_id') diff --git a/addons/account/views/account_view.xml b/addons/account/views/account_view.xml index 6d8ffdd1ac0..4c8f7845379 100644 --- a/addons/account/views/account_view.xml +++ b/addons/account/views/account_view.xml @@ -284,6 +284,7 @@ +