Ila Rana(Open ERP) has proposed merging 
lp:~openerp-dev/openobject-addons/6.0-opw-5967-ira into 
lp:openobject-addons/6.0.

Requested reviews:
  OpenERP Core Team (openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/6.0-opw-5967-ira/+merge/63846

When we add a job for a partner with the module base_contact, We get wrong 
contact for the partner.To overcome this issue, I have made changes in 
on_change_name() function, also the on_change function put in form view, and 
deleting on_change calling function from tree view.
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/6.0-opw-5967-ira/+merge/63846
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/6.0-opw-5967-ira.
=== modified file 'base_contact/base_contact.py'
--- base_contact/base_contact.py	2011-04-14 05:53:21 +0000
+++ base_contact/base_contact.py	2011-06-08 11:11:39 +0000
@@ -214,7 +214,10 @@
         'state': lambda *a: 'current',
     }
     
-    def onchange_name(self, cr, uid, ids, address_id='', name='', context=None):    
+    def onchange_name(self, cr, uid, ids, address_id='', name='', context=None):
+        if name:
+            adrs = self.pool.get('res.partner').address_get(cr, uid, [name],['default'])
+            address_id = adrs and adrs['default'] or False    
         return {'value': {'address_id': address_id}, 'domain':{'partner_id':'name'}}     
     
     def onchange_partner(self, cr, uid, _, partner_id, context=None):

=== modified file 'base_contact/base_contact_view.xml'
--- base_contact/base_contact_view.xml	2011-01-14 00:11:01 +0000
+++ base_contact/base_contact_view.xml	2011-06-08 11:11:39 +0000
@@ -53,8 +53,8 @@
                                <group colspan="4" col="4">
                                    <field name="sequence_contact" />
                                    <field name="function"/>
-                                   <field name="name" on_change="onchange_name(name,address_id)"/> 
-                                   <field name="address_id"/>                                   
+                                   <field name="name" on_change="onchange_name(address_id,name)"/> 
+                                   <field name="address_id" attrs="{'required': [('name', '!=', False)]}" on_change="onchange_address(address_id)"/>                                   
                                    <field name="date_start" />
                                    <field name="date_stop" />
                                    <field name="state" />
@@ -72,8 +72,8 @@
                             <tree string="Functions and Addresses">
                                 <field name="sequence_contact" string="Seq."/>
                                 <field name="function"/>
-                                <field name="name" on_change="onchange_partner(name)"/> 
-								<field name="address_id" attrs="{'required': [('name', '!=', False)]}" on_change="onchange_address(address_id)"/>
+                                <field name="name"/> 
+								<field name="address_id"/>
                                 <field name="phone"/>
                                 <field name="fax"/>
                                 <field name="extension"/>
@@ -373,7 +373,7 @@
         <field name="type">form</field>
         <field name="arch" type="xml">
             <form string="Contact Functions">
-	             <field name="name" select="1" on_change="onchange_partner(name)"/>
+	             <field name="name" select="1" on_change="onchange_name(address_id,name)"/>
 	             <field name="address_id" select="1" attrs="{'required': [('name', '!=', False)]}" on_change="onchange_address(address_id)"/>
 	             <field name="contact_id" select="1"/>
 	             <field name="function" select="1"/>

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to