Turkesh Patel (openERP) has proposed merging 
lp:~openerp-dev/openobject-server/trunk-bug-1009936-tpa into 
lp:openobject-server.

Requested reviews:
  OpenERP Core Team (openerp)
Related bugs:
  Bug #1009936 in OpenERP Server: "Select photo on partner form then set 
company true photo will also changed"
  https://bugs.launchpad.net/openobject-server/+bug/1009936

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-server/trunk-bug-1009936-tpa/+merge/109137

[FIX] set proper condition for photo of the partner.
-- 
https://code.launchpad.net/~openerp-dev/openobject-server/trunk-bug-1009936-tpa/+merge/109137
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-server/trunk-bug-1009936-tpa.
=== modified file 'openerp/addons/base/res/res_partner.py'
--- openerp/addons/base/res/res_partner.py	2012-05-31 07:11:34 +0000
+++ openerp/addons/base/res/res_partner.py	2012-06-07 13:08:24 +0000
@@ -208,9 +208,12 @@
         default.update({'name': _('%s (copy)')%(name), 'events':[]})
         return super(res_partner, self).copy(cr, uid, id, default, context)
 
-    def onchange_type(self, cr, uid, ids, is_company, context=None):
-        value = {'title': False,
-                 'photo': self._get_photo(cr, uid, is_company, context)}
+    def onchange_type(self, cr, uid, ids, is_company, photo, context=None):
+        value = {'title': False}
+        if photo != self._get_photo(cr, uid, True, context) and photo != self._get_photo(cr, uid, False, context):
+            value.update({'photo': photo})
+        else:
+            value.update({'photo': self._get_photo(cr, uid, is_company, context)})
         if is_company:
             value['parent_id'] = False
             domain = {'title': [('domain', '=', 'partner')]}

=== modified file 'openerp/addons/base/res/res_partner_view.xml'
--- openerp/addons/base/res/res_partner_view.xml	2012-05-31 11:26:35 +0000
+++ openerp/addons/base/res/res_partner_view.xml	2012-06-07 13:08:24 +0000
@@ -330,7 +330,7 @@
                                     on_change="onchange_address(use_parent_address, parent_id)"/>
                         </group>
                         <group col="2">
-                            <field name="is_company" on_change="onchange_type(is_company)"/>
+                            <field name="is_company" on_change="onchange_type(is_company,photo)"/>
                             <field name="customer"/>
                             <field name="supplier"/>
                         </group>

_______________________________________________
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