From 779bd1e12853e1583c6636764790c6201b6a2952 Mon Sep 17 00:00:00 2001
From: Alexis de Lattre <alexis@via.ecp.fr>
Date: Sat, 24 Jan 2015 00:37:06 +0100
Subject: [PATCH 1/5] Fix bug #17: don't raise by default on write/create As a
 consequence, we can have non E.164 values inside the phone/mobile/fax fields:
 in this case, we don't display an href link on the field and we don't display
 the Dial button. Fix show old number on paging for fax field

---
 base_phone/base_phone.py                 | 26 +++++++++++++-----------
 base_phone/static/src/js/phone_widget.js | 25 ++++++++++++++++++-----
 base_phone/static/src/xml/phone.xml      |  6 ++++--
 3 files changed, 38 insertions(+), 19 deletions(-)

diff --git a/base_phone/base_phone.py b/base_phone/base_phone.py
index 91309d7..6b9b789 100644
--- a/base_phone/base_phone.py
+++ b/base_phone/base_phone.py
@@ -61,6 +61,7 @@ class phone_common(orm.AbstractModel):
         return result
 
     def _generic_reformat_phonenumbers(self, cr, uid, vals, phonefields=None,
+                                       raise_if_parse_fails=False,
                                        context=None):
         """Reformat phone numbers in E.164 format i.e. +33141981242"""
         if phonefields is None:
@@ -89,6 +90,12 @@ class phone_common(orm.AbstractModel):
                     try:
                         res_parse = phonenumbers.parse(
                             vals.get(field), user_countrycode)
+                        vals[field] = phonenumbers.format_number(
+                            res_parse, phonenumbers.PhoneNumberFormat.E164)
+                        if init_value != vals[field]:
+                            _logger.info(
+                                "%s initial value: '%s' updated value: '%s'"
+                                % (field, init_value, vals[field]))
                     except Exception, e:
                         # I do BOTH logger and raise, because:
                         # raise is usefull when the record is created/written
@@ -96,19 +103,14 @@ class phone_common(orm.AbstractModel):
                         # logger is usefull when the record is created/written
                         #    via the webservices
                         _logger.error(
-                            "Cannot reformat the phone number %s to "
+                            "Cannot reformat the phone number '%s' to "
                             "international format" % vals.get(field))
