From 879e421368772bd30f6d683056e48995cbd182bf Mon Sep 17 00:00:00 2001
From: Ronald Portier <ronald@therp.nl>
Date: Thu, 9 Oct 2014 12:35:59 +0200
Subject: [PATCH] [ENH] Module partner_relations:     - Edit relations in
 pop-up form instead of through inline tree.

---
 partner_relations/view/res_partner.xml | 50 +++++++++++++++++++++++++-
 1 file changed, 49 insertions(+), 1 deletion(-)

diff --git a/partner_relations/view/res_partner.xml b/partner_relations/view/res_partner.xml
index c1426cc9d..74084d054 100644
--- a/partner_relations/view/res_partner.xml
+++ b/partner_relations/view/res_partner.xml
@@ -1,6 +1,7 @@
 <openerp>
     <data>
         <record id="view_res_partner_filter" model="ir.ui.view">
+            <field name="name">partner_relations.view_partner_filter</field>
             <field name="inherit_id" ref="base.view_res_partner_filter" />
             <field name="model">res.partner</field>
             <field type="xml" name="arch">
@@ -15,6 +16,7 @@
             </field>
         </record>
         <record id="view_partner_form" model="ir.ui.view">
+            <field name="name">partner_relations.view_partner_form</field>
             <field name="inherit_id" ref="base.view_partner_form" />
             <field name="model">res.partner</field>
             <field type="xml" name="arch">
@@ -30,7 +32,6 @@
                                 }"
                             >
                                 <tree
-                                    editable="top"
                                     colors="gray:is_relation_expired==True or not active;blue:is_relation_future==True"
                                 >
                                     <field
@@ -79,6 +80,53 @@
                                         invisible="True"
                                     />
                                 </tree>
+                                <form>
+                                    <field
+                                        name="type_selection_id"
+                                        required="True"
+                                        context="{
+                                            'parent_model': 'res.partner',
+                                            'parent_id': parent.id,
+                                        }"
+                                        domain="[
+                                            '|',
+                                                ('contact_type_this', '=', parent.is_company and 'c' or 'p'),
+                                                ('contact_type_this', '=', False),
+                                            ('search_partner_category_this', '=', parent.category_id),
+
+                                        ]
+                                        "
+                                        options="{'create': false, 'create_edit': false}"
+                                        on_change="on_change_type_selection_id(type_selection_id)"
+                                    /> 
+                                    <field name="type_id" invisible="True" />
+                                    <field
+                                        name="partner_id_display"
+                                        required="True"
+                                        attrs="{
+                                            'readonly': [('type_selection_id','=',False)],
+                                        }"
+                                        options="{'create': false, 'create_edit': false}"
+                                        widget="many2one_clickable"
+                                    />
+                                    <field
+                                        name="date_start"
+                                    />
+                                    <field
+                                        name="date_end"
+                                    />
+                                    <field
+                                        name="active"
+                                    />
+                                    <field
+                                        name="is_relation_expired"
+                                        invisible="True"
+                                    />
+                                    <field
+                                        name="is_relation_future"
+                                        invisible="True"
+                                    />
+                                </form>
                             </field>
                         </page>
                     </xpath>