From 12c51cd847745017f9ee82db405443eea042ee25 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Thibault=20Delavall=C3=A9e?= <tde@odoo.com>
Date: Tue, 28 Feb 2017 09:30:26 +0100
Subject: [PATCH] [FIX] mail_thread inheritance: add api.returns on override of
 message_post

As message_post is defined on mail.thread abstract class its api.returns
is not automatically added on overridden versions of the method. This
commit add missing api.returns on all overrides of the method.
---
 addons/hr/models/res_users.py              | 1 +
 addons/mail/models/res_users.py            | 1 +
 addons/website_blog/models/website_blog.py | 1 +
 addons/website_forum/models/forum.py       | 1 +
 addons/website_slides/models/slides.py     | 1 +
 5 files changed, 5 insertions(+)

diff --git a/addons/hr/models/res_users.py b/addons/hr/models/res_users.py
index f759aba30f5..ddbcc95c234 100644
--- a/addons/hr/models/res_users.py
+++ b/addons/hr/models/res_users.py
@@ -31,6 +31,7 @@ class User(models.Model):
         return self.env['hr.employee'].with_context(ctx).search([('user_id', '=', self.id)])
 
     @api.multi
+    @api.returns('self', lambda value: value.id)
     def message_post(self, **kwargs):
         """ Redirect the posting of message on res.users to the related employees.
             This is done because when giving the context of Chatter on the
diff --git a/addons/mail/models/res_users.py b/addons/mail/models/res_users.py
index 1f95c02f078..19630150a75 100644
--- a/addons/mail/models/res_users.py
+++ b/addons/mail/models/res_users.py
@@ -77,6 +77,7 @@ class Users(models.Model):
         return self.partner_id.id
 
     @api.multi
+    @api.returns('self', lambda value: value.id)
     def message_post(self, **kwargs):
         """ Redirect the posting of message on res.users as a private discussion.
             This is done because when giving the context of Chatter on the
diff --git a/addons/website_blog/models/website_blog.py b/addons/website_blog/models/website_blog.py
index 4473ede211b..3a3f82ded9e 100644
--- a/addons/website_blog/models/website_blog.py
+++ b/addons/website_blog/models/website_blog.py
@@ -33,6 +33,7 @@ class Blog(models.Model):
         return res
 
     @api.multi
+    @api.returns('self', 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 c286bfcdfb8..e53ab718b00 100644
--- a/addons/website_forum/models/forum.py
+++ b/addons/website_forum/models/forum.py
@@ -799,6 +799,7 @@ class Post(models.Model):
         return groups
 
     @api.multi
+    @api.returns('self', lambda value: value.id)
     def message_post(self, message_type='notification', subtype=None, **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 db3706a0a75..3362e32d834 100644
--- a/addons/website_slides/models/slides.py
+++ b/addons/website_slides/models/slides.py
@@ -165,6 +165,7 @@ class Channel(models.Model):
         return res
 
     @api.multi
+    @api.returns('self', 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