From 0ee404c05470098043eace3167bdf9b9e2a0fd2f Mon Sep 17 00:00:00 2001 From: Olivier Dony Date: Wed, 24 Apr 2013 15:16:25 +0200 Subject: [PATCH] [FIX] res.partner: always call write() with a list of IDs, a single ID is not valid! (even if it works sometimes) bzr revid: odo@openerp.com-20130424131625-aeljtyo5yuiyrlcz --- openerp/addons/base/res/res_company.py | 2 +- openerp/addons/base/res/res_partner.py | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/openerp/addons/base/res/res_company.py b/openerp/addons/base/res/res_company.py index c8ea8d7856e..cd17f6da3d9 100644 --- a/openerp/addons/base/res/res_company.py +++ b/openerp/addons/base/res/res_company.py @@ -253,7 +253,7 @@ class res_company(osv.osv): vals.update({'partner_id': partner_id}) self.cache_restart(cr) company_id = super(res_company, self).create(cr, uid, vals, context=context) - obj_partner.write(cr, uid, partner_id, {'company_id': company_id}, context=context) + obj_partner.write(cr, uid, [partner_id], {'company_id': company_id}, context=context) return company_id def write(self, cr, uid, ids, values, context=None): diff --git a/openerp/addons/base/res/res_partner.py b/openerp/addons/base/res/res_partner.py index 2637b922f8c..7bd7587a8d9 100644 --- a/openerp/addons/base/res/res_partner.py +++ b/openerp/addons/base/res/res_partner.py @@ -433,7 +433,7 @@ class res_partner(osv.osv, format_address): commercial_fields = self._commercial_fields(cr, uid, context=context) sync_vals = self._update_fields_values(cr, uid, partner.commercial_partner_id, commercial_fields, context=context) - return self.write(cr, uid, partner.id, sync_vals, context=context) + partner.write(sync_vals) def _commercial_sync_to_children(self, cr, uid, partner, context=None): """ Handle sync of commercial fields to descendants """ @@ -459,7 +459,7 @@ class res_partner(osv.osv, format_address): use_parent_address=partner.use_parent_address, parent_id=partner.parent_id.id, context=context).get('value', {}) - self.update_address(cr, uid, partner.id, onchange_vals, context=context) + partner.update_address(onchange_vals) # 2. To DOWNSTREAM: sync children if partner.child_ids: