From b79d05fff0cacb4d99ebc1b60f44d8dab757b806 Mon Sep 17 00:00:00 2001 From: Olivier Dony Date: Wed, 24 Jan 2018 11:25:50 +0100 Subject: [PATCH] [FIX] sms: switch `phonenumbers` warning to INFO Having it in INFO should be sufficient for its purpose, and will avoid impacting all CI builds done on a system that does not have the lib installed. For the record, this is not a hard requirement because the lib was not available in Debian stable packages at the time of release. It is only enabled on demand for those who want the feature and can install it manually. Also slightly improved the message. Fixes #22426 Closes #22459 --- addons/sms/wizard/send_sms.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/addons/sms/wizard/send_sms.py b/addons/sms/wizard/send_sms.py index b1c5a486486..844933d092a 100644 --- a/addons/sms/wizard/send_sms.py +++ b/addons/sms/wizard/send_sms.py @@ -15,8 +15,10 @@ try: except ImportError: _sms_phonenumbers_lib_imported = False - _logger.warning( - "The `phonenumbers` Python module is not installed. Try: pip install phonenumbers." + _logger.info( + "The `phonenumbers` Python module is not available. " + "Phone number validation will be skipped. " + "Try `pip3 install phonenumbers` to install it." )