From 088d00f2a7968c58ec9bafea0cd90685a5bd2933 Mon Sep 17 00:00:00 2001 From: Monica Diaz Pena Date: Thu, 22 Feb 2018 16:57:17 +0100 Subject: [PATCH] [FIX] mail: Don't duplicate uuid of mail channels. If you duplicate a channel the UUID should not be copied, otherwise it might introduce security issues and the method 'channel_minimize' will fail with an "expected singleton error". Closes #23102 --- addons/mail/models/mail_channel.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/addons/mail/models/mail_channel.py b/addons/mail/models/mail_channel.py index 06a272948ea..fb33fbf1323 100644 --- a/addons/mail/models/mail_channel.py +++ b/addons/mail/models/mail_channel.py @@ -49,7 +49,7 @@ class Channel(models.Model): ('channel', 'Channel')], 'Channel Type', default='channel') description = fields.Text('Description') - uuid = fields.Char('UUID', size=50, index=True, default=lambda self: '%s' % uuid.uuid4()) + uuid = fields.Char('UUID', size=50, index=True, default=lambda self: '%s' % uuid.uuid4(), copy=False) email_send = fields.Boolean('Send messages by email', default=False) # multi users channel channel_last_seen_partner_ids = fields.One2many('mail.channel.partner', 'channel_id', string='Last Seen')