After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 685992 - empathy strips the "+" (plus) from the numbers from the gnome-contacts
empathy strips the "+" (plus) from the numbers from the gnome-contacts
Status: RESOLVED FIXED
Product: folks
Classification: Platform
Component: libfolks
git master
Other Linux
: Normal normal
: gnome-3.8
Assigned To: folks-maint
folks-maint
Depends on:
Blocks:
 
 
Reported: 2012-10-11 17:17 UTC by Morse
Modified: 2013-01-22 16:58 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
phone-details: get_normalised: keep the leading '+' (1.58 KB, patch)
2012-12-28 14:12 UTC, Guillaume Desmottes
committed Details | Review

Description Morse 2012-10-11 17:17:56 UTC
1. open gnome-contacts and pick a contact which is in your emphathy roster
2. add a phone number for that contact (f.i. +49176.....)
3. create a sip-account and make it default for calling to "regular" phones
4. In the context menu of the contact you added the number for, you see "Call 49176..." - without plus
5. If you try and call on this number - it doesn't work.
Comment 1 Philip Withnall 2012-10-16 07:53:12 UTC
This is because Empathy uses the output of folks’ PhoneFieldDetails.get_normalised() method to give the phone number to call. That method strips off leading ‘+’ symbols.

Guillaume/Marco, should folks be doing this? Marco originally wrote the code, and I have no idea about SIP.
Comment 2 Guillaume Desmottes 2012-12-28 13:44:44 UTC
I don't think we should remove the leading '+'.

Marco: what was the rational for removing this?
Comment 3 Guillaume Desmottes 2012-12-28 14:12:32 UTC
Created attachment 232330 [details] [review]
phone-details: get_normalised: keep the leading '+'

It's needed to be able to call phone numbers containing an international
prefix.
Comment 4 Philip Withnall 2012-12-29 16:42:34 UTC
Review of attachment 232330 [details] [review]:

Committed to master (with a change to NEWS). Thanks.

commit bf25b9b6031d65569eb87836325793637b1ddec1
Author: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Date:   Fri Dec 28 15:11:25 2012 +0100

    phone-details: get_normalised: keep the leading '+'
    
    It's needed to be able to call phone numbers containing an international
    prefix.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=685992

 NEWS                     |  2 ++
 folks/phone-details.vala | 11 +++--------
 2 files changed, 5 insertions(+), 8 deletions(-)
Comment 5 Morse 2013-01-22 11:00:03 UTC
Ok, I applied this patch to my local version of libfolks-0.8.0, but telephone numbers in my roster remained the same. How can I make empathy to reload the information from gnome-contacts?
Comment 6 Travis Reitter 2013-01-22 16:58:10 UTC
(In reply to comment #5)
> Ok, I applied this patch to my local version of libfolks-0.8.0, but telephone
> numbers in my roster remained the same. How can I make empathy to reload the
> information from gnome-contacts?

It should be updated the next time you launch Empathy and it links against your patched version of the library. Both Empathy and Gnome Contacts use Folks so they get the same data sources. In most cases, any changes in the data sources are reflected immediately. But since you changed something that required the library to be reloaded, the application has to be restarted.