X-Git-Url: http://git.maemo.org/git/?a=blobdiff_plain;f=src%2Fwidgets%2Fmodest-account-view.c;h=8fa542126bc2ff3fbf0aaa7c4fa037d079563621;hb=4a1d53c83ba8c98f22a5aa5a9119f41fd6d6908b;hp=40bc5bfc4c26cb08866dc1f92dc0e4e99d381c5a;hpb=64333784e87e5ea5c6156078caacaca7ffbd6b5f;p=modest diff --git a/src/widgets/modest-account-view.c b/src/widgets/modest-account-view.c index 40bc5bf..8fa5421 100644 --- a/src/widgets/modest-account-view.c +++ b/src/widgets/modest-account-view.c @@ -35,6 +35,7 @@ #include #include #include +#include #include #include @@ -48,7 +49,14 @@ static void modest_account_view_init (ModestAccountView *obj); static void modest_account_view_finalize (GObject *obj); static void modest_account_view_select_account (ModestAccountView *account_view, - const gchar* account_name); + const gchar* account_name); + +static void on_default_account_changed (ModestAccountMgr *mgr, + gpointer user_data); + +static void on_display_name_changed (ModestAccountMgr *self, + const gchar *account, + gpointer user_data); typedef enum { MODEST_ACCOUNT_VIEW_NAME_COLUMN, @@ -66,8 +74,7 @@ struct _ModestAccountViewPrivate { ModestAccountMgr *account_mgr; /* Signal handlers */ - gulong acc_inserted_handler, acc_removed_handler, - acc_busy_changed_handler, acc_changed_handler; + GSList *sig_handlers; }; #define MODEST_ACCOUNT_VIEW_GET_PRIVATE(o) (G_TYPE_INSTANCE_GET_PRIVATE((o), \ MODEST_TYPE_ACCOUNT_VIEW, \ @@ -118,11 +125,7 @@ modest_account_view_init (ModestAccountView *obj) priv = MODEST_ACCOUNT_VIEW_GET_PRIVATE(obj); - priv->account_mgr = NULL; - priv->acc_inserted_handler = 0; - priv->acc_removed_handler = 0; - priv->acc_busy_changed_handler = 0; - priv->acc_changed_handler = 0; + priv->sig_handlers = NULL; } static void @@ -132,26 +135,10 @@ modest_account_view_finalize (GObject *obj) priv = MODEST_ACCOUNT_VIEW_GET_PRIVATE(obj); - if (priv->account_mgr) { - if (g_signal_handler_is_connected (modest_runtime_get_account_store (), - priv->acc_inserted_handler)) - g_signal_handler_disconnect (modest_runtime_get_account_store (), - priv->acc_inserted_handler); + /* Disconnect signals */ + modest_signal_mgr_disconnect_all_and_destroy (priv->sig_handlers); - if (g_signal_handler_is_connected (modest_runtime_get_account_store (), - priv->acc_removed_handler)) - g_signal_handler_disconnect (modest_runtime_get_account_store (), - priv->acc_removed_handler); - - if (g_signal_handler_is_connected (modest_runtime_get_account_store (), - priv->acc_changed_handler)) - g_signal_handler_disconnect (modest_runtime_get_account_store (), - priv->acc_changed_handler); - - if (priv->acc_busy_changed_handler) - g_signal_handler_disconnect (priv->account_mgr, priv->acc_busy_changed_handler); - - + if (priv->account_mgr) { g_object_unref (G_OBJECT(priv->account_mgr)); priv->account_mgr = NULL; } @@ -159,22 +146,23 @@ modest_account_view_finalize (GObject *obj) G_OBJECT_CLASS(parent_class)->finalize (obj); } -/* Get the string for the last updated time. Result must be g_freed */ -static gchar* +/* Get the string for the last updated time. Result must NOT be g_freed */ +static const gchar* get_last_updated_string(ModestAccountMgr* account_mgr, ModestAccountData *account_data) { /* FIXME: let's assume that 'last update' applies to the store account... */ - gchar* last_updated_string; + const gchar* last_updated_string; time_t last_updated = account_data->store_account->last_updated; if (!modest_account_mgr_account_is_busy(account_mgr, account_data->account_name)) { if (last_updated > 0) last_updated_string = modest_text_utils_get_display_date(last_updated); else - last_updated_string = g_strdup (_("mcen_va_never")); + last_updated_string = _("mcen_va_never"); } else { /* FIXME: There should be a logical name in the UI specs */ - last_updated_string = g_strdup(_("...")); + last_updated_string = _("mcen_va_refreshing"); } + return last_updated_string; } @@ -220,24 +208,24 @@ update_account_view (ModestAccountMgr *account_mgr, ModestAccountView *view) if (account_data->store_account) { GtkTreeIter iter; - - gchar *last_updated_string = get_last_updated_string(account_mgr, account_data); + + /* don't free */ + const gchar *last_updated_string = get_last_updated_string(account_mgr, account_data); if (account_data->is_enabled) { + const gchar *proto_name; + + proto_name = modest_protocol_info_get_transport_store_protocol_name (account_data->store_account->proto); gtk_list_store_insert_with_values ( model, &iter, 0, - MODEST_ACCOUNT_VIEW_NAME_COLUMN, account_name, - MODEST_ACCOUNT_VIEW_DISPLAY_NAME_COLUMN, account_data->display_name, - MODEST_ACCOUNT_VIEW_IS_ENABLED_COLUMN, account_data->is_enabled, - MODEST_ACCOUNT_VIEW_IS_DEFAULT_COLUMN, account_data->is_default, - - MODEST_ACCOUNT_VIEW_PROTO_COLUMN, - modest_protocol_info_get_transport_store_protocol_name (account_data->store_account->proto), - + MODEST_ACCOUNT_VIEW_NAME_COLUMN, account_name, + MODEST_ACCOUNT_VIEW_DISPLAY_NAME_COLUMN, account_data->display_name, + MODEST_ACCOUNT_VIEW_IS_ENABLED_COLUMN, account_data->is_enabled, + MODEST_ACCOUNT_VIEW_IS_DEFAULT_COLUMN, account_data->is_default, + MODEST_ACCOUNT_VIEW_PROTO_COLUMN, proto_name, MODEST_ACCOUNT_VIEW_LAST_UPDATED_COLUMN, last_updated_string, -1); } - g_free (last_updated_string); } modest_account_mgr_free_account_data (account_mgr, account_data); @@ -262,30 +250,34 @@ on_account_busy_changed(ModestAccountMgr *account_mgr, { GtkListStore *model = GTK_LIST_STORE(gtk_tree_view_get_model (GTK_TREE_VIEW(self))); GtkTreeIter iter; - g_message(__FUNCTION__); + gboolean found = FALSE; + if (!gtk_tree_model_get_iter_first(GTK_TREE_MODEL(model), &iter)) return; - do - { + + do { gchar* cur_name; - gtk_tree_model_get(GTK_TREE_MODEL(model), &iter, MODEST_ACCOUNT_VIEW_NAME_COLUMN, - &cur_name, -1); - if (g_str_equal(cur_name, account_name)) - { + gtk_tree_model_get(GTK_TREE_MODEL(model), &iter, + MODEST_ACCOUNT_VIEW_NAME_COLUMN, + &cur_name, -1); + + if (g_str_equal(cur_name, account_name)) { ModestAccountData* account_data = modest_account_mgr_get_account_data (account_mgr, account_name); - if (!account_data) + if (!account_data) { + g_free (cur_name); return; - gchar* last_updated_string = get_last_updated_string(account_mgr, account_data); + } + const gchar* last_updated_string = get_last_updated_string(account_mgr, account_data); gtk_list_store_set(model, &iter, MODEST_ACCOUNT_VIEW_LAST_UPDATED_COLUMN, last_updated_string, -1); - g_free (last_updated_string); modest_account_mgr_free_account_data (account_mgr, account_data); - return; + found = TRUE; } - } - while (gtk_tree_model_iter_next(GTK_TREE_MODEL(model), &iter)); + g_free (cur_name); + + } while (!found && gtk_tree_model_iter_next(GTK_TREE_MODEL(model), &iter)); } static void @@ -338,31 +330,10 @@ on_account_changed (TnyAccountStore *account_store, self = MODEST_ACCOUNT_VIEW (user_data); priv = MODEST_ACCOUNT_VIEW_GET_PRIVATE (self); - g_warning ("account changed: %s", tny_account_get_id(account)); - /* Update account view */ /* update_account_view (priv->account_mgr, self); */ } - - -static gboolean -find_default_account(ModestAccountView *self, GtkTreeIter *iter) -{ - GtkTreeModel *model = gtk_tree_view_get_model (GTK_TREE_VIEW (self)); - gboolean result; - for (result = gtk_tree_model_get_iter_first(model, iter); - result == TRUE; result = gtk_tree_model_iter_next(model, iter)) - { - gboolean is_default; - gtk_tree_model_get (model, iter, MODEST_ACCOUNT_VIEW_IS_DEFAULT_COLUMN, &is_default, -1); - if(is_default) - return TRUE; - } - - return FALSE; -} - static void on_account_default_toggled (GtkCellRendererToggle *cell_renderer, gchar *path, @@ -388,17 +359,16 @@ on_account_default_toggled (GtkCellRendererToggle *cell_renderer, MODEST_ACCOUNT_VIEW_NAME_COLUMN, &account_name, -1); - /* Set this previously-non-default account as the default */ - if (modest_account_mgr_set_default_account (priv->account_mgr, account_name)) { - GtkTreeIter old_default_iter; + /* Set this previously-non-default account as the + default. We're not updating here the value of the + DEFAULT_COLUMN because we'll do it in the + "default_account_changed" signal handler. We do it like + this because that way the signal handler is useful also + when we're inserting a new account and there is no other + one defined, in that case the change of account is provoked + by the account mgr and not by a signal toggle.*/ + modest_account_mgr_set_default_account (priv->account_mgr, account_name); - find_default_account (self, &old_default_iter); - gtk_list_store_set (GTK_LIST_STORE (model), &old_default_iter, - MODEST_ACCOUNT_VIEW_IS_DEFAULT_COLUMN, FALSE, -1); - - gtk_list_store_set (GTK_LIST_STORE (model), &iter, - MODEST_ACCOUNT_VIEW_IS_DEFAULT_COLUMN, TRUE, -1); - } g_free (account_name); } @@ -463,9 +433,12 @@ init_view (ModestAccountView *self) g_object_set(G_OBJECT(toggle_renderer), "checkbox-mode", FALSE, NULL); #endif /* MODEST_HAVE_HILDON0_WIDGETS */ - g_signal_connect (G_OBJECT(toggle_renderer), "toggled", - G_CALLBACK(on_account_default_toggled), - self); + priv->sig_handlers = + modest_signal_mgr_connect (priv->sig_handlers, + G_OBJECT(toggle_renderer), + "toggled", + G_CALLBACK(on_account_default_toggled), + self); /* account name */ column = gtk_tree_view_column_new_with_attributes (_("mcen_ti_account"), text_renderer, "text", @@ -486,21 +459,42 @@ init_view (ModestAccountView *self) */ gtk_tree_view_set_headers_visible (GTK_TREE_VIEW(self), TRUE); - priv->acc_removed_handler = g_signal_connect (G_OBJECT (modest_runtime_get_account_store ()), - "account_removed", - G_CALLBACK(on_account_removed), self); - - priv->acc_inserted_handler = g_signal_connect (G_OBJECT (modest_runtime_get_account_store ()), - "account_inserted", - G_CALLBACK(on_account_inserted), self); - - priv->acc_changed_handler = g_signal_connect (G_OBJECT (modest_runtime_get_account_store ()), - "account_changed", - G_CALLBACK(on_account_changed), self); - - priv->acc_busy_changed_handler = g_signal_connect (G_OBJECT(priv->account_mgr), - "account_busy_changed", - G_CALLBACK(on_account_busy_changed), self); + priv->sig_handlers = + modest_signal_mgr_connect (priv->sig_handlers, + G_OBJECT (modest_runtime_get_account_store ()), + "account_removed", + G_CALLBACK(on_account_removed), + self); + priv->sig_handlers = + modest_signal_mgr_connect (priv->sig_handlers, + G_OBJECT (modest_runtime_get_account_store ()), + "account_inserted", + G_CALLBACK(on_account_inserted), + self); + priv->sig_handlers = + modest_signal_mgr_connect (priv->sig_handlers, + G_OBJECT (modest_runtime_get_account_store ()), + "account_changed", + G_CALLBACK(on_account_changed), + self); + priv->sig_handlers = + modest_signal_mgr_connect (priv->sig_handlers, + G_OBJECT(priv->account_mgr), + "account_busy_changed", + G_CALLBACK(on_account_busy_changed), + self); + priv->sig_handlers = + modest_signal_mgr_connect (priv->sig_handlers, + G_OBJECT(priv->account_mgr), + "default_account_changed", + G_CALLBACK(on_default_account_changed), + self); + priv->sig_handlers = + modest_signal_mgr_connect (priv->sig_handlers, + G_OBJECT(priv->account_mgr), + "display_name_changed", + G_CALLBACK(on_display_name_changed), + self); } @@ -576,8 +570,9 @@ on_model_foreach_select_account(GtkTreeModel *model, return FALSE; /* Keep walking the tree. */ } -static void modest_account_view_select_account (ModestAccountView *account_view, - const gchar* account_name) +static void +modest_account_view_select_account (ModestAccountView *account_view, + const gchar* account_name) { /* Create a state instance so we can send two items of data to the signal handler: */ ForEachData *state = g_new0 (ForEachData, 1); @@ -592,3 +587,50 @@ static void modest_account_view_select_account (ModestAccountView *account_view, g_free (state); } +static void +on_default_account_changed (ModestAccountMgr *mgr, + gpointer user_data) +{ + GtkTreeIter iter; + gchar *default_account_name; + GtkTreeModel *model = gtk_tree_view_get_model (GTK_TREE_VIEW (user_data)); + + if (!gtk_tree_model_get_iter_first(model, &iter)) + return; + + default_account_name = modest_account_mgr_get_default_account (mgr); + + do { + gboolean is_default; + gchar *name; + + gtk_tree_model_get (model, &iter, + MODEST_ACCOUNT_VIEW_NAME_COLUMN, &name, + MODEST_ACCOUNT_VIEW_IS_DEFAULT_COLUMN, &is_default, + -1); + + /* Update the default account column */ + if ((default_account_name != NULL) && (!strcmp (name, default_account_name))) + gtk_list_store_set (GTK_LIST_STORE (model), &iter, + MODEST_ACCOUNT_VIEW_IS_DEFAULT_COLUMN, TRUE, -1); + else + gtk_list_store_set (GTK_LIST_STORE (model), &iter, + MODEST_ACCOUNT_VIEW_IS_DEFAULT_COLUMN, FALSE, -1); + + g_free (name); + + } while (gtk_tree_model_iter_next(model, &iter)); + + /* Free and force a redraw */ + g_free (default_account_name); + gtk_widget_queue_draw (GTK_WIDGET (user_data)); +} + +static void +on_display_name_changed (ModestAccountMgr *mgr, + const gchar *account, + gpointer user_data) +{ + /* Update the view */ + update_account_view (mgr, MODEST_ACCOUNT_VIEW (user_data)); +}