Work to fix recovering sort status in headers view (fixes NB#58250).
[modest] / src / modest-search.c
index d8cf19c..726715c 100644 (file)
@@ -1,4 +1,35 @@
+/* Copyright (c) 2006, Nokia Corporation
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are
+ * met:
+ *
+ * * Redistributions of source code must retain the above copyright
+ *   notice, this list of conditions and the following disclaimer.
+ * * Redistributions in binary form must reproduce the above copyright
+ *   notice, this list of conditions and the following disclaimer in the
+ *   documentation and/or other materials provided with the distribution.
+ * * Neither the name of the Nokia Corporation nor the names of its
+ *   contributors may be used to endorse or promote products derived from
+ *   this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS
+ * IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED
+ * TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A
+ * PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER
+ * OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
+ * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
+ * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
+ * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF
+ * LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
+ * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
+ * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#ifndef _GNU_SOURCE
 #define _GNU_SOURCE
+#endif
 
 #ifdef HAVE_CONFIG_H
 #include <config.h>
@@ -13,6 +44,8 @@
 #include <tny-iterator.h>
 #include <tny-simple-list.h>
 
+#include <libmodest-dbus-client/libmodest-dbus-client.h>
+
 #include "modest-text-utils.h"
 #include "modest-account-mgr.h"
 #include "modest-tny-account-store.h"
 #include "modest-search.h"
 #include "modest-runtime.h"
 
+static gchar *
+g_strdup_or_null (const gchar *str)
+{
+       gchar *string = NULL;
+
+       if  (str != NULL) {
+               string = g_strdup (str);
+       }
+
+       return string;
+}
+
 static GList*
-add_header (GList *list, TnyHeader *header, TnyFolder *folder)
+add_hit (GList *list, TnyHeader *header, TnyFolder *folder)
 {
-       TnyFolder *f;
-       
-       /* TODO: we need this call otherwise it will crash later
-        * when we try to do that call again without having the
-        * folder around, I guess that is a bug in TinyThingy */
-       f = tny_header_get_folder (header);
-
-       return g_list_prepend (list, g_object_ref (header));
+       ModestSearchHit *hit;
+       TnyHeaderFlags   flags;
+       char            *furl;
+       char            *msg_url;
+       const char      *uid;
+       const char      *subject;
+       const char      *sender;
+
+       hit = g_slice_new0 (ModestSearchHit);
+
+       furl = tny_folder_get_url_string (folder);
+       if (!furl) {
+               g_warning ("%s: tny_folder_get_url_string(): returned NULL for folder. Folder name=%s\n", __FUNCTION__, tny_folder_get_name (folder));
+       }
+       
+       /* Make sure that we use the short UID instead of the long UID,
+        * and/or find out what UID form is used when finding, in camel_data_cache_get().
+        * so we can find what we get. Philip is working on this.
+        */
+       uid = tny_header_get_uid (header);
+       if (!furl) {
+               g_warning ("%s: tny_header_get_uid(): returned NULL for message with subject=%s\n", __FUNCTION__, tny_header_get_subject (header));
+       }
+       
+       msg_url = g_strdup_printf ("%s/%s", furl, uid);
+       
+       subject = tny_header_get_subject (header);
+       sender = tny_header_get_from (header);
+
+       flags = tny_header_get_flags (header);
+
+       hit->msgid = msg_url;
+       hit->subject = g_strdup_or_null (subject);
+       hit->sender = g_strdup_or_null (sender);
+       hit->folder = furl; /* We just provide our new instance instead of copying it and freeing it. */
+       hit->msize = tny_header_get_message_size (header);
+       hit->has_attachment = flags & TNY_HEADER_FLAG_ATTACHMENTS;
+       hit->is_unread = ! (flags & TNY_HEADER_FLAG_SEEN);
+       hit->timestamp = tny_header_get_date_received (header);
+       
+       return g_list_prepend (list, hit);
 }
 
 static gboolean
@@ -111,7 +189,7 @@ search_mime_part_ogs (TnyMimePart *part, ModestSearch *search)
 
        return found;
 }
-#endif
+#endif /*MODEST_HAVE_OGS*/
 
 static gboolean
 search_mime_part_strcmp (TnyMimePart *part, ModestSearch *search)
@@ -119,8 +197,8 @@ search_mime_part_strcmp (TnyMimePart *part, ModestSearch *search)
        TnyStream *stream;
        char       buffer[8193];
        char      *chunk[2];
-       gsize      len;
-       gssize     nread;
+       gssize     len;
+       gsize     nread;
        gboolean   found;
        gboolean   res;
 
