2006-09-18 [paul] 2.4.0cvs196
[claws.git] / src / compose.c
index 340ff6fa20b484f1bba8a60ac09401d5a9e9794a..a76bf917d7cafbdb2143a1a1d3b6b856174d5687 100644 (file)
 #include "toolbar.h"
 #include "inc.h"
 #include "message_search.h"
+#include "combobox.h"
 
 enum
 {
@@ -265,21 +266,19 @@ static gboolean compose_check_entries             (Compose        *compose,
                                                 gboolean       check_subject);
 static gint compose_write_to_file              (Compose        *compose,
                                                 FILE           *fp,
-                                                gint            action);
+                                                gint            action,
+                                                gboolean        attach_parts);
 static gint compose_write_body_to_file         (Compose        *compose,
                                                 const gchar    *file);
 static gint compose_remove_reedit_target       (Compose        *compose,
                                                 gboolean        force);
 void compose_remove_draft                      (Compose        *compose);
-static gint compose_queue                      (Compose        *compose,
-                                                gint           *msgnum,
-                                                FolderItem     **item,
-                                                gchar          **msgpath);
 static gint compose_queue_sub                  (Compose        *compose,
                                                 gint           *msgnum,
                                                 FolderItem     **item,
                                                 gchar          **msgpath,
-                                                gboolean       check_subject);
+                                                gboolean       check_subject,
+                                                gboolean       remove_reedit_target);
 static void compose_add_attachments            (Compose        *compose,
                                                 MimeInfo       *parent);
 static gchar *compose_get_header               (Compose        *compose);
