A service needs to be disconnected if it's remote, not already offline,

	* mail-offline-handler.c (service_is_relevant): A service needs to
	be disconnected if it's remote, not already offline, and either
	connected OR connecting.
	(storage_go_offline): Don't put non-relevant stores offline. (Eg,
	don't force an IMAP store which you hadn't already connected to to
	connect and sync.)
	(storage_go_online): Likewise, don't "reconnect" stores that
	weren't connected before.

	* mail-ops.c (set_offline_desc): Fix the message to say
	"reconnecting" instead of "disconnecting" when appropriate.
	(mail_store_set_offline): If offline is TRUE, call
	camel_service_cancel_connect on the store. (We do this here
	because we don't want the cancel_connect request to get queued up
	behind a hanging connection attempt.)

svn path=/trunk/; revision=13192

1 files changed, 32 insertions, 24 deletions

diff --git a/mail/mail-ops.c b/mail/mail-ops.c
index 9d11277e8a..8867ecb140 100644
--- a/mail/mail-ops.c
+++ b/mail/mail-ops.c
@@ -2061,7 +2061,8 @@ static char *set_offline_desc(struct _mail_msg *mm, int done)
 	char *service_name = camel_service_get_name (CAMEL_SERVICE (m->store), TRUE);
 	char *msg;
 
-	msg = g_strdup_printf (_("Disconnecting from %s"), service_name);
+	msg = g_strdup_printf (m->offline ? _("Disconnecting from %s") :
+			       _("Reconnecting to %s"), service_name);
 	g_free (service_name);
 	return msg;
 }
@@ -2070,34 +2071,35 @@ static void set_offline_do(struct _mail_msg *mm)
 {
 	struct _set_offline_msg *m = (struct _set_offline_msg *)mm;
 
-	if (CAMEL_IS_DISCO_STORE (m->store) &&
-	    camel_disco_store_can_work_offline (CAMEL_DISCO_STORE (m->store))) {
-		if (m->offline) {
-			CamelFolder *inbox;
-
-			/* FIXME. Something more generic here... */
-			inbox = camel_store_get_inbox (m->store, NULL);
-			if (inbox) {
-				camel_disco_folder_prepare_for_offline (
-					CAMEL_DISCO_FOLDER (inbox),
-					"(match-all (not (system-flag \"Seen\")))",
-					&mm->ex);
-				camel_folder_sync (inbox, FALSE, NULL);
-				camel_object_unref (CAMEL_OBJECT (inbox));
-				if (camel_exception_is_set (&mm->ex))
-					return;
-			}
-		}
-
-		camel_disco_store_set_status (CAMEL_DISCO_STORE (m->store),
-					      m->offline ? CAMEL_DISCO_STORE_OFFLINE :
-					      CAMEL_DISCO_STORE_ONLINE, &mm->ex);
-	} else {
+	if (!CAMEL_IS_DISCO_STORE (m->store) ||
+	    !camel_disco_store_can_work_offline (CAMEL_DISCO_STORE (m->store))) {
 		if (m->offline) {
 			camel_service_disconnect (CAMEL_SERVICE (m->store),
 						  TRUE, &mm->ex);
 		}
+		return;
 	}
+
+	if (m->offline && camel_disco_store_status (CAMEL_DISCO_STORE (m->store)) == CAMEL_DISCO_STORE_ONLINE) {
+		CamelFolder *inbox;
+
+		/* FIXME. Something more generic here... (bug 10755) */
+		inbox = camel_store_get_inbox (m->store, NULL);
+		if (inbox) {
+			camel_disco_folder_prepare_for_offline (
+				CAMEL_DISCO_FOLDER (inbox),
+				"(match-all (not (system-flag \"Seen\")))",
+				&mm->ex);
+			camel_folder_sync (inbox, FALSE, NULL);
+			camel_object_unref (CAMEL_OBJECT (inbox));
+			if (camel_exception_is_set (&mm->ex))
+				return;
+		}
+	}
+
+	camel_disco_store_set_status (CAMEL_DISCO_STORE (m->store),
+				      m->offline ? CAMEL_DISCO_STORE_OFFLINE :
+				      CAMEL_DISCO_STORE_ONLINE, &mm->ex);
 }
 
 static void set_offline_done(struct _mail_msg *mm)
@@ -2129,6 +2131,12 @@ mail_store_set_offline (CamelStore *store, gboolean offline,
 {
 	struct _set_offline_msg *m;
 
+	/* Cancel any pending connect first so the set_offline_op
+	 * thread won't get queued behind a hung connect op.
+	 */
+	if (offline)
+		camel_service_cancel_connect (CAMEL_SERVICE (store));
+
 	m = mail_msg_new(&set_offline_op, NULL, sizeof(*m));
 	m->store = store;
 	camel_object_ref((CamelObject *)store);