From 916e83df09cf66b0b103ce6c0faad4fe91554518 Mon Sep 17 00:00:00 2001 From: Thomas Binsfeld Date: Wed, 8 May 2019 10:26:13 +0200 Subject: [PATCH 1/2] [IMP] Banking Mandate: deletion of bank accounts linked to mandates It should not be possible to delete a bank account linked to mandate(s). --- account_banking_mandate/models/account_banking_mandate.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/account_banking_mandate/models/account_banking_mandate.py b/account_banking_mandate/models/account_banking_mandate.py index 10cb42185..5de711fb0 100644 --- a/account_banking_mandate/models/account_banking_mandate.py +++ b/account_banking_mandate/models/account_banking_mandate.py @@ -36,7 +36,9 @@ class AccountBankingMandate(models.Model): partner_bank_id = fields.Many2one( comodel_name='res.partner.bank', string='Bank Account', track_visibility='onchange', - domain=lambda self: self._get_default_partner_bank_id_domain(),) + domain=lambda self: self._get_default_partner_bank_id_domain(), + ondelete='restrict', + ) partner_id = fields.Many2one( comodel_name='res.partner', related='partner_bank_id.partner_id', string='Partner', store=True) From ff6d00c5afeba1820fa8b8e8b5df0d1e0081bec3 Mon Sep 17 00:00:00 2001 From: Thomas Binsfeld Date: Wed, 8 May 2019 10:26:48 +0200 Subject: [PATCH 2/2] [REF] Banking Mandate: indexes on bank account and partner --- account_banking_mandate/models/account_banking_mandate.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/account_banking_mandate/models/account_banking_mandate.py b/account_banking_mandate/models/account_banking_mandate.py index 5de711fb0..18ec9532b 100644 --- a/account_banking_mandate/models/account_banking_mandate.py +++ b/account_banking_mandate/models/account_banking_mandate.py @@ -38,10 +38,11 @@ class AccountBankingMandate(models.Model): track_visibility='onchange', domain=lambda self: self._get_default_partner_bank_id_domain(), ondelete='restrict', + index=True, ) partner_id = fields.Many2one( comodel_name='res.partner', related='partner_bank_id.partner_id', - string='Partner', store=True) + string='Partner', store=True, index=True) company_id = fields.Many2one( comodel_name='res.company', string='Company', required=True, default=lambda self: self.env['res.company']._company_default_get(