From af533f8d21c6f462568cb526c932dff24e9a4a64 Mon Sep 17 00:00:00 2001
From: Veronika Kotovich <veronika.kotovich@gmail.com>
Date: Wed, 2 Sep 2015 11:53:36 +0300
Subject: [PATCH] Display Delete button only to admin

---
 mail_move_message_models.py | 25 ++++++++++++++++++++-----
 mail_move_message_views.xml |  9 ++++++---
 2 files changed, 26 insertions(+), 8 deletions(-)

diff --git a/mail_move_message_models.py b/mail_move_message_models.py
index 0e0202a..4e2c3f9 100644
--- a/mail_move_message_models.py
+++ b/mail_move_message_models.py
@@ -1,4 +1,4 @@
-import json
+from lxml import etree
 from openerp import api, models, fields, SUPERUSER_ID
 from openerp.tools import email_split
 from openerp.tools.translate import _
@@ -44,14 +44,15 @@ class wizard(models.TransientModel):
     message_moved_by_user_id = fields.Many2one('res.users', related='message_id.moved_by_user_id', string='Moved by', readonly=True)
     message_is_moved = fields.Boolean(string='Is Moved', related='message_id.is_moved', readonly=True)
     parent_id = fields.Many2one('mail.message', string='Search by name', )
-    model = fields.Selection(_model_selection, string='Model')
-    res_id = fields.Integer(string='Record ID')
+    model = fields.Selection(_model_selection, string='Model', required=True)
+    res_id = fields.Integer(string='Record ID', required=True)
     can_move = fields.Boolean('Can move', compute='get_can_move')
     move_back = fields.Boolean('Move to origin', help='Move  message and submessages to original place')
     partner_id = fields.Many2one('res.partner', string='Author', related='message_id.author_id')
     filter_by_partner = fields.Boolean('Filter Records by partner')
     message_email_from = fields.Char()
     message_name_from = fields.Char()
+    message_to_read = fields.Boolean(related='message_id.to_read')
 
     @api.depends('message_id')
     @api.one
@@ -64,8 +65,10 @@ class wizard(models.TransientModel):
         if not self.move_back:
             return
         self.parent_id = self.message_id.moved_from_parent_id
-        self.res_id = self.message_id.moved_from_res_id
-        self.model = self.message_id.moved_from_model
+        model = self.message_id.moved_from_model
+        if model in [s[0] for s in self._model_selection()]:
+            self.model = model
+            self.res_id = self.message_id.moved_from_res_id
 
     @api.onchange('parent_id', 'res_id', 'model')
     def update_move_back(self):
@@ -190,6 +193,18 @@ class wizard(models.TransientModel):
         self.message_id.set_message_read(True)
         return {'type': 'ir.actions.act_window_close'}
 
+    @api.model
+    def fields_view_get(self, view_id=None, view_type='form', context=None, toolbar=False, submenu=False):
+        res = super(wizard, self).fields_view_get(view_id=view_id, view_type=view_type, toolbar=toolbar, submenu=False)
+        doc = etree.XML(res['arch'])
+        nodes = doc.xpath("//button[@name='delete']")
+        for node in nodes:
+            if self.env.uid == 1:
+                node.set('invisible', "1")
+        res['arch'] = etree.tostring(doc)
+        return res
+
+
 
 class mail_message(models.Model):
     _inherit = 'mail.message'
diff --git a/mail_move_message_views.xml b/mail_move_message_views.xml
index 2e00462..349ab7f 100644
--- a/mail_move_message_views.xml
+++ b/mail_move_message_views.xml
@@ -18,6 +18,7 @@
             <field name="message_is_moved" invisible="1"/>
             <field name="message_name_from" invisible="1"/>
             <field name="message_email_from" invisible="1"/>
+            <field name="message_to_read" invisible="1"/>
 
             <p  attrs="{'invisible':[('can_move', '!=', False)]}">You cannot move this message. It was already moved with a message bellow. Open one and apply changes there.</p>
             <group attrs="{'invisible':[('can_move', '!=', False)]}">
@@ -42,12 +43,14 @@
                     <field name="res_id" context="{'extended_name_with_contact':1}" widget="many2one" attrs="{'readonly': [('model','=',False)]}" style="width:50%"/>
                     <button string="Create new record" name="create_record" type="object" class="oe_highlight" attrs="{'invisible':['|',('model','=',False),('res_id','!=',False)]}" special="quick_create" field="res_id" use_for_mail_move_message="True"/>
                 </div>
+                <label for="move_back" attrs="{'invisible':[('message_is_moved','=',False)]}"/>
+                <div attrs="{'invisible':[('message_is_moved','=',False)]}">
+                    <field name="move_back"/>
+                </div>
             </group>
-                
-            <field name="move_back" attrs="{'invisible':[('message_is_moved','=',False)]}"/>
             <button name="move" string="Move" type="object" class="oe_highlight" attrs="{'invisible':[('can_move', '=', False)]}"/>
             <button string="Close" class="" special="cancel" />
-            <button name="read_close" string="Mark as read and Close" type="object" class="oe_highlight"/>
+            <button name="read_close" string="Mark as read and Close" type="object" class="oe_highlight" attrs="{'invisible':[('message_to_read','=',False)]}"/>
             <separator string="Message"/>
             <div>
                 <field name="message_id" invisible="1"/>