Merge lp:~therp-nl/ocb-server/lp1210186 into lp:ocb-server

[Holger Brunn (Therp)]

[Stefan Rijnhart (Opener) (stefan-opener)]

Thanks! I checked _commercial_sync_to_children and all it does is indeed update the children's commercial fields so it seems clear that this will not be needed if only non-commercial fields are written to the commercial partner.

review: Approve

[Stefan Rijnhart (Opener) (stefan-opener)]

Rejecting, because the same proposal upstream has been merged in upstream
(http://bazaar.launchpad.net/~openerp/openobject-server/7.0/revision/5056)

Unmerged revisions

5060. By Holger Brunn (Therp) on 2013-08-08

[FIX] only descend to children when necessary

=== modified file 'openerp/addons/base/res/res_partner.py'
--- openerp/addons/base/res/res_partner.py	2013-06-12 09:53:00 +0000
+++ openerp/addons/base/res/res_partner.py	2013-08-08 16:14:37 +0000
@@ -465,7 +465,10 @@
         # 2. To DOWNSTREAM: sync children 
         if partner.child_ids:
             # 2a. Commercial Fields: sync if commercial entity
-            if partner.commercial_partner_id == partner:
+            if partner.commercial_partner_id == partner and any(
+                    field in update_values 
+                    for field in self._commercial_fields(
+                        cr, uid, context=context)):
                 self._commercial_sync_to_children(cr, uid, partner, context=context)
             # 2b. Address fields: sync if address changed
             address_fields = self._address_fields(cr, uid, context=context)