* don't do a regexp-search when doing search-in-page.
[modest] / src / dbus_api / modest-dbus-callbacks.c
index ff8de9e..f913111 100644 (file)
@@ -53,6 +53,7 @@
 #include <tny-list.h>
 #include <tny-iterator.h>
 #include <tny-simple-list.h>
+#include <tny-merge-folder.h>
 
 typedef struct 
 {
@@ -377,7 +378,7 @@ on_idle_compose_mail(gpointer user_data)
 
                                        list = g_strsplit(idle_data->attachments, ",", 0);
                                        for (i=0; list[i] != NULL; i++) {
-                                               modest_msg_edit_window_attach_file_noninteractive(
+                                               modest_msg_edit_window_attach_file_one(
                                                                (ModestMsgEditWindow *)win, list[i]);
                                        }
                                        g_strfreev(list);
@@ -634,17 +635,17 @@ static gint on_open_message(GArray * arguments, gpointer data, osso_rpc_t * retv
 static gboolean
 on_idle_delete_message (gpointer user_data)
 {
-       TnyList      *headers;
-       TnyFolder    *folder;
-       TnyIterator  *iter; 
-       TnyHeader    *header;
-       TnyHeader    *msg_header;
-       TnyMsg       *msg;
-       TnyAccount   *account;
-       GError       *error;
-       const char   *uri;
-       const char   *uid;
-       gint          res;
+       TnyList      *headers = NULL;
+       TnyFolder    *folder = NULL;
+       TnyIterator  *iter = NULL; 
+       TnyHeader    *header = NULL;
+       TnyHeader    *msg_header = NULL;
+       TnyMsg       *msg = NULL;
+       TnyAccount   *account = NULL;
+       GError       *error = NULL;
+       const char   *uri = NULL;
+       const char   *uid = NULL;
+       gint          res = 0;
 
        uri = (char *) user_data;
 
@@ -680,18 +681,22 @@ on_idle_delete_message (gpointer user_data)
 
        /* g_debug ("Searching header for msg in folder"); */
        while (!tny_iterator_is_done (iter)) {
-               const char *cur_id;
+               const char *cur_id = NULL;
 
                header = TNY_HEADER (tny_iterator_get_current (iter));
-               cur_id = tny_header_get_uid (header);
+               if (header)
+                       cur_id = tny_header_get_uid (header);
                
                if (cur_id && uid && g_str_equal (cur_id, uid)) {
                        /* g_debug ("Found corresponding header from folder"); */
                        break;
                }
 
-               header = NULL;
-               g_object_unref (header);
+               if (header) {
+                       g_object_unref (header);
+                       header = NULL;
+               }
+               
                tny_iterator_next (iter);
        }
 
@@ -706,7 +711,9 @@ on_idle_delete_message (gpointer user_data)
        msg = NULL;
 
        if (header == NULL) {
-               g_object_unref (folder);
+               if (folder)
+                       g_object_unref (folder);
+                       
                return OSSO_ERROR;
        }       
                
@@ -733,11 +740,39 @@ on_idle_delete_message (gpointer user_data)
        
        gdk_threads_leave ();
        
-       g_object_unref (header);
-       g_object_unref (folder);
+       if (header)
+               g_object_unref (header);
+       
+       if (folder) {
+               /* Trick: do a poke status in order to speed up the signaling
+                  of observers.
+                  A delete via the menu does this, in do_headers_action(), 
+                  though I don't know why.
+                */
+               tny_folder_poke_status (folder);
+       
+               g_object_unref (folder);
+       }
+       
+       if (account)
+               g_object_unref (account);
+               
+       /* Refilter the header view explicitly, to make sure that 
+        * deleted emails are really removed from view. 
+        * (They are not really deleted until contact is made with the server, 
+        * so they would appear with a strike-through until then):
+        */
+       ModestHeaderView *header_view = MODEST_HEADER_VIEW(modest_main_window_get_child_widget (
+               MODEST_MAIN_WINDOW(win), MODEST_WIDGET_TYPE_HEADER_VIEW));
+       if (header_view && MODEST_IS_HEADER_VIEW (header_view))
+               modest_header_view_refilter (header_view);
+       
        return res;
 }
 
+
+
+
 static gint
 on_delete_message (GArray *arguments, gpointer data, osso_rpc_t *retval)
 {
@@ -781,6 +816,7 @@ on_idle_send_receive(gpointer user_data)
 
 static gint on_send_receive(GArray * arguments, gpointer data, osso_rpc_t * retval)
 {      
+       printf("DEBUG: modest: %s\n", __FUNCTION__);
     /* Use g_idle to context-switch into the application's thread: */
 
     /* This method has no arguments. */
@@ -1184,6 +1220,20 @@ add_single_folder_to_list (TnyFolder *folder, GList** list)
        if (!folder)
                return;
                
+       if (TNY_IS_MERGE_FOLDER (folder)) {
+               /* Ignore these because their IDs ares
+                * a) not always unique or sensible.
+                * b) not human-readable, and currently need a human-readable 
+                *    ID here, because the osso-email-interface API does not allow 
+                *    us to return both an ID and a display name.
+                * 
+                * This is actually the merged outbox folder.
+                * We could hack our D-Bus API to understand "outbox" as the merged outboxes, 
+                * but that seems unwise. murrayc.
+                */
+                return;        
+       }
+               
        /* Add this folder to the list: */
        /*
        const gchar * folder_name = tny_folder_get_name (folder);
@@ -1292,13 +1342,9 @@ on_dbus_method_get_folders (DBusConnection *con, DBusMessage *message)
        GList *folder_names = NULL;
        add_folders_to_list (TNY_FOLDER_STORE (account), &folder_names);
 
-       printf("DEBUGa0: %s\n", __FUNCTION__);
-       
        g_object_unref (account);
        account = NULL;
        
-       printf("DEBUGa1: %s\n", __FUNCTION__);
-       
        /* Also add the folders from the local folders account,
         * because they are (currently) used with all accounts:
         * TODO: This is not working. It seems to get only the Merged Folder (with an ID of "" (not NULL)).
@@ -1311,17 +1357,12 @@ on_dbus_method_get_folders (DBusConnection *con, DBusMessage *message)
        g_object_unref (account_local);
        account_local = NULL;
 
-       printf("DEBUGa2: %s\n", __FUNCTION__);
-       
 
        /* Put the result in a DBus reply: */
        reply = dbus_message_new_method_return (message);
 
        get_folders_result_to_message (reply, folder_names);
 
-       printf("DEBUGa3: %s\n", __FUNCTION__);
-       
-       
        if (reply == NULL) {
                g_warning ("%s: Could not create reply.", __FUNCTION__);
        }
@@ -1335,8 +1376,6 @@ on_dbus_method_get_folders (DBusConnection *con, DBusMessage *message)
 
        g_list_foreach (folder_names, (GFunc)g_free, NULL);
        g_list_free (folder_names);
-       
-       printf("DEBUGa4: %s\n", __FUNCTION__);
 }