From 739cb175076f824663a308a217fd797896988cff Mon Sep 17 00:00:00 2001 From: Jeffrey Stedfast Date: Mon, 12 Apr 2004 17:43:33 +0000 Subject: Really fixes bug #56623 2004-04-12 Jeffrey Stedfast Really fixes bug #56623 * e-signature-list.c (gconf_signatures_changed): If we've had to "fix" any signatures by adding a uid, set the resave state on the list (only cared about if this is at initial load time). (e_signature_list_construct): If we've set any uids in signatures_changed(), then save the signature list back out. svn path=/trunk/; revision=25416 --- e-util/ChangeLog | 10 ++++++++++ e-util/e-signature-list.c | 20 ++++++++++++++++++-- e-util/e-signature-list.h | 8 ++++---- 3 files changed, 32 insertions(+), 6 deletions(-) diff --git a/e-util/ChangeLog b/e-util/ChangeLog index eb632ceb22..ff22477bb8 100644 --- a/e-util/ChangeLog +++ b/e-util/ChangeLog @@ -1,3 +1,13 @@ +2004-04-12 Jeffrey Stedfast + + Really fixes bug #56623 + + * e-signature-list.c (gconf_signatures_changed): If we've had to + "fix" any signatures by adding a uid, set the resave state on the + list (only cared about if this is at initial load time). + (e_signature_list_construct): If we've set any uids in + signatures_changed(), then save the signature list back out. + 2004-04-09 Jeffrey Stedfast * e-account.c (xml_set_identity): If we fail to set a sig_uid then diff --git a/e-util/e-signature-list.c b/e-util/e-signature-list.c index 90a8cd70f1..8f651f54df 100644 --- a/e-util/e-signature-list.c +++ b/e-util/e-signature-list.c @@ -27,6 +27,8 @@ #include +#include "e-uid.h" + #include "e-util-marshal.h" #include "e-signature-list.h" @@ -34,6 +36,7 @@ struct _ESignatureListPrivate { GConfClient *gconf; guint notify_id; + gboolean resave; }; enum { @@ -167,6 +170,7 @@ gconf_signatures_changed (GConfClient *client, guint cnxn_id, GConfEntry *entry, ESignatureList *signature_list = user_data; GSList *list, *l, *n, *new_sigs = NULL; gboolean have_autogen = FALSE; + gboolean resave = FALSE; ESignature *signature; EList *old_sigs; EIterator *iter; @@ -204,11 +208,17 @@ gconf_signatures_changed (GConfClient *client, guint cnxn_id, GConfEntry *entry, if (!have_autogen) { add_autogen (signature_list); have_autogen = TRUE; + resave = TRUE; } if (!found) { /* Must be a new signature */ signature = e_signature_new_from_xml (l->data); + if (!signature->uid) { + signature->uid = e_uid_new (); + resave = TRUE; + } + e_list_append (E_LIST (signature_list), signature); new_sigs = g_slist_prepend (new_sigs, signature); } @@ -219,6 +229,7 @@ gconf_signatures_changed (GConfClient *client, guint cnxn_id, GConfEntry *entry, if (!have_autogen) { add_autogen (signature_list); have_autogen = TRUE; + resave = TRUE; } if (new_sigs != NULL) { @@ -243,6 +254,8 @@ gconf_signatures_changed (GConfClient *client, guint cnxn_id, GConfEntry *entry, g_object_unref (iter); g_object_unref (old_sigs); + + signature_list->priv->resave = resave; } static void * @@ -311,6 +324,11 @@ e_signature_list_construct (ESignatureList *signature_list, GConfClient *gconf) gconf_signatures_changed (signature_list->priv->gconf, signature_list->priv->notify_id, NULL, signature_list); + + if (signature_list->priv->resave) { + e_signature_list_save (signature_list); + signature_list->priv->resave = FALSE; + } } @@ -422,7 +440,6 @@ e_signature_list_find (ESignatureList *signatures, e_signature_find_t type, cons { const ESignature *signature = NULL; EIterator *it; - char *val; /* this could use a callback for more flexibility ... ... but this makes the common cases easier */ @@ -437,7 +454,6 @@ e_signature_list_find (ESignatureList *signatures, e_signature_find_t type, cons signature = (const ESignature *) e_iterator_get (it); - val = NULL; switch (type) { case E_SIGNATURE_FIND_NAME: found = strcmp (signature->name, key) == 0; diff --git a/e-util/e-signature-list.h b/e-util/e-signature-list.h index 0e7f437041..cec82f9c6f 100644 --- a/e-util/e-signature-list.h +++ b/e-util/e-signature-list.h @@ -67,10 +67,10 @@ void e_signature_list_construct (ESignatureList *signature_list, GConfClient *gc void e_signature_list_save (ESignatureList *signature_list); -void e_signature_list_add (ESignatureList *, ESignature *); -void e_signature_list_change (ESignatureList *, ESignature *); -void e_signature_list_remove (ESignatureList *, ESignature *); +void e_signature_list_add (ESignatureList *signature_list, ESignature *signature); +void e_signature_list_change (ESignatureList *signature_list, ESignature *signature); +void e_signature_list_remove (ESignatureList *signature_list, ESignature *signature); -const ESignature *e_signature_list_find (ESignatureList *, e_signature_find_t type, const char *key); +const ESignature *e_signature_list_find (ESignatureList *signature_list, e_signature_find_t type, const char *key); #endif /* __E_SIGNATURE_LIST__ */ -- cgit v1.2.3