-                        raise orm.except_orm(
-                            _('Error:'),
-                            _("Cannot reformat the phone number '%s' to "
-                                "international format. Error message: %s")
-                            % (vals.get(field), e))
-                    vals[field] = phonenumbers.format_number(
-                        res_parse, phonenumbers.PhoneNumberFormat.E164)
-                    if init_value != vals[field]:
-                        _logger.info(
-                            "%s initial value: '%s' updated value: '%s'"
-                            % (field, init_value, vals[field]))
+                        if raise_if_parse_fails:
+                           raise orm.except_orm(
+                               _('Error:'),
+                               _("Cannot reformat the phone number '%s' to "
+                                   "international format. Error message: %s")
+                               % (vals.get(field), e))
         return vals
 
     def get_name_from_phone_number(
diff --git a/base_phone/static/src/js/phone_widget.js b/base_phone/static/src/js/phone_widget.js
index 07f358d..478a30f 100644
--- a/base_phone/static/src/js/phone_widget.js
+++ b/base_phone/static/src/js/phone_widget.js
@@ -27,9 +27,15 @@ openerp.base_phone = function (instance) {
                   href = 'tel:' + phone_num;
                   href_text = formatInternational('', phone_num) || '';
                 }
-                this.$el.find('a.oe_form_uri').attr('href', href).text(href_text);
+                if (href_text) {
+                    this.$el.find('a.oe_form_uri').attr('href', href).text(href_text);
+                    this.$el.find('span.oe_form_char_content').text('');
+                } else {
+                    this.$el.find('a.oe_form_uri').attr('href', '').text('');
+                    this.$el.find('span.oe_form_char_content').text(phone_num || '');
+                }
                 var click2dial_text = '';
-                if (phone_num && !this.options.dial_button_invisible) {
+                if (href_text && !this.options.dial_button_invisible) {
                   click2dial_text = _t('Dial');
                 }
                 this.$el.find('#click2dial')
@@ -92,10 +98,19 @@ openerp.base_phone = function (instance) {
                 this._super();
             } else {
                 var fax_num = this.get('value');
+                //console.log('BASE_PHONE fax_num = %s', fax_num);
+                var href = '#';
+                var href_text = '';
                 if (fax_num) {
-                    this.$el.find('a.oe_form_uri')
-                        .attr('href', 'fax:' + fax_num)
-                        .text(formatInternational('', fax_num) || '');
+                    href = 'fax:' + fax_num;
+                    href_text = formatInternational('', fax_num) || '';
+                }
+                if (href_text) {
+                    this.$el.find('a.oe_form_uri').attr('href', href).text(href_text);
+                    this.$el.find('span.oe_form_char_content').text('');
+                } else {
+                    this.$el.find('a.oe_form_uri').attr('href', '').text('');
+                    this.$el.find('span.oe_form_char_content').text(fax_num || '');
                 }
             }
         },
diff --git a/base_phone/static/src/xml/phone.xml b/base_phone/static/src/xml/phone.xml
index 0cd269c..d1d4a59 100644
--- a/base_phone/static/src/xml/phone.xml
+++ b/base_phone/static/src/xml/phone.xml
@@ -10,7 +10,7 @@
 <t t-name="FieldPhone">
     <span class="oe_form_field oe_form_field_email" t-att-style="widget.node.attrs.style">
         <t t-if="widget.get('effective_readonly')">
-            <a href="#" class="oe_form_uri"/>
+            <a href="#" class="oe_form_uri"/><span class="oe_form_char_content"/>
             <!--
             You should add the following "<a .. />" in your IPBX-specific
             module (such as asterisk_click2dial) via <t t-extend="FieldPhone">
@@ -33,7 +33,9 @@
 
 <t t-name="FieldFax">
     <span class="oe_form_field oe_form_field_email" t-att-style="widget.node.attrs.style">
-        <a t-if="widget.get('effective_readonly')" href="#" class="oe_form_uri"/>
+        <t t-if="widget.get('effective_readonly')">
+            <a href="#" class="oe_form_uri"/><span class="oe_form_char_content"/>
+        </t>
         <t t-if="!widget.get('effective_readonly')">
             <div>
                 <input type="text"

From 25f00cf7e981d61b4627af4e0b50ccbaca0aa56c Mon Sep 17 00:00:00 2001
From: Alexis de Lattre <alexis@via.ecp.fr>
Date: Sat, 24 Jan 2015 12:53:35 +0100
Subject: [PATCH 2/5] One possible implementation of raise when partner
 create/write is manual vs no raise with partner create/write is via code

---
 base_phone/base_phone.py        | 5 +++--
 base_phone/res_partner_view.xml | 4 ++++
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/base_phone/base_phone.py b/base_phone/base_phone.py
index 6b9b789..e2f71db 100644
--- a/base_phone/base_phone.py
+++ b/base_phone/base_phone.py
@@ -61,9 +61,10 @@ class phone_common(orm.AbstractModel):
         return result
 
     def _generic_reformat_phonenumbers(self, cr, uid, vals, phonefields=None,
-                                       raise_if_parse_fails=False,
                                        context=None):
         """Reformat phone numbers in E.164 format i.e. +33141981242"""
+        if context is None:
+            context = {}
         if phonefields is None:
             phonefields = [
                 'phone', 'partner_phone', 'work_phone', 'fax',
@@ -105,7 +106,7 @@ class phone_common(orm.AbstractModel):
                         _logger.error(
                             "Cannot reformat the phone number '%s' to "
                             "international format" % vals.get(field))
-                        if raise_if_parse_fails:
+                        if context.get('raise_if_phone_parse_fails'):
                            raise orm.except_orm(
                                _('Error:'),
                                _("Cannot reformat the phone number '%s' to "
diff --git a/base_phone/res_partner_view.xml b/base_phone/res_partner_view.xml
index fdc1996..28bf5e5 100644
--- a/base_phone/res_partner_view.xml
+++ b/base_phone/res_partner_view.xml
@@ -56,5 +56,9 @@
     </field>
 </record>
 
+<record id="base.action_partner_form" model="ir.actions.act_window">
+    <field name="context">{'search_default_customer':1, 'raise_if_phone_parse_fails': True}</field>
+</record>
+
 </data>
 </openerp>

From 03265255fd0fe9947387b9f317cb468c32a7874c Mon Sep 17 00:00:00 2001
From: Alexis de Lattre <alexis@via.ecp.fr>
Date: Sat, 24 Jan 2015 14:39:40 +0100
Subject: [PATCH 3/5] Adapt the reformat_phonenumber_wizard to the new behavior
 on raise FIX to display the result of the reformat phonenumber wizard to the
 user

---
 base_phone/wizard/reformat_all_phonenumbers.py | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/base_phone/wizard/reformat_all_phonenumbers.py b/base_phone/wizard/reformat_all_phonenumbers.py
index 8546d35..f0eace7 100644
--- a/base_phone/wizard/reformat_all_phonenumbers.py
+++ b/base_phone/wizard/reformat_all_phonenumbers.py
@@ -39,6 +39,7 @@ class reformat_all_phonenumbers(orm.TransientModel):
         phonenumbers_not_reformatted = ''
         toreformat_dict = self.pool['phone.common']._get_phone_fields(
             cr, uid, context=context)
+        ctx_raise = dict(context, raise_if_phone_parse_fails=True)
         for objname, prop in toreformat_dict.iteritems():
             fields = []
             obj = self.pool[objname]
@@ -66,7 +67,7 @@ class reformat_all_phonenumbers(orm.TransientModel):
                 # _generic_reformat_phonenumbers()
                 try:
                     obj._generic_reformat_phonenumbers(
-                        cr, uid, entry, context=context)
+                        cr, uid, entry, context=ctx_raise)
                 except Exception, e:
                     name = obj.name_get(
                         cr, uid, [init_entry['id']], context=context)[0][1]
@@ -98,4 +99,8 @@ class reformat_all_phonenumbers(orm.TransientModel):
             {'phonenumbers_not_reformatted': phonenumbers_not_reformatted},
             context=context)
         logger.info('End of the phone number reformatting wizard')
-        return True
+        action = self.pool['ir.actions.act_window'].for_xml_id(
+            cr, uid, 'base_phone', 'reformat_all_phonenumbers_action',
+            context=context)
+        action['res_id'] = ids[0]
+        return action

From 0e91470ff2a0b35020783a36427f7716fdb3968e Mon Sep 17 00:00:00 2001
From: Alexis de Lattre <alexis@via.ecp.fr>
Date: Thu, 5 Feb 2015 10:04:06 +0100
Subject: [PATCH 4/5] PEP8 fix

---
 asterisk_click2dial/asterisk_click2dial.py |  3 ++-
 base_phone/base_phone.py                   | 10 +++++-----
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/asterisk_click2dial/asterisk_click2dial.py b/asterisk_click2dial/asterisk_click2dial.py
index 5be18fe..125f910 100644
--- a/asterisk_click2dial/asterisk_click2dial.py
+++ b/asterisk_click2dial/asterisk_click2dial.py
@@ -347,7 +347,8 @@ class res_users(orm.Model):
             help="Caller ID used for the calls initiated by this user."),
         # You'd probably think: Asterisk should reuse the callerID of sip.conf!
         # But it cannot, cf
-        # http://lists.digium.com/pipermail/asterisk-users/2012-January/269787.html
+        # http://lists.digium.com/pipermail/asterisk-users/
+        # 2012-January/269787.html
         'cdraccount': fields.char(
             'CDR Account', size=50,
             help="Call Detail Record (CDR) account used for billing this "
diff --git a/base_phone/base_phone.py b/base_phone/base_phone.py
index e2f71db..51a70ee 100644
--- a/base_phone/base_phone.py
+++ b/base_phone/base_phone.py
@@ -107,11 +107,11 @@ class phone_common(orm.AbstractModel):
                             "Cannot reformat the phone number '%s' to "
                             "international format" % vals.get(field))
                         if context.get('raise_if_phone_parse_fails'):
-                           raise orm.except_orm(
-                               _('Error:'),
-                               _("Cannot reformat the phone number '%s' to "
-                                   "international format. Error message: %s")
-                               % (vals.get(field), e))
+                            raise orm.except_orm(
+                                _('Error:'),
+                                _("Cannot reformat the phone number '%s' to "
+                                    "international format. Error message: %s")
+                                % (vals.get(field), e))
         return vals
 
     def get_name_from_phone_number(

From 0ad368989a5c64c8e6575f10be6ab01236784dd2 Mon Sep 17 00:00:00 2001
From: Alexis de Lattre <alexis@via.ecp.fr>
Date: Thu, 12 Feb 2015 15:59:53 +0100
Subject: [PATCH 5/5] base_phone: Add YAML test

---
 base_phone/__openerp__.py    |  1 +
 base_phone/test/phonenum.yml | 37 ++++++++++++++++++++++++++++++++++++
 2 files changed, 38 insertions(+)
 create mode 100644 base_phone/test/phonenum.yml

diff --git a/base_phone/__openerp__.py b/base_phone/__openerp__.py
index 907c7cc..af2c459 100644
--- a/base_phone/__openerp__.py
+++ b/base_phone/__openerp__.py
@@ -77,6 +77,7 @@ for any help or question about this module.
         ],
     'qweb': ['static/src/xml/*.xml'],
     'demo': ['base_phone_demo.xml'],
+    'test': ['test/phonenum.yml'],
     'images': [],
     'installable': True,
 }
