From 1bed52ec672c5d045f9c6fe91e455389f2dcdb7f Mon Sep 17 00:00:00 2001 From: "Quentin (OpenERP)" Date: Thu, 7 Mar 2013 10:04:08 +0100 Subject: [PATCH] [FIX] account, duplicate of account.tax: better fix for problem fixed in revision 8807. fixed the duplicate of tax (wasn't working because of a unique constraint on name field). The previous fix was working only for taxes that doesn't have any child. The reason is that copy() is never called for one2many fields, only copy_data() is called (and give the same results for other fields) bzr revid: qdp-launchpad@openerp.com-20130307090408-cxb1qcfz4n9ksrg5 --- addons/account/account.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/addons/account/account.py b/addons/account/account.py index 660976389b3..0631bbfff7d 100644 --- a/addons/account/account.py +++ b/addons/account/account.py @@ -1850,13 +1850,13 @@ class account_tax(osv.osv): return result in the context Ex: result=round(price_unit*0.21,4) """ - def copy(self, cr, uid, id, default=None, context=None): + def copy_data(self, cr, uid, id, default=None, context=None): if default is None: default = {} name = self.read(cr, uid, id, ['name'], context=context)['name'] default = default.copy() default.update({'name': name + _(' (Copy)')}) - return super(account_tax, self).copy(cr, uid, id, default=default, context=context) + return super(account_tax, self).copy_data(cr, uid, id, default=default, context=context) def get_precision_tax():