X-Git-Url: http://git.maemo.org/git/?a=blobdiff_plain;f=src%2Fmodest-ui-actions.c;h=f3fb660b73895decbb5aef94c2cb66897afe2551;hb=f962dc811a8e9d38fd6cff25cbedacf1c3b0ab39;hp=26effc7456e0cfa35776285227d31b80017389c5;hpb=bd99026f08786c62dd5dac92575f34ca1756cd89;p=modest

diff --git a/src/modest-ui-actions.c b/src/modest-ui-actions.c
index 26effc7..f3fb660 100644
--- a/src/modest-ui-actions.c
+++ b/src/modest-ui-actions.c
@@ -700,12 +700,8 @@ modest_ui_actions_on_add_to_contacts (GtkAction *action, ModestWindow *win)
 	clipboard = gtk_clipboard_get (GDK_SELECTION_PRIMARY);
 	selection = gtk_clipboard_wait_for_text (clipboard);
 
-	/* Question: why is the clipboard being used here?
-	 * It doesn't really make a lot of sense. */
-
-	if (selection)
-	{
-		modest_address_book_add_address (selection);
+	if (selection) {
+		modest_address_book_add_address (selection, (GtkWindow *) win);
 		g_free (selection);
 	}
 }
@@ -994,19 +990,27 @@ modest_ui_actions_msg_retrieval_check (ModestMailOperation *mail_op,
 			      error->code == TNY_SERVICE_ERROR_MESSAGE_NOT_AVAILABLE)) {
 			gchar *subject, *msg, *format = NULL;
 			TnyAccount *account;
-			
-			subject = header?tny_header_dup_subject (header):NULL;
+
+			subject = (header) ? tny_header_dup_subject (header) : NULL;
 			if (!subject)
 				subject = g_strdup (_("mail_va_no_subject"));
 
 			account = modest_mail_operation_get_account (mail_op);
 			if (account) {
-				ModestProtocol *protocol;
-				ModestProtocolType proto;
-				proto = modest_tny_account_get_protocol_type (account);
-				protocol = modest_protocol_registry_get_protocol_by_type (modest_runtime_get_protocol_registry (), proto);
-				if (protocol)
-				  format = modest_protocol_get_translation (protocol, MODEST_PROTOCOL_TRANSLATION_MSG_NOT_AVAILABLE, subject);
+				ModestProtocolType proto = modest_tny_account_get_protocol_type (account);
+				ModestProtocol *protocol = modest_protocol_registry_get_protocol_by_type (modest_runtime_get_protocol_registry (), proto);
+
+				if (protocol) {
+					if (tny_account_get_connection_status (account) ==
+					    TNY_CONNECTION_STATUS_CONNECTED) {
+						format = modest_protocol_get_translation (protocol,
+											  MODEST_PROTOCOL_TRANSLATION_MSG_NOT_AVAILABLE,
+											  subject);
+					} else {
+						format = g_strdup_printf (_("mail_ib_backend_server_invalid"),
+									  tny_account_get_hostname (account));
+					}
+				}
 				g_object_unref (account);
 			}
 
@@ -2179,7 +2183,12 @@ new_messages_arrived (ModestMailOperation *self,
 			flags = tny_header_get_flags (header);
 
 			if (!(flags & TNY_HEADER_FLAG_SEEN)) {
-				tny_list_append (actually_new_list, G_OBJECT (header));
+				/* Messages are ordered from most
+				   recent to oldest. But we want to
+				   show notifications starting from
+				   the oldest message. That's why we
+				   reverse the list */
+				tny_list_prepend (actually_new_list, G_OBJECT (header));
 			}
 			g_object_unref (header);
 		}
@@ -4816,6 +4825,16 @@ modest_ui_actions_on_details (GtkAction *action,
 }
 
 void
+modest_ui_actions_on_limit_error (GtkAction *action,
+				  ModestWindow *win)
+{
+	g_return_if_fail (MODEST_IS_MSG_VIEW_WINDOW (win));
+
+	modest_platform_information_banner ((GtkWidget *) win, NULL, _CS("ckdg_ib_maximum_characters_reached"));
+
+}
+
+void
 modest_ui_actions_on_toggle_show_cc (GtkToggleAction *toggle,
 				     ModestMsgEditWindow *window)
 {
@@ -5664,20 +5683,20 @@ xfer_messages_error_handler (ModestMailOperation *mail_op,
 
 	win = modest_mail_operation_get_source (mail_op);
 	error = modest_mail_operation_get_error (mail_op);
-	account = modest_mail_operation_get_account (mail_op);
 
-	if (error && modest_tny_account_store_is_disk_full_error (modest_runtime_get_account_store(),
-								  (GError *) error, account)) {
-		gchar *msg = g_strdup_printf (_KR("cerm_device_memory_full"), "");
-		modest_platform_information_banner ((GtkWidget *) win, NULL, msg);
-		g_free (msg);
-	} else {
+	/* We cannot get the account from the mail op as that is the
+	   source account and for checking memory full conditions we
+	   need the destination one */
+	account = TNY_ACCOUNT (user_data);
+
+	if (error &&
+	    !modest_tny_account_store_check_disk_full_error (modest_runtime_get_account_store(),
+							     (GtkWidget *) win, (GError*) error,
+							     account, _KR("cerm_memory_card_full"))) {
 		modest_platform_run_information_dialog ((GtkWindow *) win,
 							_("mail_in_ui_folder_move_target_error"),
 							FALSE);
 	}
-	if (account)
-		g_object_unref (account);
 	if (win)
 		g_object_unref (win);
 }
@@ -5722,8 +5741,7 @@ xfer_messages_performer  (gboolean canceled,
 	/* tinymail will return NULL for local folders it seems */
 	dst_forbids_message_add = modest_protocol_registry_protocol_type_has_tag (modest_runtime_get_protocol_registry (),
 										  modest_tny_account_get_protocol_type (dst_account),
-										  MODEST_PROTOCOL_REGISTRY_STORE_FORBID_MESSAGE_ADD);
-	g_object_unref (dst_account);
+										  MODEST_PROTOCOL_REGISTRY_STORE_FORBID_INCOMING_XFERS);
 
 	if (dst_forbids_message_add) {
 		modest_platform_information_banner (GTK_WIDGET (win),
@@ -5755,7 +5773,8 @@ xfer_messages_performer  (gboolean canceled,
 	/* Perform the mail operation */
 	mail_op = modest_mail_operation_new_with_error_handling (G_OBJECT(win),
 								 xfer_messages_error_handler,
-								 movehelper, NULL);
+								 g_object_ref (dst_account),
+								 g_object_unref);
 	modest_mail_operation_queue_add (modest_runtime_get_mail_operation_queue (),
 					 mail_op);
 
@@ -5768,6 +5787,8 @@ xfer_messages_performer  (gboolean canceled,
 
 	g_object_unref (G_OBJECT (mail_op));
  end:
+	if (dst_account)
+		g_object_unref (dst_account);
 	g_object_unref (helper->dst_folder);
 	g_object_unref (helper->headers);
 	g_slice_free (XferMsgsHelper, helper);
@@ -6864,8 +6885,15 @@ modest_ui_actions_on_delete_account (GtkWindow *parent_window,
 		g_free (default_account_name);
 
 		removed = modest_account_mgr_remove_account (account_mgr, account_name);
-		if (!removed)
+		if (removed) {
+			/* Close all email notifications, we cannot
+			   distinguish if the notification belongs to
+			   this account or not, so for safety reasons
+			   we remove them all */
+			modest_platform_remove_new_mail_notifications (FALSE);
+		} else {
 			g_warning ("%s: modest_account_mgr_remove_account() failed.\n", __FUNCTION__);
+		}
 	}
 	return removed;
 }
@@ -6917,3 +6945,89 @@ modest_ui_actions_on_reload_message (const gchar *msg_id)
 
 	modest_msg_view_window_reload (MODEST_MSG_VIEW_WINDOW (window));
 }
+
+/** Check whether any connections are active, and cancel them if 
+ * the user wishes.
+ * Returns TRUE is there was no problem, 
+ * or if an operation was cancelled so we can continue.
+ * Returns FALSE if the user chose to cancel his request instead.
+ */
+
+gboolean
+modest_ui_actions_check_for_active_account (ModestWindow *self,
+					    const gchar* account_name)
+{
+	ModestTnySendQueue *send_queue;
+	ModestTnyAccountStore *acc_store;
+	ModestMailOperationQueue* queue;
+	TnyConnectionStatus store_conn_status;
+	TnyAccount *store_account = NULL, *transport_account = NULL;
+	gboolean retval = TRUE, sending = FALSE;
+
+	acc_store = modest_runtime_get_account_store ();
+	queue = modest_runtime_get_mail_operation_queue ();
+
+	store_account = 
+		modest_tny_account_store_get_server_account (acc_store,
+							     account_name,
+							     TNY_ACCOUNT_TYPE_STORE);
+
+	/* This could happen if the account was deleted before the
+	   call to this function */
+	if (!store_account)
+		return FALSE;
+
+	transport_account = 
+		modest_tny_account_store_get_server_account (acc_store,
+							     account_name,
+							     TNY_ACCOUNT_TYPE_TRANSPORT);
+
+	/* This could happen if the account was deleted before the
+	   call to this function */
+	if (!transport_account) {
+		g_object_unref (store_account);
+		return FALSE;
+	}
+
+	/* If the transport account was not used yet, then the send
+	   queue could not exist (it's created on demand) */
+	send_queue = modest_runtime_get_send_queue (TNY_TRANSPORT_ACCOUNT (transport_account), FALSE);
+	if (TNY_IS_SEND_QUEUE (send_queue))
+		sending = modest_tny_send_queue_sending_in_progress (send_queue);
+
+	store_conn_status = tny_account_get_connection_status (store_account);
+	if (store_conn_status == TNY_CONNECTION_STATUS_CONNECTED || sending) {
+		gint response;
+
+		response = modest_platform_run_confirmation_dialog (GTK_WINDOW (self), 
+								_("emev_nc_disconnect_account"));
+		if (response == GTK_RESPONSE_OK) {
+			retval = TRUE;
+		} else {
+			retval = FALSE;
+		}
+	}
+
+	if (retval) {
+
+		/* FIXME: We should only cancel those of this account */
+		modest_mail_operation_queue_cancel_all (queue);
+
+		/* Also disconnect the account */
+		if ((tny_account_get_connection_status (store_account) != TNY_CONNECTION_STATUS_DISCONNECTED) &&
+		    (tny_account_get_connection_status (store_account) != TNY_CONNECTION_STATUS_DISCONNECTED_BROKEN)) {
+			tny_camel_account_set_online (TNY_CAMEL_ACCOUNT (store_account),
+						      FALSE, NULL, NULL);
+		}
+		if (sending) {
+			tny_camel_account_set_online (TNY_CAMEL_ACCOUNT (transport_account),
+						      FALSE, NULL, NULL);
+		}
+	}
+		
+	/* Frees */
+	g_object_unref (store_account);
+	g_object_unref (transport_account);
+	
+	return retval;
+}