From ff2fa4220f2cbd1b89279e75cde75efe3a279dca Mon Sep 17 00:00:00 2001 From: Alexis de Lattre Date: Sat, 2 Jul 2016 21:01:04 +0200 Subject: [PATCH] FIX infinite loop --- base_partner_sequence/__openerp__.py | 2 +- base_partner_sequence/models/partner.py | 10 ++++++---- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/base_partner_sequence/__openerp__.py b/base_partner_sequence/__openerp__.py index be79f9951..f4799fefd 100644 --- a/base_partner_sequence/__openerp__.py +++ b/base_partner_sequence/__openerp__.py @@ -34,5 +34,5 @@ 'views/partner_view.xml', ], "active": False, - "installable": True + "installable": True, } diff --git a/base_partner_sequence/models/partner.py b/base_partner_sequence/models/partner.py index 8a6f84274..42ff65159 100644 --- a/base_partner_sequence/models/partner.py +++ b/base_partner_sequence/models/partner.py @@ -23,13 +23,15 @@ class ResPartner(models.Model): """ if not vals and not partner_id: raise Exception('Either field values or an id must be provided.') - vals = vals or {} + if vals is None: + vals = {} + values = vals.copy() # only assign a 'ref' to commercial partners if partner_id: partner = self.browse(partner_id) - vals.setdefault('is_company', partner.is_company) - vals.setdefault('parent_id', partner.parent_id.id) - return vals.get('is_company') or not vals.get('parent_id') + values.setdefault('is_company', partner.is_company) + values.setdefault('parent_id', partner.parent_id.id) + return values.get('is_company') or not values.get('parent_id') @api.model def _commercial_fields(self):