diff --git a/base_phone/test/phonenum.yml b/base_phone/test/phonenum.yml
new file mode 100644
index 0000000..5b92efb
--- /dev/null
+++ b/base_phone/test/phonenum.yml
@@ -0,0 +1,37 @@
+-
+  Write french phone numbers in national format
+-
+  !record {model: res.partner, id: partner1}:
+    name: Pierre Paillet
+    mobile: 06 42 77 42 66
+    fax: (0) 1 45 42 12 42
+-
+  Write swiss phone numbers in international format
+-
+  !record {model: res.partner, id: partner2}:
+    name: Joël Grand-Guillaume
+    parent_id: base.res_partner_12
+    phone: +41 21 619 10 10
+    mobile: +41 79 606 42 42
+-
+  Write invalid phone number
+-
+  !record {model: res.partner, id: partner3}:
+    name: Jean Badphone
+    phone: 42
+-
+  Check that valid phone numbers have been converted to E.164
+-
+  !python {model: res.partner}: |
+    partner1 = self.browse(cr, uid, ref('partner1'), context=context)
+    assert partner1.mobile == '+33642774266', 'Mobile number not written in E.164 format (partner1)'
+    assert partner1.fax == '+33145421242', 'Fax number not written in E.164 format (partner1)'
+    partner2 = self.browse(cr, uid, ref('partner2'), context=context)
+    assert partner2.phone == '+41216191010', 'Phone number not written in E.164 format (partner2)'
+    assert partner2.mobile == '+41796064242', 'Mobile number not written in E.164 format (partner2)'
+-
+  Check that invalid phone numbers are kept unchanged
+-
+  !python {model: res.partner}: |
+    partner3 = self.browse(cr, uid, ref('partner3'), context=context)
+    assert partner3.phone == '42', 'Invalid phone numbers should not be changed'