aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJeffrey Stedfast <fejj@ximian.com>2001-01-13 12:23:02 +0800
committerJeffrey Stedfast <fejj@src.gnome.org>2001-01-13 12:23:02 +0800
commit911b3978080cf82b9af95864477bfc194e8789a1 (patch)
treef583cd2f6b8bd24e5ed180a422c83c89abd2422a
parent75c4fd98b64414d2e102002a5259f554f5b27b69 (diff)
downloadgsoc2013-evolution-911b3978080cf82b9af95864477bfc194e8789a1.tar
gsoc2013-evolution-911b3978080cf82b9af95864477bfc194e8789a1.tar.gz
gsoc2013-evolution-911b3978080cf82b9af95864477bfc194e8789a1.tar.bz2
gsoc2013-evolution-911b3978080cf82b9af95864477bfc194e8789a1.tar.lz
gsoc2013-evolution-911b3978080cf82b9af95864477bfc194e8789a1.tar.xz
gsoc2013-evolution-911b3978080cf82b9af95864477bfc194e8789a1.tar.zst
gsoc2013-evolution-911b3978080cf82b9af95864477bfc194e8789a1.zip
Now takes a 'is_account_data' variable to specify whether the sources is a
2001-01-12 Jeffrey Stedfast <fejj@ximian.com> * component-factory.c (mail_load_storages): Now takes a 'is_account_data' variable to specify whether the sources is a list of accounts of a list of services. Basically, the only time you should pass in FALSE is when you are setting up NNTP storages. (add_storage): Now takes a 'name' argument that specifies the name to use in the storage. (owner_set_cb): Updated to pass TRUE for accounts and FALSE for news servers into mail_load_storages. svn path=/trunk/; revision=7469
-rw-r--r--mail/ChangeLog11
-rw-r--r--mail/component-factory.c70
-rw-r--r--mail/mail-config-druid.c2
-rw-r--r--mail/mail.h2
4 files changed, 54 insertions, 31 deletions
diff --git a/mail/ChangeLog b/mail/ChangeLog
index 2827d2d552..7c9df25b2d 100644
--- a/mail/ChangeLog
+++ b/mail/ChangeLog
@@ -1,3 +1,14 @@
+2001-01-12 Jeffrey Stedfast <fejj@ximian.com>
+
+ * component-factory.c (mail_load_storages): Now takes a
+ 'is_account_data' variable to specify whether the sources is a
+ list of accounts of a list of services. Basically, the only time
+ you should pass in FALSE is when you are setting up NNTP storages.
+ (add_storage): Now takes a 'name' argument that specifies the name
+ to use in the storage.
+ (owner_set_cb): Updated to pass TRUE for accounts and FALSE for
+ news servers into mail_load_storages.
+
2001-01-12 Christopher James Lahey <clahey@helixcode.com>
* message-list.c: Changed filter_date and filter_size to match the
diff --git a/mail/component-factory.c b/mail/component-factory.c
index 25914359ea..9e990c4d53 100644
--- a/mail/component-factory.c
+++ b/mail/component-factory.c
@@ -172,7 +172,7 @@ owner_set_cb (EvolutionShellComponent *shell_component,
gpointer user_data)
{
GNOME_Evolution_Shell corba_shell;
- GSList *sources;
+ const GSList *accounts, *news;
int i;
g_print ("evolution-mail: Yeeeh! We have an owner!\n"); /* FIXME */
@@ -186,14 +186,12 @@ owner_set_cb (EvolutionShellComponent *shell_component,
create_vfolder_storage (shell_component);
corba_shell = bonobo_object_corba_objref (BONOBO_OBJECT (shell_client));
-
- sources = mail_config_get_sources ();
- mail_load_storages (corba_shell, sources);
- /* only this one gets free'd because it's created on-the-fly */
- g_slist_free (sources);
- sources = (GSList *) mail_config_get_news ();
- mail_load_storages (corba_shell, sources);
+ accounts = mail_config_get_accounts ();
+ mail_load_storages (corba_shell, accounts, TRUE);
+
+ news = mail_config_get_news ();
+ mail_load_storages (corba_shell, news, FALSE);
mail_local_storage_startup (shell_client, evolution_dir);
@@ -296,19 +294,16 @@ create_vfolder_storage (EvolutionShellComponent *shell_component)
}
static void
-add_storage (const char *uri, CamelService *store,
+add_storage (const char *name, const char *uri, CamelService *store,
GNOME_Evolution_Shell corba_shell, CamelException *ex)
{
EvolutionStorage *storage;
EvolutionStorageResult res;
- char *name;
-
- name = camel_service_get_name (store, TRUE);
+
storage = evolution_storage_new (name, uri, "mailstorage");
- g_free (name);
-
+
res = evolution_storage_register_on_shell (storage, corba_shell);
-
+
switch (res) {
case EVOLUTION_STORAGE_OK:
g_hash_table_insert (storages_hash, store, storage);
@@ -325,14 +320,15 @@ add_storage (const char *uri, CamelService *store,
}
}
+
+/* FIXME: 'is_account_data' is an ugly hack, if we remove support for NNTP we can take it out -- fejj */
void
-mail_load_storages (GNOME_Evolution_Shell corba_shell, GSList *sources)
+mail_load_storages (GNOME_Evolution_Shell shell, const GSList *sources, gboolean is_account_data)
{
CamelException ex;
- MailConfigService *svc;
- GSList *iter;
-
- camel_exception_init (&ex);
+ const GSList *iter;
+
+ camel_exception_init (&ex);
/* Load each service (don't connect!). Check its provider and
* see if this belongs in the shell's folder list. If so, add
@@ -340,18 +336,26 @@ mail_load_storages (GNOME_Evolution_Shell corba_shell, GSList *sources)
*/
for (iter = sources; iter; iter = iter->next) {
+ const MailConfigAccount *account = NULL;
+ const MailConfigService *service = NULL;
CamelService *store;
CamelProvider *prov;
-
- svc = (MailConfigService *) iter->data;
- if (svc->url == NULL || svc->url[0] == '\0')
+
+ if (is_account_data) {
+ account = iter->data;
+ service = account->source;
+ } else {
+ service = iter->data;
+ }
+
+ if (service->url == NULL || service->url[0] == '\0')
continue;
- store = camel_session_get_service (session, svc->url,
+ store = camel_session_get_service (session, service->url,
CAMEL_PROVIDER_STORE, &ex);
if (store == NULL) {
/* FIXME: real error dialog */
- g_warning ("couldn't get service %s: %s\n", svc->url,
+ g_warning ("couldn't get service %s: %s\n", service->url,
camel_exception_get_description (&ex));
camel_exception_clear (&ex);
continue;
@@ -365,9 +369,17 @@ mail_load_storages (GNOME_Evolution_Shell corba_shell, GSList *sources)
* That issue can't be resolved on the provider level
* -- it's a per-URL problem.
*/
- if (prov->flags & CAMEL_PROVIDER_IS_STORAGE &&
- prov->flags & CAMEL_PROVIDER_IS_REMOTE) {
- add_storage (svc->url, store, corba_shell, &ex);
+ if (prov->flags & CAMEL_PROVIDER_IS_STORAGE && prov->flags & CAMEL_PROVIDER_IS_REMOTE) {
+ char *name;
+
+ if (is_account_data) {
+ name = g_strdup (account->name);
+ } else {
+ name = camel_service_get_name (store, TRUE);
+ }
+ add_storage (name, service->url, store, shell, &ex);
+ g_free (name);
+
if (camel_exception_is_set (&ex)) {
/* FIXME: real error dialog */
g_warning ("Cannot load storage: %s",
@@ -375,7 +387,7 @@ mail_load_storages (GNOME_Evolution_Shell corba_shell, GSList *sources)
camel_exception_clear (&ex);
}
}
-
+
camel_object_unref (CAMEL_OBJECT (store));
}
}
diff --git a/mail/mail-config-druid.c b/mail/mail-config-druid.c
index ba13dda948..faf2e7f2f8 100644
--- a/mail/mail-config-druid.c
+++ b/mail/mail-config-druid.c
@@ -225,7 +225,7 @@ druid_finish (GnomeDruidPage *page, gpointer arg1, gpointer user_data)
mail_config_write ();
mini = g_slist_prepend (NULL, account->source);
- mail_load_storages (druid->shell, mini);
+ mail_load_storages (druid->shell, mini, TRUE);
g_slist_free (mini);
gtk_widget_destroy (GTK_WIDGET (druid));
diff --git a/mail/mail.h b/mail/mail.h
index e85639cbf5..e23fbd2812 100644
--- a/mail/mail.h
+++ b/mail/mail.h
@@ -68,6 +68,6 @@ GtkWidget *mail_view_create (CamelFolder *source, const char *uid, CamelMimeMess
/* component factory for lack of a better place */
/*takes a GSList of MailConfigServices */
-void mail_load_storages (GNOME_Evolution_Shell corba_shell, GSList *sources);
+void mail_load_storages (GNOME_Evolution_Shell shell, const GSList *sources, gboolean is_account_data);
/* used in the subscribe dialog code */
EvolutionStorage *mail_lookup_storage (CamelStore *store);