Me card contact is normally a OssoABookContact and not a EContact
[modest] / src / hildon2 / modest-address-book.c
index 8ad0df8..b26a597 100644 (file)
 #include <gtk/gtkcellrenderertext.h>
 #include <gtk/gtktreeselection.h>
 #include <gtk/gtkentry.h>
+#include <modest-maemo-utils.h>
 
 static OssoABookContactModel *contact_model =  NULL;
 static EBook *book = NULL;
 static EBookView * book_view = NULL;
 
-static GSList *get_recipients_for_given_contact(EContact * contact);
+static GSList *get_recipients_for_given_contact (EContact * contact, gboolean *canceled);
 static void commit_contact(EContact * contact, gboolean is_new);
-static gchar *get_email_addr_from_user(const gchar * given_name);
+static gchar *get_email_addr_from_user(const gchar * given_name, gboolean *canceled);
 static gchar *ui_get_formatted_email_id(gchar * current_given_name,
                                        gchar * current_sur_name, gchar * current_email_id);
-static gchar *run_add_email_addr_to_contact_dlg(const gchar * contact_name);
+static gchar *run_add_email_addr_to_contact_dlg(const gchar * contact_name, gboolean *canceled);
 static GSList *select_email_addrs_for_contact(GList * email_addr_list);
-static gboolean resolve_address (const gchar *address, GSList **resolved_addresses, gchar **contact_id);
+static gboolean resolve_address (const gchar *address, GSList **resolved_addresses, GSList **contact_id, gboolean *canceled);
 static gchar *unquote_string (const gchar *str);
 
 
@@ -123,13 +124,79 @@ open_addressbook ()
        return TRUE; /* FIXME */        
 }
 
+typedef struct _OpenAddressbookSyncInfo {
+       gboolean retval;
+       GMainLoop *mainloop;
+} OpenAddressbookSyncInfo;
+
+static void
+get_book_view_sync_cb (EBook *book, EBookStatus status, EBookView *bookview, gpointer data)
+{
+       OpenAddressbookSyncInfo *info = (OpenAddressbookSyncInfo *) data;
+
+       if (status != E_BOOK_ERROR_OK) {
+               g_object_unref (book);
+               book = NULL;
+               info->retval = FALSE;
+               g_main_loop_quit (info->mainloop);
+               return;
+       }
+       book_view = bookview;
+
+       if (contact_model)
+#if MODEST_ABOOK_API < 4
+               osso_abook_tree_model_set_book_view (OSSO_ABOOK_TREE_MODEL (contact_model),
+                                                    book_view);
+#else /* MODEST_ABOOK_API < 4 */
+               osso_abook_list_store_set_book_view (OSSO_ABOOK_LIST_STORE (contact_model),
+                                                    book_view);
+#endif /* MODEST_ABOOK_API < 4 */
+
+       e_book_view_start (book_view);
+       info->retval = TRUE;
+       g_main_loop_quit (info->mainloop);
+}
+
+static void
+book_open_sync_cb (EBook *view, EBookStatus status, gpointer data)
+{
+       EBookQuery *query = NULL;
+       OpenAddressbookSyncInfo *info = (OpenAddressbookSyncInfo *) data;
+
+       if (status != E_BOOK_ERROR_OK) {
+               g_object_unref (book);
+               book = NULL;
+               info->retval = FALSE;
+               g_main_loop_quit (info->mainloop);
+               return;
+       }
+       query = e_book_query_any_field_contains ("");
+       e_book_async_get_book_view (book, query, NULL, -1, get_book_view_sync_cb, info);
+       e_book_query_unref (query);
+}
+
 static gboolean
 open_addressbook_sync ()
 {
+       OpenAddressbookSyncInfo *info;
+       gboolean retval;
+
        book = e_book_new_system_addressbook (NULL);
        if (!book)
                return FALSE;
 
+       info = g_slice_new (OpenAddressbookSyncInfo);
+       info->mainloop = g_main_loop_new (NULL, FALSE);
+       info->retval = FALSE;
+       if (e_book_async_open (book, FALSE, book_open_sync_cb, info) == E_BOOK_ERROR_OK) {
+               GDK_THREADS_LEAVE ();
+               g_main_loop_run (info->mainloop);
+               GDK_THREADS_ENTER ();
+       }
+       retval = info->retval;
+       g_main_loop_unref (info->mainloop);
+       g_slice_free (OpenAddressbookSyncInfo, info);
+       /* Make it launch a main loop */
        return e_book_open (book, FALSE, NULL);
 }
 
@@ -159,7 +226,8 @@ modest_address_book_add_address (const gchar *address)
 }
 
 void
-modest_address_book_select_addresses (ModestRecptEditor *recpt_editor)
+modest_address_book_select_addresses (ModestRecptEditor *recpt_editor,
+                                     GtkWindow *parent_window)
 {
 #if MODEST_ABOOK_API < 4
        GtkWidget *contact_view = NULL;
@@ -202,10 +270,13 @@ modest_address_book_select_addresses (ModestRecptEditor *recpt_editor)
        }
 #else /* MODEST_ABOOK_API < 4 */
        /* TODO: figure out how to make the contact chooser modal */
