diff options
-rw-r--r-- | addressbook/ChangeLog | 6 | ||||
-rw-r--r-- | addressbook/backend/ebook/e-card.c | 109 |
2 files changed, 106 insertions, 9 deletions
diff --git a/addressbook/ChangeLog b/addressbook/ChangeLog index da799615e8..815284871e 100644 --- a/addressbook/ChangeLog +++ b/addressbook/ChangeLog @@ -1,5 +1,11 @@ 2001-10-15 Christopher James Lahey <clahey@ximian.com> + * backend/ebook/e-card.c (addPropValueQP, addPropValueSets, + e_card_get_vobject): Properly decide whether a property needs to + be marked as QP by searching for '\n'. Fixes Ximian bug #3021. + +2001-10-15 Christopher James Lahey <clahey@ximian.com> + * gui/contact-editor/e-contact-save-as.c (e_contact_save_as, e_contact_list_save_as): Set the default file name here. Fixes Ximian bug #7053. diff --git a/addressbook/backend/ebook/e-card.c b/addressbook/backend/ebook/e-card.c index 216971c650..38116891b5 100644 --- a/addressbook/backend/ebook/e-card.c +++ b/addressbook/backend/ebook/e-card.c @@ -383,13 +383,96 @@ static VObject * addPropValueUTF8(VObject *o, const char *p, const char *v) { VObject *prop = addPropValue (o, p, v); - if (!e_utf8_is_ascii (v)) - addPropValue (prop, "CHARSET", "UTF-8"); + for (; *v; v++) { + if ((*v) & 0x80) { + addPropValue (prop, "CHARSET", "UTF-8"); + for (; *v; v++) { + if (*v == '\n') { + addProp(prop, VCQuotedPrintableProp); + return prop; + } + } + return prop; + } + if (*v == '\n') { + addProp(prop, VCQuotedPrintableProp); + for (; *v; v++) { + if ((*v) & 0x80) { + addPropValue (prop, "CHARSET", "UTF-8"); + return prop; + } + } + return prop; + } + } return prop; } -#define ADD_PROP_VALUE(o, p, v) (assumeUTF8 ? (addPropValue ((o), (p), (v))) : addPropValueUTF8 ((o), (p), (v))) -#define ADD_PROP_VALUE_SET_IS_ASCII(o, p, v) (addPropValue ((o), (p), (v)), is_ascii = is_ascii && e_utf8_is_ascii ((v))) +static VObject * +addPropValueQP(VObject *o, const char *p, const char *v) +{ + VObject *prop = addPropValue (o, p, v); + for (; *v; v++) { + if (*v == '\n') { + addProp(prop, VCQuotedPrintableProp); + break; + } + } + return prop; +} + +static void +addPropValueSets (VObject *o, char *p, char *v, gboolean assumeUTF8, gboolean *is_ascii, gboolean *has_return) +{ + addPropValue (o, p, v); + if (*has_return && (assumeUTF8 || !*is_ascii)) + return; + if (*has_return) { + for (; *v; v++) { + if (*v & 0x80) { + *is_ascii = FALSE; + return; + } + } + return; + } + if (assumeUTF8 || !*is_ascii) { + for (; *v; v++) { + if (*v == '\n') { + *has_return = TRUE; + return; + } + } + return; + } + for (; *v; v++) { + if (*v & 0x80) { + *is_ascii = FALSE; + for (; *v; v++) { + if (*v == '\n') { + *has_return = TRUE; + return; + } + } + return; + } + if (*v == '\n') { + *has_return = TRUE; + for (; *v; v++) { + if (*v & 0x80) { + *is_ascii = FALSE; + return; + } + } + return; + } + } + return; +} + +#define ADD_PROP_VALUE(o, p, v) (assumeUTF8 ? (addPropValueQP ((o), (p), (v))) : addPropValueUTF8 ((o), (p), (v))) +#define ADD_PROP_VALUE_SET_IS_ASCII(o, p, v) (addPropValueSets ((o), (p), (v), assumeUTF8, &is_ascii, &has_return)) + static VObject * e_card_get_vobject (ECard *card, gboolean assumeUTF8) @@ -398,12 +481,12 @@ e_card_get_vobject (ECard *card, gboolean assumeUTF8) vobj = newVObject (VCCardProp); - if ( card->file_as && *card->file_as ) + if (card->file_as && *card->file_as) ADD_PROP_VALUE(vobj, "X-EVOLUTION-FILE-AS", card->file_as); else if (card->file_as) addProp(vobj, "X-EVOLUTION-FILE_AS"); - if ( card->fname ) + if (card->fname && *card->fname) ADD_PROP_VALUE(vobj, VCFullNameProp, card->fname); else if (card->fname) addProp(vobj, VCFullNameProp); @@ -411,6 +494,7 @@ e_card_get_vobject (ECard *card, gboolean assumeUTF8) if ( card->name && (card->name->prefix || card->name->given || card->name->additional || card->name->family || card->name->suffix) ) { VObject *nameprop; gboolean is_ascii = TRUE; + gboolean has_return = FALSE; nameprop = addProp(vobj, VCNameProp); if ( card->name->prefix ) ADD_PROP_VALUE_SET_IS_ASCII(nameprop, VCNamePrefixesProp, card->name->prefix); @@ -422,6 +506,8 @@ e_card_get_vobject (ECard *card, gboolean assumeUTF8) ADD_PROP_VALUE_SET_IS_ASCII(nameprop, VCFamilyNameProp, card->name->family); if ( card->name->suffix ) ADD_PROP_VALUE_SET_IS_ASCII(nameprop, VCNameSuffixesProp, card->name->suffix); + if (has_return) + addProp(nameprop, VCQuotedPrintableProp); if (!(is_ascii || assumeUTF8)) addPropValue (nameprop, "CHARSET", "UTF-8"); } @@ -435,6 +521,7 @@ e_card_get_vobject (ECard *card, gboolean assumeUTF8) VObject *addressprop; ECardDeliveryAddress *address = (ECardDeliveryAddress *) e_iterator_get(iterator); gboolean is_ascii = TRUE; + gboolean has_return = FALSE; addressprop = addProp(vobj, VCAdrProp); @@ -453,7 +540,9 @@ e_card_get_vobject (ECard *card, gboolean assumeUTF8) ADD_PROP_VALUE_SET_IS_ASCII(addressprop, VCPostalCodeProp, address->code); if ( address->country ) ADD_PROP_VALUE_SET_IS_ASCII(addressprop, VCCountryNameProp, address->country); - addProp(addressprop, VCQuotedPrintableProp); + + if (has_return) + addProp(addressprop, VCQuotedPrintableProp); if (!(is_ascii || assumeUTF8)) addPropValue (addressprop, "CHARSET", "UTF-8"); } @@ -471,7 +560,6 @@ e_card_get_vobject (ECard *card, gboolean assumeUTF8) labelprop = addProp(vobj, VCDeliveryLabelProp); set_address_flags (labelprop, address_label->flags); - addProp(labelprop, VCQuotedPrintableProp); } gtk_object_unref(GTK_OBJECT(iterator)); } @@ -511,12 +599,16 @@ e_card_get_vobject (ECard *card, gboolean assumeUTF8) if (card->org || card->org_unit) { VObject *orgprop; gboolean is_ascii = TRUE; + gboolean has_return = FALSE; orgprop = addProp(vobj, VCOrgProp); if (card->org) ADD_PROP_VALUE_SET_IS_ASCII(orgprop, VCOrgNameProp, card->org); if (card->org_unit) ADD_PROP_VALUE_SET_IS_ASCII(orgprop, VCOrgUnitProp, card->org_unit); + + if (has_return) + addProp(orgprop, VCQuotedPrintableProp); if (!(is_ascii || assumeUTF8)) addPropValue (orgprop, "CHARSET", "UTF-8"); } @@ -563,7 +655,6 @@ e_card_get_vobject (ECard *card, gboolean assumeUTF8) VObject *noteprop; noteprop = ADD_PROP_VALUE(vobj, VCNoteProp, card->note); - addProp(noteprop, VCQuotedPrintableProp); } if (card->last_use) { |