From 155e81d5e7be2d67b284bf92cee76d34984e9ce6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Thibault=20Delavall=C3=A9e?= <tde@odoo.com>
Date: Thu, 9 Mar 2017 13:59:57 +0100
Subject: [PATCH] [IMP] mail: ease members management on channels

Using a m2m to partners does not allow a real management of members
because it allows to create partners, and require access rights on
partner model. Instead it now uses the o2m to channel members
allowing creation, update and removal of members without having
to work on partner records.
---
 addons/mail/models/mail_channel.py       | 1 +
 addons/mail/views/mail_channel_views.xml | 9 ++++-----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/addons/mail/models/mail_channel.py b/addons/mail/models/mail_channel.py
index 63bd82a5f87..ed1b7b771a7 100644
--- a/addons/mail/models/mail_channel.py
+++ b/addons/mail/models/mail_channel.py
@@ -20,6 +20,7 @@ class ChannelPartner(models.Model):
     _rec_name = 'partner_id'
 
     partner_id = fields.Many2one('res.partner', string='Recipient', ondelete='cascade')
+    partner_email = fields.Char('Email', related='partner_id.email')
     channel_id = fields.Many2one('mail.channel', string='Channel', ondelete='cascade')
     seen_message_id = fields.Many2one('mail.message', string='Last Seen')
     fold_state = fields.Selection([('open', 'Open'), ('folded', 'Folded'), ('closed', 'Closed')], string='Conversation Fold State', default='open')
diff --git a/addons/mail/views/mail_channel_views.xml b/addons/mail/views/mail_channel_views.xml
index e78ed6928ba..0c1ed4a8ce9 100644
--- a/addons/mail/views/mail_channel_views.xml
+++ b/addons/mail/views/mail_channel_views.xml
@@ -120,11 +120,10 @@
                                 </group>
                             </page>
                             <page string="Members">
-                                <field name="channel_partner_ids" mode="tree">
-                                    <tree string="members">
-                                        <field name="name"/>
-                                        <field name="email"/>
-                                        <field name="im_status"/>
+                                <field name="channel_last_seen_partner_ids" mode="tree" context="{'active_test': False}">
+                                    <tree string="Members" editable="bottom">
+                                        <field name="partner_id"/>
+                                        <field name="partner_email"/>
                                     </tree>
                                 </field>
                             </page>