Fix modest_tny_msg_header_get_all_recipients_list (in case from is empty)
[modest] / src / modest-widget-memory.c
index 6b4b13b..617dfd6 100644 (file)
  * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
+#include <modest-defs.h>
 #include <modest-widget-memory.h>
 #include <modest-widget-memory-priv.h>
 #include <modest-runtime.h>
 #include <modest-account-mgr-helpers.h>
 #include <modest-tny-platform-factory.h>
 #include <modest-tny-folder.h>
+#include <modest-init.h>
 #include <widgets/modest-header-view.h>
 #include <widgets/modest-msg-view.h>
 #include <widgets/modest-folder-view.h>
@@ -117,7 +119,7 @@ save_settings_widget (ModestConf *conf, GtkWidget *widget, const gchar *name)
 static gboolean
 restore_settings_widget (ModestConf *conf, GtkWidget *widget, const gchar *name)
 {
-       GtkRequisition req;
+       GtkRequisition req = {0, 0};
        gchar *key;
 
        key = _modest_widget_memory_get_keyname (name, MODEST_WIDGET_MEMORY_PARAM_HEIGHT);
@@ -193,7 +195,7 @@ restore_settings_window (ModestConf *conf, GtkWindow *win, const gchar *name)
           gtk_window_set_default_size() makes "drag-motion" signal
           report bad coordinates, so drag-and-drop do not work
           properly */
-#ifdef MODEST_PLATFORM_GNOME
+#ifdef MODEST_TOOLKIT_GTK
        if (height && width)
                gtk_window_set_default_size (win, width, height);
 #endif
@@ -218,13 +220,19 @@ static gboolean
 save_settings_paned (ModestConf *conf, GtkPaned *paned, const gchar *name)
 {
        gchar *key;
-       int pos;
+       gint pos;
+       gdouble percent;
 
-       pos = gtk_paned_get_position (paned);
-       
-       key = _modest_widget_memory_get_keyname (name, MODEST_WIDGET_MEMORY_PARAM_POS);
-       modest_conf_set_int (conf, key, pos, NULL);
-       g_free (key);
+       /* Don't save the paned position if it's not visible, 
+        * because it could not be correct: */
+       if (GTK_WIDGET_REALIZED (GTK_WIDGET (paned))) {
+               pos = gtk_paned_get_position (paned);
+               percent = (gdouble) (pos * 100) / (gdouble) GTK_WIDGET (paned)->allocation.width;
+
+               key = _modest_widget_memory_get_keyname (name, MODEST_WIDGET_MEMORY_PARAM_POS);
+               modest_conf_set_float (conf, key, percent, NULL);
+               g_free (key);
+       }
        
        return TRUE;
 }
@@ -234,16 +242,15 @@ static gboolean
 restore_settings_paned (ModestConf *conf, GtkPaned *paned, const gchar *name)
 {
        gchar *key;
-       int pos;
+       gdouble percent;
+       gint pos;
        
-       key = _modest_widget_memory_get_keyname (name, MODEST_WIDGET_MEMORY_PARAM_POS);
+       key = _modest_widget_memory_get_keyname (name, MODEST_WIDGET_MEMORY_PARAM_POS); 
+       percent = modest_conf_get_float (conf, key, NULL);
        
-       if (modest_conf_key_exists (conf, key, NULL)) {
-               pos = modest_conf_get_int (conf, key, NULL);
+       if (GTK_WIDGET_VISIBLE (GTK_WIDGET (paned)) && GTK_WIDGET_REALIZED (GTK_WIDGET (paned))) {
+               pos = GTK_WIDGET (paned)->allocation.width * percent /100;
                gtk_paned_set_position (paned, pos);
-       } else {
-               /* The initial position must follow the 30/70 rule */
-               gtk_paned_set_position (paned, GTK_WIDGET(paned)->requisition.width/3);
        }
 
        g_free (key);
@@ -256,6 +263,8 @@ save_settings_header_view (ModestConf *conf, ModestHeaderView *header_view,
                           const gchar *name)
 {
        gchar *key;
+       gchar *sort_key;
+       gchar *sort_value;
        GString *str;
        GList *cols, *cursor;
        TnyFolder *folder;
@@ -263,18 +272,32 @@ save_settings_header_view (ModestConf *conf, ModestHeaderView *header_view,
        ModestHeaderViewStyle style;
        gint sort_colid;
        GtkSortType sort_type;
+       gint sort_flag_id = 0;
        
        folder = modest_header_view_get_folder (header_view);
-       if (!folder || modest_header_view_is_empty (header_view))
+       if (!folder || modest_header_view_is_empty (header_view)) {
+               if (folder)
+                       g_object_unref (folder);
                return TRUE; /* no non-empty folder: no settings */
+       }
        
        type  = modest_tny_folder_guess_folder_type (folder);
+       if (type == TNY_FOLDER_TYPE_INVALID)
+               g_warning ("%s: BUG: TNY_FOLDER_TYPE_INVALID", __FUNCTION__);
+       
        style = modest_header_view_get_style   (header_view);
        
        key = _modest_widget_memory_get_keyname_with_double_type (name, type, style,
                                                                  MODEST_WIDGET_MEMORY_PARAM_COLUMN_WIDTH);
+       sort_key = _modest_widget_memory_get_keyname_with_double_type (name, type, style,
+                                                                      MODEST_WIDGET_MEMORY_PARAM_COLUMN_SORT);
 
        cursor = cols = modest_header_view_get_columns (header_view);
+       if (!cols) {
+               g_warning ("DEBUG: %s: modest_header_view_get_columns() returned NULL.",
+                        __FUNCTION__);
+       }
+       
        str = g_string_new (NULL);
 
        /* NOTE: the exact details of this format are important, as they
@@ -282,10 +305,12 @@ save_settings_header_view (ModestConf *conf, ModestHeaderView *header_view,
         */
        sort_colid = modest_header_view_get_sort_column_id (header_view, type); 
        sort_type = modest_header_view_get_sort_type (header_view, type); 
+
        while (cursor) {
 
                int col_id, width, sort;
                GtkTreeViewColumn *col;
+               int column_sort_flag;
                
                col    = GTK_TREE_VIEW_COLUMN (cursor->data);
                col_id = GPOINTER_TO_INT(g_object_get_data (G_OBJECT(col),
@@ -293,15 +318,34 @@ save_settings_header_view (ModestConf *conf, ModestHeaderView *header_view,
                width = gtk_tree_view_column_get_width (col);
                sort = 0;
                if (sort_colid == col_id)
-                       sort = (sort_type == GTK_SORT_ASCENDING) ? 1:-1;
-                       
+                       sort = (sort_type == GTK_SORT_ASCENDING) ? 1:0;
+               column_sort_flag = GPOINTER_TO_INT (g_object_get_data (G_OBJECT (col), MODEST_HEADER_VIEW_FLAG_SORT));
+               if (column_sort_flag != 0)
+                       sort_flag_id = column_sort_flag;
+               
                g_string_append_printf (str, "%d:%d:%d ", col_id, width, sort);
                cursor = g_list_next (cursor);
        }
 
-       modest_conf_set_string (conf, key, str->str, NULL);
+       if ((str->str == NULL) || (strlen(str->str) == 0)) {
+               /* TODO: Find out why this happens sometimes. */
+               g_warning ("DEBUG: %s: Attempting to write an empty value to "
+                       "gconf key %s. Preventing.", __FUNCTION__, key);
+       }
+       else
+               modest_conf_set_string (conf, key, str->str, NULL);
+
+       /* store current sort column for compact view */
+       if (sort_colid >= 0) {
+               sort_value = g_strdup_printf("%d:%d:%d", sort_colid, sort_type, sort_flag_id);
+               modest_conf_set_string (conf, sort_key, sort_value, NULL);
+               g_free (sort_value);
+       }
+
 
-       g_free (key);   
+
+       g_free (key);
+       g_free (sort_key);      
        g_string_free (str, TRUE);
        g_list_free (cols);
        g_object_unref (G_OBJECT (folder));
@@ -317,19 +361,34 @@ restore_settings_header_view (ModestConf *conf, ModestHeaderView *header_view,
        guint col, width;
        gint sort;
        gchar *key;
+       gchar *sort_key;
        TnyFolder *folder;
        TnyFolderType type;
        ModestHeaderViewStyle style;
-
+       gint sort_flag_id = 0;
+       gint sort_colid = -1, sort_type = GTK_SORT_DESCENDING;
+       
        folder = modest_header_view_get_folder (header_view);
-       if (!folder || modest_header_view_is_empty (header_view))
-               return TRUE; /* no non-empty folder: no settings */
+       if (!folder)
+               return TRUE; /* no folder: no settings */
        
-       type = modest_tny_folder_guess_folder_type (folder);
-       style = modest_header_view_get_style   (header_view);
+       type = modest_tny_folder_guess_folder_type (folder);    
+       if (type == TNY_FOLDER_TYPE_INVALID)
+               g_warning ("%s: BUG: TNY_FOLDER_TYPE_INVALID", __FUNCTION__);
+
+       style = modest_header_view_get_style (header_view);
 
        key = _modest_widget_memory_get_keyname_with_double_type (name, type, style,
                                                                  MODEST_WIDGET_MEMORY_PARAM_COLUMN_WIDTH);
+       sort_key = _modest_widget_memory_get_keyname_with_double_type (name, type, style,
+                                                                      MODEST_WIDGET_MEMORY_PARAM_COLUMN_SORT);
+
+       if (modest_conf_key_exists (conf, sort_key, NULL)) {
+               gchar *value = modest_conf_get_string (conf, sort_key, NULL);
+               sscanf (value, "%d:%d:%d", &sort_colid, &sort_type, &sort_flag_id);
+               g_free (value);
+       }
+
        if (modest_conf_key_exists (conf, key, NULL)) {
                
                gchar *data, *cursor;
@@ -340,6 +399,7 @@ restore_settings_header_view (ModestConf *conf, ModestHeaderView *header_view,
 
                cursor = data = modest_conf_get_string (conf, key, NULL);
                while (cursor && sscanf (cursor, "%d:%d:%d ", &col, &width, &sort) == 3) {
+
                        cols      = g_list_append (cols, GINT_TO_POINTER(col));
                        colwidths = g_list_append (colwidths, GINT_TO_POINTER(width));
                        colsortables = g_list_append (colsortables, GINT_TO_POINTER(sort));
@@ -347,32 +407,40 @@ restore_settings_header_view (ModestConf *conf, ModestHeaderView *header_view,
                }
                g_free (data);  
                
+               /* Use defaults if gconf has no, or empty information: */
+               /* We don't know why the value is empty sometimes. */
+               if (g_list_length(cols) == 0) {
+                       g_warning("%s: gconf key %s was empty. Using default column IDs.\n", 
+                               __FUNCTION__, key);
+                       g_list_free (cols);
+                       cols = NULL;
+               }
+               
+               if (!cols)
+                       cols = modest_init_get_default_header_view_column_ids (type, style);
+               
                if (cols) {
-                       GList *viewcolumns, *colcursor, *widthcursor, *sortablecursor;
+                       GList *viewcolumns, *colcursor, *widthcursor;
                        modest_header_view_set_columns (header_view, cols, type);
                        sortable = gtk_tree_view_get_model (GTK_TREE_VIEW (header_view));
 
                        widthcursor = colwidths;
-                       sortablecursor = colsortables;
                        colcursor = viewcolumns = gtk_tree_view_get_columns (GTK_TREE_VIEW(header_view));
-                       while (colcursor && widthcursor && sortablecursor) {
+                       while (colcursor && widthcursor) {
                                int width = GPOINTER_TO_INT(widthcursor->data);
-                               int sort = GPOINTER_TO_INT(sortablecursor->data);
+                               int view_column_id = GPOINTER_TO_INT (g_object_get_data (
+                                                                             G_OBJECT (colcursor->data), 
+                                                                             MODEST_HEADER_VIEW_COLUMN));
                                if (width > 0)
                                        gtk_tree_view_column_set_max_width(GTK_TREE_VIEW_COLUMN(colcursor->data),
                                                                           width);
-                               if (sort != 0) {
-                                       int colid = GPOINTER_TO_INT(g_object_get_data(G_OBJECT(colcursor->data), MODEST_HEADER_VIEW_COLUMN));
-                                       GtkSortType sort_type = (sort == 1) ? GTK_SORT_ASCENDING : GTK_SORT_DESCENDING; 
-                                       modest_header_view_set_sort_params (header_view, colid, sort_type, type);
-                                       gtk_tree_sortable_set_sort_column_id (GTK_TREE_SORTABLE(sortable),
-                                                                             colid,
-                                                                             sort_type);
-                                       gtk_tree_sortable_sort_column_changed (GTK_TREE_SORTABLE(sortable));
-                               }
+                               if (((view_column_id == MODEST_HEADER_VIEW_COLUMN_COMPACT_HEADER_IN) ||
+                                    (view_column_id == MODEST_HEADER_VIEW_COLUMN_COMPACT_HEADER_OUT)) &&
+                                   (sort_flag_id != 0))
+                                       g_object_set_data (G_OBJECT (colcursor->data), 
+                                                          MODEST_HEADER_VIEW_FLAG_SORT, GINT_TO_POINTER (sort_flag_id));
                                colcursor = g_list_next (colcursor);
                                widthcursor = g_list_next (widthcursor);
-                               sortablecursor = g_list_next (sortablecursor);
                        }
 
                        g_list_free (cols);
@@ -382,7 +450,20 @@ restore_settings_header_view (ModestConf *conf, ModestHeaderView *header_view,
                }
        }
 
+       if (sort_colid >= 0) {
+               GtkTreeModel *sortable = gtk_tree_view_get_model (GTK_TREE_VIEW (header_view));
+               if (sort_colid == TNY_GTK_HEADER_LIST_MODEL_FLAGS_COLUMN)
+                       modest_header_view_sort_by_column_id (header_view, 0, sort_type);
+               gtk_tree_sortable_set_sort_column_id (GTK_TREE_SORTABLE (sortable),
+                                                     sort_colid,
+                                                     sort_type);
+               modest_header_view_sort_by_column_id (header_view, sort_colid, sort_type);
+               gtk_tree_sortable_sort_column_changed (GTK_TREE_SORTABLE (sortable));
+       }
+
        g_free (key);
+       g_free (sort_key);
+       
        g_object_unref (G_OBJECT (folder));
 
        return TRUE;
@@ -394,19 +475,6 @@ static gboolean
 save_settings_folder_view (ModestConf *conf, ModestFolderView *folder_view,
                           const gchar *name)
 {
-       gchar *key;
-       const gchar* account_id;
-
-       /* Save the visible account */
-       key = _modest_widget_memory_get_keyname (name, "visible_server_account_id");
-
-       account_id = modest_folder_view_get_account_id_of_visible_server_account (folder_view);
-       if (account_id)
-               modest_conf_set_string (conf, key, account_id, NULL);
-       else
-               modest_conf_remove_key (conf, key, NULL);
-       g_free (key);
-
        return TRUE;
 }
 
@@ -415,44 +483,32 @@ restore_settings_folder_view (ModestConf *conf,
                              ModestFolderView *folder_view,
                              const gchar *name)
 {
-       gchar *key, *account_id;
-
-       /* Restore the visible account */
-       key = _modest_widget_memory_get_keyname (name, "visible_server_account_id");
-
-       if (modest_conf_key_exists (conf, key, NULL)) {
-               account_id = modest_conf_get_string (conf, key, NULL);
-               modest_folder_view_set_account_id_of_visible_server_account (folder_view, 
-                                                                            (const gchar *) account_id);
-               g_free (account_id);
-       } else {
-               ModestAccountMgr *mgr;
-               gchar *default_acc;
-
-               /* If there is no visible account id in the
-                  configuration then pick the default account as
-                  visible account */
-               mgr = modest_runtime_get_account_mgr ();
-               default_acc = modest_account_mgr_get_default_account (mgr);
-               if (default_acc) {
-                       ModestAccountData *acc_data;
-                       const gchar *server_acc_id;
-
-                       acc_data = modest_account_mgr_get_account_data (mgr, (const gchar*) default_acc);
-                       server_acc_id = (const gchar *) acc_data->store_account->account_name;
-
-                       modest_conf_set_string (conf, key, server_acc_id, NULL);
-                       modest_folder_view_set_account_id_of_visible_server_account (folder_view, server_acc_id);
-
-                       g_free (default_acc);
+       ModestAccountMgr *mgr;
+       gchar *default_acc;
+
+       /* Always show the default account as visible server account */
+       mgr = modest_runtime_get_account_mgr ();
+       default_acc = modest_account_mgr_get_default_account (mgr);
+       if (default_acc) {
+               ModestAccountSettings *settings;
+               const gchar *server_acc_id;
+
+               settings = modest_account_mgr_load_account_settings (mgr, (const gchar*) default_acc);
+               /* If there was any problem with the settings storage
+                  the settings could be NULL */
+               if (settings) {
+                       ModestServerAccountSettings *store_settings;
+                       store_settings = modest_account_settings_get_store_settings (settings);
+
+                       if (store_settings) {
+                               server_acc_id = modest_server_account_settings_get_account_name (store_settings);
+                               modest_folder_view_set_account_id_of_visible_server_account (folder_view, server_acc_id);
+                               g_object_unref (store_settings);
+                       }
+                       g_object_unref (settings);
                }
+               g_free (default_acc);
        }
-
-       g_free (key);
-
-       /* Select the first INBOX folder */
-       modest_folder_view_select_first_inbox_or_local (folder_view);
-
        return TRUE;
 }
 
@@ -506,7 +562,7 @@ modest_widget_memory_restore (ModestConf *conf, GObject *widget, const gchar *na
        g_return_val_if_fail (conf, FALSE);
        g_return_val_if_fail (widget, FALSE);
        g_return_val_if_fail (name, FALSE);
-       
+
        if (GTK_IS_WINDOW(widget))
                return restore_settings_window (conf, GTK_WINDOW(widget), name);
        else if (GTK_IS_PANED(widget))