From 26469cf092ecd090a2b2e35712b7aefcfff76191 Mon Sep 17 00:00:00 2001 From: "Quentin (OpenERP)" Date: Fri, 25 Mar 2011 16:38:54 +0100 Subject: [PATCH] [FIX] account/account_move_line: proper use of set() bzr revid: qdp-launchpad@openerp.com-20110325153854-jtxryt1cw7rm5a7p --- addons/account/account_move_line.py | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/addons/account/account_move_line.py b/addons/account/account_move_line.py index d0dd4b2c805..83a67df9178 100644 --- a/addons/account/account_move_line.py +++ b/addons/account/account_move_line.py @@ -1087,15 +1087,14 @@ class account_move_line(osv.osv): move_obj = self.pool.get('account.move') self._update_check(cr, uid, ids, context) result = False - move_ids = [] + move_ids = set() for line in self.browse(cr, uid, ids, context=context): - if line.move_id not in move_ids: - move_ids.append(line.move_id.id) + move_ids.add(line.move_id.id) context['journal_id'] = line.journal_id.id context['period_id'] = line.period_id.id result = super(account_move_line, self).unlink(cr, uid, [line.id], context=context) - move_ids = list(set(move_ids)) - if check: + move_ids = list(move_ids) + if check and move_ids: move_obj.validate(cr, uid, move_ids, context=context) return result