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 359816 - Contact Lookup applet support for secondary home and business phone numbers
Contact Lookup applet support for secondary home and business phone numbers
Status: RESOLVED FIXED
Product: contact-lookup-applet
Classification: Deprecated
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Contact-lookup-applet Maintainers
Contact-lookup-applet Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-10-05 07:44 UTC by Timo Aaltonen
Modified: 2006-11-15 09:44 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Timo Aaltonen 2006-10-05 07:44:33 UTC 
Forwarded from:
https://launchpad.net/distros/ubuntu/+source/contact-lookup-applet/+bug/62760

Contact lookup applet displays only the following fields in it's popup dialog.
E_CONTACT_PHONE_HOME
E_CONTACT_PHONE_BUSINESS

Since often people has multiple home and business phones, i needed support in it also for E_CONTACT_PHONE_HOME_2
E_CONTACT_PHONE_BUSINESS_2

I'm attaching a diff against contact-lookup-applet-0.14-ubuntu1 that simply adds the aforementioned fields to the "phone" label.
Comment 1 Timo Aaltonen 2006-10-05 07:45:12 UTC 
diff -U3 -r contact-lookup-applet-0.14/src/contact-dialog.c xxxx/src/contact-dialog.c
--- contact-lookup-applet-0.14/src/contact-dialog.c	2004-09-30 18:46:53.000000000 +0200
+++ xxxx/src/contact-dialog.c	2006-09-28 11:48:31.454704500 +0200
@@ -320,12 +320,22 @@
       g_string_append (s, phone);
       g_string_append (s, _(" (home)\n"));
     }
+    phone = e_contact_get_const (contact, E_CONTACT_PHONE_HOME_2);
+    if (phone) {
+      g_string_append (s, phone);
+      g_string_append (s, _(" (home)\n"));
+    }
 
     phone = e_contact_get_const (contact, E_CONTACT_PHONE_BUSINESS);
     if (phone) {
       g_string_append (s, phone);
       g_string_append (s, _(" (work)\n"));
     }
+    phone = e_contact_get_const (contact, E_CONTACT_PHONE_BUSINESS_2);
+    if (phone) {
+      g_string_append (s, phone);
+      g_string_append (s, _(" (work)\n"));
+    }
 
     phone = e_contact_get_const (contact, E_CONTACT_PHONE_MOBILE);
     if (phone) {
Comment 2 Ross Burton 2006-11-15 09:44:51 UTC 
Committed to CVS, thanks.