From 3c0c62e2958b5dccca490908145c534022a85563 Mon Sep 17 00:00:00 2001 From: Christopher James Lahey Date: Tue, 8 May 2001 04:58:41 +0000 Subject: Removed e-card-pairs.h since we're not using it. 2001-05-08 Christopher James Lahey * backend/ebook/Makefile.am (libebookinclude_HEADERS): Removed e-card-pairs.h since we're not using it. * backend/ebook/e-book-view.c (e_book_view_check_listener_queue): Added break; to default: case here. * backend/ebook/e-card-simple.c, backend/ebook/e-card-simple.h: Added changed variable so as to avoid sync_card when possible. (e_card_simple_destroy): Free all the data here properly. (e_card_simple_get_arg): Slight simplification here. (fill_in_info, e_card_simple_arbitrary_foreach, e_card_simple_get_arbitrary): Call e_card_free_empty_lists here to save a bit of memory. * backend/ebook/e-card.c, backend/ebook/e-card.h: Fixed up includes a bit. (e_card_list_get_vcard, e_card_list_send): Added these functions for acting on a group of cards. (parse_org): Cleaned up this function a bit. (e_card_free_empty_lists): Added this function to delete unnecessary ELists and save a bit of memory. (e_v_object_get_child_value): Made this return NULL if not found instead of g_strdup(""). * contact-editor/e-contact-save-as.c, contact-editor/e-contact-save-as.h (e_contact_list_save_as): Added this function to save multiple contacts. * gui/widgets/Makefile.am: Commented out reflow test. (libeminicard_a_SOURCES): Added e-minicard-view-model.c and e-minicard-view-model.h. * gui/widgets/e-minicard-view-model.c, gui/widgets/e-minicard-view-model.h: Model for use in EMinicardView. * gui/widgets/e-minicard-view.c, gui/widgets/e-minicard-view.h: Reworked this to use the new EReflow stuff. * gui/widgets/e-minicard.c (e_minicard_event): Doesn't handle right click menus now. Emits a signal on the parent canvas item instead. * printing/e-contact-print-envelope.c, printing/e-contact-print-envelope.h (e_contact_print_envelope_list_dialog_new): Added this function to print multiple envelopes (only prints first for now.) * printing/e-contact-print.c, printing/e-contact-print.h (e_contact_print_card_list_dialog_new): Added this function to print multiple cards. Only prints the first for now. svn path=/trunk/; revision=9711 --- addressbook/backend/ebook/Makefile.am | 1 - addressbook/backend/ebook/e-book-view.c | 1 + addressbook/backend/ebook/e-card-simple.c | 30 +++- addressbook/backend/ebook/e-card-simple.h | 2 + addressbook/backend/ebook/e-card.c | 238 ++++++++++++++++++++---------- addressbook/backend/ebook/e-card.h | 5 + 6 files changed, 194 insertions(+), 83 deletions(-) (limited to 'addressbook/backend') diff --git a/addressbook/backend/ebook/Makefile.am b/addressbook/backend/ebook/Makefile.am index c729d71624..da7307c1a5 100644 --- a/addressbook/backend/ebook/Makefile.am +++ b/addressbook/backend/ebook/Makefile.am @@ -59,7 +59,6 @@ libebookinclude_HEADERS = \ e-book.h \ e-book-util.h \ e-card-cursor.h \ - e-card-pairs.h \ e-card-simple.h \ e-card-types.h \ e-card.h \ diff --git a/addressbook/backend/ebook/e-book-view.c b/addressbook/backend/ebook/e-book-view.c index f20531a50a..5f8ae3e069 100644 --- a/addressbook/backend/ebook/e-book-view.c +++ b/addressbook/backend/ebook/e-book-view.c @@ -119,6 +119,7 @@ e_book_view_check_listener_queue (EBookViewListener *listener, EBookView *book_v default: g_error ("EBookView: Unknown operation %d in listener queue!\n", resp->op); + break; } g_free (resp); diff --git a/addressbook/backend/ebook/e-card-simple.c b/addressbook/backend/ebook/e-card-simple.c index 6d5619d135..f7550017a0 100644 --- a/addressbook/backend/ebook/e-card-simple.c +++ b/addressbook/backend/ebook/e-card-simple.c @@ -326,11 +326,13 @@ e_card_simple_destroy (GtkObject *object) simple->temp_fields = NULL; for(i = 0; i < E_CARD_SIMPLE_PHONE_ID_LAST; i++) - g_free(simple->phone[i]); + e_card_phone_free (simple->phone[i]); for(i = 0; i < E_CARD_SIMPLE_EMAIL_ID_LAST; i++) g_free(simple->email[i]); for(i = 0; i < E_CARD_SIMPLE_ADDRESS_ID_LAST; i++) - g_free(simple->address[i]); + e_card_address_label_free(simple->address[i]); + for(i = 0; i < E_CARD_SIMPLE_ADDRESS_ID_LAST; i++) + e_card_delivery_address_free(simple->delivery[i]); } @@ -373,10 +375,7 @@ e_card_simple_get_arg (GtkObject *object, GtkArg *arg, guint arg_id) switch (arg_id) { case ARG_CARD: e_card_simple_sync_card(simple); - if (simple->card) - GTK_VALUE_OBJECT (*arg) = GTK_OBJECT(simple->card); - else - GTK_VALUE_OBJECT (*arg) = NULL; + GTK_VALUE_OBJECT (*arg) = (GtkObject *) simple->card; break; default: arg->type = GTK_TYPE_INVALID; @@ -400,6 +399,8 @@ e_card_simple_init (ECardSimple *simple) for(i = 0; i < E_CARD_SIMPLE_ADDRESS_ID_LAST; i++) simple->address[i] = NULL; simple->temp_fields = NULL; + + simple->changed = TRUE; } static void @@ -484,6 +485,7 @@ fill_in_info(ECardSimple *simple) } } gtk_object_unref(GTK_OBJECT(iterator)); + e_card_free_empty_lists (card); } } @@ -491,7 +493,7 @@ void e_card_simple_sync_card(ECardSimple *simple) { ECard *card = simple->card; - if (card) { + if (card && simple->changed) { EList *address_list; EList *phone_list; EList *email_list; @@ -632,7 +634,10 @@ e_card_simple_sync_card(ECardSimple *simple) } } fill_in_info(simple); + e_card_free_empty_lists (card); } + + simple->changed = FALSE; } const ECardPhone *e_card_simple_get_phone (ECardSimple *simple, @@ -666,6 +671,7 @@ void e_card_simple_set_phone (ECardSimple *simple, if (simple->phone[id]) e_card_phone_free(simple->phone[id]); simple->phone[id] = e_card_phone_copy(phone); + simple->changed = TRUE; } void e_card_simple_set_email (ECardSimple *simple, @@ -675,6 +681,7 @@ void e_card_simple_set_email (ECardSimple *simple, if (simple->email[id]) g_free(simple->email[id]); simple->email[id] = g_strdup(email); + simple->changed = TRUE; } void e_card_simple_set_address (ECardSimple *simple, @@ -687,6 +694,7 @@ void e_card_simple_set_address (ECardSimple *simple, if (simple->delivery[id]) e_card_delivery_address_free(simple->delivery[id]); simple->delivery[id] = e_card_delivery_address_from_label(simple->address[id]); + simple->changed = TRUE; } void e_card_simple_set_delivery_address (ECardSimple *simple, @@ -696,6 +704,7 @@ void e_card_simple_set_delivery_address (ECardSimple *simple if (simple->delivery[id]) e_card_delivery_address_free(simple->delivery[id]); simple->delivery[id] = e_card_delivery_address_copy(delivery); + simple->changed = TRUE; } const char *e_card_simple_get_const (ECardSimple *simple, @@ -904,6 +913,7 @@ void e_card_simple_set (ECardSimple *simple, ECardAddrLabel *address; ECardPhone *phone; int style; + simple->changed = TRUE; switch (field) { case E_CARD_SIMPLE_FIELD_FULL_NAME: case E_CARD_SIMPLE_FIELD_ORG: @@ -1005,6 +1015,7 @@ void e_card_simple_arbitrary_foreach (ECardSimple if (callback) (*callback) (arbitrary, closure); } + e_card_free_empty_lists (simple->card); } } @@ -1022,6 +1033,7 @@ const ECardArbitrary *e_card_simple_get_arbitrary (ECardSimple *sim if (!strcasecmp(arbitrary->key, key)) return arbitrary; } + e_card_free_empty_lists (simple->card); } return NULL; } @@ -1033,9 +1045,11 @@ void e_card_simple_set_arbitrary (ECardSimple *sim const char *value) { if (simple->card) { - ECardArbitrary *new_arb; + ECardArbitrary *new_arb; EList *list; EIterator *iterator; + + simple->changed = TRUE; gtk_object_get(GTK_OBJECT(simple->card), "arbitrary", &list, NULL); diff --git a/addressbook/backend/ebook/e-card-simple.h b/addressbook/backend/ebook/e-card-simple.h index 608e7488df..899964fd24 100644 --- a/addressbook/backend/ebook/e-card-simple.h +++ b/addressbook/backend/ebook/e-card-simple.h @@ -138,6 +138,8 @@ struct _ECardSimple { char *email[E_CARD_SIMPLE_EMAIL_ID_LAST]; ECardAddrLabel *address[E_CARD_SIMPLE_ADDRESS_ID_LAST]; ECardDeliveryAddress *delivery[E_CARD_SIMPLE_ADDRESS_ID_LAST]; + + gboolean changed; }; struct _ECardSimpleClass { diff --git a/addressbook/backend/ebook/e-card.c b/addressbook/backend/ebook/e-card.c index ca160f0905..eb239ccd76 100644 --- a/addressbook/backend/ebook/e-card.c +++ b/addressbook/backend/ebook/e-card.c @@ -9,6 +9,11 @@ */ #include + +#include "e-card.h" + +#include + #include #include #include @@ -22,9 +27,6 @@ #include "e-util/ename/e-name-western.h" #include "e-util/ename/e-address-western.h" -#include "e-card.h" -#include "e-card-pairs.h" - #define is_a_prop_of(obj,prop) (isAPropertyOf ((obj),(prop))) #define str_val(obj) (the_str = (vObjectValueType (obj))? fakeCString (vObjectUStringZValue (obj)) : calloc (1, 1)) #define has(obj,prop) (vo = isAPropertyOf ((obj), (prop))) @@ -242,17 +244,10 @@ e_card_set_id (ECard *card, const char *id) card->id = g_strdup(id); } -/** - * e_card_get_vcard: - * @card: an #ECard - * - * Returns: a string in vCard format, which is wrapped by the @card. - */ -char -*e_card_get_vcard (ECard *card) +static VObject * +e_card_get_vobject (ECard *card) { - VObject *vobj; /*, *vprop; */ - char *temp, *ret_val; + VObject *vobj; vobj = newVObject (VCCardProp); @@ -538,6 +533,22 @@ char add_CardProperty (vprop, &crd->key.prop); } #endif + return vobj; +} + +/** + * e_card_get_vcard: + * @card: an #ECard + * + * Returns: a string in vCard format, which is wrapped by the @card. + */ +char * +e_card_get_vcard (ECard *card) +{ + VObject *vobj; + char *temp, *ret_val; + + vobj = e_card_get_vobject (card); temp = writeMemVObject(NULL, NULL, vobj); ret_val = g_strdup(temp); free(temp); @@ -545,6 +556,35 @@ char return ret_val; } +/** + * e_card_list_get_vcard: + * @list: a list of #ECards + * + * Returns: a string in vCard format. + */ +char * +e_card_list_get_vcard (GList *list) +{ + VObject *vobj; + + char *temp, *ret_val; + + vobj = NULL; + + for (; list; list = list->next) { + VObject *tempvobj; + ECard *card = list->data; + + tempvobj = e_card_get_vobject (card); + addList (&vobj, tempvobj); + } + temp = writeMemVObjects(NULL, NULL, vobj); + ret_val = g_strdup(temp); + free(temp); + cleanVObjects(vobj); + return ret_val; +} + static void parse_file_as(ECard *card, VObject *vobj) { @@ -671,17 +711,12 @@ parse_org(ECard *card, VObject *vobj) char *temp; temp = e_v_object_get_child_value(vobj, VCOrgNameProp); - if (temp) { - if (card->org) - g_free(card->org); - card->org = temp; - } + g_free(card->org); + card->org = temp; + temp = e_v_object_get_child_value(vobj, VCOrgUnitProp); - if (temp) { - if (card->org_unit) - g_free(card->org_unit); - card->org_unit = temp; - } + g_free(card->org_unit); + card->org_unit = temp; } static void @@ -1734,15 +1769,15 @@ e_card_get_arg (GtkObject *object, GtkArg *arg, guint arg_id) case ARG_ADDRESS: if (!card->address) card->address = e_list_new((EListCopyFunc) e_card_delivery_address_copy, - (EListFreeFunc) e_card_delivery_address_free, - NULL); + (EListFreeFunc) e_card_delivery_address_free, + NULL); GTK_VALUE_OBJECT(*arg) = GTK_OBJECT(card->address); break; case ARG_ADDRESS_LABEL: if (!card->address_label) card->address_label = e_list_new((EListCopyFunc) e_card_address_label_copy, - (EListFreeFunc) e_card_address_label_free, - NULL); + (EListFreeFunc) e_card_address_label_free, + NULL); GTK_VALUE_OBJECT(*arg) = GTK_OBJECT(card->address_label); break; case ARG_PHONE: @@ -1767,8 +1802,8 @@ e_card_get_arg (GtkObject *object, GtkArg *arg, guint arg_id) EIterator *iterator; if (!card->categories) card->categories = e_list_new((EListCopyFunc) g_strdup, - (EListFreeFunc) g_free, - NULL); + (EListFreeFunc) g_free, + NULL); length = e_list_length(card->categories); strs = g_new(char *, length + 1); for (iterator = e_list_get_iterator(card->categories), i = 0; e_iterator_is_valid(iterator); e_iterator_next(iterator), i++) { @@ -1782,8 +1817,8 @@ e_card_get_arg (GtkObject *object, GtkArg *arg, guint arg_id) case ARG_CATEGORY_LIST: if (!card->categories) card->categories = e_list_new((EListCopyFunc) g_strdup, - (EListFreeFunc) g_free, - NULL); + (EListFreeFunc) g_free, + NULL); GTK_VALUE_OBJECT(*arg) = GTK_OBJECT(card->categories); break; case ARG_BIRTH_DATE: @@ -1840,8 +1875,8 @@ e_card_get_arg (GtkObject *object, GtkArg *arg, guint arg_id) case ARG_ARBITRARY: if (!card->arbitrary) card->arbitrary = e_list_new((EListCopyFunc) e_card_arbitrary_copy, - (EListFreeFunc) e_card_arbitrary_free, - NULL); + (EListFreeFunc) e_card_arbitrary_free, + NULL); GTK_VALUE_OBJECT(*arg) = GTK_OBJECT(card->arbitrary); break; @@ -1960,6 +1995,40 @@ e_card_load_cards_from_file(const char *filename) return list; } +void +e_card_free_empty_lists (ECard *card) +{ + if (card->address && e_list_length (card->address) == 0) { + gtk_object_unref (GTK_OBJECT (card->address)); + card->address = NULL; + } + + if (card->address_label && e_list_length (card->address_label) == 0) { + gtk_object_unref (GTK_OBJECT (card->address_label)); + card->address_label = NULL; + } + + if (card->phone && e_list_length (card->phone) == 0) { + gtk_object_unref (GTK_OBJECT (card->phone)); + card->phone = NULL; + } + + if (card->email && e_list_length (card->email) == 0) { + gtk_object_unref (GTK_OBJECT (card->email)); + card->email = NULL; + } + + if (card->categories && e_list_length (card->categories) == 0) { + gtk_object_unref (GTK_OBJECT (card->categories)); + card->categories = NULL; + } + + if (card->arbitrary && e_list_length (card->arbitrary) == 0) { + gtk_object_unref (GTK_OBJECT (card->arbitrary)); + card->arbitrary = NULL; + } +} + static void assign_string(VObject *vobj, char **string) { @@ -3426,9 +3495,7 @@ e_v_object_get_child_value(VObject *vobj, char *name) return ret_val; } } - ret_val = g_new(char, 1); - *ret_val = 0; - return ret_val; + return NULL; } static ECardPhoneFlags @@ -3552,11 +3619,14 @@ set_address_flags (VObject *vobj, ECardAddressFlags flags) #define COMPOSER_OAFID "OAFIID:GNOME_Evolution_Mail_Composer" void -e_card_send (ECard *card, ECardDisposition disposition) +e_card_list_send (GList *cards, ECardDisposition disposition) { BonoboObjectClient *bonobo_server; GNOME_Evolution_Composer composer_server; CORBA_Environment ev; + + if (cards == NULL) + return; /* First, I obtain an object reference that represents the Composer. */ bonobo_server = bonobo_object_activate (COMPOSER_OAFID, 0); @@ -3570,33 +3640,42 @@ e_card_send (ECard *card, ECardDisposition disposition) if (disposition == E_CARD_DISPOSITION_AS_TO) { GNOME_Evolution_Composer_RecipientList *to_list, *cc_list, *bcc_list; CORBA_char *subject; - char *name; - EList *email; - EIterator *iterator; - GNOME_Evolution_Composer_Recipient *recipient; + int length; + int i; + + length = g_list_length (cards); + /* Now I have to make a CORBA sequence that represents a recipient list with one item, for the card. */ to_list = GNOME_Evolution_Composer_RecipientList__alloc (); - to_list->_maximum = 1; - to_list->_length = 1; - to_list->_buffer = CORBA_sequence_GNOME_Evolution_Composer_Recipient_allocbuf (1); - - gtk_object_get(GTK_OBJECT(card), - "full_name", &name, - "email", &email, - NULL); - - recipient = &(to_list->_buffer[0]); - - iterator = e_list_get_iterator(email); - if (e_iterator_is_valid(iterator)) { - recipient->address = CORBA_string_dup(e_iterator_get(iterator)); - } else { - recipient->address = CORBA_string_dup(""); + to_list->_maximum = length; + to_list->_length = length; + to_list->_buffer = CORBA_sequence_GNOME_Evolution_Composer_Recipient_allocbuf (length); + + for (i = 0; + cards; + i++, cards = cards->next) { + ECard *card = cards->data; + EIterator *iterator; + char *file_as; + EList *email; + GNOME_Evolution_Composer_Recipient *recipient; + + recipient = &(to_list->_buffer[i]); + gtk_object_get (GTK_OBJECT (card), + "file_as", &file_as, + "email", &email, + NULL); + + iterator = e_list_get_iterator (email); + if (e_iterator_is_valid (iterator)) { + recipient->address = CORBA_string_dup (e_iterator_get (iterator)); + } else { + recipient->address = CORBA_string_dup(""); + } + gtk_object_unref (GTK_OBJECT (iterator)); + recipient->name = CORBA_string_dup (file_as); } - gtk_object_unref(GTK_OBJECT(iterator)); - - recipient->name = CORBA_string_dup(name); cc_list = GNOME_Evolution_Composer_RecipientList__alloc (); cc_list->_maximum = cc_list->_length = 0; @@ -3604,7 +3683,7 @@ e_card_send (ECard *card, ECardDisposition disposition) bcc_list->_maximum = bcc_list->_length = 0; subject = CORBA_string_dup (""); - + GNOME_Evolution_Composer_setHeaders (composer_server, to_list, cc_list, bcc_list, subject, &ev); if (ev._major != CORBA_NO_EXCEPTION) { g_printerr ("gui/e-meeting-edit.c: I couldn't set the composer headers via CORBA! Aagh.\n"); @@ -3612,9 +3691,6 @@ e_card_send (ECard *card, ECardDisposition disposition) return; } - if (CORBA_sequence_get_release (to_list) != FALSE) - CORBA_free (to_list->_buffer); - CORBA_free (to_list); CORBA_free (cc_list); CORBA_free (bcc_list); @@ -3625,24 +3701,29 @@ e_card_send (ECard *card, ECardDisposition disposition) CORBA_char *content_type, *filename, *description, *attach_data; CORBA_boolean show_inline; char *tempstr; - char *name; - - gtk_object_get(GTK_OBJECT(card), - "full_name", &name, - NULL); content_type = CORBA_string_dup ("text/x-vcard"); filename = CORBA_string_dup (""); - tempstr = g_strdup_printf ("VCard for %s", name); - description = CORBA_string_dup (tempstr); - g_free (tempstr); + if (cards->next) { + description = CORBA_string_dup (_("Multiple VCards")); + } else { + char *file_as; + + gtk_object_get(GTK_OBJECT(cards->data), + "file_as", &file_as, + NULL); + + tempstr = g_strdup_printf (_("VCard for %s"), file_as); + description = CORBA_string_dup (tempstr); + g_free (tempstr); + } show_inline = FALSE; - tempstr = e_card_get_vcard(card); + tempstr = e_card_list_get_vcard (cards); attach_data = CORBA_string_dup (tempstr); - g_free(tempstr); + g_free (tempstr); GNOME_Evolution_Composer_attachData (composer_server, content_type, filename, description, @@ -3671,3 +3752,12 @@ e_card_send (ECard *card, ECardDisposition disposition) CORBA_exception_free (&ev); } + +void +e_card_send (ECard *card, ECardDisposition disposition) +{ + GList *list; + list = g_list_prepend (NULL, card); + e_card_list_send (list, disposition); + g_list_free (list); +} diff --git a/addressbook/backend/ebook/e-card.h b/addressbook/backend/ebook/e-card.h index 90d345b62c..3714324081 100644 --- a/addressbook/backend/ebook/e-card.h +++ b/addressbook/backend/ebook/e-card.h @@ -110,7 +110,9 @@ ECard *e_card_new (char char *e_card_get_id (ECard *card); void e_card_set_id (ECard *card, const char *character); + char *e_card_get_vcard (ECard *card); +char *e_card_list_get_vcard (GList *list); ECard *e_card_duplicate (ECard *card); /* ECardPhone manipulation */ @@ -152,6 +154,7 @@ gboolean e_card_email_match_string (const ECard /* Specialized functionality */ GList *e_card_load_cards_from_file (const char *filename); +void e_card_free_empty_lists (ECard *card); enum _ECardDisposition { E_CARD_DISPOSITION_AS_ATTACHMENT, @@ -160,6 +163,8 @@ enum _ECardDisposition { typedef enum _ECardDisposition ECardDisposition; void e_card_send (ECard *card, ECardDisposition disposition); +void e_card_list_send (GList *cards, + ECardDisposition disposition); /* Standard Gtk function */ GtkType e_card_get_type (void); -- cgit