[evolution] Bug #602425 - Search in "Current Account"/"All Accounts" depends on vfolders

[Milan Crha <mcrha src gnome org>]

commit 69e55b253eea402a16fbbb35f0e074945007c68e
Author: Milan Crha <mcrha redhat com>
Date:   Mon Jul 9 16:43:48 2012 +0200

    Bug #602425 - Search in "Current Account"/"All Accounts" depends on vfolders

 modules/mail/e-mail-shell-view.c |  205 +++++++++++++++++++-------------------
 1 files changed, 103 insertions(+), 102 deletions(-)
---
diff --git a/modules/mail/e-mail-shell-view.c b/modules/mail/e-mail-shell-view.c
index 146b89f..ad85799 100644
--- a/modules/mail/e-mail-shell-view.c
+++ b/modules/mail/e-mail-shell-view.c
@@ -44,12 +44,70 @@ static const gchar *SEARCH_RESULTS_STATE =
 "  </grouping>"
 "</ETableState>";
 
+static void
+add_folders_from_store (GList **folders,
+			CamelStore *store,
+			GCancellable *cancellable,
+			GError **error)
+{
+	CamelFolderInfo *root, *fi;
+
+	g_return_if_fail (folders != NULL);
+	g_return_if_fail (store != NULL);
+
+	if (CAMEL_IS_VEE_STORE (store))
+		return;
+
+	root = camel_store_get_folder_info_sync (
+		store, NULL,
+		CAMEL_STORE_FOLDER_INFO_RECURSIVE, cancellable, error);
+	fi = root;
+	while (fi && !g_cancellable_is_cancelled (cancellable)) {
+		CamelFolderInfo *next;
+
+		if ((fi->flags & CAMEL_FOLDER_NOSELECT) == 0) {
+			CamelFolder *fldr;
+
+			fldr = camel_store_get_folder_sync (
+				store, fi->full_name, 0, cancellable, error);
+			if (fldr) {
+				if (CAMEL_IS_VEE_FOLDER (fldr)) {
+					g_object_unref (fldr);
+				} else {
+					*folders = g_list_prepend (*folders, fldr);
+				}
+			}
+		}
+
+		/* pick the next */
+		next = fi->child;
+		if (!next)
+			next = fi->next;
+		if (!next) {
+			next = fi->parent;
+			while (next) {
+				if (next->next) {
+					next = next->next;
+					break;
+				}
+
+				next = next->parent;
+			}
+		}
+
+		fi = next;
+	}
+
+	if (root)
+		camel_store_free_folder_info_full (store, root);
+}
+
 typedef struct {
 	MailMsg base;
 
 	CamelFolder *folder;
 	GCancellable *cancellable;
-	GList *folder_list;
+	GList *stores_list;
 } SearchResultsMsg;
 
 static gchar *
@@ -63,16 +121,23 @@ search_results_exec (SearchResultsMsg *msg,
                      GCancellable *cancellable,
                      GError **error)
 {
-	GList *copied_list;
+	GList *folders = NULL, *iter;
 
-	copied_list = g_list_copy (msg->folder_list);
-	g_list_foreach (copied_list, (GFunc) g_object_ref, NULL);
+	for (iter = msg->stores_list; iter && !g_cancellable_is_cancelled (cancellable); iter = iter->next) {
+		CamelStore *store = iter->data;
+
+		add_folders_from_store (&folders, store, cancellable, error);
+	}
 
-	camel_vee_folder_set_folders (
-		CAMEL_VEE_FOLDER (msg->folder), copied_list, cancellable);
+	if (!g_cancellable_is_cancelled (cancellable)) {
+		CamelVeeFolder *vfolder = CAMEL_VEE_FOLDER (msg->folder);
 
-	g_list_foreach (copied_list, (GFunc) g_object_unref, NULL);
-	g_list_free (copied_list);
+		folders = g_list_reverse (folders);
+
+		camel_vee_folder_set_folders (vfolder, folders, cancellable);
+	}
+
+	g_list_free_full (folders, g_object_unref);
 }
 
 static void
