aboutsummaryrefslogtreecommitdiffstats
path: root/camel/providers/imap
diff options
context:
space:
mode:
authorJeffrey Stedfast <fejj@ximian.com>2001-09-21 05:28:10 +0800
committerJeffrey Stedfast <fejj@src.gnome.org>2001-09-21 05:28:10 +0800
commitd728e1e79c7a6e3d7797775de7d9ef0320937422 (patch)
tree69b216975fc8d8e5e3865a4af26201596ee277dc /camel/providers/imap
parent5052619689e904d905c8e6a27735ecc8d69638bd (diff)
downloadgsoc2013-evolution-d728e1e79c7a6e3d7797775de7d9ef0320937422.tar
gsoc2013-evolution-d728e1e79c7a6e3d7797775de7d9ef0320937422.tar.gz
gsoc2013-evolution-d728e1e79c7a6e3d7797775de7d9ef0320937422.tar.bz2
gsoc2013-evolution-d728e1e79c7a6e3d7797775de7d9ef0320937422.tar.lz
gsoc2013-evolution-d728e1e79c7a6e3d7797775de7d9ef0320937422.tar.xz
gsoc2013-evolution-d728e1e79c7a6e3d7797775de7d9ef0320937422.tar.zst
gsoc2013-evolution-d728e1e79c7a6e3d7797775de7d9ef0320937422.zip
And finally here.
2001-09-20 Jeffrey Stedfast <fejj@ximian.com> * providers/local/camel-spool-folder.c (camel_spool_folder_construct): And finally here. * providers/local/camel-local-folder.c (camel_local_folder_construct): And here. * camel-vee-store.c (vee_get_folder): ANd here too. (vee_get_folder_info): And here. * providers/imap/camel-imap-store.c (create_folder): Use camel_folder_info_build() and do better error checking. (subscribe_folder): Set the path on the folder info. * camel-store.c (camel_folder_info_build): Set the folder-info path if it hasn't already been set by the caller. (camel_folder_info_free): Free the path. * camel-store.h: Added a path member to the FolderInfo type. svn path=/trunk/; revision=13019
Diffstat (limited to 'camel/providers/imap')
-rw-r--r--camel/providers/imap/camel-imap-store.c73
1 files changed, 48 insertions, 25 deletions
diff --git a/camel/providers/imap/camel-imap-store.c b/camel/providers/imap/camel-imap-store.c
index b58b97fa83..08c871cee3 100644
--- a/camel/providers/imap/camel-imap-store.c
+++ b/camel/providers/imap/camel-imap-store.c
@@ -4,7 +4,7 @@
/*
* Authors:
* Dan Winship <danw@ximian.com>
- * Jeffrey Stedfast <fejj@danw.com>
+ * Jeffrey Stedfast <fejj@ximian.com>
*
* Copyright 2000, 2001 Ximian, Inc.
*
@@ -1043,10 +1043,12 @@ create_folder (CamelStore *store, const char *parent_name,
CamelImapStore *imap_store = CAMEL_IMAP_STORE (store);
char *full_name, *resp, *thisone;
CamelImapResponse *response;
- CamelFolderInfo *root, *fi;
+ CamelException internal_ex;
+ CamelFolderInfo *root = NULL;
gboolean need_convert;
- char **pathnames;
- int i, flags;
+ char **pathnames = NULL;
+ GPtrArray *folders = NULL;
+ int i = 0, flags;
if (!camel_disco_store_check_online (CAMEL_DISCO_STORE (store), ex))
return NULL;
@@ -1077,10 +1079,11 @@ create_folder (CamelStore *store, const char *parent_name,
camel_imap_response_free (imap_store, response);
+ camel_exception_init (&internal_ex);
+
/* if not, check if we can delete it and recreate it */
if (need_convert) {
- gchar *name;
- CamelException internal_ex;
+ char *name;
if (get_folder_status (imap_store, parent_name, "MESSAGES")) {
camel_exception_set (ex, CAMEL_EXCEPTION_FOLDER_INVALID_STATE,
@@ -1089,7 +1092,6 @@ create_folder (CamelStore *store, const char *parent_name,
}
/* delete the old parent and recreate it */
- camel_exception_init (&internal_ex);
delete_folder (store, parent_name, &internal_ex);
if (camel_exception_is_set (&internal_ex)) {
camel_exception_xfer (ex, &internal_ex);
@@ -1116,45 +1118,62 @@ create_folder (CamelStore *store, const char *parent_name,
g_free (full_name);
if (response) {
- CamelFolderInfo *parent;
- GPtrArray *folders;
-
camel_imap_response_free (imap_store, response);
/* We have to do this in case we are creating a
recursive directory structure */
+ i = 0;
pathnames = imap_parse_folder_name (imap_store, folder_name);
- full_name = imap_concat (imap_store, parent_name, pathnames[0]);
- g_free (pathnames[0]);
+ full_name = imap_concat (imap_store, parent_name, pathnames[i]);
+ g_free (pathnames[i]);
folders = g_ptr_array_new ();
+
get_folders_online (imap_store, full_name, folders, FALSE, ex);
- parent = root = folders->pdata[0];
g_free (full_name);
+ if (camel_exception_is_set (&internal_ex)) {
+ camel_exception_xfer (&internal_ex, ex);
+ goto exception;
+ }
- for (i = 1; parent && pathnames[i]; i++) {
+ for (i = 1; pathnames[i]; i++) {
full_name = imap_concat (imap_store, parent_name, pathnames[i]);
g_free (pathnames[i]);
- get_folders_online (imap_store, full_name, folders, FALSE, ex);
+ get_folders_online (imap_store, full_name, folders, FALSE, &internal_ex);
+ if (camel_exception_is_set (&internal_ex)) {
+ camel_exception_xfer (&internal_ex, ex);
+ goto exception;
+ }
g_free (full_name);
if (folders->len != i + 1)
break;
-
- fi = folders->pdata[i];
-
- fi->parent = parent;
- parent->child = fi;
- parent = fi;
}
g_free (pathnames);
+
+ root = camel_folder_info_build (folders, camel_url_get_param (CAMEL_SERVICE (store)->url, "namespace"),
+ imap_store->dir_sep, TRUE);
+
g_ptr_array_free (folders, TRUE);
- } else
- root = NULL;
+ }
return root;
+
+ exception:
+
+ for (/* i is already set */; pathnames && pathnames[i]; i++)
+ g_free (pathnames[i]);
+ g_free (pathnames);
+
+ if (folders) {
+ for (i = 0; i < folders->len; i++)
+ camel_folder_info_free (folders->pdata[i]);
+ g_ptr_array_free (folders, TRUE);
+ }
+
+ return NULL;
}
static CamelFolderInfo *
@@ -1522,13 +1541,17 @@ subscribe_folder (CamelStore *store, const char *folder_name,
name = strrchr (folder_name, imap_store->dir_sep);
if (name)
name++;
-
+
+ /* FIXME: we should probably relocate all code that generates
+ fi->path to a single location and have all code use that */
fi = g_new0 (CamelFolderInfo, 1);
fi->full_name = g_strdup (folder_name);
fi->name = g_strdup (name);
fi->url = g_strdup_printf ("%s/%s", imap_store->base_url, folder_name);
fi->unread_message_count = -1;
-
+
+ camel_folder_info_build_path (fi, imap_store->dir_sep);
+
camel_object_trigger_event (CAMEL_OBJECT (store), "folder_created", fi);
camel_folder_info_free (fi);
}