@@ -277,23 +355,24 @@ modest_search_folder (TnyFolder *folder, ModestSearch *search)
                        const char *str = tny_header_get_subject (cur);
 
                        if ((found = search_string (search->subject, str, search))) {
-                           retval = add_header (retval, cur, folder);
+                           retval = add_hit (retval, cur, folder);
                        }
                }
                
                if (!found && search->flags & MODEST_SEARCH_SENDER) {
-                       const char *str = tny_header_get_from (cur);
+                       char *str = g_strdup (tny_header_get_from (cur));
 
-                       if ((found = search_string (search->from, str, search))) {
-                               retval = add_header (retval, cur, folder);
+                       if ((found = search_string (search->from, (const gchar *) str, search))) {
+                               retval = add_hit (retval, cur, folder);
                        }
+                       g_free (str);
                }
                
                if (!found && search->flags & MODEST_SEARCH_RECIPIENT) {
                        const char *str = tny_header_get_to (cur);
 
                        if ((found = search_string (search->recipient, str, search))) {
-                               retval = add_header (retval, cur, folder);
+                               retval = add_hit (retval, cur, folder);
                        }
                }
        
@@ -313,7 +392,7 @@ modest_search_folder (TnyFolder *folder, ModestSearch *search)
                        msg = tny_folder_get_msg (folder, cur, &err);
 
                        if (err != NULL || msg == NULL) {
-                               g_warning ("Could not get message\n");
+                               g_warning ("%s: Could not get message.\n", __FUNCTION__);
                                g_error_free (err);
 
                                if (msg) {
@@ -329,7 +408,7 @@ modest_search_folder (TnyFolder *folder, ModestSearch *search)
                                TnyMimePart *pcur = (TnyMimePart *) tny_iterator_get_current (piter);
 
                                if ((found = part_search_func (pcur, search))) {
-                                       retval = add_header (retval, cur, folder);                              
+                                       retval = add_hit (retval, cur, folder);                         
                                }
 
                                g_object_unref (pcur);
@@ -341,6 +420,7 @@ modest_search_folder (TnyFolder *folder, ModestSearch *search)
                        g_object_unref (msg);
 
                }
+
 go_next:
                g_object_unref (cur);
                tny_iterator_next (iter);
@@ -403,34 +483,28 @@ modest_search_account (TnyAccount *account, ModestSearch *search)
 GList *
 modest_search_all_accounts (ModestSearch *search)
 {
-       ModestAccountMgr      *account_mgr;
        ModestTnyAccountStore *astore;
-       GSList                *accounts;
-       GSList                *iter;
+       TnyList               *accounts;
+       TnyIterator           *iter;
        GList                 *hits;
 
-       account_mgr = modest_runtime_get_account_mgr ();
-
-       accounts = modest_account_mgr_account_names (account_mgr, FALSE);
-       astore = modest_runtime_get_account_store ();
        hits = NULL;
+       astore = modest_runtime_get_account_store ();
 
-       for (iter = accounts; iter; iter = iter->next) {
-               GList      *res;
-               const char *ac_name;
-               TnyAccount *account = NULL;
+       accounts = tny_simple_list_new ();
+       tny_account_store_get_accounts (TNY_ACCOUNT_STORE (astore),
+                                       accounts,
+                                       TNY_ACCOUNT_STORE_STORE_ACCOUNTS);
 
-               ac_name = (const char *) iter->data;
+       iter = tny_list_create_iterator (accounts);
+       while (!tny_iterator_is_done (iter)) {
+               TnyAccount *account;
+               GList      *res;
 
-               account = modest_tny_account_store_get_tny_account_by_account (astore,
-                                                                              ac_name,
-                                                                              TNY_ACCOUNT_TYPE_STORE);
+               account = TNY_ACCOUNT (tny_iterator_get_current (iter));
 
-               if (account == NULL) {
-                       g_warning ("Could not get account for %s", ac_name);
-                       continue;
-               }
-               
+               g_debug ("DEBUG: %s: Searching account %s",
+                        __FUNCTION__, tny_account_get_name (account));
                res = modest_search_account (account, search);
                
                if (res != NULL) {
@@ -441,8 +515,14 @@ modest_search_all_accounts (ModestSearch *search)
                                hits = g_list_concat (hits, res);
                        }
                }
+
+               g_object_unref (account);
+               tny_iterator_next (iter);
        }
 
+       g_object_unref (accounts);
+       g_object_unref (iter);
+
        return hits;
 }