New convenience function to find the first parent node that is visible and

2002-06-18  Jeffrey Stedfast  <fejj@ximian.com>

	* message-list.c (message_list_change_first_visible_parent): New
	convenience function to find the first parent node that is visible
	and emit a changed signal on it.
	(main_folder_changed): If a message changed, call
	change_first_visible_parent() in case we are in a collapsed thread
	so that our first visible parent gets updated as well. Fixes bug
	#26263.

	* component-factory.c (storage_remove_folder): Simplify the error
	checking.

svn path=/trunk/; revision=17228

1 files changed, 5 insertions, 9 deletions

diff --git a/mail/component-factory.c b/mail/component-factory.c
index 07dcba7c32..5384bbd249 100644
--- a/mail/component-factory.c
+++ b/mail/component-factory.c
@@ -1132,7 +1132,7 @@ storage_remove_folder (EvolutionStorage *storage,
 	
 	g_warning ("storage_remove_folder: path=\"%s\"; uri=\"%s\"", path, physical_uri);
 	
-	if (!path || !physical_uri || !strncmp (physical_uri, "vtrash:", 7)) {
+	if (!path || !*path || !physical_uri || !strncmp (physical_uri, "vtrash:", 7)) {
 		notify_listener (listener, GNOME_Evolution_Storage_INVALID_URI);
 		return;
 	}
@@ -1143,21 +1143,15 @@ storage_remove_folder (EvolutionStorage *storage,
 		return;
 	}
 	
-	if (!*path) {
-		camel_url_free (url);
-		notify_listener (listener, GNOME_Evolution_Storage_INVALID_URI);
-		return;
-	}
-	
 	camel_exception_init (&ex);
-
+	
 	if (url->fragment)
 		name = url->fragment;
 	else if (url->path && url->path[0])
 		name = url->path+1;
 	else
 		name = "";
-
+	
 	if (camel_store_supports_subscriptions (store))
 		camel_store_unsubscribe_folder (store, name, NULL);
 	
@@ -1192,6 +1186,8 @@ storage_xfer_folder (EvolutionStorage *storage,
 
 	d(printf("Transfer folder on store source = '%s' dest = '%s'\n", source_path, destination_path));
 
+	/* FIXME: this is totally not gonna work once we have namespaces */
+	
 	/* Remap the 'path' to the camel friendly name based on the store dir separator */
 	sep = store->dir_sep;
 	src = g_strdup(source_path[0]=='/'?source_path+1:source_path);