Hello Folks,

Thanks for the help wrt compiling evolution!  I have identified the
issue and came up with a patch that fixes the issue for me, but probably
needs a review.

The documentation of this method claims to return the formatted name
yet it previously encoded them in quotable strings.  This patch fixes
this which results in strings containing non ASCII chracters to be
rendered correctly in the UI.  This also changes the vCard output but
vCard 3.0 specifies that quotable-printable inline ecnodings have been
eliminated.  In any case this quoting should be handled in
e_destination_export_to_vcard_attribute instead.

Please point me to the documentation on what else I need to do to get
this patch committed (or the issue fixed in another way)

See also:
https://bugs.launchpad.net/ubuntu/+source/evolution/+bug/229187

Cheers,
David

PS: Please keep be CC'ed as I'm not subscribed.

-- 
David Ayers - Team Austria
Free Software Foundation Europe (FSFE) []          (http://www.fsfe.org)
Join the Fellowship of FSFE!         [][][]      (https://fsfe.org/join)
Your donation powers our work!         ||       (http://fsfe.org/donate)
>From 44f05d8e46821f0ac88e0bd1097f5a2544c2062d Mon Sep 17 00:00:00 2001
From: David Ayers <[email protected]>
Date: Thu, 20 May 2010 18:04:57 +0200
Subject: [PATCH] Return formatted address in e_destination_get_address

The documentation of this method claims to return the formatted name
yet it previously encoded them in quotable strings.  This patch fixes
this which results in strings containing non ASCII chracters to be
rendered correctly in the UI.  This also changes the vCard output but
vCard 3.0 specifies that quotable-printable inline ecnodings have been
eliminated.  In any case this quoting should be handled in
e_destination_export_to_vcard_attribute instead.
---
 addressbook/libebook/e-destination.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/addressbook/libebook/e-destination.c b/addressbook/libebook/e-destination.c
index d4b7421..5859fe0 100644
--- a/addressbook/libebook/e-destination.c
+++ b/addressbook/libebook/e-destination.c
@@ -881,11 +881,11 @@ e_destination_get_address (const EDestination *dest)
 				iter = g_list_next (iter);
 			}
 
-			priv->addr = camel_address_encode (CAMEL_ADDRESS (addr));
+			priv->addr = camel_address_format (CAMEL_ADDRESS (addr));
 		} else if (priv->raw) {
 
 			if (camel_address_unformat (CAMEL_ADDRESS (addr), priv->raw)) {
-				priv->addr = camel_address_encode (CAMEL_ADDRESS (addr));
+				priv->addr = camel_address_format (CAMEL_ADDRESS (addr));
 			}
 		} else {
 			const gchar *name, *email;
@@ -900,7 +900,7 @@ e_destination_get_address (const EDestination *dest)
 				nothing we can do here */
 				camel_address_decode (CAMEL_ADDRESS (addr), name);
 
-			priv->addr = camel_address_encode (CAMEL_ADDRESS (addr));
+			priv->addr = camel_address_format (CAMEL_ADDRESS (addr));
 		}
 
 		g_object_unref (addr);
-- 
1.7.0.4

_______________________________________________
evolution-list mailing list
[email protected]
To change your list options or unsubscribe, visit ...
http://mail.gnome.org/mailman/listinfo/evolution-list

Reply via email to