From 8035df3e1152a8f9d9703037d9d1bee718d94331 Mon Sep 17 00:00:00 2001 From: "uco (OpenERP)" Date: Mon, 22 Nov 2010 16:07:53 +0530 Subject: [IMP]: Added context=None in function argument for rest modules. bzr revid: uco@tinyerp.com-20101122103753-idv9fdaqc28rvugu --- addons/project_long_term/project_long_term.py | 16 +++++++++++++++- 1 file changed, 15 insertions(+), 1 deletion(-) (limited to 'addons/project_long_term/project_long_term.py') diff --git a/addons/project_long_term/project_long_term.py b/addons/project_long_term/project_long_term.py index faea0916906..6d26f0dd009 100644 --- a/addons/project_long_term/project_long_term.py +++ b/addons/project_long_term/project_long_term.py @@ -70,18 +70,24 @@ class project_phase(osv.osv): return True def _check_dates(self, cr, uid, ids, context=None): + if not context: + context = {} for phase in self.read(cr, uid, ids, ['date_start', 'date_end'], context=context): if phase['date_start'] and phase['date_end'] and phase['date_start'] > phase['date_end']: return False return True def _check_constraint_start(self, cr, uid, ids, context=None): + if not context: + context = {} phase = self.read(cr, uid, ids[0], ['date_start', 'constraint_date_start'], context=context) if phase['date_start'] and phase['constraint_date_start'] and phase['date_start'] < phase['constraint_date_start']: return False return True def _check_constraint_end(self, cr, uid, ids, context=None): + if not context: + context = {} phase = self.read(cr, uid, ids[0], ['date_end', 'constraint_date_end'], context=context) if phase['date_end'] and phase['constraint_date_end'] and phase['date_end'] > phase['constraint_date_end']: return False @@ -96,6 +102,8 @@ class project_phase(osv.osv): res = {} if not ids: return res + if not context: + context = {} for phase in self.browse(cr, uid, ids, context=context): tot = 0.0 for task in phase.task_ids: @@ -137,6 +145,8 @@ class project_phase(osv.osv): def onchange_project(self, cr, uid, ids, project, context=None): result = {} + if not context: + context = {} result['date_start'] = False project_obj = self.pool.get('project.project') if project: @@ -146,6 +156,8 @@ class project_phase(osv.osv): def onchange_days(self, cr, uid, ids, project, context=None): result = {} + if not context: + context = {} for id in ids: project_id = self.browse(cr, uid, id, context=context) newdate = datetime.strptime(project_id.date_start, '%Y-%m-%d') + relativedelta(days=project_id.duration or 0.0) @@ -398,6 +410,8 @@ class project_resource_allocation(osv.osv): def get_name(self, cr, uid, ids, field_name, arg, context=None): res = {} + if not context: + context = {} for allocation in self.browse(cr, uid, ids, context=context): name = allocation.phase_id.name name += ' (%s%%)' %(allocation.useability) @@ -507,7 +521,7 @@ class resource_resource(osv.osv): context = {} if context.get('project_id',False): project_pool = self.pool.get('project.project') - project_rec = project_pool.browse(cr, uid, context['project_id']) + project_rec = project_pool.browse(cr, uid, context['project_id'], context=context) user_ids = [user_id.id for user_id in project_rec.members] args.append(('user_id','in',user_ids)) return super(resource_resource, self).search(cr, uid, args, offset, limit, order, context, count) -- cgit v1.2.3 From 6dd290e8245f4ea91ce87b0f3b5f29bafeaf12e9 Mon Sep 17 00:00:00 2001 From: "uco (OpenERP)" Date: Tue, 23 Nov 2010 12:35:05 +0530 Subject: [REF]: Replace the condition 'if not context' with 'if context is None'. bzr revid: uco@tinyerp.com-20101123070505-9i2lrqhgxzaii3zn --- addons/project_long_term/project_long_term.py | 14 -------------- 1 file changed, 14 deletions(-) (limited to 'addons/project_long_term/project_long_term.py') diff --git a/addons/project_long_term/project_long_term.py b/addons/project_long_term/project_long_term.py index 6d26f0dd009..7094eb80512 100644 --- a/addons/project_long_term/project_long_term.py +++ b/addons/project_long_term/project_long_term.py @@ -70,24 +70,18 @@ class project_phase(osv.osv): return True def _check_dates(self, cr, uid, ids, context=None): - if not context: - context = {} for phase in self.read(cr, uid, ids, ['date_start', 'date_end'], context=context): if phase['date_start'] and phase['date_end'] and phase['date_start'] > phase['date_end']: return False return True def _check_constraint_start(self, cr, uid, ids, context=None): - if not context: - context = {} phase = self.read(cr, uid, ids[0], ['date_start', 'constraint_date_start'], context=context) if phase['date_start'] and phase['constraint_date_start'] and phase['date_start'] < phase['constraint_date_start']: return False return True def _check_constraint_end(self, cr, uid, ids, context=None): - if not context: - context = {} phase = self.read(cr, uid, ids[0], ['date_end', 'constraint_date_end'], context=context) if phase['date_end'] and phase['constraint_date_end'] and phase['date_end'] > phase['constraint_date_end']: return False @@ -102,8 +96,6 @@ class project_phase(osv.osv): res = {} if not ids: return res - if not context: - context = {} for phase in self.browse(cr, uid, ids, context=context): tot = 0.0 for task in phase.task_ids: @@ -145,8 +137,6 @@ class project_phase(osv.osv): def onchange_project(self, cr, uid, ids, project, context=None): result = {} - if not context: - context = {} result['date_start'] = False project_obj = self.pool.get('project.project') if project: @@ -156,8 +146,6 @@ class project_phase(osv.osv): def onchange_days(self, cr, uid, ids, project, context=None): result = {} - if not context: - context = {} for id in ids: project_id = self.browse(cr, uid, id, context=context) newdate = datetime.strptime(project_id.date_start, '%Y-%m-%d') + relativedelta(days=project_id.duration or 0.0) @@ -410,8 +398,6 @@ class project_resource_allocation(osv.osv): def get_name(self, cr, uid, ids, field_name, arg, context=None): res = {} - if not context: - context = {} for allocation in self.browse(cr, uid, ids, context=context): name = allocation.phase_id.name name += ' (%s%%)' %(allocation.useability) -- cgit v1.2.3 From d827b2f6ea63dc04b91a7e59b3e7fecd7c5810d3 Mon Sep 17 00:00:00 2001 From: "uco (OpenERP)" Date: Mon, 13 Dec 2010 12:13:09 +0530 Subject: [REF]: Added remaining context changes in _constraints. Removed unnecessary checking for context from methods. bzr revid: uco@tinyerp.com-20101213064309-vg36f1510xv2f8zb --- addons/project_long_term/project_long_term.py | 17 ----------------- 1 file changed, 17 deletions(-) (limited to 'addons/project_long_term/project_long_term.py') diff --git a/addons/project_long_term/project_long_term.py b/addons/project_long_term/project_long_term.py index 7094eb80512..4640ea8fa48 100644 --- a/addons/project_long_term/project_long_term.py +++ b/addons/project_long_term/project_long_term.py @@ -153,8 +153,6 @@ class project_phase(osv.osv): return {'value': result} def _check_date_start(self, cr, uid, phase, date_end, context=None): - if context is None: - context = {} """ Check And Compute date_end of phase if change in date_start < older time. """ @@ -175,8 +173,6 @@ class project_phase(osv.osv): self.write(cr, uid, [phase.id], {'date_start': dt_start, 'date_end': date_end.strftime('%Y-%m-%d')}, context=context) def _check_date_end(self, cr, uid, phase, date_start, context=None): - if context is None: - context = {} """ Check And Compute date_end of phase if change in date_end > older time. """ @@ -277,8 +273,6 @@ class project_phase(osv.osv): Return a list of Resource Class objects for the resources allocated to the phase. """ res = {} - if context is None: - context = {} resource_pool = self.pool.get('resource.resource') for phase in self.browse(cr, uid, ids, context=context): user_ids = map(lambda x:x.resource_id.user_id.id, phase.resource_ids) @@ -299,8 +293,6 @@ class project_phase(osv.osv): resource_pool = self.pool.get('resource.resource') resource_allocation_pool = self.pool.get('project.resource.allocation') uom_pool = self.pool.get('product.uom') - if context is None: - context = {} default_uom_id = self._get_default_uom_id(cr, uid) for phase in self.browse(cr, uid, ids, context=context): if not phase.responsible_id: @@ -366,8 +358,6 @@ class project_phase(osv.osv): """ task_pool = self.pool.get('project.task') resource_pool = self.pool.get('resource.resource') - if context is None: - context = {} resources_list = self.generate_resources(cr, uid, ids, context=context) return_msg = {} for phase in self.browse(cr, uid, ids, context=context): @@ -431,8 +421,6 @@ class project(osv.osv): """ res = {} resource_pool = self.pool.get('resource.resource') - if context is None: - context = {} for project in self.browse(cr, uid, ids, context=context): user_ids = map(lambda x:x.id, project.members) calendar_id = project.resource_calendar_id and project.resource_calendar_id.id or False @@ -444,8 +432,6 @@ class project(osv.osv): """ Schedule the phases. """ - if context is None: - context = {} if type(ids) in (long, int,): ids = [ids] phase_pool = self.pool.get('project.phase') @@ -471,9 +457,6 @@ class project(osv.osv): user_pool = self.pool.get('res.users') task_pool = self.pool.get('project.task') resource_pool = self.pool.get('resource.resource') - if context is None: - context = {} - resources_list = self.generate_members(cr, uid, ids, context=context) return_msg = {} for project in self.browse(cr, uid, ids, context=context): -- cgit v1.2.3