2006-02-01 [paul] 2.0.0cvs4
[claws.git] / src / compose.c
index c84c75baf326e01f34fdf4ff6549b39a6bb0df5b..54e569147419f340fc138e5376bbed8926d21e3a 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * Sylpheed -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (C) 1999-2005 Hiroyuki Yamamoto
+ * Copyright (C) 1999-2006 Hiroyuki Yamamoto and the Sylpheed-Claws team
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -72,7 +72,9 @@
 #endif
 #include <signal.h>
 #include <errno.h>
+#ifndef G_OS_WIN32  /* fixme we should have a configure test. */
 #include <libgen.h>
+#endif
 
 #if (HAVE_WCTYPE_H && HAVE_WCHAR_H)
 #  include <wchar.h>
@@ -264,7 +266,8 @@ static gint compose_write_to_file           (Compose        *compose,
                                                 gint            action);
 static gint compose_write_body_to_file         (Compose        *compose,
                                                 const gchar    *file);
-static gint compose_remove_reedit_target       (Compose        *compose);
+static gint compose_remove_reedit_target       (Compose        *compose,
+                                                gboolean        force);
 void compose_remove_draft                      (Compose        *compose);
 static gint compose_queue                      (Compose        *compose,
                                                 gint           *msgnum,
@@ -695,6 +698,12 @@ static GtkItemFactoryEntry compose_entries[] =
         ENC_ACTION(C_WINDOWS_1255)},
        {N_("/_Options/Character _encoding/---"), NULL, NULL, 0, "<Separator>"},
 
+       {N_("/_Options/Character _encoding/Arabic (ISO-8859-_6)"),
+        ENC_ACTION(C_ISO_8859_6)},
+       {N_("/_Options/Character _encoding/Arabic (Windows-1256)"),
+        ENC_ACTION(C_CP1256)},
+       {N_("/_Options/Character _encoding/---"), NULL, NULL, 0, "<Separator>"},
+
        {N_("/_Options/Character _encoding/Turkish (ISO-8859-_9)"),
         ENC_ACTION(C_ISO_8859_9)},
        {N_("/_Options/Character _encoding/---"), NULL, NULL, 0, "<Separator>"},
@@ -817,6 +826,7 @@ static void compose_create_tags(GtkTextView *text, Compose *compose)
                                         "foreground-gdk", &uri_color,
                                         NULL);
        compose->no_wrap_tag = gtk_text_buffer_create_tag(buffer, "no_wrap", NULL);
+       compose->no_join_tag = gtk_text_buffer_create_tag(buffer, "no_join", NULL);
 }
 
 Compose *compose_new(PrefsAccount *account, const gchar *mailto,
@@ -1114,6 +1124,24 @@ static void compose_extract_original_charset(Compose *compose)
        }
 }
 
+#define SIGNAL_BLOCK(buffer) {                                 \
+       g_signal_handlers_block_by_func(G_OBJECT(buffer),       \
+                               G_CALLBACK(compose_changed_cb), \
+                               compose);                       \
+       g_signal_handlers_block_by_func(G_OBJECT(buffer),       \
+                               G_CALLBACK(text_inserted),      \
+                               compose);                       \
+}
+
+#define SIGNAL_UNBLOCK(buffer) {                               \
+       g_signal_handlers_unblock_by_func(G_OBJECT(buffer),     \
+                               G_CALLBACK(compose_changed_cb), \
+                               compose);                       \
+       g_signal_handlers_unblock_by_func(G_OBJECT(buffer),     \
+                               G_CALLBACK(text_inserted),      \
+                               compose);                       \
+}
+
 static Compose *compose_generic_reply(MsgInfo *msginfo, gboolean quote,
                                  gboolean to_all, gboolean to_ml,
                                  gboolean to_sender,
@@ -1206,11 +1234,15 @@ static Compose *compose_generic_reply(MsgInfo *msginfo, gboolean quote,
                compose_force_encryption(compose, account, FALSE);
        }
 
+       SIGNAL_BLOCK(textbuf);
+       
        if (account->auto_sig)
                compose_insert_sig(compose, FALSE);
 
        compose_wrap_all(compose);
 
+       SIGNAL_UNBLOCK(textbuf);
+       
        gtk_widget_grab_focus(compose->text);
 
        undo_unblock(compose->undostruct);
@@ -1307,11 +1339,15 @@ Compose *compose_forward(PrefsAccount *account, MsgInfo *msginfo,
                procmsg_msginfo_free(full_msginfo);
        }
 
+       SIGNAL_BLOCK(textbuf);
+
        if (account->auto_sig)
                compose_insert_sig(compose, FALSE);
 
        compose_wrap_all(compose);
 
+       SIGNAL_UNBLOCK(textbuf);
+       
        gtk_text_buffer_get_start_iter(textbuf, &iter);
        gtk_text_buffer_place_cursor(textbuf, &iter);
 
@@ -1408,11 +1444,15 @@ Compose *compose_forward_multiple(PrefsAccount *account, GSList *msginfo_list)
                        _("Fw: multiple emails"));
        }
 
