From f9ac993fdabacecd795037516244840c2142ec13 Mon Sep 17 00:00:00 2001 From: Matthew Barnes Date: Tue, 13 Jul 2010 08:12:59 -0400 Subject: Bug 624216 - Runtime warning in Attachment Reminder plugin Also fixes the same bug in Email Custom Header plugin. --- plugins/attachment-reminder/attachment-reminder.c | 137 +++++----------- plugins/email-custom-header/email-custom-header.c | 182 +++++++--------------- 2 files changed, 101 insertions(+), 218 deletions(-) diff --git a/plugins/attachment-reminder/attachment-reminder.c b/plugins/attachment-reminder/attachment-reminder.c index 19decd6cbf..99f8f080d8 100644 --- a/plugins/attachment-reminder/attachment-reminder.c +++ b/plugins/attachment-reminder/attachment-reminder.c @@ -59,10 +59,10 @@ enum { }; gint e_plugin_lib_enable (EPlugin *ep, gint enable); -GtkWidget *e_plugin_lib_get_configure_widget (EPlugin *epl); +GtkWidget *e_plugin_lib_get_configure_widget (EPlugin *plugin); void org_gnome_evolution_attachment_reminder (EPlugin *ep, EMEventTargetComposer *t); -GtkWidget* org_gnome_attachment_reminder_config_option (struct _EPlugin *epl, struct _EConfigHookItemFactoryData *data); +GtkWidget* org_gnome_attachment_reminder_config_option (EPlugin *plugin, struct _EConfigHookItemFactoryData *data); static gboolean ask_for_missing_attachment (EPlugin *ep, GtkWindow *widget); static gboolean check_for_attachment_clues (gchar *msg); @@ -70,11 +70,6 @@ static gboolean check_for_attachment (EMsgComposer *composer); static gchar * strip_text_msg (gchar *msg); static void commit_changes (UIData *ui); -static void cell_edited_callback (GtkCellRendererText *cell, gchar *path_string, - gchar *new_text,UIData *ui); -static gboolean clue_foreach_check_isempty (GtkTreeModel *model, GtkTreePath - *path, GtkTreeIter *iter, UIData *ui); - gint e_plugin_lib_enable (EPlugin *ep, gint enable) { @@ -227,7 +222,9 @@ commit_changes (UIData *ui) while (valid) { gchar *keyword; - gtk_tree_model_get (model, &iter, CLUE_KEYWORD_COLUMN, &keyword, -1); + gtk_tree_model_get ( + model, &iter, CLUE_KEYWORD_COLUMN, &keyword, -1); + /* Check if the keyword is not empty */ if ((keyword) && (g_utf8_strlen(g_strstrip(keyword), -1) > 0)) clue_list = g_slist_append (clue_list, keyword); @@ -241,107 +238,53 @@ commit_changes (UIData *ui) } static void -clue_check_isempty (GtkTreeModel *model, GtkTreePath *path, GtkTreeIter *iter, UIData *ui) -{ - GtkTreeSelection *selection; - gchar *keyword = NULL; - gboolean valid; - - selection = gtk_tree_view_get_selection (GTK_TREE_VIEW (ui->treeview)); - /* move to the previous node */ - valid = gtk_tree_path_prev (path); - - gtk_tree_model_get (model, iter, CLUE_KEYWORD_COLUMN, &keyword, -1); - if ((keyword) && !(g_utf8_strlen (g_strstrip (keyword), -1) > 0)) - gtk_list_store_remove (ui->store, iter); - - /* Check if we have a valid row to select. If not, then select - * the previous row */ - if (gtk_list_store_iter_is_valid (GTK_LIST_STORE (model), iter)) { - gtk_tree_selection_select_iter (selection, iter); - } else { - if (path && valid) { - gtk_tree_model_get_iter (model, iter, path); - gtk_tree_selection_select_iter (selection, iter); - } - } - - gtk_widget_grab_focus (ui->treeview); - g_free (keyword); -} - -static gboolean -clue_foreach_check_isempty (GtkTreeModel *model, GtkTreePath *path, GtkTreeIter *iter, UIData *ui) -{ - gboolean valid; - - valid = gtk_tree_model_get_iter_first (model, iter); - while (valid && gtk_list_store_iter_is_valid (ui->store, iter)) { - gchar *keyword = NULL; - gtk_tree_model_get (model, iter, CLUE_KEYWORD_COLUMN, &keyword, -1); - /* Check if the keyword is not empty and then emit the row-changed - signal (if we delete the row, then the iter gets corrupted) */ - if ((keyword) && !(g_utf8_strlen (g_strstrip (keyword), -1) > 0)) - gtk_tree_model_row_changed (model, path, iter); - - g_free (keyword); - valid = gtk_tree_model_iter_next (model, iter); - } - - return FALSE; -} - -static void -cell_edited_callback (GtkCellRendererText *cell, - gchar *path_string, - gchar *new_text, - UIData *ui) +cell_edited_cb (GtkCellRendererText *cell, + gchar *path_string, + gchar *new_text, + UIData *ui) { GtkTreeModel *model; GtkTreeIter iter; model = gtk_tree_view_get_model (GTK_TREE_VIEW (ui->treeview)); - gtk_tree_model_get_iter_from_string (model, &iter, path_string); - gtk_list_store_set (GTK_LIST_STORE (model), &iter, - CLUE_KEYWORD_COLUMN, new_text, -1); + if (new_text == NULL || *g_strstrip (new_text) == '\0') + gtk_button_clicked (GTK_BUTTON (ui->clue_remove)); + else { + gtk_list_store_set ( + GTK_LIST_STORE (model), &iter, + CLUE_KEYWORD_COLUMN, new_text, -1); + commit_changes (ui); + } +} - commit_changes (ui); +static void +cell_editing_canceled_cb (GtkCellRenderer *cell, + UIData *ui) +{ + gtk_button_clicked (GTK_BUTTON (ui->clue_remove)); } static void clue_add_clicked (GtkButton *button, UIData *ui) { GtkTreeModel *model; - GtkTreeIter iter; - gchar *new_clue = NULL; - GtkTreeViewColumn *focus_col; + GtkTreeView *tree_view; + GtkTreeViewColumn *column; GtkTreePath *path; + GtkTreeIter iter; - model = gtk_tree_view_get_model (GTK_TREE_VIEW (ui->treeview)); - gtk_tree_model_foreach (model, (GtkTreeModelForeachFunc) clue_foreach_check_isempty, ui); - - /* Disconnect from signal so that we can create an empty row */ - g_signal_handlers_disconnect_matched(G_OBJECT(model), G_SIGNAL_MATCH_FUNC, 0, 0, NULL, clue_check_isempty, ui); + tree_view = GTK_TREE_VIEW (ui->treeview); + model = gtk_tree_view_get_model (tree_view); - /* TODO : Trim and check for blank strings */ - new_clue = g_strdup (""); gtk_list_store_append (GTK_LIST_STORE (model), &iter); - gtk_list_store_set (GTK_LIST_STORE (model), &iter, - CLUE_KEYWORD_COLUMN, new_clue, -1); - focus_col = gtk_tree_view_get_column (GTK_TREE_VIEW (ui->treeview), CLUE_KEYWORD_COLUMN); path = gtk_tree_model_get_path (model, &iter); - - if (path) { - gtk_tree_view_set_cursor (GTK_TREE_VIEW (ui->treeview), path, focus_col, TRUE); - gtk_tree_view_row_activated(GTK_TREE_VIEW(ui->treeview), path, focus_col); - gtk_tree_path_free (path); - } - - /* We have done our job, connect back to the signal */ - g_signal_connect(G_OBJECT(model), "row-changed", G_CALLBACK(clue_check_isempty), ui); + column = gtk_tree_view_get_column (tree_view, CLUE_KEYWORD_COLUMN); + gtk_tree_view_set_cursor (tree_view, path, column, TRUE); + gtk_tree_view_row_activated (tree_view, path, column); + gtk_tree_path_free (path); } static void @@ -437,7 +380,7 @@ destroy_ui_data (gpointer data) } GtkWidget * -e_plugin_lib_get_configure_widget (EPlugin *epl) +e_plugin_lib_get_configure_widget (EPlugin *plugin) { GtkCellRenderer *renderer; GtkTreeSelection *selection; @@ -445,7 +388,6 @@ e_plugin_lib_get_configure_widget (EPlugin *epl) GConfClient *gconf = gconf_client_get_default(); GtkWidget *hbox; GSList *clue_list = NULL, *list; - GtkTreeModel *model; GtkWidget *reminder_configuration_box; GtkWidget *clue_container; @@ -509,7 +451,12 @@ e_plugin_lib_get_configure_widget (EPlugin *epl) gtk_tree_view_insert_column_with_attributes (GTK_TREE_VIEW (ui->treeview), -1, _("Keywords"), renderer, "text", CLUE_KEYWORD_COLUMN, NULL); g_object_set (G_OBJECT (renderer), "editable", TRUE, NULL); - g_signal_connect(renderer, "edited", (GCallback) cell_edited_callback, ui); + g_signal_connect ( + renderer, "edited", + G_CALLBACK (cell_edited_cb), ui); + g_signal_connect ( + renderer, "editing-canceled", + G_CALLBACK (cell_editing_canceled_cb), ui); selection = gtk_tree_view_get_selection (GTK_TREE_VIEW (ui->treeview)); gtk_tree_selection_set_mode (selection, GTK_SELECTION_SINGLE); @@ -527,9 +474,6 @@ e_plugin_lib_get_configure_widget (EPlugin *epl) g_signal_connect (G_OBJECT (ui->clue_edit), "clicked", G_CALLBACK (clue_edit_clicked), ui); gtk_widget_set_sensitive (ui->clue_edit, FALSE); - model = gtk_tree_view_get_model (GTK_TREE_VIEW (ui->treeview)); - g_signal_connect(G_OBJECT(model), "row-changed", G_CALLBACK(clue_check_isempty), ui); - /* Populate tree view with values from gconf */ clue_list = gconf_client_get_list ( gconf, GCONF_KEY_ATTACH_REMINDER_CLUES, GCONF_VALUE_STRING, NULL ); @@ -558,7 +502,8 @@ e_plugin_lib_get_configure_widget (EPlugin *epl) /* Configuration in Mail Prefs Page goes here */ GtkWidget * -org_gnome_attachment_reminder_config_option (struct _EPlugin *epl, struct _EConfigHookItemFactoryData *data) +org_gnome_attachment_reminder_config_option (EPlugin *plugin, + struct _EConfigHookItemFactoryData *data) { /* This function and the hook needs to be removed, once the configure code is thoroughly tested */ diff --git a/plugins/email-custom-header/email-custom-header.c b/plugins/email-custom-header/email-custom-header.c index 6b94c721f3..b78474d78b 100644 --- a/plugins/email-custom-header/email-custom-header.c +++ b/plugins/email-custom-header/email-custom-header.c @@ -630,74 +630,39 @@ commit_changes (ConfigData *cd) } static void -header_isempty (GtkTreeModel *model, GtkTreePath *path, GtkTreeIter *iter, ConfigData *cd) +cell_edited_cb (GtkCellRendererText *cell, + gchar *path_string, + gchar *new_text, + ConfigData *cd) { - GtkTreeSelection *selection; - gchar *keyword = NULL; - gchar *value = NULL; - gboolean valid; - - selection = gtk_tree_view_get_selection (GTK_TREE_VIEW (cd->treeview)); - /* move to the previous node */ - valid = gtk_tree_path_prev (path); - - gtk_tree_model_get (model, iter, HEADER_KEY_COLUMN, &keyword, -1); + GtkTreeModel *model; + GtkTreeIter iter; - if ((keyword) && !(g_utf8_strlen (g_strstrip (keyword), -1) > 0)) - gtk_list_store_remove (cd->store, iter); + model = gtk_tree_view_get_model (GTK_TREE_VIEW (cd->treeview)); + gtk_tree_model_get_iter_from_string (model, &iter, path_string); - /* Check if we have a valid row to select. If not, then select - * the previous row */ - if (gtk_list_store_iter_is_valid (GTK_LIST_STORE (model), iter)) { - gtk_tree_selection_select_iter (selection, iter); - } else { - if (path && valid) { - gtk_tree_model_get_iter (model, iter, path); - gtk_tree_selection_select_iter (selection, iter); - } + if (new_text == NULL || *g_strstrip (new_text) == '\0') + gtk_button_clicked (GTK_BUTTON (cd->header_remove)); + else { + gtk_list_store_set ( + GTK_LIST_STORE (model), &iter, + HEADER_KEY_COLUMN, new_text, -1); + commit_changes (cd); } - - gtk_widget_grab_focus (cd->treeview); - g_free (keyword); - g_free (value); } -static gboolean -header_foreach_check_isempty (GtkTreeModel *model, GtkTreePath *path, GtkTreeIter *iter, ConfigData *cd) +static void +cell_editing_canceled_cb (GtkCellRenderer *cell, + ConfigData *cd) { - gboolean valid; - - valid = gtk_tree_model_get_iter_first (model, iter); - while (valid && gtk_list_store_iter_is_valid (cd->store, iter)) { - gchar *keyword = NULL; - gchar *value = NULL; - gtk_tree_model_get (model, iter, HEADER_KEY_COLUMN, &keyword, -1); - /* Check if the keyword is not empty and then emit the row-changed - signal (if we delete the row, then the iter gets corrupted) */ - if ((keyword) && !(g_utf8_strlen (g_strstrip (keyword), -1) > 0)) - gtk_tree_model_row_changed (model, path, iter); - else { - gtk_tree_model_get (model, iter, HEADER_VALUE_COLUMN, &value, -1); - /* Check if the keyword is not empty and then emit the row-changed - signal (if we delete the row, then the iter gets corrupted) */ - if ((value) && !(g_utf8_strlen (g_strstrip (value), -1) > 0)) - gtk_tree_model_row_changed (model, path, iter); - } - - g_free (keyword); - g_free (value); - - valid = gtk_tree_model_iter_next (model, iter); - } - - return FALSE; + gtk_button_clicked (GTK_BUTTON (cd->header_remove)); } static void -cell_edited_callback (GtkCellRendererText *cell, - gchar *path_string, - gchar *new_text, - ConfigData *cd) +cell_value_edited_cb (GtkCellRendererText *cell, + gchar *path_string, + gchar *new_text, + ConfigData *cd) { GtkTreeModel *model; GtkTreeIter iter; @@ -706,62 +671,32 @@ cell_edited_callback (GtkCellRendererText *cell, gtk_tree_model_get_iter_from_string (model, &iter, path_string); - gtk_list_store_set (GTK_LIST_STORE (model), &iter, - HEADER_KEY_COLUMN, new_text, -1); + gtk_list_store_set ( + GTK_LIST_STORE (model), &iter, + HEADER_VALUE_COLUMN, new_text, -1); commit_changes (cd); } -static void -cell_value_edited_callback (GtkCellRendererText *cell, - gchar *path_string, - gchar *new_text, - ConfigData *cd) -{ - GtkTreeModel *model; - GtkTreeIter iter; - - model = gtk_tree_view_get_model (GTK_TREE_VIEW (cd->treeview)); - - gtk_tree_model_get_iter_from_string (model, &iter, path_string); - - gtk_list_store_set (GTK_LIST_STORE (model), &iter, - HEADER_VALUE_COLUMN, new_text, -1); - - commit_changes (cd); -} - static void header_add_clicked (GtkButton *button, ConfigData *cd) { GtkTreeModel *model; - GtkTreeIter iter; - GtkTreeViewColumn *focus_col; + GtkTreeView *tree_view; + GtkTreeViewColumn *column; GtkTreePath *path; + GtkTreeIter iter; - model = gtk_tree_view_get_model (GTK_TREE_VIEW (cd->treeview)); - gtk_tree_model_foreach (model, (GtkTreeModelForeachFunc) header_foreach_check_isempty, cd); - - /* Disconnect from signal so that we can create an empty row */ - g_signal_handlers_disconnect_matched(G_OBJECT(model), G_SIGNAL_MATCH_FUNC, 0, 0, NULL, header_isempty, cd); + tree_view = GTK_TREE_VIEW (cd->treeview); + model = gtk_tree_view_get_model (tree_view); gtk_list_store_append (GTK_LIST_STORE (model), &iter); - gtk_list_store_set (GTK_LIST_STORE (model), &iter, - HEADER_KEY_COLUMN, "", -1); - gtk_list_store_set (GTK_LIST_STORE (model), &iter, - HEADER_VALUE_COLUMN, "", -1); - focus_col = gtk_tree_view_get_column (GTK_TREE_VIEW (cd->treeview), HEADER_KEY_COLUMN); path = gtk_tree_model_get_path (model, &iter); - - if (path) { - gtk_tree_view_set_cursor (GTK_TREE_VIEW (cd->treeview), path, focus_col, TRUE); - gtk_tree_view_row_activated(GTK_TREE_VIEW(cd->treeview), path, focus_col); - gtk_tree_path_free (path); - } - - /* We have done our job, connect back to the signal */ - g_signal_connect(G_OBJECT(model), "row-changed", G_CALLBACK(header_isempty), cd); + column = gtk_tree_view_get_column (tree_view, HEADER_KEY_COLUMN); + gtk_tree_view_set_cursor (tree_view, path, column, TRUE); + gtk_tree_view_row_activated (tree_view, path, column); + gtk_tree_path_free (path); } static void @@ -865,10 +800,9 @@ e_plugin_lib_get_configure_widget (EPlugin *epl) GtkWidget *hbox; GSList *list; GSList *header_list = NULL; - GtkTreeModel *model; gint index; - gchar *buffer; - GtkTreeViewColumn *col; + gchar *buffer; + GtkTreeViewColumn *col; gint col_pos; GConfClient *client = gconf_client_get_default(); ConfigData *cd = g_new0 (ConfigData, 1); @@ -950,22 +884,29 @@ e_plugin_lib_get_configure_widget (EPlugin *epl) col_pos = gtk_tree_view_insert_column_with_attributes (GTK_TREE_VIEW (cd->treeview), -1, _("Key"), renderer, "text", HEADER_KEY_COLUMN, NULL); col = gtk_tree_view_get_column (GTK_TREE_VIEW (cd->treeview), col_pos -1); - gtk_tree_view_column_set_resizable (col, TRUE); - gtk_tree_view_column_set_reorderable(col, TRUE); - g_object_set (col, "min-width", 50, NULL); + gtk_tree_view_column_set_resizable (col, TRUE); + gtk_tree_view_column_set_reorderable(col, TRUE); + g_object_set (col, "min-width", 50, NULL); g_object_set (G_OBJECT (renderer), "editable", TRUE, NULL); - g_signal_connect(renderer, "edited", (GCallback) cell_edited_callback, cd); + g_signal_connect ( + renderer, "edited", + G_CALLBACK (cell_edited_cb), cd); + g_signal_connect ( + renderer, "editing-canceled", + G_CALLBACK (cell_editing_canceled_cb), cd); renderer = gtk_cell_renderer_text_new (); - col_pos = gtk_tree_view_insert_column_with_attributes (GTK_TREE_VIEW (cd->treeview), -1, _("Values"), - renderer, "text", HEADER_VALUE_COLUMN, NULL); - col = gtk_tree_view_get_column (GTK_TREE_VIEW (cd->treeview), col_pos -1); - gtk_tree_view_column_set_resizable (col, TRUE); - gtk_tree_view_column_set_reorderable(col, TRUE); - g_object_set (G_OBJECT (renderer), "editable", TRUE, NULL); + col_pos = gtk_tree_view_insert_column_with_attributes (GTK_TREE_VIEW (cd->treeview), -1, _("Values"), + renderer, "text", HEADER_VALUE_COLUMN, NULL); + col = gtk_tree_view_get_column (GTK_TREE_VIEW (cd->treeview), col_pos -1); + gtk_tree_view_column_set_resizable (col, TRUE); + gtk_tree_view_column_set_reorderable(col, TRUE); + g_object_set (G_OBJECT (renderer), "editable", TRUE, NULL); - g_signal_connect(renderer, "edited", (GCallback) cell_value_edited_callback, cd); + g_signal_connect ( + renderer, "edited", + G_CALLBACK (cell_value_edited_cb), cd); selection = gtk_tree_view_get_selection (GTK_TREE_VIEW (cd->treeview)); gtk_tree_selection_set_mode (selection, GTK_SELECTION_SINGLE); @@ -983,25 +924,22 @@ e_plugin_lib_get_configure_widget (EPlugin *epl) g_signal_connect (G_OBJECT (cd->header_edit), "clicked", G_CALLBACK (header_edit_clicked), cd); gtk_widget_set_sensitive (cd->header_edit, FALSE); - model = gtk_tree_view_get_model (GTK_TREE_VIEW (cd->treeview)); - g_signal_connect(G_OBJECT(model), "row-changed", G_CALLBACK(header_isempty), cd); - /* Populate tree view with values from gconf */ header_list = gconf_client_get_list (client,GCONF_KEY_CUSTOM_HEADER,GCONF_VALUE_STRING, NULL); for (list = header_list; list; list = g_slist_next (list)) { gchar **parse_header_list; - buffer = list->data; + buffer = list->data; gtk_list_store_append (cd->store, &iter); - parse_header_list = g_strsplit_set (buffer, "=,", -1); + parse_header_list = g_strsplit_set (buffer, "=,", -1); - gtk_list_store_set (cd->store, &iter, HEADER_KEY_COLUMN, parse_header_list[0], -1); + gtk_list_store_set (cd->store, &iter, HEADER_KEY_COLUMN, parse_header_list[0], -1); for (index = 0; parse_header_list[index+1] ; ++index) { - gtk_list_store_set (cd->store, &iter, HEADER_VALUE_COLUMN, parse_header_list[index+1], -1); - } + gtk_list_store_set (cd->store, &iter, HEADER_VALUE_COLUMN, parse_header_list[index+1], -1); + } } if (header_list) { -- cgit v1.2.3