@@ -84,9 +149,7 @@ static void
 search_results_free (SearchResultsMsg *msg)
 {
 	g_object_unref (msg->folder);
-
-	g_list_foreach (msg->folder_list, (GFunc) g_object_unref, NULL);
-	g_list_free (msg->folder_list);
+	g_list_free_full (msg->stores_list, g_object_unref);
 }
 
 static MailMsgInfo search_results_setup_info = {
@@ -99,7 +162,7 @@ static MailMsgInfo search_results_setup_info = {
 
 static gint
 mail_shell_view_setup_search_results_folder (CamelFolder *folder,
-                                             GList *folder_list,
+                                             GList *stores,
                                              GCancellable *cancellable)
 {
 	SearchResultsMsg *msg;
@@ -110,7 +173,7 @@ mail_shell_view_setup_search_results_folder (CamelFolder *folder,
 	msg = mail_msg_new (&search_results_setup_info);
 	msg->folder = folder;
 	msg->cancellable = cancellable;
-	msg->folder_list = folder_list;
+	msg->stores_list = stores;
 
 	id = msg->base.seq;
 	mail_msg_slow_ordered_push (msg);
@@ -222,7 +285,6 @@ mail_shell_view_execute_search (EShellView *shell_view)
 	EActionComboBox *combo_box;
 	EMailBackend *backend;
 	EMailSession *session;
-	MailFolderCache *cache;
 	ESourceRegistry *registry;
 	EMFolderTree *folder_tree;
 	GtkWidget *message_list;
@@ -240,7 +302,6 @@ mail_shell_view_execute_search (EShellView *shell_view)
 	GString *string;
 	GList *list, *iter;
 	GSList *search_strings = NULL;
-	GQueue queue = G_QUEUE_INIT;
 	const gchar *text;
 	gboolean valid;
 	gchar *query;
@@ -540,7 +601,7 @@ all_accounts:
 	/* Skip the search if we already have the results. */
 	if (search_folder != NULL)
 		if (g_strcmp0 (query, camel_vee_folder_get_expression (search_folder)) == 0)
-			goto execute;
+			goto all_accounts_setup;
 
 	/* Disable the scope combo while search is in progress. */
 	gtk_widget_set_sensitive (GTK_WIDGET (combo_box), FALSE);
@@ -555,13 +616,11 @@ all_accounts:
 
 		camel_vee_folder_set_expression (search_folder, query);
 
-		goto execute;
+		goto all_accounts_setup;
 	}
 
 	/* Create a new search folder. */
 
-	list = NULL;  /* list of CamelFolders */
-
 	/* FIXME Complete lack of error checking here. */
 	service = camel_session_get_service (
 		CAMEL_SESSION (session), E_MAIL_SESSION_VFOLDER_UID);
@@ -572,31 +631,16 @@ all_accounts:
 		0);
 	priv->search_account_all = search_folder;
 
-	cache = e_mail_session_get_folder_cache (session);
-	mail_folder_cache_get_local_folder_uris (cache, &queue);
-	mail_folder_cache_get_remote_folder_uris (cache, &queue);
-
-	/* Add all available local and remote folders. */
-	while (!g_queue_is_empty (&queue)) {
-		gchar *folder_uri = g_queue_pop_head (&queue);
-
-		/* FIXME Not passing a GCancellable or GError here. */
-		folder = e_mail_session_uri_to_folder_sync (
-			E_MAIL_SESSION (session), folder_uri, 0, NULL, NULL);
-
-		if (folder != NULL)
-			list = g_list_append (list, folder);
-		else
-			g_warning ("Could not open vfolder source: %s", folder_uri);
-
-		g_free (folder_uri);
-	}
-
 	camel_vee_folder_set_expression (search_folder, query);
 
+ all_accounts_setup:
+	list = em_folder_tree_model_list_stores (EM_FOLDER_TREE_MODEL (
+		gtk_tree_view_get_model (GTK_TREE_VIEW (folder_tree))));
+	g_list_foreach (list, (GFunc) g_object_ref, NULL);
+
 	priv->search_account_cancel = camel_operation_new ();
 
-	/* This takes ownership of the folder list. */
+	/* This takes ownership of the stores list. */
 	mail_shell_view_setup_search_results_folder (
 		CAMEL_FOLDER (search_folder), list,
 		priv->search_account_cancel);
@@ -653,7 +697,7 @@ current_account:
 	/* Skip the search if we already have the results. */
 	if (search_folder != NULL)
 		if (g_strcmp0 (query, camel_vee_folder_get_expression (search_folder)) == 0)
-			goto execute;
+			goto current_accout_setup;
 
 	/* Disable the scope combo while search is in progress. */
 	gtk_widget_set_sensitive (GTK_WIDGET (combo_box), FALSE);
@@ -668,70 +712,11 @@ current_account:
 
 		camel_vee_folder_set_expression (search_folder, query);
 
-		goto execute;
+		goto current_accout_setup;
 	}
 
 	/* Create a new search folder. */
 
-	if (folder != NULL) {
-		store = camel_folder_get_parent_store (folder);
-		if (store != NULL)
-			g_object_ref (store);
-	} else {
-		store = NULL;
-		em_folder_tree_get_selected (folder_tree, &store, NULL);
-	}
-
-	list = NULL;  /* list of CamelFolders */
-
-	if (store != NULL) {
-		CamelFolderInfo *root, *fi;
-
-		/* FIXME This call blocks the main loop. */
-		root = camel_store_get_folder_info_sync (
-			store, NULL,
-			CAMEL_STORE_FOLDER_INFO_RECURSIVE, NULL, NULL);
-		fi = root;
-		while (fi) {
-			CamelFolderInfo *next;
-
-			if ((fi->flags & CAMEL_FOLDER_NOSELECT) == 0) {
-				CamelFolder *fldr;
-
-				/* FIXME This call blocks the main loop. */
-				fldr = camel_store_get_folder_sync (
-					store, fi->full_name, 0, NULL, NULL);
-				if (fldr)
-					list = g_list_prepend (list, fldr);
-			}
-
-			/* pick the next */
-			next = fi->child;
-			if (!next)
-				next = fi->next;
-			if (!next) {
-				next = fi->parent;
-				while (next) {
-					if (next->next) {
-						next = next->next;
-						break;
-					}
-
-					next = next->parent;
-				}
-			}
-
-			fi = next;
-		}
-
-		if (root)
-			camel_store_free_folder_info_full (store, root);
-
-		g_object_unref (store);
-	}
-
-	list = g_list_reverse (list);
-
 	/* FIXME Complete lack of error checking here. */
 	service = camel_session_get_service (
 		CAMEL_SESSION (session), E_MAIL_SESSION_VFOLDER_UID);
@@ -744,9 +729,25 @@ current_account:
 
 	camel_vee_folder_set_expression (search_folder, query);
 
+ current_accout_setup:
+
+	if (folder != NULL && folder != CAMEL_FOLDER (search_folder)) {
+		store = camel_folder_get_parent_store (folder);
+		if (store != NULL)
+			g_object_ref (store);
+	} else {
+		store = NULL;
+		em_folder_tree_get_selected (folder_tree, &store, NULL);
+	}
+
+	list = NULL;  /* list of CamelStore-s */
+
+	if (store != NULL)
+		list = g_list_append (NULL, store);
+
 	priv->search_account_cancel = camel_operation_new ();
 
-	/* This takes ownership of the folder list. */
+	/* This takes ownership of the stores list. */
 	mail_shell_view_setup_search_results_folder (
 		CAMEL_FOLDER (search_folder), list,
 		priv->search_account_cancel);