diff options
Diffstat (limited to 'mail')
-rw-r--r-- | mail/ChangeLog | 8 | ||||
-rw-r--r-- | mail/mail-ops.c | 90 |
2 files changed, 84 insertions, 14 deletions
diff --git a/mail/ChangeLog b/mail/ChangeLog index 1a4ad5cd1b..74ffc4adce 100644 --- a/mail/ChangeLog +++ b/mail/ChangeLog @@ -1,3 +1,11 @@ +2000-04-10 Dan Winship <danw@helixcode.com> + + * mail-ops.c (fetch_mail): use camel_movemail when fetching mail + from an mbox store. This leaves behind temp files for now, + because CamelMboxFolder::delete is too confused to use, and NotZed + is rewriting CamelMboxFolder, so I'm not going to bother to try to + fix it. + 2000-04-09 Matt Loper <matt@helixcode.com> * folder-browser.c (folder_browser_new): set folder_browser->uri diff --git a/mail/mail-ops.c b/mail/mail-ops.c index 405eddec4b..edb3bf0805 100644 --- a/mail/mail-ops.c +++ b/mail/mail-ops.c @@ -24,6 +24,7 @@ */ #include <config.h> +#include <errno.h> #include <gnome.h> #include "camel/camel.h" #include "mail-ops.h" @@ -86,22 +87,83 @@ fetch_mail (GtkWidget *button, gpointer user_data) } ex = camel_exception_new (); - store = camel_session_get_store (default_session->session, url, ex); - if (!store) { - mail_exception_dialog ("Unable to get new mail", ex, window); - goto cleanup; - } - camel_service_connect_with_url (CAMEL_SERVICE (store), url, ex); - if (camel_exception_get_id (ex) != CAMEL_EXCEPTION_NONE) { - mail_exception_dialog ("Unable to get new mail", ex, window); - goto cleanup; - } - folder = camel_store_get_folder (store, "inbox", ex); - if (!folder) { - mail_exception_dialog ("Unable to get new mail", ex, window); - goto cleanup; + /* If fetching mail from an mbox store, safely copy it to a + * temporary store first. + */ + if (!strncmp (url, "mbox:", 5)) { + char *tmp_mbox, *source; + int tmpfd; + + tmp_mbox = g_strdup_printf ("%s/movemail.XXXX", + evolution_folders_dir); +#ifdef HAVE_MKSTEMP + tmpfd = mkstemp (tmp_mbox); +#else + if (mktemp (tmp_mbox)) { + tmpfd = open (tmp_mbox, O_RDWR | O_CREAT | O_EXCL, + S_IRUSR | S_IWUSR); + } else + tmpfd = -1; +#endif + if (tmpfd == -1) { + camel_exception_setv (ex, CAMEL_EXCEPTION_SYSTEM, + "Couldn't create temporary " + "mbox: %s", g_strerror (errno)); + mail_exception_dialog ("Unable to move mail", ex, + window); + goto cleanup; + } + close (tmpfd); + + /* Skip over "mbox://" plus host part (if any) or url. */ + source = strchr (url + 7, '/'); + + switch (camel_movemail (source, tmp_mbox, ex)) { + case -1: + mail_exception_dialog ("Unable to move mail", ex, + window); + /* FALL THROUGH */ + + case 0: + unlink (tmp_mbox); + g_free (tmp_mbox); + goto cleanup; + } + + folder = camel_store_get_folder (default_session->store, + strrchr (tmp_mbox, '/') + 1, + ex); + if (camel_exception_get_id (ex) != CAMEL_EXCEPTION_NONE) { + mail_exception_dialog ("Unable to move mail", ex, + window); + g_free (tmp_mbox); + goto cleanup; + } + } else { + store = camel_session_get_store (default_session->session, + url, ex); + if (!store) { + mail_exception_dialog ("Unable to get new mail", + ex, window); + goto cleanup; + } + camel_service_connect_with_url (CAMEL_SERVICE (store), + url, ex); + if (camel_exception_get_id (ex) != CAMEL_EXCEPTION_NONE) { + mail_exception_dialog ("Unable to get new mail", + ex, window); + goto cleanup; + } + + folder = camel_store_get_folder (store, "inbox", ex); + if (!folder) { + mail_exception_dialog ("Unable to get new mail", + ex, window); + goto cleanup; + } } + camel_folder_open (folder, FOLDER_OPEN_READ, ex); if (camel_exception_get_id (ex) != CAMEL_EXCEPTION_NONE) { mail_exception_dialog ("Unable to get new mail", ex, window); |