-       contact_chooser = osso_abook_contact_chooser_new_with_capabilities (NULL,
-                                                                           _("mcen_me_editor_selectrecipients"),
+       contact_chooser = osso_abook_contact_chooser_new_with_capabilities (parent_window,
+                                                                           _AB("addr_ti_dia_select_contacts"),
                                                                            OSSO_ABOOK_CAPS_EMAIL, 
                                                                            OSSO_ABOOK_CONTACT_ORDER_NAME);
+       /* Enable multiselection */
+       osso_abook_contact_chooser_set_maximum_selection (OSSO_ABOOK_CONTACT_CHOOSER (contact_chooser),
+                                                         G_MAXUINT);
 
        if (gtk_dialog_run (GTK_DIALOG (contact_chooser)) == GTK_RESPONSE_OK)
                contacts_list = osso_abook_contact_chooser_get_selection (OSSO_ABOOK_CONTACT_CHOOSER (contact_chooser));
@@ -217,8 +288,9 @@ modest_address_book_select_addresses (ModestRecptEditor *recpt_editor)
 
                for (node = contacts_list; node != NULL; node = g_list_next (node)) {
                        EContact *contact = (EContact *) node->data;
+                       gboolean canceled;
 
-                       email_addrs_per_contact = get_recipients_for_given_contact (contact);
+                       email_addrs_per_contact = get_recipients_for_given_contact (contact, &canceled);
                        if (email_addrs_per_contact) {
                                econtact_id = (gchar *) e_contact_get_const (contact, E_CONTACT_UID);
                                modest_recpt_editor_add_resolved_recipient (MODEST_RECPT_EDITOR (recpt_editor), 
@@ -256,7 +328,9 @@ modest_address_book_select_addresses (ModestRecptEditor *recpt_editor)
  * @param  Contact of type #EContact
  * @return List of resolved recipient strings, to be freed by calling function.
  */
-static GSList *get_recipients_for_given_contact(EContact * contact)
+static GSList *
+get_recipients_for_given_contact (EContact * contact,
+                                 gboolean *canceled)
 {
        gchar *givenname = NULL;
        gchar *familyname = NULL;
@@ -298,15 +372,15 @@ static GSList *get_recipients_for_given_contact(EContact * contact)
        /*Launch the 'Add e-mail addr to contact' dialog if required */
        if (email_not_present) {
 #if MODEST_ABOOK_API < 4
-               display_name = osso_abook_contact_get_display_name(contact);            
+               display_name = osso_abook_contact_get_display_name(contact);
 #else
                OssoABookContact *abook_contact;
-              
+
                abook_contact = osso_abook_contact_new_from_template (contact);
                display_name = osso_abook_contact_get_display_name(abook_contact);
 #endif
 
-               emailid = get_email_addr_from_user(display_name);
+               emailid = get_email_addr_from_user(display_name, canceled);
                if (emailid) {
                        e_contact_set(contact, E_CONTACT_EMAIL_1, emailid);
                        commit_contact(contact, FALSE);
@@ -362,17 +436,28 @@ commit_contact(EContact * contact, gboolean is_new)
 {
        g_return_if_fail (contact);
        g_return_if_fail (book);
-       
+
        if (!contact || !book)
                return;
-       
+
 #if MODEST_ABOOK_API < 4
        osso_abook_contact_commit(contact, is_new, book);
 #else
        if (OSSO_ABOOK_IS_CONTACT (contact)) {
-               osso_abook_contact_commit(contact, is_new, book, NULL);
+               osso_abook_contact_commit(OSSO_ABOOK_CONTACT(contact), is_new, book, NULL);
        } else {
-               e_book_commit_contact (book, contact, NULL);
+               GError *err = NULL;
+               if (is_new) {
+                       if (!e_book_add_contact (book, contact, &err)) {
+                               g_warning ("Failed to add contact: %s", err->message);
+                               g_error_free (err);
+                       }
+               } else {
+                       if (!e_book_commit_contact (book, contact, &err)) {
+                               g_warning ("Failed to commit contact: %s", err->message);
+                               g_error_free (err);
+                       }
+               }
        }
 #endif /* MODEST_ABOOK_API < 2 */
 }
@@ -387,7 +472,7 @@ commit_contact(EContact * contact, gboolean is_new)
  * @return E-mail address string entered by user, to be freed by calling function.
  */
 static gchar *
-get_email_addr_from_user(const gchar * given_name)
+get_email_addr_from_user(const gchar * given_name, gboolean *canceled)
 {
        gchar *notification = NULL;
        gchar *email_addr = NULL;
@@ -403,7 +488,7 @@ get_email_addr_from_user(const gchar * given_name)
        g_free(notification);
 
        if (note_response == GTK_RESPONSE_OK) {
-               email_addr = run_add_email_addr_to_contact_dlg(given_name);
+               email_addr = run_add_email_addr_to_contact_dlg (given_name, canceled);
        }
 
        return email_addr;
@@ -446,11 +531,14 @@ ui_get_formatted_email_id(gchar * current_given_name,
  * It allows user to enter an e-mail address, and shows appropriate infonote if the
  * entered string is not a valid e-mail address.
  *
+ * It must return TRUE in canceled if the dialog was canceled by the user
+ *
  * @param  contact_name  Full name of the contact
  * @return E-mail address string entered by user, to be freed by calling function.
  */
 static gchar *
-run_add_email_addr_to_contact_dlg(const gchar * contact_name)
+run_add_email_addr_to_contact_dlg(const gchar * contact_name,
+                                 gboolean *canceled)
 {
        GtkWidget *add_email_addr_to_contact_dlg = NULL;
        GtkSizeGroup *size_group = NULL;
@@ -461,27 +549,35 @@ run_add_email_addr_to_contact_dlg(const gchar * contact_name)
        gchar *new_email_addr = NULL;
        gboolean run_dialog = TRUE;
 
+       g_return_val_if_fail (canceled, NULL);
+
+       *canceled = FALSE;
+
        add_email_addr_to_contact_dlg =
            gtk_dialog_new_with_buttons(_("mcen_ti_add_email_title"), NULL,
                                        GTK_DIALOG_MODAL | GTK_DIALOG_DESTROY_WITH_PARENT,
-                                       _("mcen_bd_dialog_ok"), GTK_RESPONSE_ACCEPT,
-                                       _("mcen_bd_dialog_cancel"), GTK_RESPONSE_REJECT, NULL);
+                                       _HL("wdgt_bd_save"), GTK_RESPONSE_ACCEPT, NULL);
        gtk_dialog_set_has_separator(GTK_DIALOG(add_email_addr_to_contact_dlg), FALSE);
+#ifdef MODEST_TOOLKIT_HILDON2
+       gtk_container_set_border_width (GTK_CONTAINER (GTK_DIALOG (add_email_addr_to_contact_dlg)->vbox), 
+                                       HILDON_MARGIN_DOUBLE);
+#endif
        /*Set app_name & state_save related tags to the window */
 
        size_group = gtk_size_group_new(GTK_SIZE_GROUP_HORIZONTAL);
        name_label = gtk_label_new(contact_name);
-       gtk_misc_set_alignment(GTK_MISC(name_label), 0, 0);
+       gtk_misc_set_alignment(GTK_MISC(name_label), 0.0, 0.5);
        cptn_cntrl =
-           hildon_caption_new(size_group, _("mcen_ia_add_email_name"), name_label, NULL,
-                              HILDON_CAPTION_OPTIONAL);
+               modest_maemo_utils_create_captioned (size_group, NULL,
+                                                    _("mcen_ia_add_email_name"), FALSE,
+                                                    name_label);
        gtk_box_pack_start(GTK_BOX(GTK_DIALOG(add_email_addr_to_contact_dlg)->vbox), cptn_cntrl,
                           FALSE, FALSE, 0);
 
-       email_entry = gtk_entry_new();
-       cptn_cntrl =
-           hildon_caption_new(size_group, _("mcen_fi_add_email_name"), email_entry, NULL,
-                              HILDON_CAPTION_OPTIONAL);
+       email_entry = hildon_entry_new (HILDON_SIZE_FINGER_HEIGHT | HILDON_SIZE_AUTO_WIDTH);
+       cptn_cntrl = modest_maemo_utils_create_captioned (size_group, NULL, 
+                                                         _("mcen_fi_add_email_name"), FALSE,
+                                                         email_entry);
        hildon_gtk_entry_set_input_mode(GTK_ENTRY(email_entry), HILDON_GTK_INPUT_MODE_FULL);
        gtk_box_pack_start(GTK_BOX(GTK_DIALOG(add_email_addr_to_contact_dlg)->vbox), cptn_cntrl,
                           TRUE, TRUE, 0);
@@ -495,15 +591,16 @@ run_add_email_addr_to_contact_dlg(const gchar * contact_name)
 
                if (result == GTK_RESPONSE_ACCEPT) {
                        const gchar *invalid_char_offset = NULL;
-                       new_email_addr = g_strdup(gtk_entry_get_text(GTK_ENTRY(email_entry)));
+                       new_email_addr = g_strdup(hildon_entry_get_text(HILDON_ENTRY(email_entry)));
                        new_email_addr = g_strstrip(new_email_addr);
                        if (!modest_text_utils_validate_email_address (new_email_addr, &invalid_char_offset)) {
                                gtk_widget_grab_focus(email_entry);
                                if ((invalid_char_offset != NULL)&&(*invalid_char_offset != '\0')) {
                                        gchar *char_in_string = g_strdup_printf ("%c", *invalid_char_offset);
                                        gchar *message = g_strdup_printf(
-                                               dgettext("hildon-common-strings", "ckdg_ib_illegal_characters_entered"), 
+                                               _CS("ckdg_ib_illegal_characters_entered"), 
                                                char_in_string);
+                                       g_free (char_in_string);
                                        hildon_banner_show_information (
                                                add_email_addr_to_contact_dlg, NULL, message );
                                        g_free (message);
@@ -515,6 +612,8 @@ run_add_email_addr_to_contact_dlg(const gchar * contact_name)
                                g_free(new_email_addr);
                                new_email_addr = NULL;
                        }
+               } else {
+                       *canceled = TRUE;
                }
        }
 
@@ -535,166 +634,297 @@ static GSList *
 select_email_addrs_for_contact(GList * email_addr_list)
 {
        GtkWidget *select_email_addr_dlg = NULL;
-       GtkWidget *view = NULL, *scrolledwindow = NULL;
-       GtkTreeSelection *selection = NULL;
-       GtkCellRenderer *renderer = NULL;
-       GtkTreeViewColumn *col = NULL;
-       GtkListStore *list_store = NULL;
-       GtkTreeModel *model = NULL;
-       GtkTreeIter iter;
-       GList *pathslist = NULL, *node = NULL;
-       gint result = -1;
-       gchar *email_addr = NULL;
        GSList *selected_email_addr_list = NULL;
+       GList *node;
+       GtkWidget *selector;
+       gint result = -1;
 
        if (!email_addr_list)
                return NULL;
 
-       select_email_addr_dlg =
-           gtk_dialog_new_with_buttons(_("mcen_ti_select_email_title"),
-                                       NULL,
-                                       GTK_DIALOG_MODAL | GTK_DIALOG_DESTROY_WITH_PARENT,
-                                       _("mcen_bd_dialog_ok"), GTK_RESPONSE_ACCEPT,
-                                       _("mcen_bd_dialog_cancel"), GTK_RESPONSE_REJECT, NULL);
-       gtk_dialog_set_has_separator(GTK_DIALOG(select_email_addr_dlg), FALSE);
-
-       /* Make the window approximately big enough, because it doesn't resize to be big enough 
-        * for the window title text: */
-       gtk_window_set_default_size (GTK_WINDOW (select_email_addr_dlg), MODEST_DIALOG_WINDOW_MAX_HEIGHT, -1);
-
-       scrolledwindow = gtk_scrolled_window_new(NULL, NULL);
-       gtk_box_pack_start(GTK_BOX(GTK_DIALOG(select_email_addr_dlg)->vbox), scrolledwindow, TRUE,
-                          TRUE, 0);
-       gtk_scrolled_window_set_policy(GTK_SCROLLED_WINDOW(scrolledwindow), GTK_POLICY_AUTOMATIC,
-                                      GTK_POLICY_AUTOMATIC);
-
-       view = gtk_tree_view_new();
-       col = gtk_tree_view_column_new();
-       renderer = gtk_cell_renderer_text_new();
-       gtk_tree_view_append_column(GTK_TREE_VIEW(view), col);
-       gtk_tree_view_column_pack_start(col, renderer, TRUE);
-       gtk_tree_view_column_add_attribute(col, renderer, "text", 0);
-       selection = gtk_tree_view_get_selection(GTK_TREE_VIEW(view));
-       gtk_tree_selection_set_mode(selection, GTK_SELECTION_MULTIPLE);
-       gtk_container_add(GTK_CONTAINER(scrolledwindow), view);
-
-       list_store = gtk_list_store_new(1, G_TYPE_STRING);
-       model = GTK_TREE_MODEL(list_store);
-       gtk_tree_view_set_model(GTK_TREE_VIEW(view), model);
+       select_email_addr_dlg = hildon_picker_dialog_new (NULL);
+       gtk_window_set_title (GTK_WINDOW (select_email_addr_dlg), _("mcen_ti_select_email_title"));
 
+       selector = hildon_touch_selector_new_text ();
        for (node = email_addr_list; node != NULL && node->data != NULL; node = node->next) {
+               gchar *email_addr;
                email_addr = g_strstrip(g_strdup(node->data));
-               gtk_list_store_append(list_store, &iter);
-               gtk_list_store_set(list_store, &iter, 0, email_addr, -1);
+               hildon_touch_selector_append_text (HILDON_TOUCH_SELECTOR (selector), email_addr);
                g_free(email_addr);
        }
-       gtk_tree_model_get_iter_first (GTK_TREE_MODEL (list_store), &iter);
-       gtk_tree_selection_select_iter (selection, &iter);
+
+       hildon_picker_dialog_set_selector (HILDON_PICKER_DIALOG (select_email_addr_dlg),
+                                          HILDON_TOUCH_SELECTOR (selector));
+       gtk_window_set_default_size (GTK_WINDOW (select_email_addr_dlg), MODEST_DIALOG_WINDOW_MAX_HEIGHT, -1);
 
        gtk_widget_show_all(select_email_addr_dlg);
        result = gtk_dialog_run(GTK_DIALOG(select_email_addr_dlg));
 
-       if (result == GTK_RESPONSE_ACCEPT) {
-               pathslist = gtk_tree_selection_get_selected_rows(selection, NULL);
-               for (node = pathslist; node != NULL; node = node->next) {
-                       if (gtk_tree_model_get_iter(model, &iter, (GtkTreePath *) node->data)) {
-                               gtk_tree_model_get(model, &iter, 0, &email_addr, -1);
-                               selected_email_addr_list =
-                                   g_slist_append(selected_email_addr_list, g_strdup(email_addr));
-                               g_free(email_addr);
-                       }
-               }
+       if (result == GTK_RESPONSE_OK) {
+               gchar *current_text;
+
+               current_text = hildon_touch_selector_get_current_text (HILDON_TOUCH_SELECTOR (selector));
+               selected_email_addr_list = g_slist_append (selected_email_addr_list, current_text);
        }
 
-       gtk_list_store_clear(list_store);
        gtk_widget_destroy(select_email_addr_dlg);
        return selected_email_addr_list;
 }
 
+/* Assumes that the second argument (the user provided one) is a pure
+   email address without name */
+static gint
+compare_addresses (const gchar *address1,
+                  const gchar *mail2)
+{
+       gint retval;
+       gchar *mail1;
+
+       mail1 = modest_text_utils_get_email_address (address1);
+       retval = g_strcmp0 (mail1, mail2);
+       g_free (mail1);
+
+       return retval;
+}
 
-static gboolean /* make this public? */
-add_to_address_book (const gchar* address)
+static EContact *
+get_contact_for_address (GList *contacts,
+                        const gchar *address)
 {
-       EBookQuery *query;
-       GList *contacts = NULL;
-       GError *err = NULL;
+       EContact *retval = NULL, *contact;
+       GList *iter;
        gchar *email;
-       
-       g_return_val_if_fail (address, FALSE);
-       
-       if (!book) {
-               if (!open_addressbook ()) {
-                       g_return_val_if_reached (FALSE);
+
+       email = modest_text_utils_get_email_address (address);
+       iter = contacts;
+       while (iter && !retval) {
+               GList *emails = NULL;
+
+               contact = E_CONTACT (iter->data);
+               emails = e_contact_get (contact, E_CONTACT_EMAIL);
+               if (emails) {
+                       /* Look for the email address */
+                       if (g_list_find_custom (emails, email, (GCompareFunc) compare_addresses))
+                               retval = contact;
+
+                       /* Free the list */
+                       g_list_foreach (emails, (GFunc) g_free, NULL);
+                       g_list_free (emails);
                }
+               iter = g_list_next (iter);
        }
-       
-       g_return_val_if_fail (book, FALSE);
+       g_free (email);
 
-       email = modest_text_utils_get_email_address (address);
-       
-       query = e_book_query_field_test (E_CONTACT_EMAIL, E_BOOK_QUERY_IS, email);
-       if (!e_book_get_contacts (book, query, &contacts, &err)) {
-               g_printerr ("modest: failed to get contacts: %s",
-                           err ? err->message : "<unknown>");
-               if (err)
-                       g_error_free (err);
-               return FALSE;
+       return retval;
+}
+
+static void
+async_get_contacts_cb (EBook *book,
+                      EBookStatus status,
+                      GList *contacts,
+                      gpointer closure)
+{
+       GSList *addresses, *iter;
+       GList *to_commit_contacts, *to_add_contacts;
+       EContact *self_contact;
+
+       addresses = (GSList *) closure;
+
+       /* Check errors */
+       if (status != E_BOOK_ERROR_OK)
+               goto frees;
+
+       self_contact = (EContact *) osso_abook_self_contact_get_default ();
+       if (self_contact) {
+               contacts = g_list_prepend (contacts, self_contact);
        }
-       e_book_query_unref (query);
-       
-       /*  we need to 'commit' it, even if we already found the email
-        * address in the addressbook; thus, it will show up in the 'recent list' */
-       if (contacts)  {                
-               g_debug ("%s already in the address book", address);
-               commit_contact ((EContact*)contacts->data, FALSE);
-               
-               g_list_foreach (contacts, (GFunc)unref_gobject, NULL);
-               g_list_free (contacts);
 
-       } else {
-               /* it's not yet in the addressbook, add it now! */
-               EContact *new_contact = e_contact_new ();
-               gchar *display_address;
-               display_address = g_strdup (address);
-               if (display_address) {
-                       modest_text_utils_get_display_address (display_address);
-                       if ((display_address[0] != '\0') && (strlen (display_address) != strlen (address)))
-                               e_contact_set (new_contact, E_CONTACT_FULL_NAME, (const gpointer)display_address);
+       iter = addresses;
+       to_commit_contacts = NULL;
+       to_add_contacts = NULL;
+       while (iter) {
+               EContact *contact;
+               const gchar *address;
+
+               /* Look for a contact with such address. We perform
+                  this kind of search because we assume that users
+                  don't usually send emails to tons of addresses */
+               address = (const gchar *) iter->data;
+               contact = get_contact_for_address (contacts, address);
+
+               /* Add new or commit existing contact */
+               if (contact) {
+                       to_commit_contacts = g_list_prepend (to_commit_contacts, contact);
+                       g_debug ("----Preparing to commit contact %s", address);
+               } else {
+                       gchar *email_address, *display_address;
+
+                       /* Create new contact and add it to the list */
+                       contact = e_contact_new ();
+                       email_address = modest_text_utils_get_email_address (address);
+                       e_contact_set (contact, E_CONTACT_EMAIL_1, email_address);
+                       g_free (email_address);
+
+                       display_address = g_strdup (address);
+                       if (display_address) {
+                               modest_text_utils_get_display_address (display_address);
+                               if ((display_address[0] != '\0') && (strlen (display_address) != strlen (address)))
+                                       e_contact_set (contact, E_CONTACT_FULL_NAME, (const gpointer)display_address);
+                               g_free (display_address);
+                       }
+
+                       to_add_contacts = g_list_prepend (to_add_contacts, contact);
+                       g_debug ("----Preparing to add contact %s", address);
                }
-               e_contact_set (new_contact, E_CONTACT_EMAIL_1, (const gpointer)email);
-               g_free (display_address);
-               commit_contact (new_contact, TRUE);
-               g_debug ("%s added to address book", address);
-               g_object_unref (new_contact);
+
+               iter = g_slist_next (iter);
        }
 
-       g_free (email);
+       /* Asynchronously add contacts */
+       if (to_add_contacts)
+               e_book_async_add_contacts (book, to_add_contacts, NULL, NULL);
 
-       return TRUE;
+       /* Asynchronously commit contacts */
+       if (to_commit_contacts)
+               e_book_async_commit_contacts (book, to_commit_contacts, NULL, NULL);
+
+       /* Free lists */
+       g_list_free (to_add_contacts);
+       g_list_free (to_commit_contacts);
+
+ frees:
+       if (addresses) {
+               g_slist_foreach (addresses, (GFunc) g_free, NULL);
+               g_slist_free (addresses);
+       }
+       if (contacts) {
+               g_list_foreach (contacts, (GFunc) g_object_unref, NULL);
+               g_list_free (contacts);
+       }
 }
 
-static gboolean
-show_check_names_banner (gpointer userdata)
+
+static void
+add_to_address_book (GSList *addresses)
 {
-       GtkWidget **banner = (GtkWidget **) userdata;
+       EBookQuery **queries, *composite_query;
+       gint num_add, i;
 
-       gdk_threads_enter ();
-       *banner = modest_platform_animation_banner (NULL, NULL, _("mail_ib_checking_names"));
-       gdk_threads_leave ();
+       g_return_if_fail (addresses);
 
-       return FALSE;
+       if (!book)
+               if (!open_addressbook ())
+                       g_return_if_reached ();
+
+       /* Create the list of queries */
+       num_add = g_slist_length (addresses);
+       queries = g_malloc0 (sizeof (EBookQuery *) * num_add);
+       for (i = 0; i < num_add; i++) {
+               gchar *email;
+
+               email = modest_text_utils_get_email_address (g_slist_nth_data (addresses, i));
+               queries[i] = e_book_query_field_test (E_CONTACT_EMAIL, E_BOOK_QUERY_IS, email);
+               g_free (email);
+       }
+
+       /* Create the query */
+       composite_query = e_book_query_or (num_add, queries, TRUE);
+
+       /* Asynchronously retrieve contacts */
+       e_book_async_get_contacts (book, composite_query, async_get_contacts_cb, addresses);
+
+       /* Frees. This will unref the subqueries as well */
+       e_book_query_unref (composite_query);
 }
 
+typedef struct _CheckNamesInfo {
+       GtkWidget *banner;
+       guint show_banner_timeout;
+       guint hide_banner_timeout;
+       gboolean hide;
+       gboolean free_info;
+} CheckNamesInfo;
+
 static void
-hide_check_names_banner (GtkWidget **banner, guint banner_timeout)
+hide_check_names_banner (CheckNamesInfo *info)
+{
+       if (info->show_banner_timeout > 0) {
+               g_source_remove (info->show_banner_timeout);
+               info->show_banner_timeout = 0;
+       }
+       if (info->hide_banner_timeout > 0) {
+               info->hide = TRUE;
+               return;
+       }
+
+       if (info->banner) {
+               gtk_widget_destroy (info->banner);
+               info->banner = NULL;
+               info->hide = FALSE;
+       }
+
+       if (info->free_info) {
+               g_slice_free (CheckNamesInfo, info);
+       }
+}
+
+static gboolean hide_banner_timeout_handler (CheckNamesInfo *info)
+{
+       info->hide_banner_timeout = 0;
+       if (info->hide) {
+               gtk_widget_destroy (info->banner);
+               info->banner = NULL;
+       }
+       if (info->free_info) {
+               g_slice_free (CheckNamesInfo, info);
+       }
+       return FALSE;
+}
+
+static gboolean show_banner_timeout_handler (CheckNamesInfo *info)
+{
+       info->show_banner_timeout = 0;
+       info->banner = hildon_banner_show_animation (NULL, NULL, _("mail_ib_checking_names"));
+       info->hide_banner_timeout = g_timeout_add (1000, (GSourceFunc) hide_banner_timeout_handler, (gpointer) info);
+       return FALSE;
+}
+
+static void show_check_names_banner (CheckNamesInfo *info)
 {
-       if (*banner != NULL) {
-               gtk_widget_destroy (*banner);
-               *banner = NULL;
+       if (info->hide_banner_timeout > 0) {
+               g_source_remove (info->hide_banner_timeout);
+               info->hide_banner_timeout = 0;
+       }
+
+       info->hide = FALSE;
+       if (info->show_banner_timeout > 0)
+               return;
+
+       if (info->banner == NULL) {
+               info->show_banner_timeout = g_timeout_add (500, (GSourceFunc) show_banner_timeout_handler, (gpointer) info);
+       }
+}
+
+static void clean_check_names_banner (CheckNamesInfo *info)
+{
+       if (info->hide_banner_timeout) {
+               info->free_info = TRUE;
        } else {
-               g_source_remove (banner_timeout);
+               if (info->show_banner_timeout) {
+                       g_source_remove (info->show_banner_timeout);
+               }
+               if (info->banner)
+                       gtk_widget_destroy (info->banner);
+               g_slice_free (CheckNamesInfo, info);
        }
+}
 
+void free_resolved_addresses_list (gpointer data,
+                                  gpointer ignored)
+{
+       GSList *list = (GSList *)data;
+       g_slist_foreach (list, (GFunc) g_free, NULL);
+       g_slist_free (list);
 }
 
 gboolean
@@ -702,7 +932,7 @@ modest_address_book_check_names (ModestRecptEditor *recpt_editor, gboolean updat
 {
        const gchar *recipients = NULL;
        GSList *start_indexes = NULL, *end_indexes = NULL;
-       GSList *current_start, *current_end;
+       GSList *current_start, *current_end, *to_commit_addresses;
        gboolean result = TRUE;
        GtkTextBuffer *buffer;
        gint offset_delta = 0;
@@ -727,6 +957,7 @@ modest_address_book_check_names (ModestRecptEditor *recpt_editor, gboolean updat
        current_start = start_indexes;
        current_end = end_indexes;
        buffer = modest_recpt_editor_get_buffer (recpt_editor);
+       to_commit_addresses = NULL;
 
        while (current_start != NULL) {
                gchar *address;
@@ -749,36 +980,52 @@ modest_address_book_check_names (ModestRecptEditor *recpt_editor, gboolean updat
                if (!modest_text_utils_validate_recipient (address, &invalid_char_position)) {
                        if ((invalid_char_position != NULL) && (*invalid_char_position != '\0')) {
                                gchar *char_in_string = g_strdup_printf("%c", *invalid_char_position);
-                               gchar *message = g_strdup_printf(
-                                       dgettext("hildon-common-strings", "ckdg_ib_illegal_characters_entered"), 
-                                       char_in_string);
+                               gchar *message = 
+                                       g_strdup_printf(_CS("ckdg_ib_illegal_characters_entered"), 
+                                                       char_in_string);
                                g_free (char_in_string);
                                hildon_banner_show_information (NULL, NULL, message );
-                               g_free (message);                               
+                               g_free (message);
                                result = FALSE;
                        } else if (strstr (address, "@") == NULL) {
                                /* here goes searching in addressbook */
-                               gchar *contact_id = NULL;
+                               gboolean canceled;
+                               GSList *contact_ids = NULL;
                                GSList *resolved_addresses = NULL;
 
-                               result = resolve_address (address, &resolved_addresses, &contact_id);
+                               result = resolve_address (address, &resolved_addresses, &contact_ids, &canceled);
 
                                if (result) {
                                        gint new_length;
-                                       /* replace string */
-                                       modest_recpt_editor_replace_with_resolved_recipient (recpt_editor,
-                                                                                            &start_iter, &end_iter,
-                                                                                            resolved_addresses, 
-                                                                                            contact_id);
-                                       g_free (contact_id);
-                                       g_slist_foreach (resolved_addresses, (GFunc) g_free, NULL);
+
+                                       modest_recpt_editor_replace_with_resolved_recipients (recpt_editor,
+                                                                                             &start_iter, &end_iter,
+                                                                                             resolved_addresses,
+                                                                                             contact_ids);
+                                       g_slist_foreach (contact_ids, (GFunc) g_free, NULL);
+                                       g_slist_foreach (resolved_addresses, free_resolved_addresses_list, NULL);
+                                       g_slist_free (contact_ids);
                                        g_slist_free (resolved_addresses);
 
                                        /* update offset delta */
                                        recipients = modest_recpt_editor_get_recipients (recpt_editor);
                                        new_length = g_utf8_strlen (recipients, -1);
                                        offset_delta = offset_delta + new_length - last_length;
-                                       last_length = new_length;                                       
+                                       last_length = new_length;
+                               } else {
+                                       if (canceled) {
+                                               /* We have to remove the recipient if not resolved */
+                                               modest_recpt_editor_replace_with_resolved_recipient (recpt_editor,
+                                                                                                    &start_iter, 
+                                                                                                    &end_iter,
+                                                                                                    NULL,
+                                                                                                    NULL);
+                                       } else {
+                                               /* There is no contact with that name so it's not
+                                                  valid. Don't show any error because it'll be done
+                                                  later */
+                                               result = FALSE;
+                                       }
                                }
                        } else {
                                /* this address is not valid, select it and return control to user showing banner */
@@ -815,9 +1062,10 @@ modest_address_book_check_names (ModestRecptEditor *recpt_editor, gboolean updat
                /* note: adding it the to the addressbook if it did not exist yet,
                 * and adding it to the recent_list */
                if (result && update_addressbook && store_address)
-                       add_to_address_book (address);
+                       to_commit_addresses = g_slist_prepend (to_commit_addresses, address);
+               else
+                       g_free (address);
 
-               g_free (address);
                if (result == FALSE)
                        break;
 
@@ -825,6 +1073,10 @@ modest_address_book_check_names (ModestRecptEditor *recpt_editor, gboolean updat
                current_end = g_slist_next (current_end);
        }
 
+       /* Add addresses to address-book */
+       if (to_commit_addresses)
+               add_to_address_book (to_commit_addresses);
+
        if (current_start == NULL) {
                gtk_text_buffer_get_end_iter (buffer, &end_iter);
                gtk_text_buffer_place_cursor (buffer, &end_iter);
@@ -839,12 +1091,32 @@ modest_address_book_check_names (ModestRecptEditor *recpt_editor, gboolean updat
 
 }
 
+typedef struct _GetContactsInfo {
+       GMainLoop *mainloop;
+       GList *result;
+} GetContactsInfo;
+
+static void 
+get_contacts_for_name_cb (EBook *book, 
+                         EBookStatus status, 
+                         GList *list, 
+                         gpointer userdata)
+{
+       GetContactsInfo *info = (GetContactsInfo *) userdata;
+
+       if (status == E_BOOK_ERROR_OK)
+               info->result = list;
+
+       g_main_loop_quit (info->mainloop);
+}
+
 static GList *
 get_contacts_for_name (const gchar *name)
 {
        EBookQuery *full_name_book_query = NULL;
        GList *result;
        gchar *unquoted;
+       GetContactsInfo *info;
 
        if (name == NULL)
                return NULL;
@@ -853,8 +1125,19 @@ get_contacts_for_name (const gchar *name)
        full_name_book_query = e_book_query_field_test (E_CONTACT_FULL_NAME, E_BOOK_QUERY_CONTAINS, unquoted);
        g_free (unquoted);
 
-       e_book_get_contacts (book, full_name_book_query, &result, NULL);
+       /* TODO: Make it launch a mainloop */
+       info = g_slice_new (GetContactsInfo);
+       info->mainloop = g_main_loop_new (NULL, FALSE);
+       info->result = NULL;
+       if (e_book_async_get_contacts (book, full_name_book_query, get_contacts_for_name_cb, info) == 0) {
+               GDK_THREADS_LEAVE ();
+               g_main_loop_run (info->mainloop);
+               GDK_THREADS_ENTER ();
+       } 
+       result = info->result;
        e_book_query_unref (full_name_book_query);
+       g_main_loop_unref (info->mainloop);
+       g_slice_free (GetContactsInfo, info);
 
        return result;
 }
@@ -895,9 +1178,12 @@ select_contacts_for_name_dialog (const gchar *name)
 
                /* TODO: figure out how to make the contact chooser modal */
                contact_dialog = osso_abook_contact_chooser_new_with_capabilities (NULL,
-                                                                                  _("mcen_me_editor_selectrecipients"),
-                                                                                  OSSO_ABOOK_CAPS_EMAIL, 
+                                                                                  _AB("addr_ti_dia_select_contacts"),
+                                                                                  OSSO_ABOOK_CAPS_EMAIL,
                                                                                   OSSO_ABOOK_CONTACT_ORDER_NAME);
+               /* Enable multiselection */
+               osso_abook_contact_chooser_set_maximum_selection (OSSO_ABOOK_CONTACT_CHOOSER (contact_dialog),
+                                                                 G_MAXUINT);
                osso_abook_contact_chooser_set_model (OSSO_ABOOK_CONTACT_CHOOSER (contact_dialog),
                                                      contact_model);
 
@@ -913,30 +1199,40 @@ select_contacts_for_name_dialog (const gchar *name)
 }
 
 static gboolean
-resolve_address (const gchar *address, GSList **resolved_addresses, gchar **contact_id)
+resolve_address (const gchar *address, 
+                GSList **resolved_addresses, 
+                GSList **contact_ids,
+                gboolean *canceled)
 {
        GList *resolved_contacts;
-       guint banner_timeout;
-       GtkWidget *banner = NULL;
+       CheckNamesInfo *info;;
 
-       banner_timeout = g_timeout_add (500, show_check_names_banner, &banner);
+       g_return_val_if_fail (canceled, FALSE);
 
+       *resolved_addresses = NULL;
+       *contact_ids = NULL;
+       *canceled = FALSE;
+       info = g_slice_new0 (CheckNamesInfo);
+       show_check_names_banner (info);
+       
        contact_model = osso_abook_contact_model_new ();
        if (!open_addressbook_sync ()) {
+               hide_check_names_banner (info);
                if (contact_model) {
                        g_object_unref (contact_model);
                        contact_model = NULL;
                }
+               clean_check_names_banner (info);
                return FALSE;
        }
 
        resolved_contacts = get_contacts_for_name (address);
+       hide_check_names_banner (info);
 
        if (resolved_contacts == NULL) {
                /* no matching contacts for the search string */
                modest_platform_run_information_dialog (NULL, _("mcen_nc_no_matching_contacts"), FALSE);
-               hide_check_names_banner (&banner, banner_timeout);
-     
+               clean_check_names_banner (info);
                return FALSE;
        }
 
@@ -944,34 +1240,37 @@ resolve_address (const gchar *address, GSList **resolved_addresses, gchar **cont
                /* show a dialog to select the contact from the resolved ones */
                g_list_free (resolved_contacts);
 
-               hide_check_names_banner (&banner, banner_timeout);     
                resolved_contacts = select_contacts_for_name_dialog (address);
-               banner_timeout = g_timeout_add (500, show_check_names_banner, &banner);
-
        }
-       
+
        /* get the resolved contacts (can be no contact) */
        if (resolved_contacts) {
-               gboolean found;
-               EContact *contact = (EContact *) resolved_contacts->data;
-
-               *resolved_addresses = get_recipients_for_given_contact (contact);
-               hide_check_names_banner (&banner, banner_timeout);     
-               if (*resolved_addresses) {
-                       *contact_id = g_strdup (e_contact_get_const (contact, E_CONTACT_UID));
-                       found = TRUE;
-               } else {
-                       found = FALSE;
+               GList *node;
+               gboolean found = FALSE;
+
+               for (node = resolved_contacts; node != NULL; node = g_list_next (node)) {
+                       EContact *contact = (EContact *) node->data;
+                       GSList *resolved;
+                       gchar *contact_id;
+
+                       resolved = get_recipients_for_given_contact (contact, canceled);
+                       if (resolved) {
+                               contact_id = g_strdup (e_contact_get_const (contact, E_CONTACT_UID));
+                               *contact_ids = g_slist_append (*contact_ids, contact_id);
+                               found = TRUE;
+                               *resolved_addresses = g_slist_append (*resolved_addresses, resolved);
+                       }
                }
 
                g_list_foreach (resolved_contacts, (GFunc)unref_gobject, NULL);
                g_list_free (resolved_contacts);
+               clean_check_names_banner (info);
 
                return found;
        } else {
                /* cancelled dialog to select more than one contact or
                 * selected no contact */
-               hide_check_names_banner (&banner, banner_timeout);     
+               clean_check_names_banner (info);
                return FALSE;
        }
 
@@ -994,7 +1293,7 @@ unquote_string (const gchar *str)
                                if (*p == '\\') {
                                        g_string_append_unichar (buffer, g_utf8_get_char (p));
                                        p = g_utf8_next_char (p);
-                                       
+
                                }
                                g_string_append_unichar (buffer, g_utf8_get_char (p));
                                p = g_utf8_next_char (p);
@@ -1035,6 +1334,8 @@ modest_address_book_has_address (const gchar *address)
                            err ? err->message : "<unknown>");
                if (err)
                        g_error_free (err);
+               g_free (email);
+               e_book_query_unref (query);
                return FALSE;
        }
        e_book_query_unref (query);
@@ -1049,3 +1350,20 @@ modest_address_book_has_address (const gchar *address)
 
        return result;
 }
+
+const gchar *
+modest_address_book_get_my_name ()
+{
+       OssoABookSelfContact *self_contact = osso_abook_self_contact_get_default ();
+
+       /* We are not using osso_abook_contact_get_display_name
+          because that method fallbacks to another fields if the name
+          is not defined */
+       if (self_contact)
+               if (OSSO_ABOOK_IS_CONTACT (self_contact))
+                       return osso_abook_contact_get_name ((OssoABookContact*)self_contact);
+               else
+                       return e_contact_get ((EContact *) self_contact, E_CONTACT_NAME);
+       else
+               return NULL;
+}