From 6cb526fe91b8b341f7dc050dbcdabc89d011eabd Mon Sep 17 00:00:00 2001 From: Bhavesh Odedra Date: Wed, 4 Aug 2021 16:34:41 +0000 Subject: [PATCH] [FIX] account: check if tax name on copy method With this commit, it will check if the tax name is given by external method and take that name if any. For example, vals = {'rate': 5.2, 'name': 'Odoo New Tax'} tax_template = self.env['account.tax'].search(domain, limit=1) new_tax = tax_template.copy(default=vals) Current Behavior: new_tax.name => 'Odoo New Tax (Copy)' Instead of 'Odoo New tax' closes odoo/odoo#75406 X-original-commit: 10a5ecceead2f450822387c6f1f2d98f98317a15 Signed-off-by: oco-odoo --- addons/account/models/account_tax.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/addons/account/models/account_tax.py b/addons/account/models/account_tax.py index 7f16d1894a5..be7f55979f0 100644 --- a/addons/account/models/account_tax.py +++ b/addons/account/models/account_tax.py @@ -214,7 +214,9 @@ class AccountTax(models.Model): @api.returns('self', lambda value: value.id) def copy(self, default=None): - default = dict(default or {}, name=_("%s (Copy)", self.name)) + default = dict(default or {}) + if 'name' not in default: + default['name'] = _("%s (Copy)") % self.name return super(AccountTax, self).copy(default=default) def name_get(self):