X-Git-Url: http://git.maemo.org/git/?p=modest;a=blobdiff_plain;f=src%2Fwidgets%2Fmodest-folder-view.c;h=c4b85e6c514167a822c39133da0f9f1da2d5d2e5;hp=ec6f5c5805194d5980edccfaced2e53fbd1f971f;hb=dc6f85efbd256ef1e02449f075dd94b462dee791;hpb=64aff7ffa427145eea7436c30e0ff69793b96e7a diff --git a/src/widgets/modest-folder-view.c b/src/widgets/modest-folder-view.c index ec6f5c5..c4b85e6 100644 --- a/src/widgets/modest-folder-view.c +++ b/src/widgets/modest-folder-view.c @@ -35,12 +35,15 @@ #include #include #include +#include #include #include #include #include #include #include +#include +#include #include #include #include @@ -49,8 +52,8 @@ #include "modest-folder-view.h" #include #include -#include #include +#include /* 'private'/'protected' functions */ static void modest_folder_view_class_init (ModestFolderViewClass *klass); @@ -63,15 +66,16 @@ static void tny_account_store_view_init (gpointer g, static void modest_folder_view_set_account_store (TnyAccountStoreView *self, TnyAccountStore *account_store); -static gboolean update_model (ModestFolderView *self, - ModestTnyAccountStore *account_store); - static void on_selection_changed (GtkTreeSelection *sel, gpointer data); static void on_account_update (TnyAccountStore *account_store, const gchar *account, gpointer user_data); +static void on_account_removed (TnyAccountStore *self, + TnyAccount *account, + gpointer user_data); + static void on_accounts_reloaded (TnyAccountStore *store, gpointer user_data); @@ -121,6 +125,12 @@ static gint expand_row_timeout (gpointer data); static void setup_drag_and_drop (GtkTreeView *self); +static void _clipboard_set_selected_data (ModestFolderView *folder_view, gboolean delete); + +static void _clear_hidding_filter (ModestFolderView *folder_view); + + + enum { FOLDER_SELECTION_CHANGED_SIGNAL, FOLDER_DISPLAY_NAME_CHANGED_SIGNAL, @@ -135,14 +145,24 @@ struct _ModestFolderViewPrivate { gulong account_update_signal; gulong changed_signal; gulong accounts_reloaded_signal; + gulong account_removed_signal; + gulong conf_key_signal; - GtkTreeSelection *cur_selection; + /* not unref this object, its a singlenton */ + ModestEmailClipboard *clipboard; + + /* Filter tree model */ + gchar **hidding_ids; + guint n_selected; + TnyFolderStoreQuery *query; guint timer_expander; gchar *local_account_name; gchar *visible_account_id; ModestFolderViewStyle style; + + gboolean reselect; /* we use this to force a reselection of the INBOX */ }; #define MODEST_FOLDER_VIEW_GET_PRIVATE(o) \ (G_TYPE_INSTANCE_GET_PRIVATE((o), \ @@ -229,8 +249,6 @@ modest_folder_view_class_init (ModestFolderViewClass *klass) G_TYPE_NONE, 1, G_TYPE_STRING); } - - static void text_cell_data (GtkTreeViewColumn *column, GtkCellRenderer *renderer, GtkTreeModel *tree_model, GtkTreeIter *iter, gpointer data) @@ -238,8 +256,9 @@ text_cell_data (GtkTreeViewColumn *column, GtkCellRenderer *renderer, ModestFolderViewPrivate *priv; GObject *rendobj; gchar *fname = NULL; - gint unread, all; - TnyFolderType type; + gint unread = 0; + gint all = 0; + TnyFolderType type = TNY_FOLDER_TYPE_UNKNOWN; GObject *instance = NULL; g_return_if_fail (column); @@ -265,11 +284,13 @@ text_cell_data (GtkTreeViewColumn *column, GtkCellRenderer *renderer, priv = MODEST_FOLDER_VIEW_GET_PRIVATE (data); + gchar *item_name = NULL; + gint item_weight = 400; + if (type != TNY_FOLDER_TYPE_ROOT) { - gint number; - + gint number = 0; + if (modest_tny_folder_is_local_folder (TNY_FOLDER (instance))) { - TnyFolderType type; type = modest_tny_folder_get_local_folder_type (TNY_FOLDER (instance)); if (type != TNY_FOLDER_TYPE_UNKNOWN) { g_free (fname); @@ -277,53 +298,49 @@ text_cell_data (GtkTreeViewColumn *column, GtkCellRenderer *renderer, } } - /* Select the number to show */ + /* Select the number to show: the unread or unsent messages */ if ((type == TNY_FOLDER_TYPE_DRAFTS) || (type == TNY_FOLDER_TYPE_OUTBOX)) number = all; else number = unread; - - /* Use bold font style if there are unread messages */ - if (unread > 0) { - gchar *folder_title = g_strdup_printf ("%s (%d)", fname, unread); - g_object_set (rendobj,"text", folder_title, "weight", 800, NULL); - if (G_OBJECT (priv->cur_folder_store) == instance) - g_signal_emit (G_OBJECT(data), - signals[FOLDER_DISPLAY_NAME_CHANGED_SIGNAL], 0, - folder_title); - g_free (folder_title); + + /* Use bold font style if there are unread or unset messages */ + if (number > 0) { + item_name = g_strdup_printf ("%s (%d)", fname, number); + item_weight = 800; } else { - g_object_set (rendobj,"text", fname, "weight", 400, NULL); - if (G_OBJECT (priv->cur_folder_store) == instance) - g_signal_emit (G_OBJECT(data), - signals[FOLDER_DISPLAY_NAME_CHANGED_SIGNAL], 0, - fname); + item_name = g_strdup (fname); + item_weight = 400; } - - } else { - const gchar *account_name = NULL; - const gchar *account_id = NULL; - + + } else if (TNY_IS_ACCOUNT (instance)) { /* If it's a server account */ - account_id = tny_account_get_id (TNY_ACCOUNT (instance)); - if (!strcmp (account_id, MODEST_LOCAL_FOLDERS_ACCOUNT_ID)) { - account_name = priv->local_account_name; + if (modest_tny_account_is_virtual_local_folders ( + TNY_ACCOUNT (instance))) { + item_name = g_strdup (priv->local_account_name); + item_weight = 400; } else { - if (!strcmp (account_id, MODEST_MMC_ACCOUNT_ID)) { - /* TODO: get MMC card name */ - } else { - account_name = fname; - } + item_name = g_strdup (fname); + item_weight = 800; } - + } + + if (!item_name) + item_name = g_strdup ("unknown"); + + if (item_name && item_weight) { + /* Set the name in the treeview cell: */ + g_object_set (rendobj,"text", item_name, "weight", item_weight, NULL); + /* Notify display name observers */ - if (G_OBJECT (priv->cur_folder_store) == instance) + if (G_OBJECT (priv->cur_folder_store) == instance) { g_signal_emit (G_OBJECT(data), - signals[FOLDER_DISPLAY_NAME_CHANGED_SIGNAL], 0, - account_name); - - /* Use bold font style */ - g_object_set (rendobj,"text", account_name, "weight", 800, NULL); + signals[FOLDER_DISPLAY_NAME_CHANGED_SIGNAL], 0, + item_name); + } + + g_free (item_name); + } g_object_unref (G_OBJECT (instance)); @@ -338,10 +355,10 @@ icon_cell_data (GtkTreeViewColumn *column, GtkCellRenderer *renderer, { GObject *rendobj = NULL, *instance = NULL; GdkPixbuf *pixbuf = NULL; - TnyFolderType type; + TnyFolderType type = TNY_FOLDER_TYPE_UNKNOWN; gchar *fname = NULL; const gchar *account_id = NULL; - gint unread; + gint unread = 0; rendobj = G_OBJECT(renderer); gtk_tree_model_get (tree_model, iter, @@ -358,46 +375,57 @@ icon_cell_data (GtkTreeViewColumn *column, GtkCellRenderer *renderer, g_free (fname); return; } - - if (type == TNY_FOLDER_TYPE_NORMAL || type == TNY_FOLDER_TYPE_UNKNOWN) { - type = modest_tny_folder_guess_folder_type_from_name (fname); + + /* We include the MERGE type here because it's used to create + the local OUTBOX folder */ + if (type == TNY_FOLDER_TYPE_NORMAL || + type == TNY_FOLDER_TYPE_UNKNOWN || + type == TNY_FOLDER_TYPE_MERGE) { + type = modest_tny_folder_guess_folder_type (TNY_FOLDER (instance)); } switch (type) { case TNY_FOLDER_TYPE_ROOT: - account_id = tny_account_get_id (TNY_ACCOUNT (instance)); - if (!strcmp (account_id, MODEST_LOCAL_FOLDERS_ACCOUNT_ID)) { - pixbuf = modest_platform_get_icon (MODEST_FOLDER_ICON_LOCAL_FOLDERS); - } else { - if (!strcmp (account_id, MODEST_MMC_ACCOUNT_ID)) - pixbuf = modest_platform_get_icon (MODEST_FOLDER_ICON_MMC); - else - pixbuf = modest_platform_get_icon (MODEST_FOLDER_ICON_ACCOUNT); + if (TNY_IS_ACCOUNT (instance)) { + + if (modest_tny_account_is_virtual_local_folders ( + TNY_ACCOUNT (instance))) { + pixbuf = modest_platform_get_icon (MODEST_FOLDER_ICON_LOCAL_FOLDERS); + } + else { + account_id = tny_account_get_id (TNY_ACCOUNT (instance)); + + if (!strcmp (account_id, MODEST_MMC_ACCOUNT_ID)) + pixbuf = modest_platform_get_icon (MODEST_FOLDER_ICON_MMC); + else + pixbuf = modest_platform_get_icon (MODEST_FOLDER_ICON_ACCOUNT); + } } - break; + break; case TNY_FOLDER_TYPE_INBOX: - pixbuf = modest_platform_get_icon (MODEST_FOLDER_ICON_INBOX); - break; - case TNY_FOLDER_TYPE_OUTBOX: - pixbuf = modest_platform_get_icon (MODEST_FOLDER_ICON_OUTBOX); - break; - case TNY_FOLDER_TYPE_JUNK: - pixbuf = modest_platform_get_icon (MODEST_FOLDER_ICON_JUNK); - break; - case TNY_FOLDER_TYPE_SENT: - pixbuf = modest_platform_get_icon (MODEST_FOLDER_ICON_SENT); - break; + pixbuf = modest_platform_get_icon (MODEST_FOLDER_ICON_INBOX); + break; + case TNY_FOLDER_TYPE_OUTBOX: + pixbuf = modest_platform_get_icon (MODEST_FOLDER_ICON_OUTBOX); + break; + case TNY_FOLDER_TYPE_JUNK: + pixbuf = modest_platform_get_icon (MODEST_FOLDER_ICON_JUNK); + break; + case TNY_FOLDER_TYPE_SENT: + pixbuf = modest_platform_get_icon (MODEST_FOLDER_ICON_SENT); + break; case TNY_FOLDER_TYPE_TRASH: pixbuf = modest_platform_get_icon (MODEST_FOLDER_ICON_TRASH); - break; + break; case TNY_FOLDER_TYPE_DRAFTS: pixbuf = modest_platform_get_icon (MODEST_FOLDER_ICON_DRAFTS); - break; + break; case TNY_FOLDER_TYPE_NORMAL: - default: - pixbuf = modest_platform_get_icon (MODEST_FOLDER_ICON_NORMAL); + default: + pixbuf = modest_platform_get_icon (MODEST_FOLDER_ICON_NORMAL); break; - } + } + g_object_unref (G_OBJECT (instance)); g_free (fname); @@ -463,15 +491,18 @@ modest_folder_view_init (ModestFolderView *obj) conf = modest_runtime_get_conf(); priv->local_account_name = modest_conf_get_string (conf, MODEST_CONF_DEVICE_NAME, NULL); + /* Init email clipboard */ + priv->clipboard = modest_runtime_get_email_clipboard (); + priv->hidding_ids = NULL; + priv->n_selected = 0; + priv->reselect = FALSE; + /* Build treeview */ add_columns (GTK_WIDGET (obj)); /* Setup drag and drop */ setup_drag_and_drop (GTK_TREE_VIEW(obj)); - /* Restore conf */ - modest_widget_memory_restore (conf, G_OBJECT (obj), MODEST_CONF_FOLDER_VIEW_KEY); - /* Connect signals */ g_signal_connect (G_OBJECT (obj), "key-press-event", @@ -481,10 +512,10 @@ modest_folder_view_init (ModestFolderView *obj) * Track changes in the local account name (in the device it * will be the device name) */ - g_signal_connect (G_OBJECT(conf), - "key_changed", - G_CALLBACK(on_configuration_key_changed), obj); - + priv->conf_key_signal = + g_signal_connect (G_OBJECT(conf), + "key_changed", + G_CALLBACK(on_configuration_key_changed), obj); } static void @@ -517,6 +548,8 @@ modest_folder_view_finalize (GObject *obj) priv->account_update_signal); g_signal_handler_disconnect (G_OBJECT(priv->account_store), priv->accounts_reloaded_signal); + g_signal_handler_disconnect (G_OBJECT(priv->account_store), + priv->account_removed_signal); g_object_unref (G_OBJECT(priv->account_store)); priv->account_store = NULL; } @@ -533,6 +566,20 @@ modest_folder_view_finalize (GObject *obj) g_free (priv->local_account_name); g_free (priv->visible_account_id); + if (priv->conf_key_signal) { + g_signal_handler_disconnect (modest_runtime_get_conf (), + priv->conf_key_signal); + priv->conf_key_signal = 0; + } + + if (priv->cur_folder_store) { + g_object_unref (priv->cur_folder_store); + priv->cur_folder_store = NULL; + } + + /* Clear hidding array created by cut operation */ + _clear_hidding_filter (MODEST_FOLDER_VIEW (obj)); + G_OBJECT_CLASS(parent_class)->finalize (obj); } @@ -559,6 +606,10 @@ modest_folder_view_set_account_store (TnyAccountStoreView *self, TnyAccountStore priv->accounts_reloaded_signal)) g_signal_handler_disconnect (G_OBJECT (priv->account_store), priv->accounts_reloaded_signal); + if (g_signal_handler_is_connected (G_OBJECT (priv->account_store), + priv->account_removed_signal)) + g_signal_handler_disconnect (G_OBJECT (priv->account_store), + priv->account_removed_signal); g_object_unref (G_OBJECT (priv->account_store)); } @@ -569,23 +620,50 @@ modest_folder_view_set_account_store (TnyAccountStoreView *self, TnyAccountStore g_signal_connect (G_OBJECT(account_store), "account_update", G_CALLBACK (on_account_update), self); + priv->account_removed_signal = + g_signal_connect (G_OBJECT(account_store), "account_removed", + G_CALLBACK (on_account_removed), self); + priv->accounts_reloaded_signal = g_signal_connect (G_OBJECT(account_store), "accounts_reloaded", G_CALLBACK (on_accounts_reloaded), self); - - if (!update_model (MODEST_FOLDER_VIEW (self), - MODEST_TNY_ACCOUNT_STORE (priv->account_store))) - g_printerr ("modest: failed to update model\n"); + on_accounts_reloaded (account_store, (gpointer ) self); + g_object_unref (G_OBJECT (device)); } static void -on_account_update (TnyAccountStore *account_store, const gchar *account, +on_account_removed (TnyAccountStore *account_store, + TnyAccount *account, + gpointer user_data) +{ + ModestFolderView *self = MODEST_FOLDER_VIEW (user_data); + ModestFolderViewPrivate *priv; + + priv = MODEST_FOLDER_VIEW_GET_PRIVATE (self); + + /* If the removed account is the currently viewed one then + clear the configuration value. The new visible account will be the default account */ + if (!strcmp (priv->visible_account_id, tny_account_get_id (account))) { + modest_folder_view_set_account_id_of_visible_server_account (self, NULL); + } +} + +static void +on_account_update (TnyAccountStore *account_store, + const gchar *account, gpointer user_data) { - if (!update_model (MODEST_FOLDER_VIEW(user_data), - MODEST_TNY_ACCOUNT_STORE(account_store))) + ModestFolderView *self = MODEST_FOLDER_VIEW (user_data); + ModestFolderViewPrivate *priv; + + priv = MODEST_FOLDER_VIEW_GET_PRIVATE (self); + if (!priv->visible_account_id) + modest_widget_memory_restore (modest_runtime_get_conf(), G_OBJECT(self), + MODEST_CONF_FOLDER_VIEW_KEY); + + if (!modest_folder_view_update_model (self, account_store)) g_printerr ("modest: failed to update model for changes in '%s'", account); } @@ -594,8 +672,7 @@ static void on_accounts_reloaded (TnyAccountStore *account_store, gpointer user_data) { - update_model (MODEST_FOLDER_VIEW (user_data), - MODEST_TNY_ACCOUNT_STORE(account_store)); + modest_folder_view_update_model (MODEST_FOLDER_VIEW (user_data), account_store); } void @@ -616,6 +693,24 @@ modest_folder_view_set_title (ModestFolderView *self, const gchar *title) title != NULL); } +static gboolean +modest_folder_view_on_map (ModestFolderView *self, + GdkEventExpose *event, + gpointer data) +{ + ModestFolderViewPrivate *priv; + + priv = MODEST_FOLDER_VIEW_GET_PRIVATE (self); + + /* This won't happen often */ + if (G_UNLIKELY (priv->reselect)) { + /* Select the first inbox or the local account if not found */ + modest_folder_view_select_first_inbox_or_local (self); + priv->reselect = FALSE; + } + return FALSE; +} + GtkWidget* modest_folder_view_new (TnyFolderStoreQuery *query) { @@ -633,7 +728,9 @@ modest_folder_view_new (TnyFolderStoreQuery *query) priv->changed_signal = g_signal_connect (sel, "changed", G_CALLBACK (on_selection_changed), self); - return GTK_WIDGET(self); + g_signal_connect (self, "expose-event", G_CALLBACK (modest_folder_view_on_map), NULL); + + return GTK_WIDGET(self); } /* this feels dirty; any other way to expand all the root items? */ @@ -653,51 +750,87 @@ expand_root_items (ModestFolderView *self) /* * We use this function to implement the * MODEST_FOLDER_VIEW_STYLE_SHOW_ONE style. We only show the default - * account in this case + * account in this case, and the local folders. */ static gboolean filter_row (GtkTreeModel *model, GtkTreeIter *iter, gpointer data) { + ModestFolderViewPrivate *priv; gboolean retval = TRUE; - gint type; + TnyFolderType type = TNY_FOLDER_TYPE_UNKNOWN; GObject *instance = NULL; + const gchar *id = NULL; + guint i; + gboolean found = FALSE; + gboolean cleared = FALSE; + + g_return_val_if_fail (MODEST_IS_FOLDER_VIEW (data), FALSE); + priv = MODEST_FOLDER_VIEW_GET_PRIVATE (data); gtk_tree_model_get (model, iter, TNY_GTK_FOLDER_STORE_TREE_MODEL_TYPE_COLUMN, &type, TNY_GTK_FOLDER_STORE_TREE_MODEL_INSTANCE_COLUMN, &instance, -1); - if (type == TNY_FOLDER_TYPE_ROOT) { - TnyAccount *acc; - const gchar *account_id; - - acc = TNY_ACCOUNT (instance); - account_id = tny_account_get_id (acc); + /* Do not show if there is no instance, this could indeed + happen when the model is being modified while it's being + drawn. This could occur for example when moving folders + using drag&drop */ + if (!instance) + return FALSE; - if (strcmp (account_id, MODEST_LOCAL_FOLDERS_ACCOUNT_ID) && - strcmp (account_id, MODEST_MMC_ACCOUNT_ID)) { - ModestFolderViewPrivate *priv; + if (type == TNY_FOLDER_TYPE_ROOT) { + /* TNY_FOLDER_TYPE_ROOT means that the instance is an account instead of a folder. */ + if (TNY_IS_ACCOUNT (instance)) { + TnyAccount *acc = TNY_ACCOUNT (instance); + const gchar *account_id = tny_account_get_id (acc); + + /* If it isn't a special folder, + * don't show it unless it is the visible account: */ + if (!modest_tny_account_is_virtual_local_folders (acc) && + strcmp (account_id, MODEST_MMC_ACCOUNT_ID)) { + /* Show only the visible account id */ + if (priv->visible_account_id && strcmp (account_id, priv->visible_account_id)) + retval = FALSE; + } - /* Show only the visible account id */ - priv = MODEST_FOLDER_VIEW_GET_PRIVATE (data); - if (priv->visible_account_id && strcmp (account_id, priv->visible_account_id)) + /* Never show these to the user. They are merged into one folder + * in the local-folders account instead: */ + if (retval && MODEST_IS_TNY_OUTBOX_ACCOUNT (acc)) retval = FALSE; } } + + /* The virtual local-folders folder store is also shown by default. */ + + /* Check hiding (if necessary) */ + cleared = modest_email_clipboard_cleared (priv->clipboard); + if ((retval) && (!cleared) && (TNY_IS_FOLDER (instance))) { + id = tny_folder_get_id (TNY_FOLDER(instance)); + if (priv->hidding_ids != NULL) + for (i=0; i < priv->n_selected && !found; i++) + if (priv->hidding_ids[i] != NULL && id != NULL) + found = (!strcmp (priv->hidding_ids[i], id)); + + retval = !found; + } + /* Free */ g_object_unref (instance); return retval; } -static gboolean -update_model (ModestFolderView *self, ModestTnyAccountStore *account_store) +gboolean +modest_folder_view_update_model (ModestFolderView *self, + TnyAccountStore *account_store) { ModestFolderViewPrivate *priv; - TnyList *account_list; - GtkTreeModel *model; + GtkTreeModel *model /* , *old_model */; + /* TnyAccount *local_account; */ + TnyList *model_as_list; g_return_val_if_fail (account_store, FALSE); @@ -707,43 +840,62 @@ update_model (ModestFolderView *self, ModestTnyAccountStore *account_store) g_signal_emit (G_OBJECT(self), signals[FOLDER_SELECTION_CHANGED_SIGNAL], 0, NULL, TRUE); - + /* FIXME: the local accounts are not shown when the query selects only the subscribed folders. */ /* model = tny_gtk_folder_store_tree_model_new (TRUE, priv->query); */ model = tny_gtk_folder_store_tree_model_new (TRUE, NULL); - account_list = TNY_LIST(model); + + /* Deal with the model via its TnyList Interface, + * filling the TnyList via a get_accounts() call: */ + model_as_list = TNY_LIST(model); + /* Get the accounts: */ tny_account_store_get_accounts (TNY_ACCOUNT_STORE(account_store), - account_list, - TNY_ACCOUNT_STORE_STORE_ACCOUNTS); - if (account_list) { - GtkTreeModel *filter_model = NULL, *sortable = NULL; - - sortable = gtk_tree_model_sort_new_with_model (model); - gtk_tree_sortable_set_sort_column_id (GTK_TREE_SORTABLE(sortable), - TNY_GTK_FOLDER_STORE_TREE_MODEL_NAME_COLUMN, - GTK_SORT_ASCENDING); - gtk_tree_sortable_set_sort_func (GTK_TREE_SORTABLE (sortable), - TNY_GTK_FOLDER_STORE_TREE_MODEL_NAME_COLUMN, - cmp_rows, NULL, NULL); - - /* Create filter model */ - if (priv->style == MODEST_FOLDER_VIEW_STYLE_SHOW_ONE) { - filter_model = gtk_tree_model_filter_new (sortable, NULL); - gtk_tree_model_filter_set_visible_func (GTK_TREE_MODEL_FILTER (filter_model), - filter_row, - self, - NULL); - } + model_as_list, + TNY_ACCOUNT_STORE_STORE_ACCOUNTS); + g_object_unref (model_as_list); + model_as_list = NULL; + + GtkTreeModel *filter_model = NULL, *sortable = NULL; + + sortable = gtk_tree_model_sort_new_with_model (model); + gtk_tree_sortable_set_sort_column_id (GTK_TREE_SORTABLE(sortable), + TNY_GTK_FOLDER_STORE_TREE_MODEL_NAME_COLUMN, + GTK_SORT_ASCENDING); + gtk_tree_sortable_set_sort_func (GTK_TREE_SORTABLE (sortable), + TNY_GTK_FOLDER_STORE_TREE_MODEL_NAME_COLUMN, + cmp_rows, NULL, NULL); + + /* Create filter model */ + filter_model = gtk_tree_model_filter_new (sortable, NULL); + gtk_tree_model_filter_set_visible_func (GTK_TREE_MODEL_FILTER (filter_model), + filter_row, + self, + NULL); +/* if (priv->style == MODEST_FOLDER_VIEW_STYLE_SHOW_ONE) { */ +/* filter_model = gtk_tree_model_filter_new (sortable, NULL); */ +/* gtk_tree_model_filter_set_visible_func (GTK_TREE_MODEL_FILTER (filter_model), */ +/* filter_row, */ +/* self, */ +/* NULL); */ +/* } */ + + /* Set new model */ + gtk_tree_view_set_model (GTK_TREE_VIEW(self), filter_model); +/* gtk_tree_view_set_model (GTK_TREE_VIEW(self), */ +/* (filter_model) ? filter_model : sortable); */ + + g_object_unref (model); + g_object_unref (filter_model); +/* if (filter_model) */ +/* g_object_unref (filter_model); */ + + g_object_unref (sortable); - /* Set new model */ - gtk_tree_view_set_model (GTK_TREE_VIEW(self), - (filter_model) ? filter_model : sortable); - expand_root_items (self); /* expand all account folders */ - g_object_unref (account_list); - } - + /* Force a reselection of the INBOX next time the widget is shown */ + priv->reselect = TRUE; + return TRUE; } @@ -756,36 +908,27 @@ on_selection_changed (GtkTreeSelection *sel, gpointer user_data) GtkTreeIter iter; ModestFolderView *tree_view; ModestFolderViewPrivate *priv; - gint type; g_return_if_fail (sel); g_return_if_fail (user_data); priv = MODEST_FOLDER_VIEW_GET_PRIVATE(user_data); - priv->cur_selection = sel; - - /* folder was _un_selected if true */ - if (!gtk_tree_selection_get_selected (sel, &model, &iter)) { - if (priv->cur_folder_store) - g_object_unref (priv->cur_folder_store); - priv->cur_folder_store = NULL; - /* Notify the display name observers */ - g_signal_emit (G_OBJECT(user_data), - signals[FOLDER_DISPLAY_NAME_CHANGED_SIGNAL], 0, - NULL); + if(!gtk_tree_selection_get_selected (sel, &model, &iter)) return; - } - tree_view = MODEST_FOLDER_VIEW (user_data); + /* Notify the display name observers */ + g_signal_emit (G_OBJECT(user_data), + signals[FOLDER_DISPLAY_NAME_CHANGED_SIGNAL], 0, + NULL); + tree_view = MODEST_FOLDER_VIEW (user_data); gtk_tree_model_get (model, &iter, - TNY_GTK_FOLDER_STORE_TREE_MODEL_TYPE_COLUMN, &type, TNY_GTK_FOLDER_STORE_TREE_MODEL_INSTANCE_COLUMN, &folder, -1); /* If the folder is the same do not notify */ - if (priv->cur_folder_store == folder) { + if (priv->cur_folder_store == folder && folder) { g_object_unref (folder); return; } @@ -795,6 +938,7 @@ on_selection_changed (GtkTreeSelection *sel, gpointer user_data) g_signal_emit (G_OBJECT(tree_view), signals[FOLDER_SELECTION_CHANGED_SIGNAL], 0, priv->cur_folder_store, FALSE); g_object_unref (priv->cur_folder_store); + priv->cur_folder_store = NULL; } /* New current references */ @@ -803,7 +947,7 @@ on_selection_changed (GtkTreeSelection *sel, gpointer user_data) /* New folder has been selected */ g_signal_emit (G_OBJECT(tree_view), signals[FOLDER_SELECTION_CHANGED_SIGNAL], - 0, folder, TRUE); + 0, priv->cur_folder_store, TRUE); } TnyFolderStore * @@ -820,8 +964,32 @@ modest_folder_view_get_selected (ModestFolderView *self) return priv->cur_folder_store; } +static gint +get_cmp_rows_type_pos (GObject *folder) +{ + /* Remote accounts -> Local account -> MMC account .*/ + /* 0, 1, 2 */ + + if (TNY_IS_ACCOUNT (folder) && + modest_tny_account_is_virtual_local_folders ( + TNY_ACCOUNT (folder))) { + return 1; + } else if (TNY_IS_ACCOUNT (folder)) { + TnyAccount *account = TNY_ACCOUNT (folder); + const gchar *account_id = tny_account_get_id (account); + if (!strcmp (account_id, MODEST_MMC_ACCOUNT_ID)) + return 2; + else + return 0; + } + else { + printf ("DEBUG: %s: unexpected type.\n", __FUNCTION__); + return -1; /* Should never happen */ + } +} + /* - * This function orders the mail accounts following the next rules + * This function orders the mail accounts according to these rules: * 1st - remote accounts * 2nd - local account * 3rd - MMC account @@ -831,9 +999,11 @@ cmp_rows (GtkTreeModel *tree_model, GtkTreeIter *iter1, GtkTreeIter *iter2, gpointer user_data) { gint cmp; - gchar *name1, *name2; - TnyFolderType type; - TnyFolder *folder1, *folder2; + gchar *name1 = NULL; + gchar *name2 = NULL; + TnyFolderType type = TNY_FOLDER_TYPE_UNKNOWN; + GObject *folder1 = NULL; + GObject *folder2 = NULL; gtk_tree_model_get (tree_model, iter1, TNY_GTK_FOLDER_STORE_TREE_MODEL_NAME_COLUMN, &name1, @@ -845,29 +1015,45 @@ cmp_rows (GtkTreeModel *tree_model, GtkTreeIter *iter1, GtkTreeIter *iter2, TNY_GTK_FOLDER_STORE_TREE_MODEL_INSTANCE_COLUMN, &folder2, -1); - /* Order must be: Remote accounts -> Local account -> MMC account */ if (type == TNY_FOLDER_TYPE_ROOT) { - const gchar *account_id = tny_account_get_id (TNY_ACCOUNT (folder1)); - const gchar *account_id2 = tny_account_get_id (TNY_ACCOUNT (folder2)); - - if (!strcmp (account_id, MODEST_MMC_ACCOUNT_ID)) - cmp = +1; + /* Compare the types, so that + * Remote accounts -> Local account -> MMC account .*/ + const gint pos1 = get_cmp_rows_type_pos (folder1); + const gint pos2 = get_cmp_rows_type_pos (folder2); + /* printf ("DEBUG: %s:\n type1=%s, pos1=%d\n type2=%s, pos2=%d\n", + __FUNCTION__, G_OBJECT_TYPE_NAME(folder1), pos1, G_OBJECT_TYPE_NAME(folder2), pos2); */ + if (pos1 < pos2) + cmp = -1; + else if (pos1 > pos2) + cmp = 1; else { - if (!strcmp (account_id, MODEST_LOCAL_FOLDERS_ACCOUNT_ID)) { - if (!strcmp (account_id2, MODEST_MMC_ACCOUNT_ID)) - cmp = -1; - else - cmp = +1; - } else { - if (!strcmp (account_id2, MODEST_LOCAL_FOLDERS_ACCOUNT_ID) || - !strcmp (account_id2, MODEST_MMC_ACCOUNT_ID)) - cmp = -1; - else - cmp = modest_text_utils_utf8_strcmp (name1, name2, TRUE); - } + /* Compare items of the same type: */ + + TnyAccount *account1 = NULL; + if (TNY_IS_ACCOUNT (folder1)) + account1 = TNY_ACCOUNT (folder1); + + TnyAccount *account2 = NULL; + if (TNY_IS_ACCOUNT (folder2)) + account2 = TNY_ACCOUNT (folder2); + + const gchar *account_id = account1 ? tny_account_get_id (account1) : NULL; + const gchar *account_id2 = account2 ? tny_account_get_id (account2) : NULL; + + if (!account_id && !account_id2) + return 0; + else if (!account_id) + return -1; + else if (!account_id2) + return +1; + else if (!strcmp (account_id, MODEST_MMC_ACCOUNT_ID)) + cmp = +1; + else + cmp = modest_text_utils_utf8_strcmp (name1, name2, TRUE); } - } else + } else { cmp = modest_text_utils_utf8_strcmp (name1, name2, TRUE); + } if (folder1) g_object_unref(G_OBJECT(folder1)); @@ -898,25 +1084,18 @@ on_drag_data_get (GtkWidget *widget, gpointer data) { GtkTreeSelection *selection; - GtkTreeModel *model_sort, *model; + GtkTreeModel *model; GtkTreeIter iter; - GtkTreePath *source_row_sort; GtkTreePath *source_row; selection = gtk_tree_view_get_selection (GTK_TREE_VIEW (widget)); - gtk_tree_selection_get_selected (selection, &model_sort, &iter); - source_row_sort = gtk_tree_model_get_path (model_sort, &iter); - - /* Get the unsorted path and model */ - model = gtk_tree_model_sort_get_model (GTK_TREE_MODEL_SORT (model_sort)); - source_row = gtk_tree_model_sort_convert_path_to_child_path (GTK_TREE_MODEL_SORT (model_sort), - source_row_sort); + gtk_tree_selection_get_selected (selection, &model, &iter); + source_row = gtk_tree_model_get_path (model, &iter); gtk_tree_set_row_drag_data (selection_data, model, source_row); - gtk_tree_path_free (source_row_sort); gtk_tree_path_free (source_row); } @@ -938,18 +1117,19 @@ typedef struct _DndHelper { * and drop action */ static void -on_progress_changed (ModestMailOperation *mail_op, gpointer user_data) +on_progress_changed (ModestMailOperation *mail_op, + ModestMailOperationState *state, + gpointer user_data) { gboolean success; DndHelper *helper; helper = (DndHelper *) user_data; - if (!modest_mail_operation_is_finished (mail_op)) + if (!state->finished) return; - if (modest_mail_operation_get_status (mail_op) == - MODEST_MAIL_OPERATION_STATUS_SUCCESS) { + if (state->status == MODEST_MAIL_OPERATION_STATUS_SUCCESS) { success = TRUE; } else { success = FALSE; @@ -975,43 +1155,63 @@ drag_and_drop_from_header_view (GtkTreeModel *source_model, GtkTreePath *dest_row, DndHelper *helper) { - TnyList *headers; - TnyHeader *header; - TnyFolder *folder; - ModestMailOperation *mail_op; + TnyList *headers = NULL; + TnyHeader *header = NULL; + TnyFolder *folder = NULL; + ModestMailOperation *mail_op = NULL; GtkTreeIter source_iter, dest_iter; + g_return_if_fail (GTK_IS_TREE_MODEL(source_model)); + g_return_if_fail (GTK_IS_TREE_MODEL(dest_model)); + g_return_if_fail (dest_row); + g_return_if_fail (helper); + /* Get header */ gtk_tree_model_get_iter (source_model, &source_iter, helper->source_row); gtk_tree_model_get (source_model, &source_iter, TNY_GTK_HEADER_LIST_MODEL_INSTANCE_COLUMN, &header, -1); - + if (!TNY_IS_HEADER(header)) { + g_warning ("BUG: %s could not get a valid header", __FUNCTION__); + goto cleanup; + } + /* Get Folder */ gtk_tree_model_get_iter (dest_model, &dest_iter, dest_row); gtk_tree_model_get (dest_model, &dest_iter, TNY_GTK_FOLDER_STORE_TREE_MODEL_INSTANCE_COLUMN, &folder, -1); - /* Transfer message */ - mail_op = modest_mail_operation_new (MODEST_MAIL_OPERATION_ID_RECEIVE, NULL); + if (!TNY_IS_FOLDER(folder)) { + g_warning ("BUG: %s could not get a valid folder", __FUNCTION__); + goto cleanup; + } + /* Transfer message */ + mail_op = modest_mail_operation_new (MODEST_MAIL_OPERATION_TYPE_RECEIVE, NULL); modest_mail_operation_queue_add (modest_runtime_get_mail_operation_queue (), mail_op); g_signal_connect (G_OBJECT (mail_op), "progress-changed", G_CALLBACK (on_progress_changed), helper); - /* FIXME: I replaced this because the API changed, but D&D - should be reviewed in order to allow multiple drags*/ headers = tny_simple_list_new (); tny_list_append (headers, G_OBJECT (header)); - g_object_unref (header); - modest_mail_operation_xfer_msgs (mail_op, headers, folder, helper->delete_source); - + modest_mail_operation_xfer_msgs (mail_op, + headers, + folder, + helper->delete_source, + NULL, NULL); + /* Frees */ - g_object_unref (G_OBJECT (mail_op)); - g_object_unref (G_OBJECT (header)); - g_object_unref (G_OBJECT (folder)); +cleanup: + if (G_IS_OBJECT(mail_op)) + g_object_unref (G_OBJECT (mail_op)); + if (G_IS_OBJECT(header)) + g_object_unref (G_OBJECT (header)); + if (G_IS_OBJECT(folder)) + g_object_unref (G_OBJECT (folder)); + if (G_IS_OBJECT(headers)) + g_object_unref (headers); } /* @@ -1032,10 +1232,11 @@ drag_and_drop_from_folder_view (GtkTreeModel *source_model, TnyFolder *folder; /* Check if the drag is possible */ - if (!gtk_tree_path_compare (helper->source_row, dest_row) || - !gtk_tree_drag_dest_row_drop_possible (GTK_TREE_DRAG_DEST (dest_model), - dest_row, - selection_data)) { +/* if (!gtk_tree_path_compare (helper->source_row, dest_row) || */ +/* !gtk_tree_drag_dest_row_drop_possible (GTK_TREE_DRAG_DEST (dest_model), */ +/* dest_row, */ +/* selection_data)) { */ + if (!gtk_tree_path_compare (helper->source_row, dest_row)) { gtk_drag_finish (helper->context, FALSE, FALSE, helper->time); gtk_tree_path_free (helper->source_row); @@ -1054,7 +1255,10 @@ drag_and_drop_from_folder_view (GtkTreeModel *source_model, &folder, -1); /* Do the mail operation */ - mail_op = modest_mail_operation_new (MODEST_MAIL_OPERATION_ID_RECEIVE, NULL); + mail_op = modest_mail_operation_new_with_error_handling (MODEST_MAIL_OPERATION_TYPE_RECEIVE, + NULL, + modest_ui_actions_move_folder_error_handler, + NULL); modest_mail_operation_queue_add (modest_runtime_get_mail_operation_queue (), mail_op); g_signal_connect (G_OBJECT (mail_op), "progress-changed", @@ -1064,7 +1268,7 @@ drag_and_drop_from_folder_view (GtkTreeModel *source_model, folder, parent_folder, helper->delete_source); - + /* Frees */ g_object_unref (G_OBJECT (parent_folder)); g_object_unref (G_OBJECT (folder)); @@ -1088,35 +1292,45 @@ on_drag_data_received (GtkWidget *widget, gpointer data) { GtkWidget *source_widget; - GtkTreeModel *model_sort, *dest_model, *source_model; - GtkTreePath *source_row, *dest_row, *child_dest_row; + GtkTreeModel *dest_model, *source_model; + GtkTreePath *source_row, *dest_row; GtkTreeViewDropPosition pos; gboolean success = FALSE, delete_source = FALSE; - DndHelper *helper; + DndHelper *helper = NULL; /* Do not allow further process */ g_signal_stop_emission_by_name (widget, "drag-data-received"); + source_widget = gtk_drag_get_source_widget (context); /* Get the action */ - if (context->action == GDK_ACTION_MOVE) + if (context->action == GDK_ACTION_MOVE) { delete_source = TRUE; + /* Notify that there is no folder selected. We need to + do this in order to update the headers view (and + its monitors, because when moving, the old folder + won't longer exist. We can not wait for the end of + the operation, because the operation won't start if + the folder is in use */ + if (source_widget == widget) + g_signal_emit (G_OBJECT (widget), + signals[FOLDER_SELECTION_CHANGED_SIGNAL], 0, NULL, TRUE); + } + /* Check if the get_data failed */ if (selection_data == NULL || selection_data->length < 0) gtk_drag_finish (context, success, FALSE, time); /* Get the models */ - source_widget = gtk_drag_get_source_widget (context); gtk_tree_get_row_drag_data (selection_data, &source_model, &source_row); - model_sort = gtk_tree_view_get_model (GTK_TREE_VIEW (widget)); /* Select the destination model */ if (source_widget == widget) { dest_model = source_model; } else { - dest_model = gtk_tree_model_sort_get_model (GTK_TREE_MODEL_SORT (model_sort)); + dest_model = gtk_tree_view_get_model (GTK_TREE_VIEW (widget)); } /* Get the path to the destination row. Can not call @@ -1136,32 +1350,26 @@ on_drag_data_received (GtkWidget *widget, helper->context = context; helper->time = time; - /* Get path from the unsorted model */ - child_dest_row = - gtk_tree_model_sort_convert_path_to_child_path (GTK_TREE_MODEL_SORT (model_sort), - dest_row); - gtk_tree_path_free (dest_row); - /* Drags from the header view */ if (source_widget != widget) { drag_and_drop_from_header_view (source_model, dest_model, - child_dest_row, + dest_row, helper); } else { drag_and_drop_from_folder_view (source_model, dest_model, - child_dest_row, + dest_row, selection_data, helper); } /* Frees */ gtk_tree_path_free (source_row); - gtk_tree_path_free (child_dest_row); + gtk_tree_path_free (dest_row); } /* @@ -1266,7 +1474,7 @@ on_drag_motion (GtkWidget *widget, /* Do not allow drops between folders */ if (!dest_row || - pos == GTK_TREE_VIEW_DROP_BEFORE || + pos == GTK_TREE_VIEW_DROP_BEFORE || pos == GTK_TREE_VIEW_DROP_AFTER) { gtk_tree_view_set_drag_dest_row(GTK_TREE_VIEW (widget), NULL, 0); gdk_drag_status(context, 0, time); @@ -1281,7 +1489,6 @@ on_drag_motion (GtkWidget *widget, gtk_tree_view_set_drag_dest_row (GTK_TREE_VIEW (widget), dest_row, pos); priv->timer_expander = g_timeout_add (500, expand_row_timeout, widget); } - gtk_tree_path_free (dest_row); /* Select the desired action. By default we pick MOVE */ suggested_action = GDK_ACTION_MOVE; @@ -1296,6 +1503,8 @@ on_drag_motion (GtkWidget *widget, gdk_drag_status(context, GDK_ACTION_DEFAULT, time); out: + if (dest_row) + gtk_tree_path_free (dest_row); g_signal_stop_emission_by_name (widget, "drag-motion"); return valid_location; } @@ -1408,6 +1617,7 @@ on_configuration_key_changed (ModestConf* conf, if (!key) return; + g_return_if_fail (MODEST_IS_FOLDER_VIEW (self)); priv = MODEST_FOLDER_VIEW_GET_PRIVATE(self); if (!strcmp (key, MODEST_CONF_DEVICE_NAME)) { @@ -1446,25 +1656,29 @@ modest_folder_view_set_account_id_of_visible_server_account (ModestFolderView *s const gchar *account_id) { ModestFolderViewPrivate *priv; - ModestConf *conf; GtkTreeModel *model; g_return_if_fail (self); priv = MODEST_FOLDER_VIEW_GET_PRIVATE(self); - if (priv->visible_account_id) + /* This will be used by the filter_row callback, + * to decided which rows to show: */ + if (priv->visible_account_id) { g_free (priv->visible_account_id); - priv->visible_account_id = g_strdup (account_id); - - /* Save preferences */ - conf = modest_runtime_get_conf (); - modest_widget_memory_save (conf, G_OBJECT (self), MODEST_CONF_FOLDER_VIEW_KEY); + priv->visible_account_id = NULL; + } + if (account_id) + priv->visible_account_id = g_strdup (account_id); /* Refilter */ model = gtk_tree_view_get_model (GTK_TREE_VIEW (self)); if (GTK_IS_TREE_MODEL_FILTER (model)) gtk_tree_model_filter_refilter (GTK_TREE_MODEL_FILTER (model)); + + /* Save settings to gconf */ + modest_widget_memory_save (modest_runtime_get_conf (), G_OBJECT(self), + MODEST_CONF_FOLDER_VIEW_KEY); } const gchar * @@ -1478,3 +1692,152 @@ modest_folder_view_get_account_id_of_visible_server_account (ModestFolderView *s return (const gchar *) priv->visible_account_id; } + +static gboolean +find_inbox_iter (GtkTreeModel *model, GtkTreeIter *iter, GtkTreeIter *inbox_iter) +{ + do { + GtkTreeIter child; + TnyFolderType type = TNY_FOLDER_TYPE_UNKNOWN; + gchar *name = NULL; + + gtk_tree_model_get (model, iter, + TNY_GTK_FOLDER_STORE_TREE_MODEL_NAME_COLUMN, &name, + TNY_GTK_FOLDER_STORE_TREE_MODEL_TYPE_COLUMN, + &type, -1); + + /* + printf ("DEBUG: %s: name=%s, type=%d, TNY_FOLDER_TYPE_INBOX=%d\n", + __FUNCTION__, name, type, TNY_FOLDER_TYPE_INBOX); + */ + + gboolean result = FALSE; + if (type == TNY_FOLDER_TYPE_INBOX) { + result = TRUE; + } else if (type == TNY_FOLDER_TYPE_NORMAL) { + /* tinymail's camel implementation only provides TNY_FOLDER_TYPE_NORMAL + * when getting folders from the cache, before connectin, so we do + * an extra check. We could fix this in tinymail, but it's easier + * to do here. + */ + if (strcmp (name, "Inbox") == 0) + result = TRUE; + } + + g_free (name); + + if (result) { + *inbox_iter = *iter; + return TRUE; + } + + if (gtk_tree_model_iter_children (model, &child, iter)) { + if (find_inbox_iter (model, &child, inbox_iter)) + return TRUE; + } + + } while (gtk_tree_model_iter_next (model, iter)); + + return FALSE; +} + +void +modest_folder_view_select_first_inbox_or_local (ModestFolderView *self) +{ + GtkTreeModel *model; + GtkTreeIter iter, inbox_iter; + GtkTreeSelection *sel; + +/* /\* Do not set it if the folder view was not painted *\/ */ +/* if (!GTK_WIDGET_MAPPED (self)) */ +/* return; */ + + model = gtk_tree_view_get_model (GTK_TREE_VIEW (self)); + if (!model) + return; + + expand_root_items (self); + sel = gtk_tree_view_get_selection (GTK_TREE_VIEW (self)); + + gtk_tree_model_get_iter_first (model, &iter); + if (find_inbox_iter (model, &iter, &inbox_iter)) { + gtk_tree_selection_select_iter (sel, &inbox_iter); + } + else { + gtk_tree_model_get_iter_first (model, &iter); + gtk_tree_selection_select_iter (sel, &iter); + } +} + +void +modest_folder_view_copy_selection (ModestFolderView *folder_view) +{ + /* Copy selection */ + _clipboard_set_selected_data (folder_view, FALSE); +} + +void +modest_folder_view_cut_selection (ModestFolderView *folder_view) +{ + ModestFolderViewPrivate *priv = NULL; + GtkTreeModel *model = NULL; + const gchar **hidding = NULL; + guint i, n_selected; + + g_return_if_fail (MODEST_IS_FOLDER_VIEW (folder_view)); + priv = MODEST_FOLDER_VIEW_GET_PRIVATE (folder_view); + + /* Copy selection */ + _clipboard_set_selected_data (folder_view, TRUE); + + /* Get hidding ids */ + hidding = modest_email_clipboard_get_hidding_ids (priv->clipboard, &n_selected); + + /* Clear hidding array created by previous cut operation */ + _clear_hidding_filter (MODEST_FOLDER_VIEW (folder_view)); + + /* Copy hidding array */ + priv->n_selected = n_selected; + priv->hidding_ids = g_malloc0(sizeof(gchar *) * n_selected); + for (i=0; i < n_selected; i++) + priv->hidding_ids[i] = g_strdup(hidding[i]); + + /* Hide cut folders */ + model = gtk_tree_view_get_model (GTK_TREE_VIEW (folder_view)); + gtk_tree_model_filter_refilter (GTK_TREE_MODEL_FILTER (model)); +} + +static void +_clipboard_set_selected_data (ModestFolderView *folder_view, + gboolean delete) +{ + ModestFolderViewPrivate *priv = NULL; + TnyFolderStore *folder = NULL; + + g_return_if_fail (MODEST_IS_FOLDER_VIEW (folder_view)); + priv = MODEST_FOLDER_VIEW_GET_PRIVATE (folder_view); + + /* Set selected data on clipboard */ + g_return_if_fail (MODEST_IS_EMAIL_CLIPBOARD (priv->clipboard)); + folder = modest_folder_view_get_selected (folder_view); + modest_email_clipboard_set_data (priv->clipboard, TNY_FOLDER(folder), NULL, delete); + + /* Free */ + g_object_unref (folder); +} + +static void +_clear_hidding_filter (ModestFolderView *folder_view) +{ + ModestFolderViewPrivate *priv; + guint i; + + g_return_if_fail (MODEST_IS_FOLDER_VIEW (folder_view)); + priv = MODEST_FOLDER_VIEW_GET_PRIVATE(folder_view); + + if (priv->hidding_ids != NULL) { + for (i=0; i < priv->n_selected; i++) + g_free (priv->hidding_ids[i]); + g_free(priv->hidding_ids); + } +}