diff options
| author | Thibault Francois <tfr@openerp.com> | 2020-08-14 12:29:32 +0000 |
|---|---|---|
| committer | Thibault Francois <tfr@openerp.com> | 2020-08-14 15:12:21 +0000 |
| commit | f7d478999a5172b0fb726774e2d95ff2631b1e17 (patch) | |
| tree | dc9627210184228629cb8005c614bc39efecead7 /addons | |
| parent | eee563ae1a0c17dbecefbbaa64bd4765c8a80350 (diff) | |
[FIX] website_crm_partner_assign: date_partner_assign
Problem:
Now that date_partner_assign is a computed field, a
value should be set each time there is a value
set for partner_assigned_id. Still, it was not the case
Solution:
Use the proper decorator: depends instead of onchange.
closes odoo/odoo#55950
X-original-commit: 56d859d85f1e00029a601e9b1daa6bbcbdb7dc43
Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
Diffstat (limited to 'addons')
| -rw-r--r-- | addons/website_crm_partner_assign/models/crm_lead.py | 2 | ||||
| -rw-r--r-- | addons/website_crm_partner_assign/tests/test_partner_assign.py | 1 |
2 files changed, 2 insertions, 1 deletions
diff --git a/addons/website_crm_partner_assign/models/crm_lead.py b/addons/website_crm_partner_assign/models/crm_lead.py index 09735aed057..cb24e1a462d 100644 --- a/addons/website_crm_partner_assign/models/crm_lead.py +++ b/addons/website_crm_partner_assign/models/crm_lead.py @@ -30,7 +30,7 @@ class CrmLead(models.Model): fields += ['partner_latitude', 'partner_longitude', 'partner_assigned_id', 'date_partner_assign'] return super(CrmLead, self)._merge_data(fields) - @api.onchange("partner_assigned_id") + @api.depends("partner_assigned_id") def _compute_date_partner_assign(self): for lead in self: if not lead.partner_assigned_id: diff --git a/addons/website_crm_partner_assign/tests/test_partner_assign.py b/addons/website_crm_partner_assign/tests/test_partner_assign.py index 4a376b4b132..5869baf7dc1 100644 --- a/addons/website_crm_partner_assign/tests/test_partner_assign.py +++ b/addons/website_crm_partner_assign/tests/test_partner_assign.py @@ -74,6 +74,7 @@ class TestPartnerAssign(TransactionCase): self.assertEqual(lead.partner_assigned_id, partner_uk, "Opportuniy is not assigned nearest partner") self.assertTrue(50 < lead.partner_latitude < 55, "Latitude is wrong: 50 < %s < 55" % lead.partner_latitude) self.assertTrue(-4 < lead.partner_longitude < -1, "Longitude is wrong: -4 < %s < -1" % lead.partner_longitude) + self.assertTrue(lead.date_partner_assign, "Partner Assignment Date should be set") # I forward this opportunity to its nearest partner. context = dict(self.env.context, default_model='crm.lead', default_res_id=lead.id, active_ids=lead.ids) |