From 06e451cd0f205125498033dac323cba67807b803 Mon Sep 17 00:00:00 2001 From: Ronald Portier Date: Tue, 15 Mar 2022 15:10:13 +0100 Subject: [PATCH] [FIX] web_advanced_filter: prevent error on filter delete The wizard that adds criteria to a filter maintains a reference to that filter. That makes it impossible to delete the filter pointed to. By changing the filter_id field to delete cascade, this problem is solved. --- .../wizards/ir_filters_combine_with_existing.py | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/web_advanced_filter/wizards/ir_filters_combine_with_existing.py b/web_advanced_filter/wizards/ir_filters_combine_with_existing.py index 285b16ae6..87ff5a83d 100644 --- a/web_advanced_filter/wizards/ir_filters_combine_with_existing.py +++ b/web_advanced_filter/wizards/ir_filters_combine_with_existing.py @@ -18,7 +18,13 @@ class IrFiltersCombineWithExisting(models.TransientModel): domain = fields.Char('Domain', required=True) context = fields.Char('Context', required=True, default='{}') model = fields.Char('Model', required=True) - filter_id = fields.Many2one('ir.filters', 'Filter', required=True) + filter_id = fields.Many2one( + comodel_name='ir.filters', + string='Filter', + ondelete='cascade', + index=True, + required=True + ) @api.multi def button_save(self):