From f338b963b896847e40a8a561c513dd69be84527d Mon Sep 17 00:00:00 2001 From: "ARA (OpenERP)" Date: Tue, 11 Jan 2011 11:33:18 +0530 Subject: [PATCH] [REF]account:improve variable name bzr revid: ara@tinyerp.com-20110111060318-ctjohf91qc6xotfe --- addons/account/account.py | 20 ++++++++++++-------- 1 file changed, 12 insertions(+), 8 deletions(-) diff --git a/addons/account/account.py b/addons/account/account.py index f5c9392723d..bc8c2a0f775 100755 --- a/addons/account/account.py +++ b/addons/account/account.py @@ -420,15 +420,17 @@ class account_account(osv.osv): return True def _check_type(self, cr, uid, ids, context=None): - type = self.browse(cr, uid, ids, context=context) - for t in type: - if t.parent_id.type != 'view' and t.parent_id: + if context is None: + context = {} + accounts = self.browse(cr, uid, ids, context=context) + for account in accounts: + if account.parent_id.type != 'view' and account.parent_id: return False return True _constraints = [ (_check_recursion, 'Error ! You can not create recursive accounts.', ['parent_id']), - (_check_type, 'You cannot create a account! \nMake sure if the account has children it should be type "View"! ', ['parent_id']), + (_check_type, 'You cannot create a account! \nMake sure if the account has parent it should be type "View"! ', ['parent_id']), ] _sql_constraints = [ ('code_company_uniq', 'unique (code,company_id)', 'The code of the account must be unique per company !') @@ -2304,16 +2306,18 @@ class account_account_template(osv.osv): } def _check_type(self, cr, uid, ids, context=None): - type = self.browse(cr, uid, ids, context=context) - for t in type: - if t.parent_id.type != 'view' and t.parent_id: + if context is None: + context = {} + accounts = self.browse(cr, uid, ids, context=context) + for account in accounts: + if account.parent_id.type != 'view' and account.parent_id: return False return True _check_recursion = check_cycle _constraints = [ (_check_recursion, 'Error ! You can not create recursive account templates.', ['parent_id']), - (_check_type, 'You cannot create a account template! \nMake sure if the account template has children it should be type "View"! ', ['parent_id']), + (_check_type, 'You cannot create a account template! \nMake sure if the account template has parent it should be type "View"! ', ['parent_id']), ]