X-Git-Url: http://git.maemo.org/git/?p=modest;a=blobdiff_plain;f=src%2Fmodest-ui-actions.c;h=549854cba658f9ab0588d02571d85a3b00f228e6;hp=c5aa3d826aa00a44e0dd73f278bf481a8eefef0c;hb=da606c2c7e6a4aa46790dcd692f57fbd8966ccfa;hpb=3c9ac384ee1bf67629b26a96940c305ae2447036

diff --git a/src/modest-ui-actions.c b/src/modest-ui-actions.c
index c5aa3d8..549854c 100644
--- a/src/modest-ui-actions.c
+++ b/src/modest-ui-actions.c
@@ -177,6 +177,12 @@ static void modest_ui_actions_on_main_window_move_to (GtkAction *action,
 						      GtkWidget *folder_view,
 						      TnyFolderStore *dst_folder,
 						      ModestMainWindow *win);
+#ifdef MODEST_TOOLKIT_HILDON2
+static void modest_ui_actions_on_folder_window_move_to (GtkWidget *folder_view,
+							TnyFolderStore *dst_folder,
+							TnyList *selection,
+							GtkWindow *win);
+#endif
 
 static void modest_ui_actions_on_window_move_to (GtkAction *action,
 						 TnyList *list_to_move,
@@ -3120,11 +3126,10 @@ modest_ui_actions_on_select_editor_background_color (GtkAction *action,
 }
 
 void 
-modest_ui_actions_on_insert_image (GtkAction *action,
+modest_ui_actions_on_insert_image (GObject *object,
 				   ModestMsgEditWindow *window)
 {
 	g_return_if_fail (MODEST_IS_MSG_EDIT_WINDOW (window));
-	g_return_if_fail (GTK_IS_ACTION (action));
 
 
 	if (modest_platform_check_memory_low (MODEST_WINDOW(window), TRUE))
@@ -3154,7 +3159,6 @@ modest_ui_actions_on_remove_attachments (GtkAction *action,
 					 ModestMsgEditWindow *window)
 {
 	g_return_if_fail (MODEST_IS_MSG_EDIT_WINDOW (window));
-	g_return_if_fail (GTK_IS_ACTION (action));
 
 	modest_msg_edit_window_remove_attachments (window, NULL);
 }
@@ -3258,20 +3262,22 @@ do_create_folder_cb (ModestMailOperation *mail_op,
 
 static void
 do_create_folder (GtkWindow *parent_window, 
-		  TnyFolderStore *parent_folder, 
+		  TnyFolderStore *suggested_parent, 
 		  const gchar *suggested_name)
 {
 	gint result;
 	gchar *folder_name = NULL;
+	TnyFolderStore *parent_folder = NULL;
 
 	result = modest_platform_run_new_folder_dialog (GTK_WINDOW (parent_window),
-							parent_folder,
+							suggested_parent,
 							(gchar *) suggested_name,
-							&folder_name);
-	
-	if (result == GTK_RESPONSE_ACCEPT) {
+							&folder_name,
+							&parent_folder);
+
+	if (result == GTK_RESPONSE_ACCEPT && parent_folder) {
 		ModestMailOperation *mail_op;
-		
+
 		mail_op  = modest_mail_operation_new ((GObject *) parent_window);
 		modest_mail_operation_queue_add (modest_runtime_get_mail_operation_queue (), 
 						 mail_op);
@@ -3282,6 +3288,9 @@ do_create_folder (GtkWindow *parent_window,
 						     folder_name);
 		g_object_unref (mail_op);
 	}
+
+	if (parent_folder)
+		g_object_unref (parent_folder);
 }
 
 static void
@@ -3312,22 +3321,41 @@ modest_ui_actions_create_folder(GtkWidget *parent_window,
 {
 	TnyFolderStore *parent_folder;
 
+#ifdef MODEST_TOOLKIT_HILDON2
+	const gchar *active_account;
+	TnyAccount *account;
+	ModestTnyAccountStore *acc_store;
+
+	/* In hildon 2.2 we use the current account as default parent */
+	acc_store = modest_runtime_get_account_store ();
+	active_account = modest_window_get_active_account (MODEST_WINDOW (parent_window));
+	if (active_account) {
+		account = modest_tny_account_store_get_server_account (acc_store,
+								       active_account,
+								       TNY_ACCOUNT_TYPE_STORE);
+		parent_folder = TNY_FOLDER_STORE (account);
+	} else {
+		parent_folder = (TnyFolderStore *) 
+			modest_tny_account_store_get_local_folders_account (acc_store);
+	}
+#else
 	parent_folder = modest_folder_view_get_selected (MODEST_FOLDER_VIEW(folder_view));
-	
+#endif
+
 	if (parent_folder) {
 		/* The parent folder will be freed in the callback */
-		modest_platform_connect_if_remote_and_perform (GTK_WINDOW (parent_window), 
+		modest_platform_connect_if_remote_and_perform (GTK_WINDOW (parent_window),
 							       TRUE,
 							       parent_folder,
-							       create_folder_performer, 
+							       create_folder_performer,
 							       parent_folder);
 	}
 }
 
-void 
+void
 modest_ui_actions_on_new_folder (GtkAction *action, ModestWindow *window)
 {
-	
+
 	g_return_if_fail (MODEST_IS_WINDOW(window));
 
 	if (MODEST_IS_MAIN_WINDOW (window)) {
@@ -3392,8 +3420,8 @@ on_rename_folder_cb (ModestMailOperation *mail_op,
 {
 	ModestFolderView *folder_view;
 
-	/* If the window was closed when renaming a folder this could
-	   happen */
+	/* If the window was closed when renaming a folder, or if
+	 * it's not a main window this will happen */
 	if (!MODEST_IS_FOLDER_VIEW (user_data))
 		return;
 
@@ -3422,11 +3450,7 @@ on_rename_folder_performer (gboolean canceled,
 	if (canceled || err) {
 		/* In memory full conditions we could get this error here */
 		check_memory_full_error ((GtkWidget *) parent_window, err);
-	} else if (MODEST_IS_MAIN_WINDOW(parent_window)) {
-
-		folder_view = modest_main_window_get_child_widget (
-				MODEST_MAIN_WINDOW (parent_window),
-				MODEST_MAIN_WINDOW_WIDGET_TYPE_FOLDER_VIEW);
+	} else {
 
 		mail_op = 
 			modest_mail_operation_new_with_error_handling (G_OBJECT(parent_window),
@@ -3436,9 +3460,18 @@ on_rename_folder_performer (gboolean canceled,
 		modest_mail_operation_queue_add (modest_runtime_get_mail_operation_queue (),
 				mail_op);
 
-		/* Clear the headers view */
-		sel = gtk_tree_view_get_selection (GTK_TREE_VIEW (folder_view));
- 		gtk_tree_selection_unselect_all (sel);
+		if (MODEST_IS_MAIN_WINDOW(parent_window)) {
+
+			folder_view = modest_main_window_get_child_widget (
+				MODEST_MAIN_WINDOW (parent_window),
+				MODEST_MAIN_WINDOW_WIDGET_TYPE_FOLDER_VIEW);
+
+			/* Clear the headers view */
+			sel = gtk_tree_view_get_selection (GTK_TREE_VIEW (folder_view));
+			gtk_tree_selection_unselect_all (sel);
+		} else {
+			folder_view = NULL;
+		}
 
 		/* Actually rename the folder */
 		modest_mail_operation_rename_folder (mail_op,
@@ -3456,40 +3489,48 @@ on_rename_folder_performer (gboolean canceled,
 
 void 
 modest_ui_actions_on_rename_folder (GtkAction *action,
-				     ModestMainWindow *main_window)
+				     ModestWindow *window)
+{
+	modest_ui_actions_on_edit_mode_rename_folder (window);
+}
+
+gboolean 
+modest_ui_actions_on_edit_mode_rename_folder (ModestWindow *window)
 {
 	TnyFolderStore *folder;
 	GtkWidget *folder_view;
-	GtkWidget *header_view;	
+	gboolean do_rename = TRUE;
 
-	g_return_if_fail (MODEST_IS_MAIN_WINDOW(main_window));
+	g_return_val_if_fail (MODEST_IS_WINDOW(window), FALSE);
 
-	folder_view = modest_main_window_get_child_widget (main_window,
-							   MODEST_MAIN_WINDOW_WIDGET_TYPE_FOLDER_VIEW);
-	if (!folder_view)
-		return;
+	if (MODEST_IS_MAIN_WINDOW (window)) {
+		folder_view = modest_main_window_get_child_widget (MODEST_MAIN_WINDOW (window),
+								   MODEST_MAIN_WINDOW_WIDGET_TYPE_FOLDER_VIEW);
+		if (!folder_view)
+			return FALSE;
 
-	header_view = modest_main_window_get_child_widget (main_window,
-							   MODEST_MAIN_WINDOW_WIDGET_TYPE_HEADER_VIEW);
-	
-	if (!header_view)
-		return;
+#ifdef MODEST_TOOLKIT_HILDON2
+	} else if (MODEST_IS_FOLDER_WINDOW (window)) {
+		folder_view = GTK_WIDGET (modest_folder_window_get_folder_view (MODEST_FOLDER_WINDOW (window)));
+#endif
+	} else {
+		return FALSE;
+	}
 
 	folder = modest_folder_view_get_selected (MODEST_FOLDER_VIEW(folder_view));
 
 	if (!folder)
-		return;
+		return FALSE;
 
 	if (TNY_IS_FOLDER (folder)) {
 		gchar *folder_name = NULL;
 		gint response;
 		const gchar *current_name;
 		TnyFolderStore *parent;
-		gboolean do_rename = TRUE;
 
 		current_name = tny_folder_get_name (TNY_FOLDER (folder));
 		parent = tny_folder_get_folder_store (TNY_FOLDER (folder));
-		response = modest_platform_run_rename_folder_dialog (GTK_WINDOW (main_window), 
+		response = modest_platform_run_rename_folder_dialog (GTK_WINDOW (window),
 								     parent, current_name, 
 								     &folder_name);
 		g_object_unref (parent);
@@ -3500,11 +3541,12 @@ modest_ui_actions_on_rename_folder (GtkAction *action,
 			RenameFolderInfo *rename_folder_data = g_new0 (RenameFolderInfo, 1);
 			rename_folder_data->folder = g_object_ref (folder);
 			rename_folder_data->new_name = folder_name;
-			modest_platform_connect_if_remote_and_perform (GTK_WINDOW(main_window), TRUE,
+			modest_platform_connect_if_remote_and_perform (GTK_WINDOW(window), TRUE,
 					folder, on_rename_folder_performer, rename_folder_data);
 		}
 	}
 	g_object_unref (folder);
+	return do_rename;
 }
 
 static void
@@ -3550,6 +3592,10 @@ on_delete_folder_cb (gboolean canceled,
 	} else if (MODEST_IS_FOLDER_WINDOW (parent_window)) {
 		folder_view = GTK_WIDGET (modest_folder_window_get_folder_view (MODEST_FOLDER_WINDOW (parent_window)));
 #endif
+	} else {
+		g_object_unref (G_OBJECT (info->folder));
+		g_free (info);
+		return;
 	}
 
 	/* Unselect the folder before deleting it to free the headers */
@@ -3591,6 +3637,8 @@ delete_folder (ModestWindow *window, gboolean move_to_trash)
 	} else if (MODEST_IS_FOLDER_WINDOW (window)) {
 		folder_view = GTK_WIDGET (modest_folder_window_get_folder_view (MODEST_FOLDER_WINDOW (window)));
 #endif
+	} else {
+		return FALSE;
 	}
 	if (!folder_view)
 		return FALSE;
@@ -4760,7 +4808,7 @@ on_move_to_dialog_response (GtkDialog *dialog,
 		TnyFolderStore *dst_folder;
 
 	case MODEST_GTK_RESPONSE_NEW_FOLDER:
-		modest_ui_actions_create_folder (GTK_WIDGET (dialog), folder_view);
+		modest_ui_actions_create_folder (GTK_WIDGET (parent_win), folder_view);
 		return;
 	case GTK_RESPONSE_NONE:
 	case GTK_RESPONSE_CANCEL:
@@ -4774,6 +4822,13 @@ on_move_to_dialog_response (GtkDialog *dialog,
 								  folder_view,
 								  dst_folder,
 								  MODEST_MAIN_WINDOW (parent_win));
+#ifdef MODEST_TOOLKIT_HILDON2
+		} else if (MODEST_IS_FOLDER_WINDOW (parent_win)) {
+			modest_ui_actions_on_folder_window_move_to (folder_view,
+								    dst_folder,
+								    helper->list,
+								    GTK_WINDOW (parent_win));
+#endif
 		} else {
 			/* Moving from headers window in edit mode */
 			modest_ui_actions_on_window_move_to (NULL, helper->list,
@@ -4790,7 +4845,8 @@ on_move_to_dialog_response (GtkDialog *dialog,
 	}
 
 	/* Free the helper and exit */
-	g_object_unref (helper->list);
+	if (helper->list)
+		g_object_unref (helper->list);
 	g_slice_free (MoveToInfo, helper);
 	gtk_widget_destroy (GTK_WIDGET (dialog));
 }
@@ -4958,40 +5014,42 @@ move_to_cb (ModestMailOperation *mail_op,
 	    gpointer user_data)
 {
 	MoveToHelper *helper = (MoveToHelper *) user_data;
+	GObject *object = modest_mail_operation_get_source (mail_op);
 
 	/* Note that the operation could have failed, in that case do
 	   nothing */
-	if (modest_mail_operation_get_status (mail_op) == 
-	    MODEST_MAIL_OPERATION_STATUS_SUCCESS) {
+	if (modest_mail_operation_get_status (mail_op) != 
+	    MODEST_MAIL_OPERATION_STATUS_SUCCESS)
+		goto frees;
 
-		GObject *object = modest_mail_operation_get_source (mail_op);
-		if (MODEST_IS_MSG_VIEW_WINDOW (object)) {
-			ModestMsgViewWindow *self = MODEST_MSG_VIEW_WINDOW (object);
+	if (MODEST_IS_MSG_VIEW_WINDOW (object)) {
+		ModestMsgViewWindow *self = MODEST_MSG_VIEW_WINDOW (object);
 
-			if (!modest_msg_view_window_select_next_message (self) &&
-			    !modest_msg_view_window_select_previous_message (self)) {
-				/* No more messages to view, so close this window */
-				modest_ui_actions_on_close_window (NULL, MODEST_WINDOW(self));
-			}
-		} else if (MODEST_IS_MAIN_WINDOW (object) && 
-			   gtk_tree_row_reference_valid (helper->reference)) {
-			GtkWidget *header_view;
-			GtkTreePath *path;
-			GtkTreeSelection *sel;
-
-			header_view = modest_main_window_get_child_widget (MODEST_MAIN_WINDOW(object),
-									   MODEST_MAIN_WINDOW_WIDGET_TYPE_HEADER_VIEW);
-			sel = gtk_tree_view_get_selection (GTK_TREE_VIEW (header_view));
-			path = gtk_tree_row_reference_get_path (helper->reference);
-			/* We need to unselect the previous one
-			   because we could be copying instead of
-			   moving */
-			gtk_tree_selection_unselect_all (sel);
-			gtk_tree_selection_select_path (sel, path);
-			gtk_tree_path_free (path);
+		if (!modest_msg_view_window_select_next_message (self) &&
+		    !modest_msg_view_window_select_previous_message (self)) {
+			/* No more messages to view, so close this window */
+			modest_ui_actions_on_close_window (NULL, MODEST_WINDOW(self));
 		}
-		g_object_unref (object);
-        }
+	} else if (MODEST_IS_MAIN_WINDOW (object) && 
+		   gtk_tree_row_reference_valid (helper->reference)) {
+		GtkWidget *header_view;
+		GtkTreePath *path;
+		GtkTreeSelection *sel;
+
+		header_view = modest_main_window_get_child_widget (MODEST_MAIN_WINDOW(object),
+								   MODEST_MAIN_WINDOW_WIDGET_TYPE_HEADER_VIEW);
+		sel = gtk_tree_view_get_selection (GTK_TREE_VIEW (header_view));
+		path = gtk_tree_row_reference_get_path (helper->reference);
+		/* We need to unselect the previous one
+		   because we could be copying instead of
+		   moving */
+		gtk_tree_selection_unselect_all (sel);
+		gtk_tree_selection_select_path (sel, path);
+		gtk_tree_path_free (path);
+	}
+	g_object_unref (object);
+
+ frees:
 	/* Destroy the helper */
 	move_to_helper_destroyer (helper);
 }
@@ -5005,13 +5063,17 @@ folder_move_to_cb (ModestMailOperation *mail_op,
 	GObject *object;
 
 	object = modest_mail_operation_get_source (mail_op);
-	folder_view = modest_main_window_get_child_widget (MODEST_MAIN_WINDOW(object),
-							   MODEST_MAIN_WINDOW_WIDGET_TYPE_FOLDER_VIEW);
-	g_object_ref (folder_view);
-	g_object_unref (object);
-	move_to_cb (mail_op, user_data);
-	modest_folder_view_select_folder (MODEST_FOLDER_VIEW (folder_view), new_folder, FALSE);
-	g_object_unref (folder_view);
+	if (MODEST_IS_MAIN_WINDOW (object)) {
+		folder_view = modest_main_window_get_child_widget (MODEST_MAIN_WINDOW(object),
+								   MODEST_MAIN_WINDOW_WIDGET_TYPE_FOLDER_VIEW);
+		g_object_ref (folder_view);
+		g_object_unref (object);
+		move_to_cb (mail_op, user_data);
+		modest_folder_view_select_folder (MODEST_FOLDER_VIEW (folder_view), new_folder, FALSE);
+		g_object_unref (folder_view);
+	} else {
+		move_to_cb (mail_op, user_data);
+	}
 }
 
 static void
@@ -5280,20 +5342,12 @@ static void
 xfer_messages_error_handler (ModestMailOperation *mail_op, 
 			     gpointer user_data)
 {
-	ModestWindow *main_window = NULL;
-
-	/* Disable next automatic folder selection */
-	main_window = modest_window_mgr_get_main_window (modest_runtime_get_window_mgr (),
-							 FALSE); /* don't create */
-	if (main_window) {
-		GObject *win = modest_mail_operation_get_source (mail_op);
-		modest_platform_run_information_dialog ((GtkWindow *) win, 
-							_("mail_in_ui_folder_move_target_error"), 
-							FALSE);
-		if (win)
-			g_object_unref (win);
-	}
-	move_to_helper_destroyer (user_data);
+	GObject *win = modest_mail_operation_get_source (mail_op);
+	modest_platform_run_information_dialog ((GtkWindow *) win, 
+						_("mail_in_ui_folder_move_target_error"), 
+						FALSE);
+	if (win)
+		g_object_unref (win);
 }
 
 typedef struct {
@@ -5347,12 +5401,15 @@ xfer_messages_performer  (gboolean canceled,
 	}
 
 	movehelper = g_new0 (MoveToHelper, 1);
+
+#ifndef MODEST_TOOLKIT_HILDON2
 	movehelper->banner = modest_platform_animation_banner (GTK_WIDGET (win), NULL,
 							       _CS("ckct_nw_pasting"));
 	if (movehelper->banner != NULL)  {
 		g_object_ref (movehelper->banner);
 		gtk_widget_show (GTK_WIDGET (movehelper->banner));
 	}
+#endif
 
 	if (MODEST_IS_MAIN_WINDOW (win)) {
 		GtkWidget *header_view = 
@@ -5396,14 +5453,14 @@ on_move_folder_cb (gboolean canceled, GError *err, GtkWindow *parent_window,
 	MoveFolderInfo *info = (MoveFolderInfo*)user_data;
 	GtkTreeSelection *sel;
 	ModestMailOperation *mail_op = NULL;
-	
-	if (canceled || err || !MODEST_IS_MAIN_WINDOW (parent_window)) {
+
+	if (canceled || err || !MODEST_IS_WINDOW (parent_window)) {
 		g_object_unref (G_OBJECT (info->src_folder));
 		g_object_unref (G_OBJECT (info->dst_folder));
 		g_free (info);
 		return;
 	}
-	
+
 	MoveToHelper *helper = g_new0 (MoveToHelper, 1);
 	helper->banner = modest_platform_animation_banner (GTK_WIDGET (parent_window), NULL,
 			_CS("ckct_nw_pasting"));
@@ -5436,8 +5493,10 @@ on_move_folder_cb (gboolean canceled, GError *err, GtkWindow *parent_window,
 	/* 			modest_folder_view_select_folder (MODEST_FOLDER_VIEW(folder_view), */
 	/* 							  TNY_FOLDER (src_folder), TRUE); */
 
-	modest_folder_view_select_folder (MODEST_FOLDER_VIEW(info->folder_view),
-					  TNY_FOLDER (info->dst_folder), TRUE);
+	if (MODEST_IS_MAIN_WINDOW (parent_window)) {
+		modest_folder_view_select_folder (MODEST_FOLDER_VIEW(info->folder_view),
+						  TNY_FOLDER (info->dst_folder), TRUE);
+	}
 	modest_mail_operation_xfer_folder (mail_op,
 			TNY_FOLDER (info->src_folder),
 			info->dst_folder,
@@ -5533,6 +5592,62 @@ modest_ui_actions_on_main_window_move_to (GtkAction *action,
 	g_object_unref (src_folder);
 }
 
+#ifdef MODEST_TOOLKIT_HILDON2
+/*
+ * UI handler for the "Move to" action when invoked from the
+ * ModestFolderWindow
+ */
+static void 
+modest_ui_actions_on_folder_window_move_to (GtkWidget *folder_view,
+					    TnyFolderStore *dst_folder,
+					    TnyList *selection,
+					    GtkWindow *win)
+{
+	TnyFolderStore *src_folder = NULL;
+	TnyIterator *iterator;
+
+	if (tny_list_get_length (selection) != 1)
+		return;
+
+	iterator = tny_list_create_iterator (selection);
+	src_folder = TNY_FOLDER_STORE (tny_iterator_get_current (iterator));
+	g_object_unref (iterator);
+
+
+	gboolean do_xfer = TRUE;
+
+	/* Allow only to transfer folders to the local root folder */
+	if (TNY_IS_ACCOUNT (dst_folder) && 
+	    !MODEST_IS_TNY_LOCAL_FOLDERS_ACCOUNT (dst_folder) &&
+	    !modest_tny_account_is_memory_card_account (TNY_ACCOUNT (dst_folder))) {
+		do_xfer = FALSE;
+	} else if (!TNY_IS_FOLDER (src_folder)) {
+		g_warning ("%s: src_folder is not a TnyFolder.\n", __FUNCTION__);
+		do_xfer = FALSE;
+	}
+
+	if (do_xfer) {
+		MoveFolderInfo *info = g_new0 (MoveFolderInfo, 1);
+		DoubleConnectionInfo *connect_info = g_slice_new (DoubleConnectionInfo);
+		
+		info->src_folder = g_object_ref (src_folder);
+		info->dst_folder = g_object_ref (dst_folder);
+		info->delete_original = TRUE;
+		info->folder_view = folder_view;
+
+		connect_info->callback = on_move_folder_cb;
+		connect_info->dst_account = get_account_from_folder_store (TNY_FOLDER_STORE (dst_folder));
+		connect_info->data = info;
+
+		modest_platform_double_connect_and_perform(GTK_WINDOW (win), TRUE,
+							   TNY_FOLDER_STORE (src_folder), 
+							   connect_info);
+	}
+	/* Frees */
+	g_object_unref (src_folder);
+}
+#endif
+
 
 void
 modest_ui_actions_transfer_messages_helper (GtkWindow *win,
@@ -5617,11 +5732,17 @@ void
 modest_ui_actions_on_move_to (GtkAction *action, 
 			      ModestWindow *win)
 {
+	modest_ui_actions_on_edit_mode_move_to (win);
+}
+
+gboolean 
+modest_ui_actions_on_edit_mode_move_to (ModestWindow *win)
+{
 	GtkWidget *dialog = NULL, *folder_view = NULL;
 	ModestMainWindow *main_window;
 	MoveToInfo *helper = NULL;
 
-	g_return_if_fail (MODEST_IS_WINDOW (win));
+	g_return_val_if_fail (MODEST_IS_WINDOW (win), FALSE);
 
 	/* Get the main window if exists */
 	if (MODEST_IS_MAIN_WINDOW (win))
@@ -5654,6 +5775,8 @@ modest_ui_actions_on_move_to (GtkAction *action,
 
 	/* Show the dialog */
 	gtk_widget_show (dialog);
+
+	return TRUE;
 }
 
 /*
@@ -6025,12 +6148,19 @@ modest_ui_actions_on_open_addressbook (GtkAction *action, ModestWindow *win)
 
 
 void
-modest_ui_actions_on_toggle_find_in_page (GtkToggleAction *action,
+modest_ui_actions_on_toggle_find_in_page (GtkAction *action,
 					  ModestWindow *window)
 {
+	gboolean active;
 	g_return_if_fail (MODEST_IS_MSG_EDIT_WINDOW (window));
 
-	modest_msg_edit_window_toggle_find_toolbar (MODEST_MSG_EDIT_WINDOW (window), gtk_toggle_action_get_active (action));
+	if (GTK_IS_TOGGLE_ACTION (action))
+		active = gtk_toggle_action_get_active (GTK_TOGGLE_ACTION (action));
+	else
+		active = TRUE;
+
+	modest_msg_edit_window_toggle_find_toolbar (MODEST_MSG_EDIT_WINDOW (window), 
+						    active);
 }
 
 static void