aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--ChangeLog5
-rw-r--r--addressbook/contact-editor/e-contact-editor.c94
-rw-r--r--addressbook/gui/contact-editor/e-contact-editor.c94
3 files changed, 117 insertions, 76 deletions
diff --git a/ChangeLog b/ChangeLog
index 8aef8ed4f2..0db0865385 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,10 @@
2000-04-13 Christopher James Lahey <clahey@helixcode.com>
+ * addressbook/contact-editor/e-contact-editor.c (extract_info):
+ Check for 0 length fields when building the outgoing ECard.
+
+2000-04-13 Christopher James Lahey <clahey@helixcode.com>
+
* addressbook/backend/pas/pas-book-view.c: Give correct warnings.
* addressbook/backend/ebook/e-card.c (e_card_set_arg): g_strdup
diff --git a/addressbook/contact-editor/e-contact-editor.c b/addressbook/contact-editor/e-contact-editor.c
index ebd3611f2e..2e0d2219a5 100644
--- a/addressbook/contact-editor/e-contact-editor.c
+++ b/addressbook/contact-editor/e-contact-editor.c
@@ -654,6 +654,7 @@ extract_info(EContactEditor *editor)
ECard *card = editor->card;
if (card) {
char *fname;
+ char *string;
ECardList *address_list;
ECardList *phone_list;
ECardList *email_list;
@@ -678,69 +679,86 @@ extract_info(EContactEditor *editor)
position = 0;
editable = GTK_EDITABLE(glade_xml_get_widget(editor->gui, "entry-fullname"));
fname = gtk_editable_get_chars(editable, 0, -1);
+ if (fname && *fname)
+ gtk_object_set(GTK_OBJECT(card),
+ "full_name", fname,
+ NULL);
+ g_free(fname);
iterator = e_card_list_get_iterator(address_list);
address = e_card_iterator_get(iterator);
editable = GTK_EDITABLE(glade_xml_get_widget(editor->gui, "text-address"));
- if (address) {
- address_copy = e_card_delivery_address_copy(address);
- if (address_copy->city)
- g_free(address_copy->city);
- address_copy->city = gtk_editable_get_chars(editable, 0, -1);
- e_card_iterator_set(iterator, address_copy);
- e_card_delivery_address_free(address_copy);
- } else {
- address_copy = g_new0(ECardDeliveryAddress, 1);
- address_copy->city = gtk_editable_get_chars(editable, 0, -1);
- e_card_list_append(address_list, address_copy);
- e_card_delivery_address_free(address_copy);
- }
+ string = gtk_editable_get_chars(editable, 0, -1);
+ if (string && *string) {
+ if (address) {
+ address_copy = e_card_delivery_address_copy(address);
+ if (address_copy->city)
+ g_free(address_copy->city);
+ address_copy->city = string;
+ e_card_iterator_set(iterator, address_copy);
+ e_card_delivery_address_free(address_copy);
+ } else {
+ address_copy = g_new0(ECardDeliveryAddress, 1);
+ address_copy->city = string;
+ e_card_list_append(address_list, address_copy);
+ e_card_delivery_address_free(address_copy);
+ }
+ } else
+ g_free(string);
gtk_object_unref(GTK_OBJECT(iterator));
position = 0;
iterator = e_card_list_get_iterator(phone_list);
phone = e_card_iterator_get(iterator);
editable = GTK_EDITABLE(glade_xml_get_widget(editor->gui, "entry-phone1"));
- if (phone) {
- phone_copy = e_card_phone_copy(phone);
- if (phone_copy->number)
- g_free(phone_copy->number);
- phone_copy->number = gtk_editable_get_chars(editable, 0, -1);
- e_card_iterator_set(iterator, phone_copy);
- e_card_phone_free(phone_copy);
- } else {
- phone_copy = g_new0(ECardPhone, 1);
- phone_copy->number = gtk_editable_get_chars(editable, 0, -1);
- e_card_list_append(phone_list, phone_copy);
- e_card_phone_free(phone_copy);
- }
+ string = gtk_editable_get_chars(editable, 0, -1);
+ if (string && *string) {
+ if (phone) {
+ phone_copy = e_card_phone_copy(phone);
+ if (phone_copy->number)
+ g_free(phone_copy->number);
+ phone_copy->number = string;
+ e_card_iterator_set(iterator, phone_copy);
+ e_card_phone_free(phone_copy);
+ } else {
+ phone_copy = g_new0(ECardPhone, 1);
+ phone_copy->number = string;
+ e_card_list_append(phone_list, phone_copy);
+ e_card_phone_free(phone_copy);
+ }
+ } else
+ g_free(string);
gtk_object_unref(GTK_OBJECT(iterator));
position = 0;
iterator = e_card_list_get_iterator(email_list);
editable = GTK_EDITABLE(glade_xml_get_widget(editor->gui, "entry-email1"));
email = gtk_editable_get_chars(editable, 0, -1);
- if (e_card_iterator_is_valid(iterator))
- e_card_iterator_set(iterator, email);
- else
- e_card_list_append(email_list, email);
+ if (email && *email) {
+ if (e_card_iterator_is_valid(iterator))
+ e_card_iterator_set(iterator, email);
+ else
+ e_card_list_append(email_list, email);
+ }
g_free(email);
gtk_object_unref(GTK_OBJECT(iterator));
position = 0;
editable = GTK_EDITABLE(glade_xml_get_widget(editor->gui, "entry-web"));
url = gtk_editable_get_chars(editable, 0, -1);
+ if (url && *url)
+ gtk_object_set(GTK_OBJECT(card),
+ "url", url,
+ NULL);
+ g_free(url);
editable = GTK_EDITABLE(glade_xml_get_widget(editor->gui, "entry-jobtitle"));
title = gtk_editable_get_chars(editable, 0, -1);
+ if (title && *title)
+ gtk_object_set(GTK_OBJECT(card),
+ "title", title,
+ NULL);
+ g_free(title);
- gtk_object_set(GTK_OBJECT(card),
- "full_name", fname,
- "url", url,
- "title", title,
- NULL);
-
- g_free(fname);
- g_free(url);
}
}
diff --git a/addressbook/gui/contact-editor/e-contact-editor.c b/addressbook/gui/contact-editor/e-contact-editor.c
index ebd3611f2e..2e0d2219a5 100644
--- a/addressbook/gui/contact-editor/e-contact-editor.c
+++ b/addressbook/gui/contact-editor/e-contact-editor.c
@@ -654,6 +654,7 @@ extract_info(EContactEditor *editor)
ECard *card = editor->card;
if (card) {
char *fname;
+ char *string;
ECardList *address_list;
ECardList *phone_list;
ECardList *email_list;
@@ -678,69 +679,86 @@ extract_info(EContactEditor *editor)
position = 0;
editable = GTK_EDITABLE(glade_xml_get_widget(editor->gui, "entry-fullname"));
fname = gtk_editable_get_chars(editable, 0, -1);
+ if (fname && *fname)
+ gtk_object_set(GTK_OBJECT(card),
+ "full_name", fname,
+ NULL);
+ g_free(fname);
iterator = e_card_list_get_iterator(address_list);
address = e_card_iterator_get(iterator);
editable = GTK_EDITABLE(glade_xml_get_widget(editor->gui, "text-address"));
- if (address) {
- address_copy = e_card_delivery_address_copy(address);
- if (address_copy->city)
- g_free(address_copy->city);
- address_copy->city = gtk_editable_get_chars(editable, 0, -1);
- e_card_iterator_set(iterator, address_copy);
- e_card_delivery_address_free(address_copy);
- } else {
- address_copy = g_new0(ECardDeliveryAddress, 1);
- address_copy->city = gtk_editable_get_chars(editable, 0, -1);
- e_card_list_append(address_list, address_copy);
- e_card_delivery_address_free(address_copy);
- }
+ string = gtk_editable_get_chars(editable, 0, -1);
+ if (string && *string) {
+ if (address) {
+ address_copy = e_card_delivery_address_copy(address);
+ if (address_copy->city)
+ g_free(address_copy->city);
+ address_copy->city = string;
+ e_card_iterator_set(iterator, address_copy);
+ e_card_delivery_address_free(address_copy);
+ } else {
+ address_copy = g_new0(ECardDeliveryAddress, 1);
+ address_copy->city = string;
+ e_card_list_append(address_list, address_copy);
+ e_card_delivery_address_free(address_copy);
+ }
+ } else
+ g_free(string);
gtk_object_unref(GTK_OBJECT(iterator));
position = 0;
iterator = e_card_list_get_iterator(phone_list);
phone = e_card_iterator_get(iterator);
editable = GTK_EDITABLE(glade_xml_get_widget(editor->gui, "entry-phone1"));
- if (phone) {
- phone_copy = e_card_phone_copy(phone);
- if (phone_copy->number)
- g_free(phone_copy->number);
- phone_copy->number = gtk_editable_get_chars(editable, 0, -1);
- e_card_iterator_set(iterator, phone_copy);
- e_card_phone_free(phone_copy);
- } else {
- phone_copy = g_new0(ECardPhone, 1);
- phone_copy->number = gtk_editable_get_chars(editable, 0, -1);
- e_card_list_append(phone_list, phone_copy);
- e_card_phone_free(phone_copy);
- }
+ string = gtk_editable_get_chars(editable, 0, -1);
+ if (string && *string) {
+ if (phone) {
+ phone_copy = e_card_phone_copy(phone);
+ if (phone_copy->number)
+ g_free(phone_copy->number);
+ phone_copy->number = string;
+ e_card_iterator_set(iterator, phone_copy);
+ e_card_phone_free(phone_copy);
+ } else {
+ phone_copy = g_new0(ECardPhone, 1);
+ phone_copy->number = string;
+ e_card_list_append(phone_list, phone_copy);
+ e_card_phone_free(phone_copy);
+ }
+ } else
+ g_free(string);
gtk_object_unref(GTK_OBJECT(iterator));
position = 0;
iterator = e_card_list_get_iterator(email_list);
editable = GTK_EDITABLE(glade_xml_get_widget(editor->gui, "entry-email1"));
email = gtk_editable_get_chars(editable, 0, -1);
- if (e_card_iterator_is_valid(iterator))
- e_card_iterator_set(iterator, email);
- else
- e_card_list_append(email_list, email);
+ if (email && *email) {
+ if (e_card_iterator_is_valid(iterator))
+ e_card_iterator_set(iterator, email);
+ else
+ e_card_list_append(email_list, email);
+ }
g_free(email);
gtk_object_unref(GTK_OBJECT(iterator));
position = 0;
editable = GTK_EDITABLE(glade_xml_get_widget(editor->gui, "entry-web"));
url = gtk_editable_get_chars(editable, 0, -1);
+ if (url && *url)
+ gtk_object_set(GTK_OBJECT(card),
+ "url", url,
+ NULL);
+ g_free(url);
editable = GTK_EDITABLE(glade_xml_get_widget(editor->gui, "entry-jobtitle"));
title = gtk_editable_get_chars(editable, 0, -1);
+ if (title && *title)
+ gtk_object_set(GTK_OBJECT(card),
+ "title", title,
+ NULL);
+ g_free(title);
- gtk_object_set(GTK_OBJECT(card),
- "full_name", fname,
- "url", url,
- "title", title,
- NULL);
-
- g_free(fname);
- g_free(url);
}
}