X-Git-Url: http://git.maemo.org/git/?p=modest;a=blobdiff_plain;f=src%2Fmodest-ui-actions.c;h=98d57c6682ef06b8fe1d0a46206f136559386087;hp=3b63ab39fc813b00da4fc2a57ee963abd2db0cb5;hb=de8ac9aa1f0d03dffda84e55ad31c094923ad70f;hpb=30e15b4d88be32b5633e900d048cf13415a22a24;ds=sidebyside

diff --git a/src/modest-ui-actions.c b/src/modest-ui-actions.c
index 3b63ab3..98d57c6 100644
--- a/src/modest-ui-actions.c
+++ b/src/modest-ui-actions.c
@@ -122,8 +122,6 @@ static void     reply_forward_cb       (ModestMailOperation *mail_op,
 
 static void     reply_forward          (ReplyForwardAction action, ModestWindow *win);
 
-static gchar*   ask_for_folder_name    (GtkWindow *parent_window, const gchar *title);
-
 
 static void     _on_send_receive_progress_changed (ModestMailOperation  *mail_op, 
 						   ModestMailOperationState *state,
@@ -272,7 +270,7 @@ modest_ui_actions_on_delete (GtkAction *action, ModestWindow *win)
 	if (MODEST_IS_MAIN_WINDOW (win)) {
 		header_view = modest_main_window_get_child_widget (MODEST_MAIN_WINDOW (win),
 								   MODEST_WIDGET_TYPE_HEADER_VIEW);
-		if (!GTK_WIDGET_HAS_FOCUS (header_view))
+		if (!gtk_widget_is_focus (header_view))
 			return;
 	}
 
@@ -307,16 +305,15 @@ modest_ui_actions_on_delete (GtkAction *action, ModestWindow *win)
 	}
 
 	/* Select message */
-	if (tny_list_get_length(header_list) > 1)
-		message = g_strdup(_("emev_nc_delete_messages"));
-	else {
+	if (tny_list_get_length(header_list) == 1) {
 		iter = tny_list_create_iterator (header_list);
 		header = TNY_HEADER (tny_iterator_get_current (iter));
 		desc = g_strdup_printf ("%s", tny_header_get_subject (header)); 
-		message = g_strdup_printf(_("emev_nc_delete_message"), desc);
 		g_object_unref (header);
 		g_object_unref (iter);
 	}
+	message = g_strdup_printf(ngettext("emev_nc_delete_message", "emev_nc_delete_messages", 
+					   tny_list_get_length(header_list)), desc);
 
 	/* Confirmation dialog */		
 	response = modest_platform_run_confirmation_dialog (GTK_WINDOW (win),
@@ -348,10 +345,11 @@ modest_ui_actions_on_delete (GtkAction *action, ModestWindow *win)
 void
 modest_ui_actions_on_quit (GtkAction *action, ModestWindow *win)
 {
-	#ifdef MODEST_PLATFORM_MAEMO
+#ifdef MODEST_PLATFORM_MAEMO
 	modest_osso_save_state();
-	#endif /* MODEST_PLATFORM_MAEMO */
-	
+#endif /* MODEST_PLATFORM_MAEMO */
+
+	g_message ("quiting...");	
 	gtk_main_quit ();
 }
 
@@ -475,9 +473,9 @@ modest_ui_actions_on_smtp_servers (GtkAction *action, ModestWindow *win)
 	/* Show the window: */	
 	gtk_window_set_transient_for (GTK_WINDOW (specific_window), GTK_WINDOW (win));
 	gtk_window_set_modal (GTK_WINDOW (specific_window), TRUE);
-    gtk_widget_show (specific_window);
+    	gtk_widget_show (specific_window);
     
-    /* Save changes when the window is hidden: */
+    	/* Save changes when the window is hidden: */
 	g_signal_connect (specific_window, "hide", 
 		G_CALLBACK (on_smtp_servers_window_hide), win);
 #endif /* MODEST_PLATFORM_MAEMO */
@@ -659,6 +657,7 @@ open_msg_cb (ModestMailOperation *mail_op,
 cleanup:
 	/* Free */
 	g_free(account);
+	g_object_unref (parent_win);
 	g_object_unref (msg);
 	g_object_unref (folder);
 	g_object_unref (header);
@@ -680,6 +679,7 @@ modest_ui_actions_get_msgs_full_error_handler (ModestMailOperation *mail_op,
 
 		modest_platform_run_information_dialog ((win) ? GTK_WINDOW (win) : NULL,
 							error->message);
+		g_object_unref (win);
 	}
 }
 
@@ -725,11 +725,20 @@ _modest_ui_actions_open (TnyList *headers, ModestWindow *win)
 								 modest_ui_actions_get_msgs_full_error_handler, 
 								 NULL);
 	modest_mail_operation_queue_add (modest_runtime_get_mail_operation_queue (), mail_op);
-	modest_mail_operation_get_msgs_full (mail_op, 
-					     not_opened_headers, 
-					     open_msg_cb, 
-					     NULL, 
-					     NULL);
+	if (tny_list_get_length (not_opened_headers) > 1) {
+		modest_mail_operation_get_msgs_full (mail_op, 
+						     not_opened_headers, 
+						     open_msg_cb, 
+						     NULL, 
+						     NULL);
+	} else {
+		TnyIterator *iter = tny_list_create_iterator (not_opened_headers);
+		TnyHeader *header = TNY_HEADER (tny_iterator_get_current (iter));
+		modest_mail_operation_get_msg (mail_op, header, open_msg_cb, NULL);
+		g_object_unref (header);
+		g_object_unref (iter);
+	}
+
 	/* Clean */
 	g_object_unref (not_opened_headers);
 	g_object_unref (iter);
@@ -1071,6 +1080,17 @@ modest_ui_actions_on_sort (GtkAction *action,
 	}
 }
 
+static void
+new_messages_arrived (ModestMailOperation *self, 
+		      gint new_messages,
+		      gpointer user_data)
+{
+	if (new_messages == 0)
+		return;
+
+	modest_platform_on_new_msg ();
+}
+
 /*
  * This function performs the send & receive required actions. The
  * window is used to create the mail operation. Typically it should
@@ -1113,7 +1133,7 @@ modest_ui_actions_do_send_receive (const gchar *account_name, ModestWindow *win)
 	   internally, so the progress objects will receive the proper
 	   progress information */
 	modest_mail_operation_queue_add (modest_runtime_get_mail_operation_queue (), mail_op);
-	modest_mail_operation_update_account (mail_op, acc_name);
+	modest_mail_operation_update_account (mail_op, acc_name, new_messages_arrived, NULL);
 	g_object_unref (G_OBJECT (mail_op));
 	
 	/* Free */
@@ -1207,7 +1227,7 @@ modest_ui_actions_on_header_selected (ModestHeaderView *header_view,
 		return;
 	
 	/* Update Main window title */
-	if (GTK_WIDGET_HAS_FOCUS (header_view)) {
+	if (gtk_widget_is_focus (GTK_WIDGET(header_view))) {
 		const gchar *subject = tny_header_get_subject (header);
 		if (subject && strlen(subject) > 0)
 			gtk_window_set_title (GTK_WINDOW (main_window), subject);
@@ -1265,7 +1285,6 @@ folder_refreshed_cb (const GObject *obj,
 		     TnyFolder *folder, 
 		     gpointer user_data)
 {
-/* 	printf ("DEBUG: %s\n", __FUNCTION__); */
 	ModestMainWindow *win = NULL;
 	GtkWidget *header_view;
 
@@ -1277,24 +1296,11 @@ folder_refreshed_cb (const GObject *obj,
 
 	/* Check if folder is empty and set headers view contents style */
 	if (tny_folder_get_all_count (folder) == 0) {
-/* 	printf ("DEBUG: %s: tny_folder_get_all_count() returned 0.\n", __FUNCTION__); */
+	printf ("DEBUG: %s: tny_folder_get_all_count() returned 0.\n", __FUNCTION__);
 		modest_main_window_set_contents_style (win,
 						       MODEST_MAIN_WINDOW_CONTENTS_STYLE_EMPTY);
 	} else {
-/* 		printf ("DEBUG: %s: tny_folder_get_all_count() returned >0.\n", __FUNCTION__); */
-
-		/* Set the header view, we could change it to
-		   the empty view after the refresh. We do not
-		   need to save the configuration because we
-		   have already done that when unselecting the
-		   folder */
-		modest_main_window_set_contents_style (win, 
-						       MODEST_MAIN_WINDOW_CONTENTS_STYLE_HEADERS);
-
-		/* Restore configuration */
-		modest_widget_memory_restore (modest_runtime_get_conf (), 
-					      G_OBJECT(header_view),
-					      MODEST_CONF_HEADER_VIEW_KEY);
+		printf ("DEBUG: %s: tny_folder_get_all_count() returned >0.\n", __FUNCTION__);
 	}
 }
 
@@ -1326,13 +1332,20 @@ modest_ui_actions_on_folder_selection_changed (ModestFolderView *folder_view,
 	} else {
 		if (TNY_IS_FOLDER (folder_store) && selected) {
 			
-			if (!TNY_IS_MERGE_FOLDER (folder_store)) { /* TnyMergeFolder can have no get_account() implementation. */
-				/* Update the active account */
-				TnyAccount *account = modest_tny_folder_get_account (TNY_FOLDER (folder_store));
+			/* Update the active account */
+			TnyAccount *account = modest_tny_folder_get_account (TNY_FOLDER (folder_store));
+			if (account) {
 				set_active_account_from_tny_account (account, MODEST_WINDOW (main_window));
 				g_object_unref (account);
+				account = NULL;
 			}
 
+			/* Set the header style by default, it could
+			   be changed later by the refresh callback to
+			   empty */
+			modest_main_window_set_contents_style (main_window, 
+							       MODEST_MAIN_WINDOW_CONTENTS_STYLE_HEADERS);
+
 			/* Set folder on header view. This function
 			   will call tny_folder_refresh_async so we
 			   pass a callback that will be called when
@@ -1342,11 +1355,22 @@ modest_ui_actions_on_folder_selection_changed (ModestFolderView *folder_view,
 						       TNY_FOLDER (folder_store),
 						       folder_refreshed_cb,
 						       main_window);
+			
+			/* Restore configuration. We need to do this
+			   *after* the set_folder because the widget
+			   memory asks the header view about its
+			   folder  */
+			modest_widget_memory_restore (modest_runtime_get_conf (), 
+						      G_OBJECT(header_view),
+						      MODEST_CONF_HEADER_VIEW_KEY);
 		} else {
 			/* Update the active account */
 			modest_window_set_active_account (MODEST_WINDOW (main_window), NULL);
-			/* Do not show folder */
-			modest_widget_memory_save (conf, G_OBJECT (header_view), MODEST_CONF_HEADER_VIEW_KEY);
+			/* Save only if we're seeing headers */
+			if (modest_main_window_get_contents_style (main_window) ==
+			    MODEST_MAIN_WINDOW_CONTENTS_STYLE_HEADERS)
+				modest_widget_memory_save (conf, G_OBJECT (header_view), 
+							   MODEST_CONF_HEADER_VIEW_KEY);
 			modest_header_view_clear (MODEST_HEADER_VIEW(header_view));
 		}
 	}
@@ -1449,6 +1473,7 @@ modest_ui_actions_on_save_to_drafts (GtkWidget *widget, ModestMsgEditWindow *edi
 	MsgData *data;
 	gchar *account_name, *from;
 	ModestAccountMgr *account_mgr;
+	gchar *info_text = NULL;
 
 	g_return_if_fail (MODEST_IS_MSG_EDIT_WINDOW(edit_window));
 	
@@ -1505,6 +1530,10 @@ modest_ui_actions_on_save_to_drafts (GtkWidget *widget, ModestMsgEditWindow *edi
 
 	modest_msg_edit_window_free_msg_data (edit_window, data);
 
+	info_text = g_strdup_printf (_("mail_va_saved_to_drafts"), _("mcen_me_folder_drafts"));
+	modest_platform_information_banner (NULL, NULL, info_text);
+	g_free (info_text);
+
 	/* Save settings and close the window */
 	gtk_widget_destroy (GTK_WIDGET (edit_window));
 }
@@ -1578,6 +1607,8 @@ modest_ui_actions_on_send (GtkWidget *widget, ModestMsgEditWindow *edit_window)
 		}
 	}
 
+	modest_platform_information_banner (NULL, NULL, _("mcen_ib_outbox_waiting_to_be_sent"));
+
 	/* Create the mail operation */
 	ModestMailOperation *mail_operation = modest_mail_operation_new (MODEST_MAIL_OPERATION_TYPE_SEND, G_OBJECT(edit_window));
 	modest_mail_operation_queue_add (modest_runtime_get_mail_operation_queue (), mail_operation);
@@ -1752,46 +1783,6 @@ modest_ui_actions_on_remove_attachments (GtkAction *action,
 	modest_msg_edit_window_remove_attachments (window, NULL);
 }
 
-/*
- * Shows a dialog with an entry that asks for some text. The returned
- * value must be freed by the caller. The dialog window title will be
- * set to @title.
- */
-static gchar *
-ask_for_folder_name (GtkWindow *parent_window,
-		     const gchar *title)
-{
-	GtkWidget *dialog, *entry;
-	gchar *folder_name = NULL;
-
-	/* Ask for folder name */
-	dialog = gtk_dialog_new_with_buttons (_("New Folder Name"),
-					      parent_window,
-					      GTK_DIALOG_MODAL,
-					      GTK_STOCK_CANCEL,
-					      GTK_RESPONSE_REJECT,
-					      GTK_STOCK_OK,
-					      GTK_RESPONSE_ACCEPT,
-					      NULL);
-	gtk_box_pack_start (GTK_BOX(GTK_DIALOG(dialog)->vbox), 
-			    gtk_label_new(title),
-			    FALSE, FALSE, 0);
-		
-	entry = gtk_entry_new_with_max_length (40);
-	gtk_box_pack_start (GTK_BOX(GTK_DIALOG(dialog)->vbox), 
-			    entry,
-			    TRUE, FALSE, 0);	
-	
-	gtk_widget_show_all (GTK_WIDGET(GTK_DIALOG(dialog)->vbox));
-	
-	if (gtk_dialog_run (GTK_DIALOG(dialog)) == GTK_RESPONSE_ACCEPT)		
-		folder_name = g_strdup (gtk_entry_get_text (GTK_ENTRY (entry)));
-
-	gtk_widget_destroy (dialog);
-
-	return folder_name;
-}
-
 void 
 modest_ui_actions_on_new_folder (GtkAction *action, ModestMainWindow *main_window)
 {
@@ -1871,10 +1862,14 @@ modest_ui_actions_on_rename_folder (GtkAction *action,
 	
 	if (folder && TNY_IS_FOLDER (folder)) {
 		gchar *folder_name;
-		folder_name = ask_for_folder_name (GTK_WINDOW (main_window),
-						   _("Please enter a new name for the folder"));
+		gint response;
+		const gchar *current_name;
+
+		current_name = tny_folder_get_name (TNY_FOLDER (folder));
+		response = modest_platform_run_rename_folder_dialog (GTK_WINDOW (main_window), NULL,
+								     current_name, &folder_name);
 
-		if (folder_name != NULL && strlen (folder_name) > 0) {
+		if (response == GTK_RESPONSE_OK && strlen (folder_name) > 0) {
 			ModestMailOperation *mail_op;
 
 			mail_op = modest_mail_operation_new (MODEST_MAIL_OPERATION_TYPE_INFO, G_OBJECT(main_window));
@@ -1902,6 +1897,7 @@ modest_ui_actions_delete_folder_error_handler (ModestMailOperation *mail_op,
 
 	modest_platform_run_information_dialog ((win) ? GTK_WINDOW (win) : NULL,
 						_("mail_in_ui_folder_delete_error"));
+	g_object_unref (win);
 }
 
 static void
@@ -2104,12 +2100,8 @@ modest_ui_actions_on_password_requested (TnyAccountStore *account_store,
 			const gboolean username_was_changed = 
 				(strcmp (*username, initial_username) != 0);
 			if (username_was_changed) {
-				/* To actually use a changed username, 
-				 * we must reset the connection, according to pvanhoof.
-				 * This _might_ be a sensible way to do that: */
-				 TnyDevice *device = modest_runtime_get_device();
-				 tny_device_force_offline (device);
-				 tny_device_force_online (device);
+				g_warning ("%s: tinymail does not yet support changing the "
+					"username in the get_password() callback.\n", __FUNCTION__);
 			}
 		}
 			
@@ -2333,6 +2325,10 @@ modest_ui_actions_on_select_all (GtkAction *action,
 		/* Select all messages */
 		selection = gtk_tree_view_get_selection (GTK_TREE_VIEW(header_view));
 		gtk_tree_selection_select_all (selection);
+
+		/* Set focuse on header view */
+		gtk_widget_grab_focus (header_view);
+
 	} else if (GTK_IS_HTML (focused_widget)) {
 		gtk_html_select_all (GTK_HTML (focused_widget));
 	}
@@ -2527,16 +2523,20 @@ modest_ui_actions_on_details (GtkAction *action,
 		folder_view = modest_main_window_get_child_widget (MODEST_MAIN_WINDOW (win),
 								    MODEST_WIDGET_TYPE_FOLDER_VIEW);
 		if (gtk_widget_is_focus (folder_view)) {
-			TnyFolder *folder;
-
-			folder = (TnyFolder *) modest_folder_view_get_selected (MODEST_FOLDER_VIEW (folder_view));
-
+			TnyFolderStore *folder_store
+				= modest_folder_view_get_selected (MODEST_FOLDER_VIEW (folder_view));
+			if (!folder_store) {
+				g_warning ("%s: No item was selected.\n", __FUNCTION__);
+				return;	
+			}
 			/* Show only when it's a folder */
-			if (!folder || !TNY_IS_FOLDER (folder))
-				return;
+			/* This function should not be called for account items, 
+			 * because we dim the menu item for them. */
+			if (TNY_IS_FOLDER (folder_store)) {
+				show_folder_details (TNY_FOLDER (folder_store), GTK_WINDOW (win));
+			}
 
-			show_folder_details (folder, GTK_WINDOW (win));
-			g_object_unref (folder);
+			g_object_unref (folder_store);
 
 		} else {
 			header_view = modest_main_window_get_child_widget (MODEST_MAIN_WINDOW (win),
@@ -2749,17 +2749,13 @@ msgs_move_to_confirmation (GtkWindow *win,
 		if (!modest_tny_folder_is_local_folder (src_folder)) {
 			const gchar *message;
 			
-			if (tny_list_get_length (headers) == 1)
-				if (has_retrieved_msgs (headers))
-					message = _("mcen_nc_move_retrieve");
-				else
-					message = _("mcen_nc_move_header");
-			else
-				if (has_retrieved_msgs (headers))
-					message = _("mcen_nc_move_retrieves");
-				else
-					message = _("mcen_nc_move_headers");
-			
+			if (has_retrieved_msgs (headers))
+				message = ngettext ("mcen_nc_move_retrieve", "mcen_nc_move_retrieves",
+						    tny_list_get_length (headers));
+			else 
+				message = ngettext ("mcen_nc_move_header", "mcen_nc_move_headers",
+						    tny_list_get_length (headers));
+
 			response = modest_platform_run_confirmation_dialog (GTK_WINDOW (win),
 									    (const gchar *) message);
 		}
@@ -2792,6 +2788,7 @@ modest_ui_actions_move_folder_error_handler (ModestMailOperation *mail_op,
 	/* TODO: show error message */
 	modest_platform_run_information_dialog ((win) ? GTK_WINDOW (win) : NULL,
 						_("mail_in_ui_folder_move_target_error"));
+	g_object_unref (win);
 }
 
 /*
@@ -3050,7 +3047,7 @@ modest_ui_actions_remove_attachments (GtkAction *action,
 				      ModestWindow *window)
 {
 	if (MODEST_IS_MSG_VIEW_WINDOW (window)) {
-		modest_msg_view_window_remove_attachments (MODEST_MSG_VIEW_WINDOW (window), NULL);
+		modest_msg_view_window_remove_attachments (MODEST_MSG_VIEW_WINDOW (window));
 	} else {
 		/* not supported window for this action */
 		g_return_if_reached ();