@@ -326,7 +325,9 @@ static void compose_undo_state_changed              (UndoMain       *undostruct,
 static void compose_create_header_entry        (Compose *compose);
 static void compose_add_header_entry   (Compose *compose, gchar *header, gchar *text);
 static void compose_update_priority_menu_item(Compose * compose);
-
+#if USE_ASPELL
+static void compose_spell_menu_changed (void *data);
+#endif
 static void compose_add_field_list     ( Compose *compose,
                                          GList *listAddress );
 
@@ -335,7 +336,7 @@ static void compose_add_field_list  ( Compose *compose,
 static gboolean compose_edit_size_alloc (GtkEditable   *widget,
                                         GtkAllocation  *allocation,
                                         GtkSHRuler     *shruler);
-static void account_activated          (GtkMenuItem    *menuitem,
+static void account_activated          (GtkComboBox *optmenu,
                                         gpointer        data);
 static void attach_selected            (GtkTreeView    *tree_view, 
                                         GtkTreePath    *tree_path,
@@ -506,8 +507,6 @@ static void compose_check_forwards_go          (Compose *compose);
 static gint compose_defer_auto_save_draft      (Compose        *compose);
 static PrefsAccount *compose_guess_forward_account_from_msginfo        (MsgInfo *msginfo);
 
-static gboolean compose_close  (Compose *compose);
-
 static GtkItemFactoryEntry compose_popup_entries[] =
 {
        {N_("/_Add..."),        NULL, compose_attach_cb, 0, NULL},
@@ -766,8 +765,8 @@ static GtkItemFactoryEntry compose_entries[] =
 static GtkTargetEntry compose_mime_types[] =
 {
        {"text/uri-list", 0, 0},
-       {"text/plain", 0, 0},
-       {"STRING", 0, 0}
+       {"UTF8_STRING", 0, 0},
+       {"text/plain", 0, 0}
 };
 
 static gboolean compose_put_existing_to_front(MsgInfo *info)
@@ -961,6 +960,7 @@ Compose *compose_generic_new(PrefsAccount *account, const gchar *mailto, FolderI
            compose->gtkaspell) 
                gtkaspell_change_dict(compose->gtkaspell, 
                    item->prefs->default_dictionary);
+       compose_spell_menu_changed(compose);
 #endif
 
        if (account->auto_sig)
@@ -982,7 +982,7 @@ Compose *compose_generic_new(PrefsAccount *account, const gchar *mailto, FolderI
        } else {
                if (mailto) {
                        compose_entry_append(compose, mailto, COMPOSE_NEWSGROUPS);
-               } else if (item) {
+               } else if (item && FOLDER_CLASS(item->folder) == news_get_class()) {
                        compose_entry_append(compose, item->path, COMPOSE_NEWSGROUPS);
                }
                /*
@@ -1316,6 +1316,7 @@ static Compose *compose_generic_reply(MsgInfo *msginfo, gboolean quote,
                compose_quote_fmt(compose, compose->replyinfo,
                                  prefs_common.quotefmt,
                                  qmark, body, FALSE);
+               quote_fmt_reset_vartable();
        }
        if (procmime_msginfo_is_encrypted(compose->replyinfo)) {
                compose_force_encryption(compose, account, FALSE);
@@ -1431,6 +1432,7 @@ Compose *compose_forward(PrefsAccount *account, MsgInfo *msginfo,
                compose_quote_fmt(compose, full_msginfo,
                                  prefs_common.fw_quotefmt,
                                  qmark, body, FALSE);
+               quote_fmt_reset_vartable();
                compose_attach_parts(compose, msginfo);
 
                procmsg_msginfo_free(full_msginfo);
@@ -1626,7 +1628,7 @@ static void compose_colorize_signature(Compose *compose)
                }
 }
 
-void compose_reedit(MsgInfo *msginfo)
+Compose *compose_reedit(MsgInfo *msginfo, gboolean batch)
 {
        Compose *compose = NULL;
        PrefsAccount *account = NULL;
@@ -1639,12 +1641,14 @@ void compose_reedit(MsgInfo *msginfo)
        gboolean use_signing = FALSE;
        gboolean use_encryption = FALSE;
        gchar *privacy_system = NULL;
+       int priority = PRIORITY_NORMAL;
+       MsgInfo *replyinfo = NULL, *fwdinfo = NULL;
 
-       g_return_if_fail(msginfo != NULL);
-       g_return_if_fail(msginfo->folder != NULL);
+       g_return_val_if_fail(msginfo != NULL, NULL);
+       g_return_val_if_fail(msginfo->folder != NULL, NULL);
 
        if (compose_put_existing_to_front(msginfo)) {
-               return;
+               return NULL;
        }
 
         if (folder_has_parent_of_type(msginfo->folder, F_QUEUE) ||
@@ -1690,7 +1694,29 @@ void compose_reedit(MsgInfo *msginfo)
                if (!procheader_get_header_from_msginfo(msginfo, queueheader_buf, 
                                             sizeof(queueheader_buf), "X-Priority: ")) {
                        param = atoi(&queueheader_buf[strlen("X-Priority: ")]); /* mind the space */
-                       compose->priority = param;
+                       priority = param;
+               }
+               if (!procheader_get_header_from_msginfo(msginfo, queueheader_buf, 
+                                            sizeof(queueheader_buf), "RMID:")) {
+                       gchar **tokens = g_strsplit(&queueheader_buf[strlen("RMID:")], "\t", 0);
+                       if (tokens[0] && tokens[1] && tokens[2]) {
+                               FolderItem *orig_item = folder_find_item_from_identifier(tokens[0]);
+                               if (orig_item != NULL) {
+                                       replyinfo = folder_item_get_msginfo_by_msgid(orig_item, tokens[2]);
+                               }
+                       }
+                       g_strfreev(tokens);
+               }
+               if (!procheader_get_header_from_msginfo(msginfo, queueheader_buf, 
+                                            sizeof(queueheader_buf), "FMID:")) {
+                       gchar **tokens = g_strsplit(&queueheader_buf[strlen("FMID:")], "\t", 0);
+                       if (tokens[0] && tokens[1] && tokens[2]) {
+                               FolderItem *orig_item = folder_find_item_from_identifier(tokens[0]);
+                               if (orig_item != NULL) {
+                                       fwdinfo = folder_item_get_msginfo_by_msgid(orig_item, tokens[2]);
+                               }
+                       }
+                       g_strfreev(tokens);
                }
        } else {
                account = msginfo->folder->folder->account;
@@ -1706,12 +1732,16 @@ void compose_reedit(MsgInfo *msginfo)
         if (!account) {
                account = cur_account;
         }
-       g_return_if_fail(account != NULL);
+       g_return_val_if_fail(account != NULL, NULL);
 
-       compose = compose_create(account, COMPOSE_REEDIT, FALSE);
+       compose = compose_create(account, COMPOSE_REEDIT, batch);
        
+       compose->replyinfo = replyinfo;
+       compose->fwdinfo = fwdinfo;
+
        compose->updating = TRUE;
-       
+       compose->priority = priority;
+
        if (privacy_system != NULL) {
                compose->privacy_system = privacy_system;
                compose_use_signing(compose, use_signing);
@@ -1750,7 +1780,7 @@ void compose_reedit(MsgInfo *msginfo)
        if (compose_parse_header(compose, msginfo) < 0) {
                compose->updating = FALSE;
                compose_destroy(compose);
-               return;
+               return NULL;
        }
        compose_reedit_set_entry(compose, msginfo);
 
@@ -1810,7 +1840,12 @@ void compose_reedit(MsgInfo *msginfo)
 
        if (compose->deferred_destroy) {
                compose_destroy(compose);
+               return NULL;
        }
+       
+       compose->sig_str = compose_get_signature_str(compose);
+       
+       return compose;
 }
 
 Compose *compose_redirect(PrefsAccount *account, MsgInfo *msginfo,
@@ -1871,6 +1906,7 @@ Compose *compose_redirect(PrefsAccount *account, MsgInfo *msginfo,
        gtk_editable_set_editable(GTK_EDITABLE(compose->subject_entry), FALSE);
 
        compose_quote_fmt(compose, msginfo, "%M", NULL, NULL, FALSE);
+       quote_fmt_reset_vartable();
        gtk_text_view_set_editable(GTK_TEXT_VIEW(compose->text), FALSE);
 
        compose_colorize_signature(compose);
@@ -2075,7 +2111,7 @@ static void compose_entries_set(Compose *compose, const gchar *mailto)
        gchar *subject = NULL;
        gchar *body = NULL;
        gchar *temp = NULL;
-       gint  len = 0;
+       guint  len = 0;
 
        scan_mailto_url(mailto, &to, &cc, NULL, &subject, &body);
 
@@ -2571,7 +2607,7 @@ static void compose_reply_set_entry(Compose *compose, MsgInfo *msginfo,
 
        if (msginfo->subject && *msginfo->subject) {
                gchar *buf, *buf2;
-               guchar *p;
+               gchar *p;
 
                buf = p = g_strdup(msginfo->subject);
                p += subject_get_prefix_length(p);
@@ -2681,7 +2717,9 @@ static void compose_insert_sig(Compose *compose, gboolean replace)
        GtkTextMark *mark;
        GtkTextIter iter, iter_end;
        gint cur_pos;
+       gchar *search = NULL;
        gboolean prev_autowrap;
+       gboolean found = FALSE, shift = FALSE;
 
        
        g_return_if_fail(compose->account != NULL);
@@ -2699,8 +2737,9 @@ static void compose_insert_sig(Compose *compose, gboolean replace)
 
        gtk_text_buffer_get_end_iter(buffer, &iter);
 
-       if (replace && compose->sig_str) {
-               gboolean found;
+       search = compose->sig_str;
+again:
+       if (replace && search) {
                GtkTextIter first_iter, start_iter, end_iter;
 
                gtk_text_buffer_get_start_iter(buffer, &first_iter);
@@ -2709,7 +2748,7 @@ static void compose_insert_sig(Compose *compose, gboolean replace)
                        found = FALSE;
                else
                        found = gtk_text_iter_forward_search(&first_iter,
-                                                            compose->sig_str,
+                                                            search,
                                                             GTK_TEXT_SEARCH_TEXT_ONLY,
                                                             &start_iter, &end_iter,
                                                             NULL);
@@ -2718,6 +2757,12 @@ static void compose_insert_sig(Compose *compose, gboolean replace)
                        gtk_text_buffer_delete(buffer, &start_iter, &end_iter);
                        iter = start_iter;
                }
+       } 
+       if (replace && !found && search && strlen(search) > 2
+       &&  search[0] == '\n' && search[1] == '\n') {
+               search ++;
+               shift = TRUE;
+               goto again;
        }
 
        g_free(compose->sig_str);
@@ -2726,7 +2771,10 @@ static void compose_insert_sig(Compose *compose, gboolean replace)
                compose->sig_str = g_strdup("");
 
        cur_pos = gtk_text_iter_get_offset(&iter);
-       gtk_text_buffer_insert(buffer, &iter, compose->sig_str, -1);
+       if (shift && found)
+               gtk_text_buffer_insert(buffer, &iter, compose->sig_str + 1, -1);
+       else
+               gtk_text_buffer_insert(buffer, &iter, compose->sig_str, -1);
        /* skip \n\n */
        gtk_text_buffer_get_iter_at_offset(buffer, &iter, cur_pos);
        gtk_text_iter_forward_char(&iter);
@@ -3503,6 +3551,7 @@ static void compose_beautify_paragraph(Compose *compose, GtkTextIter *par_iter,
                        {"http://",  strcasestr, get_uri_part,   make_uri_string},
                        {"https://", strcasestr, get_uri_part,   make_uri_string},
                        {"ftp://",   strcasestr, get_uri_part,   make_uri_string},
+                       {"sftp://",  strcasestr, get_uri_part,   make_uri_string},
                        {"www.",     strcasestr, get_uri_part,   make_http_string},
                        {"mailto:",  strcasestr, get_uri_part,   make_uri_string},
                        {"@",        strcasestr, get_email_part, make_email_string}
@@ -3879,8 +3928,11 @@ static void compose_select_account(Compose *compose, PrefsAccount *account,
                                       
        activate_privacy_system(compose, account, FALSE);
 
-       if (!init && compose->mode != COMPOSE_REDIRECT)
+       if (!init && compose->mode != COMPOSE_REDIRECT) {
+               undo_block(compose->undostruct);
                compose_insert_sig(compose, TRUE);
+               undo_unblock(compose->undostruct);
+       }
 }
 
 gboolean compose_check_for_valid_recipient(Compose *compose) {
@@ -3949,6 +4001,9 @@ static gboolean compose_check_for_set_recipients(Compose *compose)
                }
                if (!found_other) {
                        AlertValue aval;
+                       if (compose->batch) {
+                               gtk_widget_show_all(compose->window);
+                       }
                        aval = alertpanel(_("Send"),
                                          _("The only recipient is the default CC address. Send anyway?"),
                                          GTK_STOCK_CANCEL, _("+_Send"), NULL);
@@ -3966,7 +4021,7 @@ static gboolean compose_check_for_set_recipients(Compose *compose)
                        entry = gtk_editable_get_chars(GTK_EDITABLE(((ComposeHeaderEntry *)list->data)->entry), 0, -1);
                        header = gtk_editable_get_chars(GTK_EDITABLE(GTK_COMBO(((ComposeHeaderEntry *)list->data)->combo)->entry), 0, -1);
                        g_strstrip(entry);
-                       if (strcmp(entry, compose->account->auto_cc)
+                       if (strcmp(entry, compose->account->auto_bcc)
                        ||  strcmp(header, (prefs_common.trans_hdr ? gettext("Bcc:") : "Bcc:"))) {
                                found_other = TRUE;
                                g_free(entry);
@@ -3977,6 +4032,9 @@ static gboolean compose_check_for_set_recipients(Compose *compose)
                }
                if (!found_other) {
                        AlertValue aval;
+                       if (compose->batch) {
+                               gtk_widget_show_all(compose->window);
+                       }
                        aval = alertpanel(_("Send"),
                                          _("The only recipient is the default BCC address. Send anyway?"),
                                          GTK_STOCK_CANCEL, _("+_Send"), NULL);
@@ -3992,6 +4050,9 @@ static gboolean compose_check_entries(Compose *compose, gboolean check_subject)
        const gchar *str;
 
        if (compose_check_for_valid_recipient(compose) == FALSE) {
+               if (compose->batch) {
+                       gtk_widget_show_all(compose->window);
+               }
                alertpanel_error(_("Recipient is not specified."));
                return FALSE;
        }
@@ -4019,10 +4080,12 @@ static gboolean compose_check_entries(Compose *compose, gboolean check_subject)
 gint compose_send(Compose *compose)
 {
        gint msgnum;
-       FolderItem *folder;
+       FolderItem *folder = NULL;
        gint val = -1;
        gchar *msgpath = NULL;
        gboolean discard_window = FALSE;
+       gchar *errstr = NULL;
+       MainWindow *mainwin = mainwindow_get_mainwindow();
 
        if (prefs_common.send_dialog_mode != SEND_DIALOG_ALWAYS
                        || compose->batch == TRUE)
@@ -4038,7 +4101,8 @@ gint compose_send(Compose *compose)
                goto bail;
        }
 
-       val = compose_queue(compose, &msgnum, &folder, &msgpath);
+       inc_lock();
+       val = compose_queue(compose, &msgnum, &folder, &msgpath, TRUE);
 
        if (val) {
                if (compose->batch) {
@@ -4047,9 +4111,13 @@ gint compose_send(Compose *compose)
                if (val == -4) {
                        alertpanel_error(_("Could not queue message for sending:\n\n"
                                           "Charset conversion failed."));
+               } else if (val == -5) {
+                       alertpanel_error(_("Could not queue message for sending:\n\n"
+                                          "Couldn't get recipient encryption key."));
                } else if (val == -3) {
+                       if (privacy_peek_error())
                        alertpanel_error(_("Could not queue message for sending:\n\n"
-                                          "Signature failed."));
+                                          "Signature failed: %s"), privacy_get_error());
                } else if (val == -2 && errno != 0) {
                        alertpanel_error(_("Could not queue message for sending:\n\n%s."), strerror(errno));
                } else {
@@ -4073,15 +4141,15 @@ gint compose_send(Compose *compose)
                if (!discard_window) {
                        goto bail;
                }
+               inc_unlock();
                return -1;
        }
-       
        if (msgpath == NULL) {
                msgpath = folder_item_fetch_msg(folder, msgnum);
-               val = procmsg_send_message_queue(msgpath);
+               val = procmsg_send_message_queue(msgpath, &errstr, folder, msgnum);
                g_free(msgpath);
        } else {
-               val = procmsg_send_message_queue(msgpath);
+               val = procmsg_send_message_queue(msgpath, &errstr, folder, msgnum);
                g_unlink(msgpath);
                g_free(msgpath);
        }
@@ -4100,21 +4168,35 @@ gint compose_send(Compose *compose)
                if (!discard_window)
                        compose_close(compose);
        } else {
-               alertpanel_error(_("The message was queued but could not be "
+               if (errstr) {
+                       gchar *tmp = g_strdup_printf(_("%s\nUse \"Send queued messages\" from "
+                                  "the main window to retry."), errstr);
+                       g_free(errstr);
+                       alertpanel_error_log(tmp);
+                       g_free(tmp);
+               } else {
+                       alertpanel_error_log(_("The message was queued but could not be "
                                   "sent.\nUse \"Send queued messages\" from "
                                   "the main window to retry."));
+               }
                if (!discard_window) {
                        goto bail;              
                }
+               inc_unlock();
                return -1;
        }
-
+       inc_unlock();
+       toolbar_main_set_sensitive(mainwin);
+       main_window_set_menu_sensitive(mainwin);
        return 0;
 
 bail:
+       inc_unlock();
        compose_allow_user_actions (compose, TRUE);
        compose->sending = FALSE;
        compose->modified = TRUE; 
+       toolbar_main_set_sensitive(mainwin);
+       main_window_set_menu_sensitive(mainwin);
 
        return -1;
 }
@@ -4324,7 +4406,7 @@ error:
        return -1;
 }
 
-static gint compose_write_to_file(Compose *compose, FILE *fp, gint action)
+static gint compose_write_to_file(Compose *compose, FILE *fp, gint action, gboolean attach_parts)
 {
        GtkTextBuffer *buffer;
        GtkTextIter start, end;
@@ -4335,6 +4417,9 @@ static gint compose_write_to_file(Compose *compose, FILE *fp, gint action)
        MimeInfo *mimemsg, *mimetext;
        gint line;
 
+       if (action == COMPOSE_WRITE_FOR_SEND)
+               attach_parts = TRUE;
+
        /* create message MimeInfo */
        mimemsg = procmime_mimeinfo_new();
         mimemsg->type = MIMETYPE_MESSAGE;
@@ -4452,12 +4537,16 @@ static gint compose_write_to_file(Compose *compose, FILE *fp, gint action)
        mimetext->subtype = g_strdup("plain");
        g_hash_table_insert(mimetext->typeparameters, g_strdup("charset"),
                            g_strdup(out_codeset));
+                           
        /* protect trailing spaces when signing message */
        if (action == COMPOSE_WRITE_FOR_SEND && compose->use_signing && 
            privacy_system_can_sign(compose->privacy_system)) {
                encoding = ENC_QUOTED_PRINTABLE;
        }
        
+       debug_print("main text: %d bytes encoded as %s in %d\n",
+               strlen(buf), out_codeset, encoding);
+
        /* check for line length limit */
        if (action == COMPOSE_WRITE_FOR_SEND &&
            encoding != ENC_QUOTED_PRINTABLE && encoding != ENC_BASE64 &&
@@ -4481,7 +4570,7 @@ static gint compose_write_to_file(Compose *compose, FILE *fp, gint action)
                procmime_encode_content(mimetext, encoding);
 
        /* append attachment parts */
-       if (compose_use_attach(compose)) {
+       if (compose_use_attach(compose) && attach_parts) {
                MimeInfo *mimempart;
                gchar *boundary = NULL;
                mimempart = procmime_mimeinfo_new();
@@ -4608,11 +4697,14 @@ void compose_remove_draft(Compose *compose)
 
 }
 
-static gint compose_queue(Compose *compose, gint *msgnum, FolderItem **item, gchar **msgpath)
+gint compose_queue(Compose *compose, gint *msgnum, FolderItem **item, gchar **msgpath,
+                  gboolean remove_reedit_target)
 {
-       return compose_queue_sub (compose, msgnum, item, msgpath, FALSE);
+       return compose_queue_sub (compose, msgnum, item, msgpath, FALSE, remove_reedit_target);
 }
-static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item, gchar **msgpath, gboolean check_subject)
+static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item, 
+                             gchar **msgpath, gboolean check_subject,
+                             gboolean remove_reedit_target)
 {
        FolderItem *queue;
        gchar *tmp;
@@ -4629,6 +4721,9 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
        
        if (compose_check_entries(compose, check_subject) == FALSE) {
                 lock = FALSE;
+               if (compose->batch) {
+                       gtk_widget_show_all(compose->window);
+               }
                 return -1;
        }
 
@@ -4666,6 +4761,7 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
        if ((fp = g_fopen(tmp, "wb")) == NULL) {
                FILE_OP_ERROR(tmp, "fopen");
                g_free(tmp);
+               lock = FALSE;
                return -2;
        }
 
@@ -4744,6 +4840,11 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
                                fprintf(fp, "X-Sylpheed-Encrypt:%d\n", compose->use_encryption);
                                /* and if encdata was null, it means there's been a problem in 
                                 * key selection */
+                               lock = FALSE;
+                               fclose(fp);
+                               g_unlink(tmp);
+                               g_free(tmp);
+                               return -5;
                        }
                        g_free(encdata);
                }
@@ -4777,7 +4878,9 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
                fprintf(fp, "FMID:%s\t%d\t%s\n", folderid, compose->fwdinfo->msgnum, compose->fwdinfo->msgid);
                g_free(folderid);
        }
-       fprintf(fp, "\n");
+
+       /* end of headers */
+       fprintf(fp, "X-Sylpheed-End-Special-Headers: 1\n");
 
        if (compose->redirect_filename != NULL) {
                if (compose_redirect_write_to_file(compose, fp) < 0) {
@@ -4789,7 +4892,7 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
                }
        } else {
                gint result = 0;
-               if ((result = compose_write_to_file(compose, fp, COMPOSE_WRITE_FOR_SEND)) < 0) {
+               if ((result = compose_write_to_file(compose, fp, COMPOSE_WRITE_FOR_SEND, TRUE)) < 0) {
                        lock = FALSE;
                        fclose(fp);
                        g_unlink(tmp);
@@ -4802,14 +4905,20 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
                FILE_OP_ERROR(tmp, "fclose");
                g_unlink(tmp);
                g_free(tmp);
+               lock = FALSE;
                return -2;
        }
 
-       queue = account_get_special_folder(compose->account, F_QUEUE);
+       if (item && *item) {
+               queue = *item;
+       } else {
+               queue = account_get_special_folder(compose->account, F_QUEUE);
+       }
        if (!queue) {
                g_warning("can't find queue folder\n");
                g_unlink(tmp);
                g_free(tmp);
+               lock = FALSE;
                return -1;
        }
        folder_item_scan(queue);
@@ -4817,6 +4926,7 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
                g_warning("can't queue the message\n");
                g_unlink(tmp);
                g_free(tmp);
+               lock = FALSE;
                return -1;
        }
        
@@ -4826,7 +4936,7 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
        } else
                *msgpath = tmp;
 
-       if (compose->mode == COMPOSE_REEDIT) {
+       if (compose->mode == COMPOSE_REEDIT && remove_reedit_target) {
                compose_remove_reedit_target(compose, FALSE);
        }
 
@@ -4980,10 +5090,8 @@ static gchar *compose_get_header(Compose *compose)
        header = g_string_sized_new(64);
 
        /* Date */
-       if (compose->account->add_date) {
-               get_rfc822_date(buf, sizeof(buf));
-               g_string_append_printf(header, "Date: %s\n", buf);
-       }
+       get_rfc822_date(buf, sizeof(buf));
+       g_string_append_printf(header, "Date: %s\n", buf);
 
        /* From */
        
@@ -5462,7 +5570,7 @@ GtkWidget *compose_create_attach(Compose *compose)
        gtk_tree_view_append_column(GTK_TREE_VIEW(attach_clist), column);
 
        gtk_tree_view_set_rules_hint(GTK_TREE_VIEW(attach_clist),
-                                    prefs_common.enable_rules_hint);
+                                    prefs_common.use_stripes_everywhere);
        selection = gtk_tree_view_get_selection(GTK_TREE_VIEW(attach_clist));
        gtk_tree_selection_set_mode(selection, GTK_SELECTION_MULTIPLE);
 
@@ -6114,7 +6222,9 @@ static GtkWidget *compose_account_option_menu_create(Compose *compose)
        GList *accounts;
        GtkWidget *hbox;
        GtkWidget *optmenu;
-       GtkWidget *menu;
+       GtkWidget *optmenubox;
+       GtkListStore *menu;
+       GtkTreeIter iter;
        GtkWidget *from_name = NULL;
 
        gint num = 0, def_menu = 0;
@@ -6122,8 +6232,9 @@ static GtkWidget *compose_account_option_menu_create(Compose *compose)
        accounts = account_get_list();
        g_return_val_if_fail(accounts != NULL, NULL);
 
-       optmenu = gtk_option_menu_new();
-       menu = gtk_menu_new();
+       optmenubox = gtk_event_box_new();
+       optmenu = gtkut_sc_combobox_create(optmenubox, FALSE);
+       menu = GTK_LIST_STORE(gtk_combo_box_get_model(GTK_COMBO_BOX(optmenu)));
 
        hbox = gtk_hbox_new(FALSE, 6);
        from_name = gtk_entry_new();
@@ -6133,12 +6244,11 @@ static GtkWidget *compose_account_option_menu_create(Compose *compose)
 
        for (; accounts != NULL; accounts = accounts->next, num++) {
                PrefsAccount *ac = (PrefsAccount *)accounts->data;
-               GtkWidget *menuitem;
                gchar *name, *from = NULL;
 
                if (ac == compose->account) def_menu = num;
 
-               name = g_markup_printf_escaped("From: <i>%s</i>",
+               name = g_markup_printf_escaped(_("From: <i>%s</i>"),
                                       ac->account_name);
                
                if (ac == compose->account) {
@@ -6154,24 +6264,21 @@ static GtkWidget *compose_account_option_menu_create(Compose *compose)
                                gtk_entry_set_text(GTK_ENTRY(from_name), from);
                        }
                }
-               MENUITEM_ADD(menu, menuitem, name, ac->account_id);
-               gtk_label_set_use_markup (
-                               GTK_LABEL (gtk_bin_get_child (GTK_BIN (menuitem))),
-                               TRUE);
+               COMBOBOX_ADD(menu, name, ac->account_id);
                g_free(name);
                g_free(from);
-               g_signal_connect(G_OBJECT(menuitem), "activate",
-                                G_CALLBACK(account_activated),
-                                compose);
        }
 
-       gtk_option_menu_set_menu(GTK_OPTION_MENU(optmenu), menu);
-       gtk_option_menu_set_history(GTK_OPTION_MENU(optmenu), def_menu);
+       gtk_combo_box_set_active(GTK_COMBO_BOX(optmenu), def_menu);
 
-       gtk_box_pack_start(GTK_BOX(hbox), optmenu, FALSE, FALSE, 0);
+       g_signal_connect(G_OBJECT(optmenu), "changed",
+                       G_CALLBACK(account_activated),
+                       compose);
+
+       gtk_box_pack_start(GTK_BOX(hbox), optmenubox, FALSE, FALSE, 0);
        gtk_box_pack_start(GTK_BOX(hbox), from_name, TRUE, TRUE, 0);
        
-       gtk_tooltips_set_tip(compose->tooltips, optmenu,
+       gtk_tooltips_set_tip(compose->tooltips, optmenubox,
                _("Account to use for this email"), NULL);
        gtk_tooltips_set_tip(compose->tooltips, from_name,
                _("Sender address to be used"), NULL);
@@ -6513,7 +6620,7 @@ static void compose_template_apply(Compose *compose, Template *tmpl,
 
        /* process the other fields */
        compose_template_apply_fields(compose, tmpl);
-       
+       quote_fmt_reset_vartable();
        compose_changed_cb(NULL, compose);
 }
 
@@ -6721,7 +6828,7 @@ static void compose_attach_property(Compose *compose)
 {
        GtkTreeView *tree_view = GTK_TREE_VIEW(compose->attach_clist);
        AttachInfo *ainfo;
-       GtkOptionMenu *optmenu;
+       GtkComboBox *optmenu;
        GtkTreeSelection *selection;
        GList *sel;
        GtkTreeModel *model;
@@ -6755,13 +6862,11 @@ static void compose_attach_property(Compose *compose)
        gtk_widget_show(attach_prop.window);
        manage_window_set_transient(GTK_WINDOW(attach_prop.window));
 
-       optmenu = GTK_OPTION_MENU(attach_prop.encoding_optmenu);
+       optmenu = GTK_COMBO_BOX(attach_prop.encoding_optmenu);
        if (ainfo->encoding == ENC_UNKNOWN)
-               menu_select_by_data(GTK_MENU(gtk_option_menu_get_menu(optmenu)),
-                                   GINT_TO_POINTER(ENC_BASE64));
+               combobox_select_by_data(optmenu, ENC_BASE64);
        else
-               menu_select_by_data(GTK_MENU(gtk_option_menu_get_menu(optmenu)),
-                                   GINT_TO_POINTER(ainfo->encoding));
+               combobox_select_by_data(optmenu, ainfo->encoding);
 
        gtk_entry_set_text(GTK_ENTRY(attach_prop.mimetype_entry),
                           ainfo->content_type ? ainfo->content_type : "");
@@ -6776,8 +6881,6 @@ static void compose_attach_property(Compose *compose)
                gchar *cnttype = NULL;
                gchar *file = NULL;
                off_t size = 0;
-               GtkWidget *menu;
-               GtkWidget *menuitem;
 
                cancelled = FALSE;
                gtk_main();
@@ -6802,10 +6905,7 @@ static void compose_attach_property(Compose *compose)
                        }
                }
 
-               menu = gtk_option_menu_get_menu(optmenu);
-               menuitem = gtk_menu_get_active(GTK_MENU(menu));
-               ainfo->encoding = GPOINTER_TO_INT
-                       (g_object_get_data(G_OBJECT(menuitem), MENU_VAL_ID));
+               ainfo->encoding = combobox_get_active_data(optmenu);
 
                entry_text = gtk_entry_get_text(GTK_ENTRY(attach_prop.path_entry));
                if (*entry_text != '\0') {
@@ -6873,14 +6973,14 @@ static void compose_attach_property_create(gboolean *cancelled)
        GtkWidget *mimetype_entry;
        GtkWidget *hbox;
        GtkWidget *optmenu;
-       GtkWidget *optmenu_menu;
-       GtkWidget *menuitem;
+       GtkListStore *optmenu_menu;
        GtkWidget *path_entry;
        GtkWidget *filename_entry;
        GtkWidget *hbbox;
        GtkWidget *ok_btn;
        GtkWidget *cancel_btn;
        GList     *mime_type_list, *strlist;
+       GtkTreeIter iter;
 
        debug_print("Creating attach_property window...\n");
 
@@ -6947,21 +7047,16 @@ static void compose_attach_property_create(gboolean *cancelled)
        gtk_table_attach(GTK_TABLE(table), hbox, 1, 2, 1, 2,
                         GTK_EXPAND|GTK_SHRINK|GTK_FILL, 0, 0, 0);
 
-       optmenu = gtk_option_menu_new();
-       gtk_box_pack_start(GTK_BOX(hbox), optmenu, TRUE, TRUE, 0);
-
-       optmenu_menu = gtk_menu_new();
-       MENUITEM_ADD(optmenu_menu, menuitem, "7bit", ENC_7BIT);
-       gtk_option_menu_set_menu(GTK_OPTION_MENU(optmenu), optmenu_menu);
-       MENUITEM_ADD(optmenu_menu, menuitem, "8bit", ENC_8BIT);
-       gtk_option_menu_set_menu(GTK_OPTION_MENU(optmenu), optmenu_menu);
-       MENUITEM_ADD(optmenu_menu, menuitem, "quoted-printable",
-                    ENC_QUOTED_PRINTABLE);
-       gtk_option_menu_set_menu(GTK_OPTION_MENU(optmenu), optmenu_menu);
+       optmenu = gtkut_sc_combobox_create(NULL, TRUE);
+       optmenu_menu = GTK_LIST_STORE(gtk_combo_box_get_model(GTK_COMBO_BOX(optmenu)));
 
-       MENUITEM_ADD(optmenu_menu, menuitem, "base64", ENC_BASE64);
+       COMBOBOX_ADD(optmenu_menu, "7bit", ENC_7BIT);
+       COMBOBOX_ADD(optmenu_menu, "8bit", ENC_8BIT);
+       COMBOBOX_ADD(optmenu_menu, "quoted-printable",  ENC_QUOTED_PRINTABLE);
+       COMBOBOX_ADD(optmenu_menu, "base64", ENC_BASE64);
+       gtk_combo_box_set_active(GTK_COMBO_BOX(optmenu), 0);
 
-       gtk_option_menu_set_menu(GTK_OPTION_MENU(optmenu), optmenu_menu);
+       gtk_box_pack_start(GTK_BOX(hbox), optmenu, TRUE, TRUE, 0);
 
        SET_LABEL_AND_ENTRY(_("Path"),      path_entry,     2);
        SET_LABEL_AND_ENTRY(_("File name"), filename_entry, 3);
@@ -7357,15 +7452,22 @@ static gboolean compose_edit_size_alloc(GtkEditable *widget,
        return TRUE;
 }
 
-static void account_activated(GtkMenuItem *menuitem, gpointer data)
+static void account_activated(GtkComboBox *optmenu, gpointer data)
 {
        Compose *compose = (Compose *)data;
 
        PrefsAccount *ac;
        gchar *folderidentifier;
+       gint account_id = 0;
+       GtkTreeModel *menu;
+       GtkTreeIter iter;
+
+       /* Get ID of active account in the combo box */
+       menu = gtk_combo_box_get_model(optmenu);
+       gtk_combo_box_get_active_iter(optmenu, &iter);
+       gtk_tree_model_get(menu, &iter, 1, &account_id, -1);
 
-       ac = account_find_from_id(
-               GPOINTER_TO_INT(g_object_get_data(G_OBJECT(menuitem), MENU_VAL_ID)));
+       ac = account_find_from_id(account_id);
        g_return_if_fail(ac != NULL);
 
        if (ac != compose->account)
@@ -7481,18 +7583,28 @@ static void compose_send_later_cb(gpointer data, guint action,
        Compose *compose = (Compose *)data;
        gint val;
 
-       val = compose_queue_sub(compose, NULL, NULL, NULL, TRUE);
-       if (!val) 
+       inc_lock();
+       val = compose_queue_sub(compose, NULL, NULL, NULL, TRUE, TRUE);
+       inc_unlock();
+
+       if (!val) {
                compose_close(compose);
-       else if (val == -2) {
+       } else if (val == -1) {
+               alertpanel_error(_("Could not queue message."));
+       } else if (val == -2) {
                alertpanel_error(_("Could not queue message:\n\n%s."), strerror(errno));
        } else if (val == -3) {
+               if (privacy_peek_error())
                alertpanel_error(_("Could not queue message for sending:\n\n"
-                                  "Signature failed."));
+                                  "Signature failed: %s"), privacy_get_error());
        } else if (val == -4) {
                alertpanel_error(_("Could not queue message for sending:\n\n"
                                   "Charset conversion failed."));
+       } else if (val == -5) {
+               alertpanel_error(_("Could not queue message for sending:\n\n"
+                                  "Couldn't get recipient encryption key."));
        }
+       toolbar_main_set_sensitive(mainwindow_get_mainwindow());
 }
 
 void compose_draft (gpointer data) 
@@ -7560,9 +7672,28 @@ static void compose_draft_cb(gpointer data, guint action, GtkWidget *widget)
                fprintf(fp, "X-Sylpheed-Encrypt:%d\n", compose->use_encryption);
                fprintf(fp, "X-Sylpheed-Privacy-System:%s\n", compose->privacy_system);
        }
-       fprintf(fp, "\n");
 
-       if (compose_write_to_file(compose, fp, COMPOSE_WRITE_FOR_STORE) < 0) {
+       /* Message-ID of message replying to */
+       if ((compose->replyinfo != NULL) && (compose->replyinfo->msgid != NULL)) {
+               gchar *folderid;
+               
+               folderid = folder_item_get_identifier(compose->replyinfo->folder);
+               fprintf(fp, "RMID:%s\t%d\t%s\n", folderid, compose->replyinfo->msgnum, compose->replyinfo->msgid);
+               g_free(folderid);
+       }
+       /* Message-ID of message forwarding to */
+       if ((compose->fwdinfo != NULL) && (compose->fwdinfo->msgid != NULL)) {
+               gchar *folderid;
+               
+               folderid = folder_item_get_identifier(compose->fwdinfo->folder);
+               fprintf(fp, "FMID:%s\t%d\t%s\n", folderid, compose->fwdinfo->msgnum, compose->fwdinfo->msgid);
+               g_free(folderid);
+       }
+
+       /* end of headers */
+       fprintf(fp, "X-Sylpheed-End-Special-Headers: 1\n");
+
+       if (compose_write_to_file(compose, fp, COMPOSE_WRITE_FOR_STORE, action != COMPOSE_AUTO_SAVE) < 0) {
                fclose(fp);
                g_unlink(tmp);
                g_free(tmp);
@@ -7585,7 +7716,6 @@ static void compose_draft_cb(gpointer data, guint action, GtkWidget *widget)
                goto unlock;
        }
        g_free(tmp);
-       draft->mtime = 0;       /* force updating */
 
        if (compose->mode == COMPOSE_REEDIT) {
                compose_remove_reedit_target(compose, TRUE);
@@ -7598,7 +7728,7 @@ static void compose_draft_cb(gpointer data, guint action, GtkWidget *widget)
                        procmsg_msginfo_set_flags(newmsginfo, MSG_LOCKED, MSG_DRAFT);
                else
                        procmsg_msginfo_set_flags(newmsginfo, 0, MSG_DRAFT);
-               if (compose_use_attach(compose))
+               if (compose_use_attach(compose) && action != COMPOSE_AUTO_SAVE)
                        procmsg_msginfo_set_flags(newmsginfo, 0,
                                                  MSG_HAS_ATTACHMENT);
 
@@ -7856,16 +7986,18 @@ static void entry_paste_clipboard(Compose *compose, GtkWidget *entry,
                GtkTextMark *mark_start = gtk_text_buffer_get_insert(buffer);
                GtkTextIter start_iter, end_iter;
                gint start, end;
-               
                gchar *contents = gtk_clipboard_wait_for_text(gtk_clipboard_get(clip));
 
                if (contents == NULL)
                        return;
 
+               undo_paste_clipboard(GTK_TEXT_VIEW(compose->text), compose->undostruct);
+
                /* we shouldn't delete the selection when middle-click-pasting, or we
                 * can't mid-click-paste our own selection */
-               if (clip != GDK_SELECTION_PRIMARY)
+               if (clip != GDK_SELECTION_PRIMARY) {
                        gtk_text_buffer_delete_selection(buffer, FALSE, TRUE);
+               }
                
                if (insert_place == NULL) {
                        /* if insert_place isn't specified, insert at the cursor.
@@ -7893,7 +8025,6 @@ static void entry_paste_clipboard(Compose *compose, GtkWidget *entry,
                        gtk_text_iter_backward_char(&start_iter);
                        compose_beautify_paragraph(compose, &start_iter, TRUE);
                }
-               
        } else if (GTK_IS_EDITABLE(entry))
                gtk_editable_paste_clipboard (GTK_EDITABLE(entry));
        
@@ -8475,6 +8606,43 @@ static void compose_insert_drag_received_cb (GtkWidget           *widget,
        /* strangely, testing data->type == gdk_atom_intern("text/uri-list", TRUE)
         * does not work */
        if (gdk_atom_name(data->type) && !strcmp(gdk_atom_name(data->type), "text/uri-list")) {
+               AlertValue val = G_ALERTDEFAULT;
+
+               switch (prefs_common.compose_dnd_mode) {
+                       case COMPOSE_DND_ASK:
+                               val = alertpanel_full(_("Insert or attach?"),
+                                        _("Do you want to insert the contents of the file(s) "
+                                          "into the message body, or attach it to the email?"),
+                                         GTK_STOCK_CANCEL, _("+_Insert"), _("_Attach"),
+                                         TRUE, NULL, ALERT_QUESTION, G_ALERTALTERNATE);
+                               break;
+                       case COMPOSE_DND_INSERT:
+                               val = G_ALERTALTERNATE;
+                               break;
+                       case COMPOSE_DND_ATTACH:
+                               val = G_ALERTOTHER;
+                               break;
+                       default:
+                               /* unexpected case */
+                               g_warning("error: unexpected compose_dnd_mode option value in compose_insert_drag_received_cb()");
+               }
+
+               if (val & G_ALERTDISABLE) {
+                       val &= ~G_ALERTDISABLE;
+                       /* remember what action to perform by default, only if we don't click Cancel */
+                       if (val == G_ALERTALTERNATE)
+                               prefs_common.compose_dnd_mode = COMPOSE_DND_INSERT;
+                       else if (val == G_ALERTOTHER)
+                                       prefs_common.compose_dnd_mode = COMPOSE_DND_ATTACH;
+               }
+
+               if (val == G_ALERTDEFAULT || val == G_ALERTCANCEL) {
+                       gtk_drag_finish(drag_context, FALSE, FALSE, time);
+                       return;
+               } else if (val == G_ALERTOTHER) {
+                       compose_attach_drag_received_cb(widget, drag_context, x, y, data, info, time, user_data);
+                       return;
+               } 
                list = uri_list_extract_filenames((const gchar *)data->data);
                for (tmp = list; tmp != NULL; tmp = tmp->next) {
                        compose_insert_file(compose, (const gchar *)tmp->data);
@@ -8644,6 +8812,7 @@ static void text_inserted(GtkTextBuffer *buffer, GtkTextIter *iter,
                gtk_text_buffer_place_cursor(buffer, iter);
 
                compose_quote_fmt(compose, NULL, "%Q", qmark, new_text, TRUE);
+               quote_fmt_reset_vartable();
                g_free(new_text);
                g_object_set_data(G_OBJECT(compose->text), "paste_as_quotation",
                                  GINT_TO_POINTER(paste_as_quotation - 1));
@@ -8760,7 +8929,7 @@ static PrefsAccount *compose_guess_forward_account_from_msginfo(MsgInfo *msginfo
        return account;
 }
 
-static gboolean compose_close(Compose *compose)
+gboolean compose_close(Compose *compose)
 {
        gint x, y;