[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
This commit is contained in:
parent
2fc7eed0d8
commit
f6986c8cff
|
@ -642,7 +642,6 @@ class account_account(osv.osv):
|
||||||
done_list = []
|
done_list = []
|
||||||
account = self.browse(cr, uid, id, context=context)
|
account = self.browse(cr, uid, id, context=context)
|
||||||
new_child_ids = []
|
new_child_ids = []
|
||||||
default.setdefault('code', _("%s (copy)") % (account['code'] or ''))
|
|
||||||
if not local:
|
if not local:
|
||||||
done_list = []
|
done_list = []
|
||||||
if account.id in 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)]
|
default['child_parent_ids'] = [(6, 0, new_child_ids)]
|
||||||
else:
|
else:
|
||||||
default['child_parent_ids'] = False
|
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)
|
return super(account_account, self).copy(cr, uid, id, default, context=context)
|
||||||
|
|
||||||
def _check_moves(self, cr, uid, ids, method, context=None):
|
def _check_moves(self, cr, uid, ids, method, context=None):
|
||||||
|
|
Loading…
Reference in New Issue