* moved most quote-related funtions to (new) modest-text-utils.[ch]
[modest] / src / gtk-glade / modest-ui.c
index 49acd25..843edea 100644 (file)
@@ -24,6 +24,7 @@
 #include "../modest-tny-msg-view.h"
 #include "../modest-tny-transport-actions.h"
 
+#include "../modest-text-utils.h"
 
 #define MODEST_GLADE          PREFIX "/share/modest/glade/modest.glade"
 #define MODEST_GLADE_MAIN_WIN "main"
@@ -197,9 +198,9 @@ modest_ui_new (ModestConf *modest_conf)
        g_signal_connect (account_store_iface, "password_requested",
                          G_CALLBACK(on_password_requested),
                          NULL);
+       
        glade_init ();
-       priv->glade_xml = glade_xml_new (MODEST_GLADE,
-                                        NULL,NULL);
+       priv->glade_xml = glade_xml_new (MODEST_GLADE, NULL,NULL);
        if (!priv->glade_xml) {
                g_warning ("failed to do glade stuff");
                g_object_unref (obj);
@@ -374,8 +375,11 @@ modest_ui_show_edit_window (ModestUI *modest_ui, const gchar* to,
        gtk_entry_set_text(GTK_ENTRY(to_entry), to);
 
        buf = gtk_text_view_get_buffer(GTK_TEXT_VIEW(body_view));
-       gtk_text_buffer_set_text(buf, body, -1);
-
+       if (body) {
+               gtk_text_buffer_set_text(buf, body, -1);
+       } else {
+               gtk_text_buffer_set_text(buf, "", -1);
+       }
        g_signal_connect (win, "destroy", G_CALLBACK(hide_edit_window),
                          NULL);
 
@@ -416,7 +420,7 @@ modest_ui_last_window_closed (GObject *obj, gpointer data)
 
 void
 on_account_selector_selection_changed (GtkWidget *widget,
-       gpointer user_data)
+                                      gpointer user_data)
 {
        GtkTreeModel *model = gtk_combo_box_get_model(GTK_COMBO_BOX(widget));
        GtkTreeIter iter;
@@ -436,10 +440,8 @@ on_account_selector_selection_changed (GtkWidget *widget,
        }
 
        g_message("Value: '%s'\n", account_name);
-       fill_advanced_setup_dialog(GTK_DIALOG(user_data), account_name);
 }
 
-
 void
 on_account_settings1_activate (GtkMenuItem *menuitem,
                               gpointer user_data)
@@ -501,7 +503,6 @@ on_account_settings1_activate (GtkMenuItem *menuitem,
        g_object_unref(glade_xml);
 }
 
-
 static void
 on_folder_clicked (ModestTnyFolderTreeView *folder_tree,
                   TnyMsgFolderIface *folder,
@@ -664,7 +665,8 @@ on_new_mail_clicked (GtkWidget *widget, ModestUI *modest_ui)
        modest_ui_show_edit_window (modest_ui, "", "", "", "", "", NULL);
 }
 
-static gchar*
+
+static gchar *
 modest_ui_quote_msg(const TnyMsgIface *src, const gchar *from, time_t sent_date)
 {
        GList *parts;
@@ -673,23 +675,17 @@ modest_ui_quote_msg(const TnyMsgIface *src, const gchar *from, time_t sent_date)
        TnyTextBufferStream *dest;
        TnyMsgMimePartIface *body = NULL;
        GtkTextBuffer *buf;
-       GtkTextIter begin, end, iter1, iter2, iter3;
-       gchar *txt, *line;
-       gint tmp;
-       gint indent;
-       gboolean break_line;
-       gchar sent_str[101];
-       gchar from_cut[82];
-       gchar reply_head[202];
-
+       gint limit = 76; /* TODO: make configurable */
+       gchar *quoted;
+       
        buf = gtk_text_buffer_new(NULL);
        dest = tny_text_buffer_stream_new(buf);
+       
+       /* is the warning in this line due to a bug in tinymail? */
        parts  = (GList*) tny_msg_iface_get_parts (src);
 
        while (parts) {
-               /* TODO: maybe we'd like to quote more than one part?
-                *       cleanup, fix leaks
-                */
+               /* TODO: maybe we'd like to quote more than one part? */
                TnyMsgMimePartIface *part =
                        TNY_MSG_MIME_PART_IFACE(parts->data);
                if (tny_msg_mime_part_iface_content_type_is (part, "text/plain")) {
@@ -699,7 +695,7 @@ modest_ui_quote_msg(const TnyMsgIface *src, const gchar *from, time_t sent_date)
                parts = parts->next;
        }
        if (!body) {
-               return "";
+               return NULL;
        }
        buf    = gtk_text_buffer_new (NULL);
        stream = TNY_STREAM_IFACE(tny_text_buffer_stream_new (buf));
@@ -708,151 +704,23 @@ modest_ui_quote_msg(const TnyMsgIface *src, const gchar *from, time_t sent_date)
        tny_msg_mime_part_iface_decode_to_stream (body, stream);
        tny_stream_iface_reset (stream);
 
-       /* format sent_date */
-       strftime(sent_str, 100, "%c", localtime(&sent_date));
-       strncpy(from_cut, from, 80);
-       sprintf(reply_head, "On %s, %s wrote:\n", sent_str, from_cut);
-
-       gtk_text_buffer_get_iter_at_line(buf, &iter1, 0);
-       gtk_text_buffer_insert(buf, &iter1, reply_head, -1);
-       gtk_text_buffer_get_iter_at_line(buf, &iter1, 1);
-       while (TRUE) {
-               /* at each beginning of this while, iter1 must be at the beginning of
-                  the (next) line to quote */
-
-               iter2 = iter1;
-               if (gtk_text_iter_get_chars_in_line(&iter1) > 1) {
-                       /* check whether line is already quoted */
-                       iter2 = iter1;
-                       gtk_text_iter_forward_char (&iter2);
-                       txt = gtk_text_buffer_get_text (buf, &iter1, &iter2, FALSE);
-               } else {
-                       txt = "";
-               }
-               /* insert quotation mark */
-               tmp = gtk_text_iter_get_offset(&iter1);
-               gtk_text_buffer_insert(buf, &iter1, "> ", -1);
-
-               /* still at the beginning of the line */
-               gtk_text_buffer_get_iter_at_offset(buf, &iter1, tmp);
-               iter2 = iter1;
-
-               if (strcmp(txt, ">") != 0) {
-
-                       /* line was not already quoted */
-
-                       /* now check whether the line must be broken: */
-                       if (gtk_text_iter_get_chars_in_line(&iter2) >= 79) {
-
-                               gtk_text_iter_set_line_offset(&iter2, 79);
-
-                               /* move iter1 behind quote mark at the beginnig of the line */
-                               gtk_text_iter_forward_word_end(&iter1);
-
-                               /* save iter2 position */
-                               iter3 = iter2;
-
-                               /* move iter2 back one word (from breakpoint in line) */
-                               gtk_text_iter_backward_word_start(&iter2);
-
-                               /* check for one-word line (up to iter2) */
-                               if (!gtk_text_iter_compare(&iter1, &iter2) < 0) {
-                                       gtk_text_iter_forward_word_end(&iter2); /* BUG? */
-                               }
-
-                               /* insert linebreak */
-                               tmp = gtk_text_iter_get_offset(&iter2);
-                               gtk_text_buffer_insert(buf, &iter2, "\n", -1);
-                               gtk_text_buffer_get_iter_at_offset(buf, &iter1, tmp);
-
-                               /* move to the beginning of the "new" line */
-                               gtk_text_iter_forward_line(&iter1);
-
-                               /* try to kill 1 space */
-                               iter2 = iter1;
-                               gtk_text_iter_forward_char(&iter2);
-                               txt = gtk_text_buffer_get_text(buf, &iter1, &iter2, FALSE);
-                               if (strcmp(txt, " ") == 0) {
-                                       tmp = gtk_text_iter_get_offset(&iter1);
-                                       gtk_text_buffer_delete(buf, &iter1, &iter2);
-                                       gtk_text_buffer_get_iter_at_offset(buf, &iter1, tmp);
-                               }
-
-                               /* check whether there is a next line to merge */
-                               iter3 = iter1;
-                               if (!gtk_text_iter_forward_line(&iter3)) {
-                                       continue;
-                               }
-                               /* iter3 is now at the beginning of the next line.*/
-
-                               /* check for empty line */
-                               if (gtk_text_iter_get_chars_in_line(&iter3) < 2) {
-                                       continue;
-                               }
-
-                               /* check for quote */
-                               iter2 = iter3;
-                               gtk_text_iter_forward_char (&iter2);
-                               txt = gtk_text_buffer_get_text(buf, &iter3, &iter2, FALSE);
-                               if (strcmp(txt, ">") == 0) {
-                                       /* iter1 is still at the beginning of the newly broken
-                                        * so we don't have to cleanup */
-                                       continue;
-                               }
-
-                               if (!gtk_text_iter_forward_to_line_end(&iter1)) {
-                                       /* no further lines to merge */
-                                       continue;
-                               }
-
-                               /* "mark" newline */
-                               iter2 = iter1;
-                               gtk_text_iter_forward_char(&iter2);
-
-                               /* do the merge */
-                               tmp = gtk_text_iter_get_offset(&iter1);
-                               gtk_text_buffer_delete (buf, &iter1, &iter2);
-                               gtk_text_buffer_get_iter_at_offset (buf, &iter1, tmp);
-
-                               /* insert space */
-                               gtk_text_buffer_insert(buf, &iter1, " ", -1);
-                               gtk_text_buffer_get_iter_at_offset (buf, &iter1, tmp);
-
-                               /* move to beginning of line and continue */
-                               gtk_text_iter_set_line_offset(&iter1, 0);
-                               continue;
-
-                       } else {
-                               /* line doesn't have to be broken, we're done. */
-                               if (!gtk_text_iter_forward_line(&iter1)) {
-                                       break;
-                               }
-                               continue;
-                       }
-               } else {
-                       /* line was already quoted */
-                       if (!gtk_text_iter_forward_line(&iter1)) {
-                               break;
-                       }
-                       continue;
-               }
-       }
-
-       gtk_text_buffer_get_bounds (buf, &begin, &end);
-       txt = gtk_text_buffer_get_text (buf, &begin, &end, FALSE);
-
-       return txt;
+       quoted = modest_text_utils_quote (buf, from, sent_date, limit);
+               
+       g_object_unref(stream);
+       g_object_unref(buf);
+       return quoted;
 }
 
 static void
 modest_ui_reply_to_msg (ModestUI *modest_ui, TnyMsgHeaderIface *header,
                                                ModestTnyMsgView *msg_view) {
-       const gchar *subject, *from, *quoted;
-       time_t sent_date;
        const TnyMsgIface *msg;
        const TnyMsgFolderIface *folder;
-       gchar *re_sub;
-
+       GString *re_sub;
+       const gchar *subject, *from;
+       gchar *quoted;
+       time_t sent_date;
+       
        quoted = "";
        if (header) {
                folder = tny_msg_header_iface_get_folder (TNY_MSG_HEADER_IFACE(header));
@@ -867,25 +735,23 @@ modest_ui_reply_to_msg (ModestUI *modest_ui, TnyMsgHeaderIface *header,
                        return;
                }
                subject = tny_msg_header_iface_get_subject(header);
-               /* TODO: checks, free */
-               re_sub = malloc(strlen(subject) + 5);
-               strcpy (re_sub, "Re: ");
-               strcat (re_sub, subject);
+               re_sub = g_string_new(subject);
+               g_string_prepend(re_sub, "Re: ");
                /* FIXME: honor replyto, cc */
                from = tny_msg_header_iface_get_from(header);
                sent_date = tny_msg_header_iface_get_date_sent(header);
                quoted = modest_ui_quote_msg(msg, from, sent_date);
-
        } else {
-               printf("no header\n");
+               g_warning("no header");
                return;
        }
 
-       modest_ui_show_edit_window (modest_ui, from, "FIXME:cc", /* bcc */ "", re_sub, quoted, NULL);
+       modest_ui_show_edit_window (modest_ui, from, /* cc */ "", /* bcc */ "", re_sub->str, quoted, NULL);
+       g_free(quoted);
+       g_string_free(re_sub, TRUE);
 }
 
 
-/* WIP, testing az */
 static void
 on_reply_clicked (GtkWidget *widget, ModestUI *modest_ui)
 {
@@ -920,9 +786,11 @@ on_reply_clicked (GtkWidget *widget, ModestUI *modest_ui)
        sel = gtk_tree_view_get_selection (GTK_TREE_VIEW(header_view));
        g_return_if_fail (sel);
 
-       if (!gtk_tree_selection_get_selected (sel, &model, &iter))
+       if (!gtk_tree_selection_get_selected (sel, &model, &iter)) {
                /* no message was selected. TODO: disable reply button in this case */
+               g_warning("nothing to reply to");
                return;
+       }
 
        gtk_tree_model_get (model, &iter,
                            TNY_MSG_HEADER_LIST_MODEL_INSTANCE_COLUMN,
@@ -932,7 +800,6 @@ on_reply_clicked (GtkWidget *widget, ModestUI *modest_ui)
 }
 
 
-
 /* FIXME: truly evil --> we cannot really assume that
  * there is only one edit window open...
  */