diff --git a/addons/crm/wizard/crm_lead_to_opportunity.py b/addons/crm/wizard/crm_lead_to_opportunity.py index ccf85a36422..4043adcfb37 100644 --- a/addons/crm/wizard/crm_lead_to_opportunity.py +++ b/addons/crm/wizard/crm_lead_to_opportunity.py @@ -26,7 +26,7 @@ from tools.translate import _ class crm_lead2opportunity(osv.osv_memory): _name = 'crm.lead2opportunity' _description = 'Lead To Opportunity' - + def action_cancel(self, cr, uid, ids, context=None): """ Closes Lead To Opportunity form @@ -38,74 +38,80 @@ class crm_lead2opportunity(osv.osv_memory): """ return {'type': 'ir.actions.act_window_close'} - + def action_apply(self, cr, uid, ids, context=None): """ This converts lead to opportunity and opens Opportunity view - @param self: The object pointer - @param cr: the current row, from the database cursor, - @param uid: the current user’s ID for security checks, - @param ids: List of Lead to Opportunity IDs - @param context: A standard dictionary for contextual values + @param ids: ids of the leads to convert to opportunities - @return : Dictionary value for created Opportunity form + @return : View dictionary opening the Opportunity form view """ - record_id = context and context.get('active_id', False) or False + record_id = context and context.get('active_id') or False if not record_id: return {} - lead_obj = self.pool.get('crm.lead') - data_obj = self.pool.get('ir.model.data') + leads = self.pool.get('crm.lead') + models_data = self.pool.get('ir.model.data') # Get Opportunity views - result = data_obj._get_id(cr, uid, 'crm', 'view_crm_case_opportunities_filter') - res = data_obj.read(cr, uid, result, ['res_id']) - id2 = data_obj._get_id(cr, uid, 'crm', 'crm_case_form_view_oppor') - id3 = data_obj._get_id(cr, uid, 'crm', 'crm_case_tree_view_oppor') - if id2: - id2 = data_obj.browse(cr, uid, id2, context=context).res_id - if id3: - id3 = data_obj.browse(cr, uid, id3, context=context).res_id + result = models_data._get_id( + cr, uid, 'crm', 'view_crm_case_opportunities_filter') + opportunity_view_search = models_data.browse( + cr, uid, result, context=context).res_id + opportunity_view_form = models_data._get_id( + cr, uid, 'crm', 'crm_case_form_view_oppor') + opportunity_view_tree = models_data._get_id( + cr, uid, 'crm', 'crm_case_tree_view_oppor') + if opportunity_view_form: + opportunity_view_form = models_data.browse( + cr, uid, opportunity_view_form, context=context).res_id + if opportunity_view_tree: + opportunity_view_tree = models_data.browse( + cr, uid, opportunity_view_tree, context=context).res_id - lead = lead_obj.browse(cr, uid, record_id, context=context) + lead = leads.browse(cr, uid, record_id, context=context) for this in self.browse(cr, uid, ids, context=context): vals ={ - 'planned_revenue': this.planned_revenue, + 'planned_revenue': this.planned_revenue, 'probability': this.probability, - 'name': this.name, - 'partner_id': this.partner_id.id, - 'user_id': (this.partner_id.user_id and this.partner_id.user_id.id) or (lead.user_id and lead.user_id.id), + 'name': this.name, + 'partner_id': this.partner_id.id, + 'user_id': (this.partner_id.user_id and this.partner_id.user_id.id) or (lead.user_id and lead.user_id.id), 'type': 'opportunity' } - lead_obj.write(cr, uid, lead.id, vals, context=context) - lead_obj.history(cr, uid, [lead], _('Opportunity'), details='Converted to Opportunity', context=context) + lead.write(vals, context=context) + leads.history(cr, uid, [lead], _('Opportunity'), details='Converted to Opportunity', context=context) if lead.partner_id: msg_ids = [ x.id for x in lead.message_ids] - self.pool.get('mailgate.message').write(cr, uid, msg_ids, {'partner_id': lead.partner_id.id}, context=context) - message = _('Lead ') + " '" + lead.name + "' "+ _("is converted to Opportunity.") - self.log(cr, uid, lead.id, message) - value = { - 'name': _('Opportunity'), - 'view_type': 'form', - 'view_mode': 'form,tree', - 'res_model': 'crm.lead', - 'domain': [('type', '=', 'opportunity')], - 'res_id': int(lead.id), - 'view_id': False, - 'views': [(id2, 'form'), (id3, 'tree'), (False, 'calendar'), (False, 'graph')], - 'type': 'ir.actions.act_window', - 'search_view_id': res['res_id'] + self.pool.get('mailgate.message').write(cr, uid, msg_ids, { + 'partner_id': lead.partner_id.id + }, context=context) + self.log(cr, uid, lead.id, + _("Lead '%s' has been converted to an opportunity.") % lead.name) + + return { + 'name': _('Opportunity'), + 'view_type': 'form', + 'view_mode': 'form,tree', + 'res_model': 'crm.lead', + 'domain': [('type', '=', 'opportunity')], + 'res_id': int(lead.id), + 'view_id': False, + 'views': [(opportunity_view_form, 'form'), + (opportunity_view_tree, 'tree'), + (False, 'calendar'), (False, 'graph')], + 'type': 'ir.actions.act_window', + 'search_view_id': opportunity_view_search } - return value _columns = { - 'name' : fields.char('Opportunity', size=64, required=True, select=1), - 'probability': fields.float('Success Rate (%)'), - 'planned_revenue': fields.float('Expected Revenue'), - 'partner_id': fields.many2one('res.partner', 'Partner'), + 'name' : fields.char('Opportunity', size=64, required=True, select=1), + 'probability': fields.float('Success Rate (%)'), + 'planned_revenue': fields.float('Expected Revenue'), + 'partner_id': fields.many2one('res.partner', 'Partner'), } - + def view_init(self, cr, uid, fields, context=None): """ This function checks for precondition before wizard executes @@ -154,8 +160,8 @@ class crm_lead2opportunity_partner(osv.osv_memory): _inherit = 'crm.lead2partner' _columns = { - 'partner_id': fields.many2one('res.partner', 'Partner'), - 'action': fields.selection([('exist', 'Link to an existing partner'), ('create', 'Create a new partner'), ('no','Do not create a partner')], 'Action'), + 'partner_id': fields.many2one('res.partner', 'Partner'), + 'action': fields.selection([('exist', 'Link to an existing partner'), ('create', 'Create a new partner'), ('no','Do not create a partner')], 'Action'), } def make_partner(self, cr, uid, ids, context=None): @@ -171,7 +177,7 @@ class crm_lead2opportunity_partner(osv.osv_memory): """ if not context: context = {} - + partner_ids = self._create_partner(cr, uid, ids, context) value = {} data_obj = self.pool.get('ir.model.data') @@ -179,18 +185,18 @@ class crm_lead2opportunity_partner(osv.osv_memory): view_id = False if data_id: view_id = data_obj.browse(cr, uid, data_id, context=context).res_id - + context.update({'partner_id': partner_ids}) - value = { - 'name': _('Create Opportunity'), - 'view_type': 'form', - 'view_mode': 'form,tree', - 'res_model': 'crm.lead2opportunity.action', - 'view_id': False, - 'context': context, - 'views': [(view_id, 'form')], - 'type': 'ir.actions.act_window', - 'target': 'new', + value = { + 'name': _('Create Opportunity'), + 'view_type': 'form', + 'view_mode': 'form,tree', + 'res_model': 'crm.lead2opportunity.action', + 'view_id': False, + 'context': context, + 'views': [(view_id, 'form')], + 'type': 'ir.actions.act_window', + 'target': 'new', } return value @@ -213,19 +219,19 @@ class crm_lead2opportunity_partner(osv.osv_memory): view_id = data_obj.browse(cr, uid, data_id, context=context).res_id context.update({'partner_id': False}) - value = { - 'name': _('Create Opportunity'), - 'view_type': 'form', - 'view_mode': 'form,tree', - 'res_model': 'crm.lead2opportunity', - 'view_id': False, - 'context': context, - 'views': [(view_id, 'form')], - 'type': 'ir.actions.act_window', - 'target': 'new', + value = { + 'name': _('Create Opportunity'), + 'view_type': 'form', + 'view_mode': 'form,tree', + 'res_model': 'crm.lead2opportunity', + 'view_id': False, + 'context': context, + 'views': [(view_id, 'form')], + 'type': 'ir.actions.act_window', + 'target': 'new', } return value - + def view_init(self, cr, uid, fields, context=None): """ @@ -278,30 +284,30 @@ class crm_lead2opportunity_action(osv.osv_memory): if data_id: view_id = data_obj.browse(cr, uid, data_id, context=context).res_id value = { - 'name': _('Create Opportunity'), - 'view_type': 'form', - 'view_mode': 'form,tree', - 'res_model': 'crm.lead2opportunity', - 'view_id': False, - 'context': context, - 'views': [(view_id, 'form')], - 'type': 'ir.actions.act_window', - 'target': 'new', + 'name': _('Create Opportunity'), + 'view_type': 'form', + 'view_mode': 'form,tree', + 'res_model': 'crm.lead2opportunity', + 'view_id': False, + 'context': context, + 'views': [(view_id, 'form')], + 'type': 'ir.actions.act_window', + 'target': 'new', } elif this.name == 'merge': data_id = data_obj._get_id(cr, uid, 'crm', 'merge_opportunity_form') if data_id: view_id = data_obj.browse(cr, uid, data_id, context=context).res_id value = { - 'name': _('Merge with Existing Opportunity'), - 'view_type': 'form', - 'view_mode': 'form,tree', - 'res_model': 'crm.merge.opportunity', - 'view_id': False, - 'context': context, - 'views': [(view_id, 'form')], - 'type': 'ir.actions.act_window', - 'target': 'new', + 'name': _('Merge with Existing Opportunity'), + 'view_type': 'form', + 'view_mode': 'form,tree', + 'res_model': 'crm.merge.opportunity', + 'view_id': False, + 'context': context, + 'views': [(view_id, 'form')], + 'type': 'ir.actions.act_window', + 'target': 'new', } return value