View reorganization

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 = []

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():

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,14 +23,26 @@
                 <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"/>
+
+            <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"/>
-                <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"/>
+                    <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>
+                
+            <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"/>