diff --git a/addons/crm/tests/test_crm_lead_convert.py b/addons/crm/tests/test_crm_lead_convert.py index 48502f57154..d20a616d47f 100644 --- a/addons/crm/tests/test_crm_lead_convert.py +++ b/addons/crm/tests/test_crm_lead_convert.py @@ -29,6 +29,32 @@ class TestLeadConvertForm(crm_common.TestLeadConvertCommon): self.assertEqual(wizard.action, 'exist') self.assertEqual(wizard.partner_id, customer) + @users('user_sales_manager') + def test_form_name_onchange(self): + """ Test Lead._find_matching_partner() """ + lead = self.env['crm.lead'].browse(self.lead_1.ids) + lead_dup = lead.copy({'name': 'Duplicate'}) + customer = self.env['res.partner'].create({ + "name": "Amy Wong", + "email": '"Amy, PhD Student, Wong" Tiny ' + }) + + wizard = Form(self.env['crm.lead2opportunity.partner'].with_context({ + 'active_model': 'crm.lead', + 'active_id': lead.id, + 'active_ids': lead.ids, + })) + + self.assertEqual(wizard.name, 'merge') + self.assertEqual(wizard.action, 'exist') + self.assertEqual(wizard.partner_id, customer) + self.assertEqual(wizard.duplicated_lead_ids[:], lead + lead_dup) + + wizard.name = 'convert' + wizard.action = 'create' + self.assertEqual(wizard.action, 'create', 'Should keep user input') + self.assertEqual(wizard.name, 'convert', 'Should keep user input') + @tagged('lead_manage') class TestLeadConvert(crm_common.TestLeadConvertCommon): diff --git a/addons/crm/wizard/crm_lead_to_opportunity.py b/addons/crm/wizard/crm_lead_to_opportunity.py index 72d0e236f64..b24314c75af 100644 --- a/addons/crm/wizard/crm_lead_to_opportunity.py +++ b/addons/crm/wizard/crm_lead_to_opportunity.py @@ -50,7 +50,8 @@ class Lead2OpportunityPartner(models.TransientModel): @api.depends('duplicated_lead_ids') def _compute_name(self): for convert in self: - convert.name = 'merge' if convert.duplicated_lead_ids and len(convert.duplicated_lead_ids) >= 2 else 'convert' + if not convert.name: + convert.name = 'merge' if convert.duplicated_lead_ids and len(convert.duplicated_lead_ids) >= 2 else 'convert' @api.depends('lead_id') def _compute_action(self):