From 53e1fc5d87e5d03f0a7dc2ca8aa37336f47d895d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Thibault=20Delavall=C3=A9e?= <tde@openerp.com>
Date: Wed, 29 Apr 2015 13:47:47 +0200
Subject: [PATCH] [FIX] mail: mail_template: issues linked to the migration -
 do not browse a browse record - removed onchange from the form view, as they
 are managed directly by the new API

---
 addons/mail/models/mail_template.py       |  2 +-
 addons/mail/views/mail_template_views.xml | 11 ++++-------
 2 files changed, 5 insertions(+), 8 deletions(-)

diff --git a/addons/mail/models/mail_template.py b/addons/mail/models/mail_template.py
index 3b654d6ec5c..a7e02b6ee63 100644
--- a/addons/mail/models/mail_template.py
+++ b/addons/mail/models/mail_template.py
@@ -164,7 +164,7 @@ class MailTemplate(models.Model):
     def onchange_model_id(self):
         # TDE CLEANME: should'nt it be a stored related ?
         if self.model_id:
-            self.model = self.env['ir.model'].browse(self.model_id).model
+            self.model = self.model_id.model
         else:
             self.model = False
 
diff --git a/addons/mail/views/mail_template_views.xml b/addons/mail/views/mail_template_views.xml
index 4eada82f810..5ba30950ef0 100644
--- a/addons/mail/views/mail_template_views.xml
+++ b/addons/mail/views/mail_template_views.xml
@@ -10,7 +10,7 @@
                         <div class="oe_title">
                             <label for="name" class="oe_edit_only"/><h1><field name="name" required="1"/></h1>
                             <group>
-                                <field name="model_id" required="1" on_change="onchange_model_id(model_id)" options="{'no_create': True}"/>
+                                <field name="model_id" required="1" options="{'no_create': True}"/>
                                 <field name="model" invisible="1"/>
                             </group>
                         </div>
@@ -67,15 +67,12 @@
                             <page string="Dynamic Placeholder Generator">
                                 <group>
                                     <field name="model_object_field"
-                                            domain="[('model_id','=',model_id),('ttype','!=','one2many'),('ttype','!=','many2many')]"
-                                            on_change="onchange_sub_model_object_value_field(model_object_field)"/>
+                                            domain="[('model_id','=',model_id),('ttype','!=','one2many'),('ttype','!=','many2many')]"/>
                                     <field name="sub_object" readonly="1"/>
                                     <field name="sub_model_object_field"
                                             domain="[('model_id','=',sub_object),('ttype','!=','one2many'),('ttype','!=','many2many')]"
-                                            attrs="{'readonly':[('sub_object','=',False)],'required':[('sub_object','!=',False)]}"
-                                            on_change="onchange_sub_model_object_value_field(model_object_field,sub_model_object_field)"/>
-                                    <field name="null_value"
-                                            on_change="onchange_sub_model_object_value_field(model_object_field,sub_model_object_field,null_value)"/>
+                                            attrs="{'readonly':[('sub_object','=',False)],'required':[('sub_object','!=',False)]}"/>
+                                    <field name="null_value"/>
                                     <field name="copyvalue"/>
                                 </group>
                             </page>