From 4809d8f596dd5bb63e25ec5f27f81bc40b3825d7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Thibault=20Delavall=C3=A9e?= Date: Fri, 15 Apr 2022 08:03:27 +0000 Subject: [PATCH] [FW][FIX] sms: remove sms notifications at uninstall Currently when uninstalling SMS notifications linked to SMS are kept but set as email notifications. This makes no sense as they are not linked to any outgoing mail.mail or email address. As the SMS are removed and as the SMS feature is not available anymore it is better to simply remove existing SMS notifications. Task-2778461 closes odoo/odoo#88970 X-original-commit: f307a6c19a2dbe64ba6120a00610063f256102d5 Signed-off-by: Thibault Delavallee (tde) --- addons/sms/models/mail_notification.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/addons/sms/models/mail_notification.py b/addons/sms/models/mail_notification.py index 5fdfc33f477..b203abec10d 100644 --- a/addons/sms/models/mail_notification.py +++ b/addons/sms/models/mail_notification.py @@ -9,7 +9,7 @@ class MailNotification(models.Model): notification_type = fields.Selection(selection_add=[ ('sms', 'SMS') - ], ondelete={'sms': 'set default'}) + ], ondelete={'sms': 'cascade'}) sms_id = fields.Many2one('sms.sms', string='SMS', index='btree_not_null', ondelete='set null') sms_number = fields.Char('SMS Number') failure_type = fields.Selection(selection_add=[