+       SIGNAL_BLOCK(textbuf);
+       
        if (account->auto_sig)
                compose_insert_sig(compose, FALSE);
 
        compose_wrap_all(compose);
 
+       SIGNAL_UNBLOCK(textbuf);
+       
        gtk_text_buffer_get_start_iter(textbuf, &iter);
        gtk_text_buffer_place_cursor(textbuf, &iter);
 
@@ -1788,26 +1828,33 @@ void compose_entry_append(Compose *compose, const gchar *address,
 
 void compose_entry_mark_default_to(Compose *compose, const gchar *mailto)
 {
-       static GtkStyle *bold_style = NULL;
-       static GdkColor bold_color;
+       static GdkColor yellow;
+       static GdkColor black;
+       static gboolean yellow_initialised = FALSE;
        GSList *h_list;
        GtkEntry *entry;
                
+       if (!yellow_initialised) {
+               gdk_color_parse("#f5f6be", &yellow);
+               gdk_color_parse("#000000", &black);
+               yellow_initialised = gdk_colormap_alloc_color(
+                       gdk_colormap_get_system(), &yellow, FALSE, TRUE);
+               yellow_initialised &= gdk_colormap_alloc_color(
+                       gdk_colormap_get_system(), &black, FALSE, TRUE);
+       }
+
        for (h_list = compose->header_list; h_list != NULL; h_list = h_list->next) {
                entry = GTK_ENTRY(((ComposeHeaderEntry *)h_list->data)->entry);
                if (gtk_entry_get_text(entry) && 
                    !g_utf8_collate(gtk_entry_get_text(entry), mailto)) {
-                       gtk_widget_ensure_style(GTK_WIDGET(entry));
-                       if (!bold_style) {
-                               gtkut_convert_int_to_gdk_color
-                                       (prefs_common.color_new, &bold_color);
-                               bold_style = gtk_style_copy(gtk_widget_get_style
-                                       (GTK_WIDGET(entry)));
-                               pango_font_description_set_weight
-                                       (bold_style->font_desc, PANGO_WEIGHT_BOLD);
-                               bold_style->fg[GTK_STATE_NORMAL] = bold_color;
+                       if (yellow_initialised) {
+                               gtk_widget_modify_base(
+                                       GTK_WIDGET(((ComposeHeaderEntry *)h_list->data)->entry),
+                                       GTK_STATE_NORMAL, &yellow);
+                               gtk_widget_modify_text(
+                                       GTK_WIDGET(((ComposeHeaderEntry *)h_list->data)->entry),
+                                       GTK_STATE_NORMAL, &black);
                        }
-                       gtk_widget_set_style(GTK_WIDGET(entry), bold_style);
                }
        }
 }
@@ -2120,6 +2167,9 @@ static gchar *compose_quote_fmt(Compose *compose, MsgInfo *msginfo,
        GtkTextIter iter;
        GtkTextMark *mark;
        
+
+       SIGNAL_BLOCK(buffer);
+
        if (!msginfo)
                msginfo = &dummyinfo;
 
@@ -2132,7 +2182,7 @@ static gchar *compose_quote_fmt(Compose *compose, MsgInfo *msginfo,
                if (buf == NULL)
                        alertpanel_error(_("Quote mark format error."));
                else
-                       Xstrdup_a(quote_str, buf, return NULL)
+                       Xstrdup_a(quote_str, buf, goto error)
        }
 
        if (fmt && *fmt != '\0') {
@@ -2147,7 +2197,7 @@ static gchar *compose_quote_fmt(Compose *compose, MsgInfo *msginfo,
                buf = quote_fmt_get_buffer();
                if (buf == NULL) {
                        alertpanel_error(_("Message reply/forward format error."));
-                       return NULL;
+                       goto error;
                }
        } else
                buf = "";
@@ -2155,13 +2205,6 @@ static gchar *compose_quote_fmt(Compose *compose, MsgInfo *msginfo,
        prev_autowrap = compose->autowrap;
        compose->autowrap = FALSE;
 
-       g_signal_handlers_block_by_func(G_OBJECT(buffer),
-                               G_CALLBACK(compose_changed_cb),
-                               compose);
-       g_signal_handlers_block_by_func(G_OBJECT(buffer),
-                               G_CALLBACK(text_inserted),
-                               compose);
-
        mark = gtk_text_buffer_get_insert(buffer);
        gtk_text_buffer_get_iter_at_mark(buffer, &iter, mark);
        if (g_utf8_validate(buf, -1, NULL)) { 
@@ -2193,12 +2236,12 @@ static gchar *compose_quote_fmt(Compose *compose, MsgInfo *msginfo,
        if (compose->autowrap && rewrap)
                compose_wrap_all(compose);
 
-       g_signal_handlers_unblock_by_func(G_OBJECT(buffer),
-                               G_CALLBACK(compose_changed_cb),
-                               compose);
-       g_signal_handlers_unblock_by_func(G_OBJECT(buffer),
-                               G_CALLBACK(text_inserted),
-                               compose);
+       goto ok;
+
+error:
+       buf = NULL;
+ok:
+       SIGNAL_UNBLOCK(buffer);
 
 
        return buf;
@@ -3161,8 +3204,16 @@ static gboolean compose_join_next_line(Compose *compose,
        if (quote_len > 0)
                gtk_text_buffer_delete(buffer, &iter_, &end);
 
-       /* delete linebreak and extra spaces */
+       /* don't join line breaks put by the user */
        prev = cur = iter_;
+       gtk_text_iter_backward_char(&cur);
+       if (gtk_text_iter_has_tag(&cur, compose->no_join_tag)) {
+               gtk_text_iter_forward_char(&cur);
+               *iter = cur;
+               return FALSE;
+       }
+       gtk_text_iter_forward_char(&cur);
+       /* delete linebreak and extra spaces */
        while (gtk_text_iter_backward_char(&cur)) {
                wc1 = gtk_text_iter_get_char(&cur);
                if (!g_unichar_isspace(wc1))
@@ -3218,6 +3269,7 @@ static gboolean compose_join_next_line(Compose *compose,
                g_warning("alloc error scanning URIs\n"); \
        }
 
+static gboolean automatic_break = FALSE;
 static void compose_beautify_paragraph(Compose *compose, GtkTextIter *par_iter, gboolean force)
 {
        GtkTextView *text = GTK_TEXT_VIEW(compose->text);
@@ -3259,6 +3311,7 @@ static void compose_beautify_paragraph(Compose *compose, GtkTextIter *par_iter,
        }
 
        /* go until paragraph end (empty line) */
+       
        while (!gtk_text_iter_ends_line(&iter)) {
                gchar *scanpos = NULL;
                /* parse table - in order of priority */
@@ -3293,10 +3346,16 @@ static void compose_beautify_paragraph(Compose *compose, GtkTextIter *par_iter,
                gchar *o_walk = NULL, *walk = NULL, *bp = NULL, *ep = NULL;
                gint walk_pos;
                
+               if (!prev_autowrap)
+                       g_signal_handlers_block_by_func(G_OBJECT(buffer),
+                                       G_CALLBACK(text_inserted),
+                                       compose);
+               
                if (gtk_text_iter_has_tag(&iter, compose->no_wrap_tag) && !force)
                        goto colorize;
 
                uri_start = uri_stop = -1;
+               quote_len = 0;
                quote_str = compose_get_quote_str(buffer, &iter, &quote_len);
 
                if (quote_str) {
@@ -3322,11 +3381,15 @@ static void compose_beautify_paragraph(Compose *compose, GtkTextIter *par_iter,
                                               quote_len)) {
                        GtkTextIter prev, next, cur;
                        
-                       if (prev_autowrap != FALSE || force)
+                       if (prev_autowrap != FALSE || force) {
+                               automatic_break = TRUE;
                                gtk_text_buffer_insert(buffer, &break_pos, "\n", 1);
-                       else if (quote_str && wrap_quote)
+                               automatic_break = FALSE;
+                       } else if (quote_str && wrap_quote) {
+                               automatic_break = TRUE;
                                gtk_text_buffer_insert(buffer, &break_pos, "\n", 1);
-                       else 
+                               automatic_break = FALSE;
+                       } else 
                                goto colorize;
                        /* remove trailing spaces */
                        cur = break_pos;
@@ -3356,6 +3419,10 @@ static void compose_beautify_paragraph(Compose *compose, GtkTextIter *par_iter,
 
                        /* move iter to current line start */
                        gtk_text_iter_set_line_offset(&iter, 0);
+                       if (quote_str) {
+                               g_free(quote_str);
+                               quote_str = NULL;
+                       }
                        continue;
                } else {
                        /* move iter to next line start */
@@ -3363,6 +3430,10 @@ static void compose_beautify_paragraph(Compose *compose, GtkTextIter *par_iter,
                }
 
 colorize:
+               if (!prev_autowrap)
+                       g_signal_handlers_unblock_by_func(G_OBJECT(buffer),
+                                       G_CALLBACK(text_inserted),
+                                       compose);
                end_of_line = iter;
                while (!gtk_text_iter_ends_line(&end_of_line)) {
                        gtk_text_iter_forward_char(&end_of_line);
@@ -3450,6 +3521,12 @@ colorize:
        compose->autowrap = prev_autowrap;
 }
 
+void compose_action_cb(void *data)
+{
+       Compose *compose = (Compose *)data;
+       compose_wrap_all(compose);
+}
+
 static void compose_wrap_all(Compose *compose)
 {
        compose_wrap_all_full(compose, FALSE);
@@ -3613,8 +3690,8 @@ static gboolean compose_check_for_set_recipients(Compose *compose)
                        AlertValue aval;
                        aval = alertpanel(_("Send"),
                                          _("The only recipient is the default CC address. Send anyway?"),
-                                         GTK_STOCK_YES, GTK_STOCK_NO, NULL);
-                       if (aval != G_ALERTDEFAULT)
+                                         GTK_STOCK_CANCEL, _("+_Send"), NULL);
+                       if (aval != G_ALERTALTERNATE)
                                return FALSE;
                }
        }
@@ -3641,8 +3718,8 @@ static gboolean compose_check_for_set_recipients(Compose *compose)
                        AlertValue aval;
                        aval = alertpanel(_("Send"),
                                          _("The only recipient is the default BCC address. Send anyway?"),
-                                         GTK_STOCK_YES, GTK_STOCK_NO, NULL);
-                       if (aval != G_ALERTDEFAULT)
+                                         GTK_STOCK_CANCEL, _("+_Send"), NULL);
+                       if (aval != G_ALERTALTERNATE)
                                return FALSE;
                }
        }
@@ -3668,8 +3745,8 @@ static gboolean compose_check_entries(Compose *compose, gboolean check_subject)
 
                aval = alertpanel(_("Send"),
                                  _("Subject is empty. Send it anyway?"),
-                                 GTK_STOCK_YES, GTK_STOCK_NO, NULL);
-               if (aval != G_ALERTDEFAULT)
+                                 GTK_STOCK_CANCEL, _("+_Send"), NULL);
+               if (aval != G_ALERTALTERNATE)
                        return FALSE;
        }
 
@@ -3698,7 +3775,7 @@ gint compose_send(Compose *compose)
                } else if (val == -3) {
                        alertpanel_error(_("Could not queue message for sending:\n\n"
                                           "Signature failed."));
-               } else if (val == -2) {
+               } else if (val == -2 && errno != 0) {
                        alertpanel_error(_("Could not queue message for sending:\n\n%s."), strerror(errno));
                } else {
                        alertpanel_error(_("Could not queue message for sending."));
@@ -3721,6 +3798,7 @@ gint compose_send(Compose *compose)
                                   "the main window to retry."));
                if (prefs_common.send_dialog_mode == SEND_DIALOG_ALWAYS) {
                        compose->sending = FALSE;
+                       compose->modified = TRUE; 
                        compose_allow_user_actions (compose, TRUE);
                }
                return 0;
@@ -3750,6 +3828,7 @@ gint compose_send(Compose *compose)
                                   "the main window to retry."));
                if (prefs_common.send_dialog_mode == SEND_DIALOG_ALWAYS) {
                        compose_allow_user_actions (compose, TRUE);
+                       compose->modified = TRUE; 
                        compose->sending = FALSE;               
                }
                return -1;
@@ -3760,6 +3839,7 @@ gint compose_send(Compose *compose)
 bail:
        compose_allow_user_actions (compose, TRUE);
        compose->sending = FALSE;
+       compose->modified = TRUE; 
 
        return -1;
 }
@@ -3985,6 +4065,7 @@ static gint compose_write_to_file(Compose *compose, FILE *fp, gint action)
         mimemsg->type = MIMETYPE_MESSAGE;
         mimemsg->subtype = g_strdup("rfc822");
        mimemsg->content = MIMECONTENT_MEM;
+       mimemsg->tmp = TRUE; /* must free content later */
        mimemsg->data.mem = compose_get_header(compose);
 
        /* Create text part MimeInfo */
@@ -4016,7 +4097,8 @@ static gint compose_write_to_file(Compose *compose, FILE *fp, gint action)
                                test_conv_global_out = conv_codeset_strdup(chars, src_codeset, out_codeset);
                        }
                        
-                       if (!test_conv_global_out && compose->orig_charset) {
+                       if (!test_conv_global_out && compose->orig_charset
+                       &&  strcmp(compose->orig_charset, CS_US_ASCII)) {
                                out_codeset = compose->orig_charset;
                                debug_print("failure; trying to convert to %s\n", out_codeset);
                                test_conv_reply = conv_codeset_strdup(chars, src_codeset, out_codeset);
@@ -4025,7 +4107,7 @@ static gint compose_write_to_file(Compose *compose, FILE *fp, gint action)
                        if (!test_conv_global_out && !test_conv_reply) {
                                /* we're lost */
                                out_codeset = CS_INTERNAL;
-                               debug_print("finally using %s\n", out_codeset);
+                               debug_print("failure; finally using %s\n", out_codeset);
                        }
                        g_free(test_conv_global_out);
                        g_free(test_conv_reply);
@@ -4059,11 +4141,11 @@ static gint compose_write_to_file(Compose *compose, FILE *fp, gint action)
                                msg = g_strdup_printf(_("Can't convert the character encoding of the message \n"
                                                        "to the specified %s charset.\n"
                                                        "Send it as %s?"), out_codeset, src_codeset);
-                               aval = alertpanel_full(_("Error"), msg, GTK_STOCK_YES, GTK_STOCK_NO, NULL, FALSE,
-                                                     NULL, ALERT_ERROR, G_ALERTALTERNATE);
+                               aval = alertpanel_full(_("Error"), msg, GTK_STOCK_CANCEL, _("+_Send"), NULL, FALSE,
+                                                     NULL, ALERT_ERROR, G_ALERTDEFAULT);
                                g_free(msg);
 
-                               if (aval != G_ALERTDEFAULT) {
+                               if (aval != G_ALERTALTERNATE) {
                                        g_free(chars);
                                        return -3;
                                } else {
@@ -4089,6 +4171,7 @@ static gint compose_write_to_file(Compose *compose, FILE *fp, gint action)
 
        mimetext = procmime_mimeinfo_new();
        mimetext->content = MIMECONTENT_MEM;
+       mimetext->tmp = TRUE; /* must free content later */
        mimetext->data.mem = buf;
        mimetext->type = MIMETYPE_TEXT;
        mimetext->subtype = g_strdup("plain");
@@ -4112,9 +4195,9 @@ static gint compose_write_to_file(Compose *compose, FILE *fp, gint action)
                           "The contents of the message might be broken on the way to the delivery.\n"
                           "\n"
                           "Send it anyway?"), line + 1);
-               aval = alertpanel(_("Warning"), msg, GTK_STOCK_OK, GTK_STOCK_CANCEL, NULL);
+               aval = alertpanel(_("Warning"), msg, GTK_STOCK_CANCEL, GTK_STOCK_OK, NULL);
                g_free(msg);
-               if (aval != G_ALERTDEFAULT) {
+               if (aval != G_ALERTALTERNATE) {
                        return -1;
                }
        }
@@ -4212,7 +4295,7 @@ static gint compose_write_body_to_file(Compose *compose, const gchar *file)
        return 0;
 }
 
-static gint compose_remove_reedit_target(Compose *compose)
+static gint compose_remove_reedit_target(Compose *compose, gboolean force)
 {
        FolderItem *item;
        MsgInfo *msginfo = compose->targetinfo;
@@ -4220,6 +4303,9 @@ static gint compose_remove_reedit_target(Compose *compose)
        g_return_val_if_fail(compose->mode == COMPOSE_REEDIT, -1);
        if (!msginfo) return -1;
 
+       if (!force && MSG_IS_LOCKED(msginfo->flags))
+               return 0;
+
        item = msginfo->folder;
        g_return_val_if_fail(item != NULL, -1);
 
@@ -4300,9 +4386,6 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
                }                       
        }
 
-        if (prefs_common.linewrap_at_send)
-               compose_wrap_all(compose);
-
        /* write queue header */
        tmp = g_strdup_printf("%s%cqueue.%p", get_tmp_dir(),
                              G_DIR_SEPARATOR, compose);
@@ -4459,7 +4542,7 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
        g_free(tmp);
 
        if (compose->mode == COMPOSE_REEDIT) {
-               compose_remove_reedit_target(compose);
+               compose_remove_reedit_target(compose, FALSE);
        }
 
        if ((msgnum != NULL) && (item != NULL)) {
@@ -4492,6 +4575,7 @@ static void compose_add_attachments(Compose *compose, MimeInfo *parent)
                mimepart = procmime_mimeinfo_new();
                mimepart->content = MIMECONTENT_FILE;
                mimepart->data.filename = g_strdup(ainfo->file);
+               mimepart->tmp = FALSE; /* or we destroy our attachment */
                mimepart->offset = 0;
 
                stat(ainfo->file, &statbuf);
@@ -5231,6 +5315,8 @@ static gboolean text_clicked(GtkWidget *text, GdkEventButton *event,
 #if USE_ASPELL
        if (event->button == 3) {
                GtkTextIter iter;
+               GtkTextIter sel_start, sel_end;
+               gboolean stuff_selected;
                gint x, y;
                /* move the cursor to allow GtkAspell to check the word
                 * under the mouse */
@@ -5239,7 +5325,18 @@ static gboolean text_clicked(GtkWidget *text, GdkEventButton *event,
                        &x, &y);
                gtk_text_view_get_iter_at_location (GTK_TEXT_VIEW(text),
                        &iter, x, y);
+               /* get selection */
+               stuff_selected = gtk_text_buffer_get_selection_bounds(
+                               GTK_TEXT_VIEW(text)->buffer,
+                               &sel_start, &sel_end);
+
                gtk_text_buffer_place_cursor (GTK_TEXT_VIEW(text)->buffer, &iter);
+               /* reselect stuff */
+               if (stuff_selected 
+               && gtk_text_iter_in_range(&iter, &sel_start, &sel_end)) {
+                       gtk_text_buffer_select_range(GTK_TEXT_VIEW(text)->buffer,
+                               &sel_start, &sel_end);
+               }
                return FALSE; /* pass the event so that the right-click goes through */
        }
 #endif
@@ -5445,6 +5542,8 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode)
                         G_CALLBACK(text_inserted), compose);
        g_signal_connect(G_OBJECT(text), "button_press_event",
                         G_CALLBACK(text_clicked), compose);
+       g_signal_connect(G_OBJECT(subject_entry), "changed",
+                        G_CALLBACK(compose_changed_cb), compose);
 
        /* drag and drop */
        gtk_drag_dest_set(text, GTK_DEST_DEFAULT_ALL, compose_mime_types, 
@@ -5979,9 +6078,9 @@ static void compose_template_apply(Compose *compose, Template *tmpl,
        GtkTextMark *mark;
        GtkTextIter iter;
        gchar *qmark;
-       gchar *parsed_str;
+       gchar *parsed_str = NULL;
        gint cursor_pos = 0;
-       if (!tmpl || !tmpl->value) return;
+       if (!tmpl) return;
 
        text = GTK_TEXT_VIEW(compose->text);
        buffer = gtk_text_view_get_buffer(text);
@@ -6003,25 +6102,26 @@ static void compose_template_apply(Compose *compose, Template *tmpl,
        mark = gtk_text_buffer_get_insert(buffer);
        gtk_text_buffer_get_iter_at_mark(buffer, &iter, mark);
 
-       if ((compose->replyinfo == NULL) && (compose->fwdinfo == NULL)) {
-               parsed_str = compose_quote_fmt(compose, NULL, tmpl->value,
-                                              NULL, NULL, FALSE);
-       } else {
-               if (prefs_common.quotemark && *prefs_common.quotemark)
-                       qmark = prefs_common.quotemark;
-               else
-                       qmark = "> ";
-
-               if (compose->replyinfo != NULL)
-                       parsed_str = compose_quote_fmt(compose, compose->replyinfo,
-                                                      tmpl->value, qmark, NULL, FALSE);
-               else if (compose->fwdinfo != NULL)
-                       parsed_str = compose_quote_fmt(compose, compose->fwdinfo,
-                                                      tmpl->value, qmark, NULL, FALSE);
-               else
-                       parsed_str = NULL;
+       if (tmpl->value) {
+               if ((compose->replyinfo == NULL) && (compose->fwdinfo == NULL)) {
+                       parsed_str = compose_quote_fmt(compose, NULL, tmpl->value,
+                                                      NULL, NULL, FALSE);
+               } else {
+                       if (prefs_common.quotemark && *prefs_common.quotemark)
+                               qmark = prefs_common.quotemark;
+                       else
+                               qmark = "> ";
+
+                       if (compose->replyinfo != NULL)
+                               parsed_str = compose_quote_fmt(compose, compose->replyinfo,
+                                                              tmpl->value, qmark, NULL, FALSE);
+                       else if (compose->fwdinfo != NULL)
+                               parsed_str = compose_quote_fmt(compose, compose->fwdinfo,
+                                                              tmpl->value, qmark, NULL, FALSE);
+                       else
+                               parsed_str = NULL;
+               }
        }
-
        if (replace && parsed_str && compose->account->auto_sig)
                compose_insert_sig(compose, FALSE);
 
@@ -6040,8 +6140,7 @@ static void compose_template_apply(Compose *compose, Template *tmpl,
                gtk_text_buffer_place_cursor(buffer, &iter);
        }
 
-       if (parsed_str)
-               compose_changed_cb(NULL, compose);
+       compose_changed_cb(NULL, compose);
 }
 
 static void compose_destroy(Compose *compose)
@@ -6374,11 +6473,17 @@ static void compose_attach_property_create(gboolean *cancelled)
        strlist = NULL;
        for (; mime_type_list != NULL; mime_type_list = mime_type_list->next) {
                MimeType *type = (MimeType *) mime_type_list->data;
-               strlist = g_list_append(strlist, 
-                               g_strdup_printf("%s/%s",
-                                       type->type, type->sub_type));
+               gchar *tmp;
+
+               tmp = g_strdup_printf("%s/%s", type->type, type->sub_type);
+
+               if (g_list_find_custom(strlist, tmp, (GCompareFunc)strcmp2))
+                       g_free(tmp);
+               else
+                       strlist = g_list_insert_sorted(strlist, (gpointer)tmp,
+                                       (GCompareFunc)strcmp2);
        }
-       
+
        gtk_combo_set_popdown_strings(GTK_COMBO(mimetype_entry), strlist);
 
        for (mime_type_list = strlist; mime_type_list != NULL; 
@@ -6416,8 +6521,8 @@ static void compose_attach_property_create(gboolean *cancelled)
        SET_LABEL_AND_ENTRY(_("Path"),      path_entry,     2);
        SET_LABEL_AND_ENTRY(_("File name"), filename_entry, 3);
 
-       gtkut_stock_button_set_create(&hbbox, &ok_btn, GTK_STOCK_OK,
-                                     &cancel_btn, GTK_STOCK_CANCEL,
+       gtkut_stock_button_set_create(&hbbox, &cancel_btn, GTK_STOCK_CANCEL,
+                                     &ok_btn, GTK_STOCK_OK,
                                      NULL, NULL);
        gtk_box_pack_end(GTK_BOX(vbox), hbbox, FALSE, FALSE, 0);
        gtk_widget_grab_default(ok_btn);
@@ -6601,12 +6706,12 @@ static gboolean compose_ext_editor_kill(Compose *compose)
                        (_("The external editor is still working.\n"
                           "Force terminating the process?\n"
                           "process group id: %d"), -pgid);
-               val = alertpanel_full(_("Notice"), msg, GTK_STOCK_YES, GTK_STOCK_NO,
-                                     NULL, FALSE, NULL, ALERT_WARNING, G_ALERTALTERNATE);
+               val = alertpanel_full(_("Notice"), msg, GTK_STOCK_NO, GTK_STOCK_YES,
+                                     NULL, FALSE, NULL, ALERT_WARNING, G_ALERTDEFAULT);
                        
                g_free(msg);
 
-               if (val == G_ALERTDEFAULT) {
+               if (val == G_ALERTALTERNATE) {
                        g_source_remove(compose->exteditor_tag);
                        g_io_channel_shutdown(compose->exteditor_ch,
                                              FALSE, NULL);
@@ -6944,6 +7049,7 @@ static void compose_draft_cb(gpointer data, guint action, GtkWidget *widget)
        static gboolean lock = FALSE;
        MsgInfo *newmsginfo;
        FILE *fp;
+       gboolean target_locked = FALSE;
        
        if (lock) return;
 
@@ -7001,6 +7107,12 @@ static void compose_draft_cb(gpointer data, guint action, GtkWidget *widget)
                goto unlock;
        }
        fclose(fp);
+       
+       if (compose->targetinfo) {
+               target_locked = MSG_IS_LOCKED(compose->targetinfo->flags);
+               flag.perm_flags = target_locked?MSG_LOCKED:0;
+       }
+       flag.tmp_flags = MSG_DRAFT;
 
        folder_item_scan(draft);
        if ((msgnum = folder_item_add_msg(draft, tmp, &flag, TRUE)) < 0) {
@@ -7014,13 +7126,16 @@ static void compose_draft_cb(gpointer data, guint action, GtkWidget *widget)
        draft->mtime = 0;       /* force updating */
 
        if (compose->mode == COMPOSE_REEDIT) {
-               compose_remove_reedit_target(compose);
+               compose_remove_reedit_target(compose, TRUE);
        }
 
        newmsginfo = folder_item_get_msginfo(draft, msgnum);
        if (newmsginfo) {
                procmsg_msginfo_unset_flags(newmsginfo, ~0, ~0);
-               procmsg_msginfo_set_flags(newmsginfo, 0, MSG_DRAFT);
+               if (target_locked)
+                       procmsg_msginfo_set_flags(newmsginfo, MSG_LOCKED, MSG_DRAFT);
+               else
+                       procmsg_msginfo_set_flags(newmsginfo, 0, MSG_DRAFT);
                if (compose_use_attach(compose))
                        procmsg_msginfo_set_flags(newmsginfo, 0,
                                                  MSG_HAS_ATTACHMENT);
@@ -7040,7 +7155,10 @@ static void compose_draft_cb(gpointer data, guint action, GtkWidget *widget)
                gchar *path;
 
                path = folder_item_fetch_msg(draft, msgnum);
-               g_return_if_fail(path != NULL);
+               if (path == NULL) {
+                       debug_print("can't fetch %s:%d\n",draft->path, msgnum);
+                       goto unlock;
+               }
                if (g_stat(path, &s) < 0) {
                        FILE_OP_ERROR(path, "stat");
                        g_free(path);
@@ -7054,6 +7172,8 @@ static void compose_draft_cb(gpointer data, guint action, GtkWidget *widget)
                compose->targetinfo->size = s.st_size;
                compose->targetinfo->mtime = s.st_mtime;
                compose->targetinfo->folder = draft;
+               if (target_locked)
+                       procmsg_msginfo_set_flags(compose->targetinfo, MSG_LOCKED, 0);
                compose->mode = COMPOSE_REEDIT;
                
                if (action == COMPOSE_AUTO_SAVE) {
@@ -7748,6 +7868,7 @@ static void compose_toggle_autowrap_cb(gpointer data, guint action,
        compose->autowrap = GTK_CHECK_MENU_ITEM(widget)->active;
        if (compose->autowrap)
                compose_wrap_all_full(compose, TRUE);
+       compose->autowrap = GTK_CHECK_MENU_ITEM(widget)->active;
 }
 
 static void compose_toggle_sign_cb(gpointer data, guint action,
@@ -8046,9 +8167,17 @@ static void text_inserted(GtkTextBuffer *buffer, GtkTextIter *iter,
                                  
                gtk_text_buffer_get_iter_at_mark(buffer, iter, mark);
                gtk_text_buffer_place_cursor(buffer, iter);
-       } else
-               gtk_text_buffer_insert(buffer, iter, text, len);
-
+       } else {
+               if (strcmp(text, "\n") || automatic_break
+               || gtk_text_iter_starts_line(iter))
+                       gtk_text_buffer_insert(buffer, iter, text, len);
+               else {
+                       debug_print("insert nowrap \\n\n");
+                       gtk_text_buffer_insert_with_tags_by_name(buffer, 
+                               iter, text, len, "no_join", NULL);
+               }
+       }
+       
        mark = gtk_text_buffer_create_mark(buffer, NULL, iter, FALSE);
        
        compose_beautify_paragraph(compose, iter, FALSE);