[evolution-patches] 311639, mh rename dir issues

[Not Zed <notzed ximian com>]

changelog says most of it, mainly fixes rename in mh

-- 
adfa(evolution-2.4:20087): gtkhtml-WARNING **: cannot find icon:
'stock_insert-url' in gnome 

Index: camel/providers/local/ChangeLog
===================================================================
RCS file: /cvs/gnome/evolution-data-server/camel/providers/local/ChangeLog,v
retrieving revision 1.8
diff -u -p -r1.8 ChangeLog
--- camel/providers/local/ChangeLog	8 Jun 2005 15:33:24 -0000	1.8
+++ camel/providers/local/ChangeLog	8 Aug 2005 10:31:30 -0000
@@ -1,3 +1,13 @@
+2005-08-08  Not Zed  <NotZed Ximian com>
+
+	** See bug #311639.
+
+	* camel-mh-store.c (folders_update): add a rename mode, fix
+	callers.
+	(folders_scan): pass top to folder_info_build, otherwise
+	get_folder_info doesn't work right.  Fix the check for subfolder
+	test.
+
 2005-06-08  Not Zed  <NotZed Ximian com>
 
 	** See bug #303225
Index: camel/providers/local/camel-mh-store.c
===================================================================
RCS file: /cvs/gnome/evolution-data-server/camel/providers/local/camel-mh-store.c,v
retrieving revision 1.22
diff -u -p -r1.22 camel-mh-store.c
--- camel/providers/local/camel-mh-store.c	12 Nov 2004 05:53:12 -0000	1.22
+++ camel/providers/local/camel-mh-store.c	8 Aug 2005 10:31:31 -0000
@@ -108,14 +108,16 @@ enum {
 	UPDATE_NONE,
 	UPDATE_ADD,
 	UPDATE_REMOVE,
+	UPDATE_RENAME,
 };
 
 /* update the .folders file if it exists, or create it if it doesn't */
 static void
-folders_update(const char *root, const char *folder, int mode)
+folders_update(const char *root, int mode, const char *folder, const char *new)
 {
 	char *tmp, *tmpnew, *line = NULL;
 	CamelStream *stream, *in = NULL, *out = NULL;
+	int flen = strlen(folder);
 
 	tmpnew = g_alloca (strlen (root) + 16);
 	sprintf (tmpnew, "%s.folders~", root);
@@ -145,6 +147,16 @@ folders_update(const char *root, const c
 			if (strcmp(line, folder) == 0)
 				copy = FALSE;
 			break;
+		case UPDATE_RENAME:
+			if (strncmp(line, folder, flen) == 0
+			    && (line[flen] == 0 || line[flen] == '/')) {
+				if (camel_stream_write(out, new, strlen(new)) == -1
+				    || camel_stream_write(out, line+flen, strlen(line)-flen) == -1
+				    || camel_stream_write(out, "\n", 1) == -1)
+					goto fail;
+				copy = FALSE;
+			}
+			break;
 		case UPDATE_ADD: {
 			int cmp = strcmp(line, folder);
 
@@ -226,7 +238,7 @@ get_folder(CamelStore * store, const cha
 		/* add to .folders if we are supposed to */
 		/* FIXME: throw exception on error */
 		if (((CamelMhStore *)store)->flags & CAMEL_MH_DOTFOLDERS)
-			folders_update(((CamelLocalStore *)store)->toplevel_dir, folder_name, UPDATE_ADD);
+			folders_update(((CamelLocalStore *)store)->toplevel_dir, UPDATE_ADD, folder_name, NULL);
 	} else if (!S_ISDIR(st.st_mode)) {
 		camel_exception_setv(ex, CAMEL_EXCEPTION_STORE_NO_FOLDER,
 				     _("Cannot get folder `%s': not a directory."), folder_name);
@@ -267,7 +279,7 @@ static void delete_folder(CamelStore * s
 
 	/* remove from .folders if we are supposed to */
 	if (((CamelMhStore *)store)->flags & CAMEL_MH_DOTFOLDERS)
-		folders_update(((CamelLocalStore *)store)->toplevel_dir, folder_name, UPDATE_REMOVE);
+		folders_update(((CamelLocalStore *)store)->toplevel_dir, UPDATE_REMOVE, folder_name, NULL);
 
 	/* and remove metadata */
 	((CamelStoreClass *)parent_class)->delete_folder(store, folder_name, ex);
@@ -288,8 +300,7 @@ rename_folder(CamelStore *store, const c
 
 	if (((CamelMhStore *)store)->flags & CAMEL_MH_DOTFOLDERS) {
 		/* yeah this is messy, but so is mh! */
-		folders_update(((CamelLocalStore *)store)->toplevel_dir, new, UPDATE_ADD);
-		folders_update(((CamelLocalStore *)store)->toplevel_dir, old, UPDATE_REMOVE);
+		folders_update(((CamelLocalStore *)store)->toplevel_dir, UPDATE_RENAME, old, new);
 	}
 }
 
@@ -482,8 +493,9 @@ folders_scan(CamelStore *store, CamelURL
 		if (top && top[0]) {
 			int toplen = strlen(top);
 
-			/* check is subdir */
-			if (strncmp(top, line, len) != 0)
+			/* check is dir or subdir */
+			if (strncmp(top, line, toplen) != 0
+			    || (line[toplen] != 0 && line[toplen] != '/'))
 				continue;
 		
 			/* check is not sub-subdir if not recursive */
@@ -508,7 +520,7 @@ folders_scan(CamelStore *store, CamelURL
 	}
 
 	if (folders->len)
-		*fip = camel_folder_info_build(folders, NULL, '/', TRUE);
+		*fip = camel_folder_info_build(folders, top, '/', TRUE);
 	g_ptr_array_free(folders, TRUE);
 
 	g_hash_table_foreach(visited, (GHFunc)g_free, NULL);