evcard_to_string() only took one TEL attrib from a vcard. But multiple TEL attribs are supported. This patch will convert all TEL attribs to strings. --- plugins/phonebook-ebook.c | 22 +++++++++++++--------- 1 files changed, 13 insertions(+), 9 deletions(-) diff --git a/plugins/phonebook-ebook.c b/plugins/phonebook-ebook.c index 089b956..d2482c0 100644 --- a/plugins/phonebook-ebook.c +++ b/plugins/phonebook-ebook.c @@ -97,6 +97,7 @@ static char *evcard_to_string(EVCard *evcard, unsigned int format, uint64_t filter) { EVCard *evcard2; + GList *l; char *vcard; unsigned int i; @@ -109,18 +110,21 @@ static char *evcard_to_string(EVCard *evcard, unsigned int format, */ filter = format == EVC_FORMAT_VCARD_30 ? filter | 0x87: filter | 0x85; + l = e_vcard_get_attributes(evcard); evcard2 = e_vcard_new(); - for (i = 0; i < 29; i++) { - EVCardAttribute *attrib; - - if (!(filter & (1 << i))) - continue; - - attrib = e_vcard_get_attribute(evcard, attribute_mask[i]); + for (; l; l = g_list_next(l)) { + EVCardAttribute *attrib = l->data; if (!attrib) continue; - - e_vcard_add_attribute(evcard2, e_vcard_attribute_copy(attrib)); + const char *name = e_vcard_attribute_get_name(attrib); + for (i = 0; attribute_mask[i] != NULL; i++) { + if (!(filter & (1 << i))) + continue; + if (g_strcmp0(name, attribute_mask[i]) != 0) + continue; + e_vcard_add_attribute(evcard2, + e_vcard_attribute_copy(attrib)); + } } vcard = e_vcard_to_string(evcard2, format); -- 1.7.2 -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html