From 5701d936249ef97bfa3fdf4af5553d933273d03a Mon Sep 17 00:00:00 2001 From: Raphael Collet Date: Thu, 23 Aug 2018 13:45:41 +0200 Subject: [PATCH] [FIX] *: decorator `returns` on `message_post` --- addons/account/models/account_invoice.py | 2 +- addons/account_facturx/models/account_invoice.py | 2 +- addons/calendar/models/calendar.py | 2 +- addons/l10n_ch/models/account_invoice.py | 2 +- addons/purchase/models/purchase.py | 2 +- addons/rating/models/mail_thread.py | 2 +- addons/sale/models/sale.py | 2 +- addons/website_blog/models/website_blog.py | 2 +- addons/website_forum/models/forum.py | 2 +- addons/website_slides/models/slides.py | 2 +- 10 files changed, 10 insertions(+), 10 deletions(-) diff --git a/addons/account/models/account_invoice.py b/addons/account/models/account_invoice.py index d80a4750d3b..160e37952c2 100644 --- a/addons/account/models/account_invoice.py +++ b/addons/account/models/account_invoice.py @@ -614,7 +614,7 @@ class AccountInvoice(models.Model): } @api.multi - @api.returns('self', lambda value: value.id) + @api.returns('mail.message', lambda value: value.id) def message_post(self, **kwargs): if self.env.context.get('mark_invoice_as_sent'): self.filtered(lambda inv: not inv.sent).write({'sent': True}) diff --git a/addons/account_facturx/models/account_invoice.py b/addons/account_facturx/models/account_invoice.py index 3d87880d69f..9be1376867e 100644 --- a/addons/account_facturx/models/account_invoice.py +++ b/addons/account_facturx/models/account_invoice.py @@ -202,7 +202,7 @@ class AccountInvoice(models.Model): return invoice_form.save() @api.multi - @api.returns('self', lambda value: value.id) + @api.returns('mail.message', lambda value: value.id) def message_post(self, **kwargs): # OVERRIDE # /!\ 'default_res_id' in self._context is used to don't process attachment when using a form view. diff --git a/addons/calendar/models/calendar.py b/addons/calendar/models/calendar.py index f9be94f5dfc..471de145472 100644 --- a/addons/calendar/models/calendar.py +++ b/addons/calendar/models/calendar.py @@ -1455,7 +1455,7 @@ class Meeting(models.Model): event.message_needaction = rec.message_needaction @api.multi - @api.returns('self', lambda value: value.id) + @api.returns('mail.message', lambda value: value.id) def message_post(self, **kwargs): thread_id = self.id if isinstance(self.id, pycompat.string_types): diff --git a/addons/l10n_ch/models/account_invoice.py b/addons/l10n_ch/models/account_invoice.py index 6b2b4d7fdf7..6afa172d617 100644 --- a/addons/l10n_ch/models/account_invoice.py +++ b/addons/l10n_ch/models/account_invoice.py @@ -169,7 +169,7 @@ class AccountInvoice(models.Model): return rslt @api.multi - @api.returns('self', lambda value: value.id) + @api.returns('mail.message', lambda value: value.id) def message_post(self, **kwargs): if self.env.context.get('l10n_ch_mark_isr_as_sent'): self.filtered(lambda inv: not inv.l10n_ch_isr_sent).write({'l10n_ch_isr_sent': True}) diff --git a/addons/purchase/models/purchase.py b/addons/purchase/models/purchase.py index a672cdcb748..61202d30e7d 100644 --- a/addons/purchase/models/purchase.py +++ b/addons/purchase/models/purchase.py @@ -277,7 +277,7 @@ class PurchaseOrder(models.Model): } @api.multi - @api.returns('self', lambda value: value.id) + @api.returns('mail.message', lambda value: value.id) def message_post(self, **kwargs): if self.env.context.get('mark_rfq_as_sent'): self.filtered(lambda o: o.state == 'draft').write({'state': 'sent'}) diff --git a/addons/rating/models/mail_thread.py b/addons/rating/models/mail_thread.py index 1b91b819969..18f2155edd8 100644 --- a/addons/rating/models/mail_thread.py +++ b/addons/rating/models/mail_thread.py @@ -8,7 +8,7 @@ class MailThread(models.AbstractModel): _inherit = 'mail.thread' @api.multi - @api.returns('self', lambda value: value.id) + @api.returns('mail.message', lambda value: value.id) def message_post(self, **kwargs): rating_value = kwargs.pop('rating_value', False) message = super(MailThread, self).message_post(**kwargs) diff --git a/addons/sale/models/sale.py b/addons/sale/models/sale.py index ef1c8b1c1e5..aeb55ce8920 100644 --- a/addons/sale/models/sale.py +++ b/addons/sale/models/sale.py @@ -593,7 +593,7 @@ class SaleOrder(models.Model): } @api.multi - @api.returns('self', lambda value: value.id) + @api.returns('mail.message', lambda value: value.id) def message_post(self, **kwargs): if self.env.context.get('mark_so_as_sent'): self.filtered(lambda o: o.state == 'draft').with_context(tracking_disable=True).write({'state': 'sent'}) diff --git a/addons/website_blog/models/website_blog.py b/addons/website_blog/models/website_blog.py index a1b59a83e6f..dc6f2d16736 100644 --- a/addons/website_blog/models/website_blog.py +++ b/addons/website_blog/models/website_blog.py @@ -36,7 +36,7 @@ class Blog(models.Model): return res @api.multi - @api.returns('self', lambda value: value.id) + @api.returns('mail.message', lambda value: value.id) def message_post(self, parent_id=False, subtype=None, **kwargs): """ Temporary workaround to avoid spam. If someone replies on a channel through the 'Presentation Published' email, it should be considered as a diff --git a/addons/website_forum/models/forum.py b/addons/website_forum/models/forum.py index 812404e65de..64b37ea3b51 100644 --- a/addons/website_forum/models/forum.py +++ b/addons/website_forum/models/forum.py @@ -818,7 +818,7 @@ class Post(models.Model): return groups @api.multi - @api.returns('self', lambda value: value.id) + @api.returns('mail.message', lambda value: value.id) def message_post(self, message_type='notification', **kwargs): question_followers = self.env['res.partner'] if self.ids and message_type == 'comment': # user comments have a restriction on karma diff --git a/addons/website_slides/models/slides.py b/addons/website_slides/models/slides.py index bc7db7ef124..50d6641f964 100644 --- a/addons/website_slides/models/slides.py +++ b/addons/website_slides/models/slides.py @@ -168,7 +168,7 @@ class Channel(models.Model): return res @api.multi - @api.returns('self', lambda value: value.id) + @api.returns('mail.message', lambda value: value.id) def message_post(self, parent_id=False, subtype=None, **kwargs): """ Temporary workaround to avoid spam. If someone replies on a channel through the 'Presentation Published' email, it should be considered as a