Call e_select_names_model_clean after adding address. This should deal

2001-03-31  Jon Trowbridge  <trow@ximian.com>

	* gui/component/select-names/e-select-names.c
	(real_add_address_cb): Call e_select_names_model_clean after
	adding address.  This should deal with the bug reports related
	to stray commas.

	* gui/component/select-names/e-select-names-model.c
	(e_select_names_model_clean): Added.  Remove all empty destinations.

	* backend/ebook/e-destination.c (e_destination_is_empty): Added.  Check
	if an EDestination is in essentially a null state.

svn path=/trunk/; revision=9067

3 files changed, 35 insertions, 1 deletions

diff --git a/addressbook/gui/component/select-names/e-select-names-model.c b/addressbook/gui/component/select-names/e-select-names-model.c
index 5dd11496d2..59d7bc7c24 100644
--- a/addressbook/gui/component/select-names/e-select-names-model.c
+++ b/addressbook/gui/component/select-names/e-select-names-model.c
@@ -433,9 +433,40 @@ e_select_names_model_delete (ESelectNamesModel *model, gint index)
 }
 
 void
+e_select_names_model_clean (ESelectNamesModel *model)
+{
+	GList *iter, *next;
+	gboolean changed = FALSE;
+
+	g_return_if_fail (model != NULL && E_IS_SELECT_NAMES_MODEL (model));
+
+	iter = model->priv->data;
+
+	while (iter) {
+		EDestination *dest;
+
+		next = g_list_next (iter);
+		dest = iter->data ? E_DESTINATION (iter->data) : NULL;
+
+		if (dest == NULL || e_destination_is_empty (dest)) {
+			if (dest)
+				gtk_object_unref (GTK_OBJECT (dest));
+			model->priv->data = g_list_remove_link (model->priv->data, iter);
+			g_list_free_1 (iter);
+			changed = TRUE;
+		}
+		
+		iter = next;
+	}
+
+	if (changed)
+		e_select_names_model_changed (model);
+}
+
+void
 e_select_names_model_delete_all (ESelectNamesModel *model)
 {
-	g_return_if_fail (model != NULL);
+	g_return_if_fail (model != NULL && E_IS_SELECT_NAMES_MODEL (model));
 
 	g_list_foreach (model->priv->data, (GFunc) gtk_object_unref, NULL);
 	g_list_free (model->priv->data);
diff --git a/addressbook/gui/component/select-names/e-select-names-model.h b/addressbook/gui/component/select-names/e-select-names-model.h
index 9d8c66f063..ab7477762c 100644
--- a/addressbook/gui/component/select-names/e-select-names-model.h
+++ b/addressbook/gui/component/select-names/e-select-names-model.h
@@ -59,6 +59,8 @@ void          e_select_names_model_replace    (ESelectNamesModel *model, gint in
 void          e_select_names_model_delete     (ESelectNamesModel *model, gint index);
 void          e_select_names_model_delete_all (ESelectNamesModel *model);
 
+void          e_select_names_model_clean      (ESelectNamesModel *model);
+
 void          e_select_names_model_name_pos (ESelectNamesModel *model, gint index, gint *pos, gint *length);
 void          e_select_names_model_text_pos (ESelectNamesModel *model, gint pos, gint *index, gint *start_pos, gint *length);
 
diff --git a/addressbook/gui/component/select-names/e-select-names.c b/addressbook/gui/component/select-names/e-select-names.c
index cbe9624be7..30dc482e85 100644
--- a/addressbook/gui/component/select-names/e-select-names.c
+++ b/addressbook/gui/component/select-names/e-select-names.c
@@ -149,6 +149,7 @@ real_add_address_cb (int model_row,
 	e_select_names_model_insert (child->source,
 				     e_select_names_model_count (child->source),
 				     dest);
+	e_select_names_model_clean (child->source);
 
 	gtk_object_unref(GTK_OBJECT(card));
 }