From f6986c8cffcfc9f15bdb75010a8bc6ae0ec46951 Mon Sep 17 00:00:00 2001 From: "Stefan Rijnhart (Opener)" Date: Tue, 30 May 2017 13:12:47 +0100 Subject: [PATCH] [FIX] account: fix copy() of accounts having children Don't pass default code to child accounts to avoid raising wrongly the constraint: duplicate key value violates unique constraint "account_account_code_company_uniq" Courtesy of Stefan Rijnhart (Opener). Was PR #16804 --- addons/account/account.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/addons/account/account.py b/addons/account/account.py index 824751a9a2e..655db7065d5 100644 --- a/addons/account/account.py +++ b/addons/account/account.py @@ -642,7 +642,6 @@ class account_account(osv.osv): done_list = [] account = self.browse(cr, uid, id, context=context) new_child_ids = [] - default.setdefault('code', _("%s (copy)") % (account['code'] or '')) if not local: done_list = [] if account.id in done_list: @@ -656,6 +655,7 @@ class account_account(osv.osv): default['child_parent_ids'] = [(6, 0, new_child_ids)] else: default['child_parent_ids'] = False + default.setdefault('code', _("%s (copy)") % (account['code'] or '')) return super(account_account, self).copy(cr, uid, id, default, context=context) def _check_moves(self, cr, uid, ids, method, context=None):