diff options
| author | Florimond Husquinet (fhu) <fhu@odoo.com> | 2020-09-29 14:56:38 +0000 |
|---|---|---|
| committer | Florimond Husquinet (fhu) <fhu@odoo.com> | 2020-09-29 14:56:38 +0000 |
| commit | e497fa0f6de6ae3c0ab3ffe3c3dec43037944d17 (patch) | |
| tree | 679a6f96bab508224db890650bdb480b57e2d8b5 /addons/mail_client_extension | |
| parent | 563eafc0ec2eed7deedd65fc92cdcc687492e3f3 (diff) | |
[FIX] mail_client_extension: checks for empty list in iap data
Instead of not returning the key, the IAP returns an empty list
when there is no phone number or no email.
closes odoo/odoo#58840
Signed-off-by: Xavier Morel (xmo) <xmo@odoo.com>
Diffstat (limited to 'addons/mail_client_extension')
| -rw-r--r-- | addons/mail_client_extension/controllers/main.py | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/addons/mail_client_extension/controllers/main.py b/addons/mail_client_extension/controllers/main.py index fe97d2f592e..d435406d945 100644 --- a/addons/mail_client_extension/controllers/main.py +++ b/addons/mail_client_extension/controllers/main.py @@ -160,15 +160,17 @@ class MailClientExtensionController(http.Controller): if 'enrichment_info' in iap_data: return None, iap_data['enrichment_info'] + phone_numbers = iap_data.get('phone_numbers') + emails = iap_data.get('email') new_company_info = { 'is_company': True, 'name': iap_data.get("name"), 'street': iap_data.get("street_name"), 'city': iap_data.get("city"), 'zip': iap_data.get("postal_code"), - 'phone': iap_data.get('phone_numbers', [''])[0], + 'phone': phone_numbers[0] if phone_numbers else None, 'website': iap_data.get("domain"), - 'email': iap_data.get('email', [''])[0] + 'email': emails[0] if emails else None } if iap_data.get('country_code'): country = request.env['res.country'].search([('code', '=', iap_data['country_code'].upper())]) |
