From 39ace35f396d7ad27e50a304592113cbe2a355b3 Mon Sep 17 00:00:00 2001
From: Veronika Kotovich <veronika.kotovich@gmail.com>
Date: Fri, 28 Aug 2015 13:37:19 +0300
Subject: [PATCH] View reorganization

---
 controllers/main.py         |  1 -
 mail_move_message_models.py |  4 ++--
 mail_move_message_views.xml | 31 ++++++++++++++++++++++---------
 3 files changed, 24 insertions(+), 12 deletions(-)

diff --git a/controllers/main.py b/controllers/main.py
index 16cbca1..bbc0436 100644
--- a/controllers/main.py
+++ b/controllers/main.py
@@ -16,7 +16,6 @@ class DataSetCustom(DataSet):
                     break
             partner_info = {}
             if contact_field:
-                print res
                 partner_info = model.read([r[0] for r in res], [contact_field])
                 partner_info = dict([(p['id'], p[contact_field]) for p in partner_info])
             final_res = []
diff --git a/mail_move_message_models.py b/mail_move_message_models.py
index 3d4501f..677d6ab 100644
--- a/mail_move_message_models.py
+++ b/mail_move_message_models.py
@@ -67,10 +67,10 @@ class wizard(models.TransientModel):
 
         self.record_url = '/web#id=%s&model=%s' % (self.res_id, self.model)
 
-    @api.onchange('filter_by_partner', 'partner_id')
+    @api.onchange('model', 'filter_by_partner', 'partner_id')
     def on_change_partner(self):
         domain = {'res_id': []}
-        if self.filter_by_partner and self.partner_id:
+        if self.model and self.filter_by_partner and self.partner_id:
             fields = self.env[self.model].fields_get(False)
             contact_field = False
             for n, f in fields.iteritems():
diff --git a/mail_move_message_views.xml b/mail_move_message_views.xml
index b6967c9..28b7fb1 100644
--- a/mail_move_message_views.xml
+++ b/mail_move_message_views.xml
@@ -13,6 +13,7 @@
     <field name="model">mail_move_message.wizard</field>
     <field name="arch" type="xml">
         <form string="Move Message">
+
             <field name="can_move" invisible="1"/>
             <field name="message_is_moved" invisible="1"/>
 
@@ -22,16 +23,28 @@
                 <field name="message_moved_by_user_id"/>
                 <button name="open_moved_by_message_id" string="Open message" type="object" class="oe_highlight"/>
             </group>
-            <group attrs="{'invisible':[('can_move', '=', False)]}" col="4">
-                <field name="move_back" attrs="{'invisible':[('message_is_moved','=',False)]}" colspan="4"/>
-                <field name="model" class="oe_inline" widget="polymorphic" polymorphic="res_id"/>
-                <field name="filter_by_partner"/>
-                <field name="partner_id"/>
-                <field name="res_id" colspan="2" context="{'extended_name_with_contact':1}" widget="many2one" attrs="{'readonly': [('model','=',False)]}"/>
-                <field name="record_url" colspan="4" widget="url"/>
+
+            <group attrs="{'invisible':[('can_move', '=', False)]}" colspan="2">
+                <label for="model"/>
+                <div>
+                    <field name="model" widget="polymorphic" polymorphic="res_id" style="width:100%"/>
+                </div>
+                <label for="filter_by_partner"/>
+                <div>
+                    <field name="filter_by_partner"/>
+                    <field name="partner_id" style="width:50%"/>
+                    <button name="create_partner" type="object" string="Create Partner" attrs="{'invisible':[('partner_id','!=',False)]}"/>
+                </div>
+                <label for="res_id"/>
+                <div>
+                    <field name="res_id" context="{'extended_name_with_contact':1}" widget="many2one" attrs="{'readonly': [('model','=',False)]}" class="oe_inline"/>
+                    <field name="record_url" widget="url" class="oe_inline"/>
+                </div>
             </group>
-                <button name="move" string="Move" type="object" class="oe_highlight" attrs="{'invisible':[('can_move', '=', False)]}"/>
-                <button string="Cancel" class="" special="cancel" />
+                
+            <field name="move_back" attrs="{'invisible':[('message_is_moved','=',False)]}" colspan="4"/>
+            <button name="move" string="Move" type="object" class="oe_highlight" attrs="{'invisible':[('can_move', '=', False)]}"/>
+            <button string="Cancel" class="" special="cancel" />
             <separator string="Message"/>
             <div>
                 <field name="message_id" invisible="1"/>