remove unneeded code
[claws.git] / src / compose.c
index 5b32266d34b8a84801b1b295994fa04af3fa6989..1534742770d52203643f8053cd39361228eebb43 100644 (file)
 #include "template.h"
 #include "undo.h"
 #include "foldersel.h"
+#include "prefs_actions.h"
 
 #if USE_GPGME
 #  include "rfc2015.h"
@@ -171,7 +172,8 @@ static gchar *compose_parse_references              (const gchar    *ref,
 static gchar *compose_quote_fmt                        (Compose        *compose,
                                                 MsgInfo        *msginfo,
                                                 const gchar    *fmt,
-                                                const gchar    *qmark);
+                                                const gchar    *qmark,
+                                                const gchar    *body);
 
 static void compose_reply_set_entry            (Compose        *compose,
                                                 MsgInfo        *msginfo,
@@ -186,24 +188,24 @@ static void compose_insert_file                   (Compose        *compose,
                                                 const gchar    *file);
 static void compose_attach_append              (Compose        *compose,
                                                 const gchar    *file,
-                                                ContentType     cnttype);
-static void compose_attach_append_with_type(Compose *compose,
-                                           const gchar *file,
-                                           const gchar *type,
-                                           ContentType cnttype);
+                                                const gchar    *type,
+                                                const gchar    *content_type);
+static void compose_attach_parts               (Compose        *compose,
+                                                MsgInfo        *msginfo);
 static void compose_wrap_line                  (Compose        *compose);
 static void compose_wrap_line_all              (Compose        *compose);
 static void compose_set_title                  (Compose        *compose);
 
 static PrefsAccount *compose_current_mail_account(void);
 /* static gint compose_send                    (Compose        *compose); */
+static gboolean compose_check_for_valid_recipient
+                                               (Compose        *compose);
+static gboolean compose_check_entries          (Compose        *compose);
 static gint compose_write_to_file              (Compose        *compose,
                                                 const gchar    *file,
                                                 gboolean        is_draft);
 static gint compose_write_body_to_file         (Compose        *compose,
                                                 const gchar    *file);
-static gint compose_save_to_outbox             (Compose        *compose,
-                                                const gchar    *file);
 static gint compose_remove_reedit_target       (Compose        *compose);
 static gint compose_queue                      (Compose        *compose,
                                                 gint           *msgnum,
@@ -434,13 +436,11 @@ static void followupto_activated  (GtkWidget      *widget,
                                         Compose        *compose);
 #endif
 
-static void compose_attach_parts       (Compose        *compose,
-                                        MsgInfo        *msginfo);
-
 static void compose_generic_reply(MsgInfo *msginfo, gboolean quote,
                                  gboolean to_all,
                                  gboolean ignore_replyto,
-                                 gboolean followup_and_reply_to);
+                                 gboolean followup_and_reply_to,
+                                 const gchar *body);
 
 void compose_headerentry_changed_cb       (GtkWidget          *entry,
                                            ComposeHeaderEntry *headerentry);
@@ -624,10 +624,11 @@ static GtkItemFactoryEntry compose_entries[] =
 #endif /* USE_GPGME */
        {N_("/_Message/---"),           NULL,           NULL,   0, "<Separator>"},
        {N_("/_Message/_Request Return Receipt"),       NULL, compose_toggle_return_receipt_cb, 0, "<ToggleItem>"},
-       {N_("/_Tool"),                  NULL, NULL, 0, "<Branch>"},
-       {N_("/_Tool/Show _ruler"),      NULL, compose_toggle_ruler_cb, 0, "<ToggleItem>"},
-       {N_("/_Tool/_Address book"),    "<shift><control>A", compose_address_cb , 0, NULL},
-       {N_("/_Tool/_Template"),        NULL, NULL, 0, "<Branch>"},
+       {N_("/_Tools"),                 NULL, NULL, 0, "<Branch>"},
+       {N_("/_Tools/Show _ruler"),     NULL, compose_toggle_ruler_cb, 0, "<ToggleItem>"},
+       {N_("/_Tools/_Address book"),   "<shift><control>A", compose_address_cb , 0, NULL},
+       {N_("/_Tools/_Template"),       NULL, NULL, 0, "<Branch>"},
+       {N_("/_Tools/Actio_ns"),        NULL, NULL, 0, "<Branch>"},
        {N_("/_Help"),                  NULL, NULL, 0, "<Branch>"},
        {N_("/_Help/_About"),           NULL, about_show, 0, NULL}
 };
@@ -661,7 +662,7 @@ Compose *compose_bounce(PrefsAccount *account, MsgInfo *msginfo)
                                   msginfo->subject);
        gtk_editable_set_editable(GTK_EDITABLE(c->subject_entry), FALSE);
 
-       compose_quote_fmt(c, msginfo, "%M", NULL);
+       compose_quote_fmt(c, msginfo, "%M", NULL, NULL);
        gtk_editable_set_editable(GTK_EDITABLE(c->text), FALSE);
 
        ifactory = gtk_item_factory_from_widget(c->popupmenu);
@@ -683,7 +684,7 @@ Compose *compose_bounce(PrefsAccount *account, MsgInfo *msginfo)
        menu_set_sensitive(ifactory, "/Message/Encrypt", FALSE);
 #endif
        menu_set_sensitive(ifactory, "/Message/Request Return Receipt", FALSE);
-       menu_set_sensitive(ifactory, "/Tool/Template", FALSE);
+       menu_set_sensitive(ifactory, "/Tools/Template", FALSE);
        
        gtk_widget_set_sensitive(c->insert_btn, FALSE);
        gtk_widget_set_sensitive(c->attach_btn, FALSE);
@@ -794,22 +795,26 @@ Compose *compose_new_followup_and_replyto(PrefsAccount *account,
 */
 
 void compose_reply(MsgInfo *msginfo, gboolean quote, gboolean to_all,
-                  gboolean ignore_replyto)
+                  gboolean ignore_replyto, const gchar *body)
 {
-       compose_generic_reply(msginfo, quote, to_all, ignore_replyto, FALSE);
+       compose_generic_reply(msginfo, quote, to_all, ignore_replyto, FALSE,
+                             body);
 }
 
 void compose_followup_and_reply_to(MsgInfo *msginfo, gboolean quote,
                                   gboolean to_all,
-                                  gboolean ignore_replyto)
+                                  gboolean ignore_replyto,
+                                  const gchar *body)
 {
-       compose_generic_reply(msginfo, quote, to_all, ignore_replyto, TRUE);
+       compose_generic_reply(msginfo, quote, to_all, ignore_replyto, TRUE,
+                             body);
 }
 
 static void compose_generic_reply(MsgInfo *msginfo, gboolean quote,
                                  gboolean to_all,
                                  gboolean ignore_replyto,
-                                 gboolean followup_and_reply_to)
+                                 gboolean followup_and_reply_to,
+                                 const gchar *body)
 {
        Compose *compose;
        PrefsAccount *account;
@@ -916,7 +921,7 @@ static void compose_generic_reply(MsgInfo *msginfo, gboolean quote,
 
                quote_str = compose_quote_fmt(compose, msginfo,
                                              prefs_common.quotefmt,
-                                             qmark);
+                                             qmark, body);
        }
 
        if (prefs_common.auto_sig)
@@ -931,10 +936,11 @@ static void compose_generic_reply(MsgInfo *msginfo, gboolean quote,
        gtk_stext_thaw(text);
        gtk_widget_grab_focus(compose->text);
 
-        if (prefs_common.auto_exteditor)
+       if (prefs_common.auto_exteditor)
                compose_exec_ext_editor(compose);
 }
 
+static void set_toolbar_style(Compose *compose);
 
 static gchar *procmime_get_file_name(MimeInfo *mimeinfo)
 {
@@ -986,167 +992,6 @@ static gchar *mime_extract_file(gchar *source, MimeInfo *partinfo)
        return filename;
 }
 
-static void compose_attach_parts(Compose *compose, MsgInfo *msginfo)
-{
-       FILE *fp;
-       gchar *file;
-       MimeInfo *mimeinfo;
-       MsgInfo *tmpmsginfo;
-       gchar *p;
-       gchar *boundary;
-       gint boundary_len = 0;
-       gchar buf[BUFFSIZE];
-       glong fpos, prev_fpos;
-       gint npart;
-       gchar *source;
-       gchar *filename;
-
-       g_return_if_fail(msginfo != NULL);
-       
-#if USE_GPGME
-       for (;;) {
-               if ((fp = procmsg_open_message(msginfo)) == NULL) return;
-               mimeinfo = procmime_scan_mime_header(fp, MIME_TEXT);
-               if (!mimeinfo) break;
-
-               if (!MSG_IS_ENCRYPTED(msginfo->flags) &&
-                   rfc2015_is_encrypted(mimeinfo)) {
-                       MSG_SET_TMP_FLAGS(msginfo->flags, MSG_ENCRYPTED);
-               }
-               if (MSG_IS_ENCRYPTED(msginfo->flags) &&
-                   !msginfo->plaintext_file  &&
-                   !msginfo->decryption_failed) {
-                       rfc2015_decrypt_message(msginfo, mimeinfo, fp);
-                       if (msginfo->plaintext_file &&
-                           !msginfo->decryption_failed) {
-                               fclose(fp);
-                               continue;
-                       }
-               }
-               
-               break;
-       }
-#else /* !USE_GPGME */
-       if ((fp = procmsg_open_message(msginfo)) == NULL) return;
-       mimeinfo = procmime_scan_mime_header(fp, MIME_TEXT);
-#endif /* USE_GPGME */
-
-       fclose(fp);
-       if (!mimeinfo) return;
-       if (mimeinfo->mime_type == MIME_TEXT)
-               return;
-
-       if ((fp = procmsg_open_message(msginfo)) == NULL) return;
-
-       g_return_if_fail(mimeinfo != NULL);
-       g_return_if_fail(mimeinfo->mime_type != MIME_TEXT);
-
-       if (mimeinfo->mime_type == MIME_MULTIPART) {
-               g_return_if_fail(mimeinfo->boundary != NULL);
-               g_return_if_fail(mimeinfo->sub == NULL);
-       }
-       g_return_if_fail(fp != NULL);
-
-       boundary = mimeinfo->boundary;
-
-       if (boundary) {
-               boundary_len = strlen(boundary);
-
-               /* look for first boundary */
-               while ((p = fgets(buf, sizeof(buf), fp)) != NULL)
-                       if (IS_BOUNDARY(buf, boundary, boundary_len)) break;
-               if (!p) {
-                       fclose(fp);
-                       return;
-               }
-       }
-
-       if ((fpos = ftell(fp)) < 0) {
-               perror("ftell");
-               fclose(fp);
-               return;
-       }
-
-       for (npart = 0;; npart++) {
-               MimeInfo *partinfo;
-               gboolean eom = FALSE;
-
-               prev_fpos = fpos;
-
-               partinfo = procmime_scan_mime_header(fp, MIME_TEXT);
-               if (!partinfo) break;
-
-               if (npart != 0)
-                       procmime_mimeinfo_insert(mimeinfo, partinfo);
-               else
-                       procmime_mimeinfo_free(partinfo);
-
-               /* look for next boundary */
-               buf[0] = '\0';
-               while ((p = fgets(buf, sizeof(buf), fp)) != NULL) {
-                       if (IS_BOUNDARY(buf, boundary, boundary_len)) {
-                               if (buf[2 + boundary_len]     == '-' &&
-                                   buf[2 + boundary_len + 1] == '-')
-                                       eom = TRUE;
-                               break;
-                       }
-               }
-               if (p == NULL)
-                       eom = TRUE;     /* broken MIME message */
-               fpos = ftell(fp);
-
-               partinfo->size = fpos - prev_fpos - strlen(buf);
-
-               if (eom) break;
-       }
-
-       source = procmsg_get_message_file_path(msginfo);
-
-       g_return_if_fail(mimeinfo != NULL);
-
-       if (!mimeinfo->main && mimeinfo->parent)
-               {
-                       filename = mime_extract_file(source, mimeinfo);
-
-                       compose_attach_append_with_type(compose, filename,
-                                                       mimeinfo->content_type,
-                                                       mimeinfo->mime_type);
-
-                       g_free(filename);
-               }
-
-       if (mimeinfo->sub && mimeinfo->sub->children)
-               {
-                       filename = mime_extract_file(source, mimeinfo->sub);
-
-                       compose_attach_append_with_type(compose, filename,
-                                                       mimeinfo->content_type,
-                                                       mimeinfo->sub->mime_type);
-
-                       g_free(filename);
-               }
-
-       if (mimeinfo->children) {
-               MimeInfo *child;
-
-               child = mimeinfo->children;
-               while (child) {
-                       filename = mime_extract_file(source, child);
-
-                       compose_attach_append_with_type(compose, filename,
-                                                       child->content_type,
-                                                       child->mime_type);
-
-                       g_free(filename);
-
-                       child = child->next;
-               }
-       }
-
-       fclose(fp);
-
-       procmime_mimeinfo_free_all(mimeinfo);
-}
 
 
 #define INSERT_FW_HEADER(var, hdr) \
@@ -1157,7 +1002,7 @@ if (msginfo->var && *msginfo->var) { \
 }
 
 Compose *compose_forward(PrefsAccount *account, MsgInfo *msginfo,
-                        gboolean as_attach)
+                        gboolean as_attach, const gchar *body)
 {
        Compose *compose;
        /*      PrefsAccount *account; */
@@ -1208,30 +1053,33 @@ Compose *compose_forward(PrefsAccount *account, MsgInfo *msginfo,
        text = GTK_STEXT(compose->text);
        gtk_stext_freeze(text);
 
-       if (as_attach) {
-               gchar *msgfile;
 
-               msgfile = procmsg_get_message_file_path(msginfo);
-               if (!is_file_exist(msgfile))
-                       g_warning(_("%s: file not exist\n"), msgfile);
-               else
-                       compose_attach_append(compose, msgfile,
-                                             MIME_MESSAGE_RFC822);
+               if (as_attach) {
+                       gchar *msgfile;
 
-               g_free(msgfile);
-       } else {
-               gchar *qmark;
-               gchar *quote_str;
+                       msgfile = procmsg_get_message_file_path(msginfo);
+                       if (!is_file_exist(msgfile))
+                               g_warning(_("%s: file not exist\n"), msgfile);
+                       else
+                               compose_attach_append(compose, msgfile, msgfile,
+                                                     "message/rfc822");
 
-               if (prefs_common.fw_quotemark && *prefs_common.fw_quotemark)
-                       qmark = prefs_common.fw_quotemark;
-               else
-                       qmark = "> ";
+                       g_free(msgfile);
+               } else {
+                       gchar *qmark;
+                       gchar *quote_str;
 
-               quote_str = compose_quote_fmt(compose, msginfo,
-                                             prefs_common.fw_quotefmt, qmark);
-               compose_attach_parts(compose, msginfo);
-       }
+                       if (prefs_common.fw_quotemark &&
+                           *prefs_common.fw_quotemark)
+                               qmark = prefs_common.fw_quotemark;
+                       else
+                               qmark = "> ";
+
+                       quote_str = compose_quote_fmt(compose, msginfo,
+                                                     prefs_common.fw_quotefmt,
+                                                     qmark, body);
+                       compose_attach_parts(compose, msginfo);
+               }
 
        if (prefs_common.auto_sig)
                compose_insert_sig(compose);
@@ -1298,8 +1146,8 @@ Compose *compose_forward_multiple(PrefsAccount *account, GSList *msginfo_list)
                if (!is_file_exist(msgfile))
                        g_warning(_("%s: file not exist\n"), msgfile);
                else
-                       compose_attach_append(compose, msgfile,
-                               MIME_MESSAGE_RFC822);
+                       compose_attach_append(compose, msgfile, msgfile,
+                               "message/rfc822");
                g_free(msgfile);
        }
 
@@ -1699,7 +1547,8 @@ static gchar *compose_parse_references(const gchar *ref, const gchar *msgid)
 }
 
 static gchar *compose_quote_fmt(Compose *compose, MsgInfo *msginfo,
-                               const gchar *fmt, const gchar *qmark)
+                               const gchar *fmt, const gchar *qmark,
+                               const gchar *body)
 {
        GtkSText *text = GTK_STEXT(compose->text);
        gchar *quote_str = NULL;
@@ -1708,7 +1557,7 @@ static gchar *compose_quote_fmt(Compose *compose, MsgInfo *msginfo,
        gint len;
 
        if (qmark != NULL) {
-               quote_fmt_init(msginfo, NULL);
+               quote_fmt_init(msginfo, NULL, NULL);
                quote_fmt_scan_string(qmark);
                quote_fmt_parse();
 
@@ -1720,7 +1569,7 @@ static gchar *compose_quote_fmt(Compose *compose, MsgInfo *msginfo,
        }
 
        if (fmt && *fmt != '\0') {
-               quote_fmt_init(msginfo, quote_str);
+               quote_fmt_init(msginfo, quote_str, body);
                quote_fmt_scan_string(fmt);
                quote_fmt_parse();
 
@@ -1753,9 +1602,10 @@ static void compose_reply_set_entry(Compose *compose, MsgInfo *msginfo,
                                    gboolean to_all, gboolean ignore_replyto,
                                    gboolean followup_and_reply_to)
 {
-       GSList *cc_list;
+       GSList *cc_list = NULL;
        GSList *cur;
-       gchar *from;
+       gchar *from = NULL;
+       gchar *replyto = NULL;
        GHashTable *to_table;
 
        g_return_if_fail(compose->account != NULL);
@@ -1770,11 +1620,6 @@ static void compose_reply_set_entry(Compose *compose, MsgInfo *msginfo,
                                     : msginfo->from ? msginfo->from : ""),
                                     COMPOSE_TO);
 
-       if (compose->replyto && to_all)
-               compose_entry_append
-                       (compose, compose->replyto, COMPOSE_CC);
-
-
        if (compose->account->protocol == A_NNTP) {
                if (ignore_replyto)
                        compose_entry_append
@@ -1808,15 +1653,30 @@ static void compose_reply_set_entry(Compose *compose, MsgInfo *msginfo,
 
        if (!to_all || compose->account->protocol == A_NNTP) return;
 
-       from = g_strdup(compose->replyto ? compose->replyto :
-                       msginfo->from ? msginfo->from : "");
-       extract_address(from);
+       if (compose->replyto) {
+               Xstrdup_a(replyto, compose->replyto, return);
+               extract_address(replyto);
+       }
+       if (msginfo->from) {
+               Xstrdup_a(from, msginfo->from, return);
+               extract_address(from);
+       }
+
+       if (compose->mailinglist && from) {
+               cc_list = address_list_append(cc_list, from);
+       }
+
+       if (replyto && from)
+               cc_list = address_list_append(cc_list, from);
+
+       if (!compose->mailinglist)
+               cc_list = address_list_append(cc_list, msginfo->to);
 
-       cc_list = address_list_append(NULL, msginfo->to);
        cc_list = address_list_append(cc_list, compose->cc);
 
        to_table = g_hash_table_new(g_str_hash, g_str_equal);
-       g_hash_table_insert(to_table, from, GINT_TO_POINTER(1));
+       if (replyto)
+               g_hash_table_insert(to_table, replyto, GINT_TO_POINTER(1));
        if (compose->account)
                g_hash_table_insert(to_table, compose->account->address,
                                    GINT_TO_POINTER(1));
@@ -1833,7 +1693,6 @@ static void compose_reply_set_entry(Compose *compose, MsgInfo *msginfo,
                cur = next;
        }
        g_hash_table_destroy(to_table);
-       g_free(from);
 
        if (cc_list) {
                for (cur = cc_list; cur != NULL; cur = cur->next)
@@ -1976,7 +1835,7 @@ static void compose_insert_file(Compose *compose, const gchar *file)
 
        g_return_if_fail(file != NULL);
 
-       if ((fp = fopen(file, "r")) == NULL) {
+       if ((fp = fopen(file, "rb")) == NULL) {
                FILE_OP_ERROR(file, "fopen");
                return;
        }
@@ -2000,30 +1859,15 @@ static void compose_insert_file(Compose *compose, const gchar *file)
        fclose(fp);
 }
 
-static void compose_attach_info(Compose * compose, AttachInfo * ainfo,
-                               ContentType cnttype)
-{
-       gchar *text[N_ATTACH_COLS];
-       gint row;
-
-       text[COL_MIMETYPE] = ainfo->content_type;
-       text[COL_SIZE] = to_human_readable(ainfo->size);
-       text[COL_NAME] = ainfo->name;
-
-       row = gtk_clist_append(GTK_CLIST(compose->attach_clist), text);
-       gtk_clist_set_row_data(GTK_CLIST(compose->attach_clist), row, ainfo);
-
-       if (cnttype != MIME_MESSAGE_RFC822)
-               compose_changed_cb(NULL, compose);
-}
-
-static void compose_attach_append_with_type(Compose *compose,
-                                           const gchar *file,
-                                           const gchar *type,
-                                           ContentType cnttype)
+static void compose_attach_append(Compose *compose, const gchar *file,
+                                 const gchar *filename,
+                                 const gchar *content_type)
 {
        AttachInfo *ainfo;
+       gchar *text[N_ATTACH_COLS];
+       FILE *fp;
        off_t size;
+       gint row;
 
        if (!is_file_exist(file)) {
                g_warning(_("File %s doesn't exist\n"), file);
@@ -2034,9 +1878,15 @@ static void compose_attach_append_with_type(Compose *compose,
                return;
        }
        if (size == 0) {
-               alertpanel_notice(_("File %s is empty\n"), file);
+               alertpanel_notice(_("File %s is empty."), file);
+               return;
+       }
+       if ((fp = fopen(file, "rb")) == NULL) {
+               alertpanel_error(_("Can't read %s."), file);
                return;
        }
+       fclose(fp);
+
 #if 0 /* NEW COMPOSE GUI */
        if (!compose->use_attach) {
                GtkItemFactory *ifactory;
@@ -2044,7 +1894,7 @@ static void compose_attach_append_with_type(Compose *compose,
 
                ifactory = gtk_item_factory_from_widget(compose->menubar);
                menuitem = gtk_item_factory_get_item(ifactory,
-                                                    "/Message/Attach");
+                                                    "/View/Attachment");
                gtk_check_menu_item_set_active(GTK_CHECK_MENU_ITEM(menuitem),
                                               TRUE);
        }
@@ -2052,74 +1902,91 @@ static void compose_attach_append_with_type(Compose *compose,
        ainfo = g_new0(AttachInfo, 1);
        ainfo->file = g_strdup(file);
 
-       if (cnttype == MIME_MESSAGE_RFC822) {
-               ainfo->encoding = ENC_7BIT;
-               ainfo->name = g_strdup_printf(_("Message: %s"),
-                                             g_basename(file));
+       if (content_type) {
+               ainfo->content_type = g_strdup(content_type);
+               if (!strcasecmp(content_type, "message/rfc822")) {
+                       ainfo->encoding = ENC_7BIT;
+                       ainfo->name = g_strdup_printf
+                               (_("Message: %s"),
+                                g_basename(filename ? filename : file));
+               } else {
+                       ainfo->encoding = ENC_BASE64;
+                       ainfo->name = g_strdup
+                               (g_basename(filename ? filename : file));
+               }
        } else {
+               ainfo->content_type = procmime_get_mime_type(file);
+               if (!ainfo->content_type)
+                       ainfo->content_type =
+                               g_strdup("application/octet-stream");
                ainfo->encoding = ENC_BASE64;
-               ainfo->name = g_strdup(g_basename(file));
+               ainfo->name = g_strdup(g_basename(filename ? filename : file)); 
        }
-
-       ainfo->content_type = g_strdup(type);
        ainfo->size = size;
 
-       compose_attach_info(compose, ainfo, cnttype);
+       text[COL_MIMETYPE] = ainfo->content_type;
+       text[COL_SIZE] = to_human_readable(size);
+       text[COL_NAME] = ainfo->name;
+
+       row = gtk_clist_append(GTK_CLIST(compose->attach_clist), text);
+       gtk_clist_set_row_data(GTK_CLIST(compose->attach_clist), row, ainfo);
 }
 
-static void compose_attach_append(Compose *compose, const gchar *file,
-                                 ContentType cnttype)
+#define IS_FIRST_PART_TEXT(info) \
+       ((info->mime_type == MIME_TEXT || info->mime_type == MIME_TEXT_HTML || \
+         info->mime_type == MIME_TEXT_ENRICHED) || \
+        (info->mime_type == MIME_MULTIPART && info->content_type && \
+         !strcasecmp(info->content_type, "multipart/alternative") && \
+         (info->children && \
+          (info->children->mime_type == MIME_TEXT || \
+           info->children->mime_type == MIME_TEXT_HTML || \
+           info->children->mime_type == MIME_TEXT_ENRICHED))))
+
+static void compose_attach_parts(Compose *compose, MsgInfo *msginfo)
 {
-       AttachInfo *ainfo;
-       gchar *text[N_ATTACH_COLS];
-       off_t size;
-       gint row;
+       MimeInfo *mimeinfo;
+       MimeInfo *child;
+       gchar *infile;
+       gchar *outfile;
 
-       if (!is_file_exist(file)) {
-               g_warning(_("File %s doesn't exist\n"), file);
-               return;
-       }
-       if ((size = get_file_size(file)) < 0) {
-               g_warning(_("Can't get file size of %s\n"), file);
-               return;
-       }
-       if (size == 0) {
-               alertpanel_notice(_("File %s is empty\n"), file);
+       mimeinfo = procmime_scan_message(msginfo);
+       if (!mimeinfo) return;
+
+       /* skip first text (presumably message body) */
+       child = mimeinfo->children;
+       if (!child || IS_FIRST_PART_TEXT(mimeinfo)) {
+               procmime_mimeinfo_free_all(mimeinfo);
                return;
        }
-#if 0 /* NEW COMPOSE GUI */
-       if (!compose->use_attach) {
-               GtkItemFactory *ifactory;
-               GtkWidget *menuitem;
+       if (IS_FIRST_PART_TEXT(child))
+               child = child->next;
 
-               ifactory = gtk_item_factory_from_widget(compose->menubar);
-               menuitem = gtk_item_factory_get_item(ifactory,
-                                                    "/View/Attachment");
-               gtk_check_menu_item_set_active(GTK_CHECK_MENU_ITEM(menuitem),
-                                              TRUE);
-       }
-#endif
-       ainfo = g_new0(AttachInfo, 1);
-       ainfo->file = g_strdup(file);
+       infile = procmsg_get_message_file_path(msginfo);
 
-       if (cnttype == MIME_MESSAGE_RFC822) {
-               ainfo->content_type = g_strdup("message/rfc822");
-               ainfo->encoding = ENC_7BIT;
-               ainfo->name = g_strdup_printf(_("Message: %s"),
-                                             g_basename(file));
-       } else {
-               ainfo->content_type = procmime_get_mime_type(file);
-               if (!ainfo->content_type)
-                       ainfo->content_type =
-                               g_strdup("application/octet-stream");
-               ainfo->encoding = ENC_BASE64;
-               ainfo->name = g_strdup(g_basename(file));
+       while (child != NULL) {
+               if (child->children || child->mime_type == MIME_MULTIPART) {
+                       child = procmime_mimeinfo_next(child);
+                       continue;
+               }
+
+               outfile = procmime_get_tmp_file_name(child);
+               if (procmime_get_part(outfile, infile, child) < 0)
+                       g_warning(_("Can't get the part of multipart message."));
+               else
+                       compose_attach_append
+                               (compose, outfile,
+                                child->filename ? child->filename : child->name,
+                                child->content_type);
+
+               child = child->next;
        }
-       ainfo->size = size;
 
-       compose_attach_info(compose, ainfo, cnttype);
+       g_free(infile);
+       procmime_mimeinfo_free_all(mimeinfo);
 }
 
+#undef IS_FIRST_PART_TEXT
+
 #define GET_CHAR(pos, buf, len)                                                     \
 {                                                                           \
        if (text->use_wchar)                                                 \
@@ -2170,7 +2037,7 @@ static void compose_wrap_line(Compose *compose)
                        }
                        line_end = 1;
                } else {
-                       if (ch_len == 1 && strchr(">:#", *cbuf))
+                       if (ch_len == 1 && strchr(">|:#", *cbuf))
                                quoted = 1;
                        else if (ch_len != 1 || !isspace(*cbuf))
                                quoted = 0;
@@ -2192,7 +2059,7 @@ static void compose_wrap_line(Compose *compose)
                        }
                        line_end = 1;
                } else {
-                       if (line_end && ch_len == 1 && strchr(">:#", *cbuf))
+                       if (line_end && ch_len == 1 && strchr(">|:#", *cbuf))
                                goto compose_end; /* quoted part */
 
                        line_end = 0;
@@ -2300,9 +2167,8 @@ static guint get_indent_length(GtkSText *text, guint start_pos, guint text_len)
                GET_CHAR(i, cbuf, ch_len);
                if (ch_len > 1)
                        break;
-               /* allow space, tab, > or | */
-               if (cbuf[0] != ' ' && cbuf[0] != '\t' &&
-                   cbuf[0] != '>' && cbuf[0] != '|')
+               /* allow space, tab, >, |, : or # */
+               if (!strchr(" \t>|:#", *cbuf))
                        break;
                indent_len++;
        }
@@ -2707,6 +2573,29 @@ gboolean compose_check_for_valid_recipient(Compose *compose) {
        return recipient_found;
 }
 
+static gboolean compose_check_entries(Compose *compose)
+{
+       gchar *str;
+
+       if (compose_check_for_valid_recipient(compose) == FALSE) {
+               alertpanel_error(_("Recipient is not specified."));
+               return FALSE;
+       }
+
+       str = gtk_entry_get_text(GTK_ENTRY(compose->subject_entry));
+       if (*str == '\0') {
+               AlertValue aval;
+
+               aval = alertpanel(_("Send"),
+                                 _("Subject is empty. Send it anyway?"),
+                                 _("Yes"), _("No"), NULL);
+               if (aval != G_ALERTDEFAULT)
+                       return FALSE;
+       }
+
+       return TRUE;
+}
+
 gint compose_send(Compose *compose)
 {
        gint msgnum;
@@ -2720,10 +2609,9 @@ gint compose_send(Compose *compose)
        }
        
        val = procmsg_send_message_queue(folder_item_fetch_msg(folder, msgnum));
-       if(!val) {
-               folder_item_remove_msg(folder, msgnum);
-               folderview_update_item(folder, TRUE);
-       }
+
+       folder_item_remove_msg(folder, msgnum);
+       folderview_update_item(folder, TRUE);
 
        return val;
 }
@@ -2742,8 +2630,7 @@ gint compose_send(Compose *compose)
 
        lock = TRUE;
 
-       if(!compose_check_for_valid_recipient(compose)) {
-               alertpanel_error(_("Recipient is not specified."));
+       if (compose_check_entries(compose) == FALSE) {
                lock = FALSE;
                return 1;
        }
@@ -2834,13 +2721,17 @@ gint compose_send(Compose *compose)
                                folderview_update_item
                                        (compose->targetinfo->folder, TRUE);
                }
-       }
-
-       /* save message to outbox */
-       if (ok == 0 && prefs_common.savemsg) {
-               if (compose_save_to_outbox(compose, tmp) < 0)
-                       alertpanel_error
-                               (_("Can't save the message to outbox."));
+               /* save message to outbox */
+               if (prefs_common.savemsg) {
+                       Folder *folder = FOLDER(compose->account->folder);
+                       FolderItem *outbox = NULL;
+
+                       if (folder)
+                               outbox = folder->outbox;
+                       if (procmsg_save_to_outbox(outbox, tmp, FALSE) < 0)
+                               alertpanel_error
+                                       (_("Can't save the message to outbox."));
+               }
        }
 
        unlink(tmp);
@@ -2944,12 +2835,12 @@ static gint compose_bounce_write_to_file(Compose *compose, const gchar *file)
        size_t len;
        gchar buf[BUFFSIZE];
 
-       if ((fp = fopen(compose->bounce_filename, "r")) == NULL) {
+       if ((fp = fopen(compose->bounce_filename, "rb")) == NULL) {
                FILE_OP_ERROR(file, "fopen");
                return -1;
        }
 
-       if ((fdest = fopen(file, "w")) == NULL) {
+       if ((fdest = fopen(file, "wb")) == NULL) {
                FILE_OP_ERROR(file, "fopen");
                fclose(fp);
                return -1;
@@ -2972,20 +2863,24 @@ static gint compose_bounce_write_to_file(Compose *compose, const gchar *file)
                if (fputs(buf, fdest) == -1)
                        goto error;
 
-               if (g_strncasecmp(buf, "From:", strlen("From:")) == 0) {
-                       fputs(" (by way of ", fdest);
-                       if (compose->account->name
-                           && *compose->account->name) {
-                               compose_convert_header
-                                       (buf, sizeof(buf),
-                                        compose->account->name,
-                                        strlen("From: "));
-                               fprintf(fdest, "%s <%s>",
-                                       buf, compose->account->address);
-                       } else
-                               fprintf(fdest, "%s",
-                                       compose->account->address);
-                       fputs(")", fdest);
+               if (!prefs_common.bounce_keep_from) {
+                       if (g_strncasecmp(buf, "From:",
+                                         strlen("From:")) == 0) {
+                               fputs(" (by way of ", fdest);
+                               if (compose->account->name
+                                   && *compose->account->name) {
+                                       compose_convert_header
+                                               (buf, sizeof(buf),
+                                                compose->account->name,
+                                                strlen("From: "));
+                                       fprintf(fdest, "%s <%s>",
+                                               buf,
+                                               compose->account->address);
+                               } else
+                                       fprintf(fdest, "%s",
+                                               compose->account->address);
+                               fputs(")", fdest);
+                       }
                }
 
                if (fputs("\n", fdest) == -1)
@@ -3060,7 +2955,7 @@ static gint compose_write_to_file(Compose *compose, const gchar *file,
        const gchar *out_codeset;
        EncodingType encoding;
 
-       if ((fp = fopen(file, "w")) == NULL) {
+       if ((fp = fopen(file, "wb")) == NULL) {
                FILE_OP_ERROR(file, "fopen");
                return -1;
        }
@@ -3097,11 +2992,20 @@ static gint compose_write_to_file(Compose *compose, const gchar *file,
 
                buf = conv_codeset_strdup(chars, src_codeset, out_codeset);
                if (!buf) {
-                       g_free(chars);
-                       fclose(fp);
-                       unlink(file);
-                       alertpanel_error(_("Can't convert the codeset of the message."));
-                       return -1;
+                       AlertValue aval;
+
+                       aval = alertpanel
+                               (_("Error"),
+                                _("Can't convert the character encoding of the message.\n"
+                                  "Send it anyway?"), _("Yes"), _("+No"), NULL);
+                       if (aval != G_ALERTDEFAULT) {
+                               g_free(chars);
+                               fclose(fp);
+                               unlink(file);
+                               return -1;
+                       } else {
+                               buf = g_strdup(chars);
+                       }
                }
        }
        g_free(chars);
@@ -3191,7 +3095,7 @@ static gint compose_write_body_to_file(Compose *compose, const gchar *file)
        size_t len;
        gchar *chars;
 
-       if ((fp = fopen(file, "w")) == NULL) {
+       if ((fp = fopen(file, "wb")) == NULL) {
                FILE_OP_ERROR(file, "fopen");
                return -1;
        }
@@ -3224,43 +3128,6 @@ static gint compose_write_body_to_file(Compose *compose, const gchar *file)
        return 0;
 }
 
-static gint compose_save_to_outbox(Compose *compose, const gchar *file)
-{
-       FolderItem *outbox;
-       gchar *path;
-       gint num;
-       FILE *fp;
-
-       debug_print(_("saving sent message...\n"));
-
-       outbox = folder_get_default_outbox();
-       path = folder_item_get_path(outbox);
-       if (!is_dir_exist(path))
-               make_dir_hier(path);
-
-       folder_item_scan(outbox);
-       if ((num = folder_item_add_msg(outbox, file, FALSE)) < 0) {
-               g_free(path);
-               g_warning(_("can't save message\n"));
-               return -1;
-       }
-
-       if ((fp = procmsg_open_mark_file(path, TRUE)) == NULL)
-               g_warning(_("can't open mark file\n"));
-       else {
-               MsgInfo newmsginfo;
-
-               newmsginfo.msgnum = num;
-               newmsginfo.flags.perm_flags = 0;
-               newmsginfo.flags.tmp_flags = 0;
-               procmsg_write_flags(&newmsginfo, fp);
-               fclose(fp);
-       }
-       g_free(path);
-
-       return 0;
-}
-
 static gint compose_remove_reedit_target(Compose *compose)
 {
        FolderItem *item;
@@ -3302,12 +3169,11 @@ static gint compose_queue(Compose *compose, gint *msgnum, FolderItem **item)
 
         lock = TRUE;
        
-        if(!compose_check_for_valid_recipient(compose)) {
-                alertpanel_error(_("Recipient is not specified."));
+       if (compose_check_entries(compose) == FALSE) {
                 lock = FALSE;
                 return -1;
-        }
-                                                                       
+       }
+
        if (!compose->to_list && !compose->newsgroup_list) {
                g_warning(_("can't get recipient list."));
                lock = FALSE;
@@ -3362,13 +3228,13 @@ static gint compose_queue(Compose *compose, gint *msgnum, FolderItem **item)
 
        /* add queue header */
        tmp = g_strdup_printf("%s%cqueue.%d", g_get_tmp_dir(),
-                                     G_DIR_SEPARATOR, (gint)compose);
-       if ((fp = fopen(tmp, "w")) == NULL) {
+                             G_DIR_SEPARATOR, (gint)compose);
+       if ((fp = fopen(tmp, "wb")) == NULL) {
                FILE_OP_ERROR(tmp, "fopen");
                g_free(tmp);
                return -1;
        }
-       if ((src_fp = fopen(tmp2, "r")) == NULL) {
+       if ((src_fp = fopen(tmp2, "rb")) == NULL) {
                FILE_OP_ERROR(tmp2, "fopen");
                fclose(fp);
                unlink(tmp);
@@ -3460,9 +3326,12 @@ static gint compose_queue(Compose *compose, gint *msgnum, FolderItem **item)
                g_free(tmp2);
                return -1;
        }
-                                               
-       /* queue message */
-       queue = folder_get_default_queue();
+
+       if (compose->account->folder &&
+           FOLDER(compose->account->folder)->queue)
+               queue = FOLDER(compose->account->folder)->queue;
+       else
+               queue = folder_get_default_queue();
 
        folder_item_scan(queue);
        queue_path = folder_item_get_path(queue);
@@ -3523,7 +3392,10 @@ static void compose_write_attach(Compose *compose, FILE *fp)
 
        for (row = 0; (ainfo = gtk_clist_get_row_data(clist, row)) != NULL;
             row++) {
-               if ((attach_fp = fopen(ainfo->file, "r")) == NULL) {
+               gchar buf[BUFFSIZE];
+               gchar inbuf[B64_LINE_SIZE], outbuf[B64_BUFFSIZE];
+
+               if ((attach_fp = fopen(ainfo->file, "rb")) == NULL) {
                        g_warning(_("Can't open file %s\n"), ainfo->file);
                        continue;
                }
@@ -3546,15 +3418,19 @@ static void compose_write_attach(Compose *compose, FILE *fp)
                fprintf(fp, "Content-Transfer-Encoding: %s\n\n",
                        procmime_get_encoding_str(ainfo->encoding));
 
-               if (ainfo->encoding == ENC_7BIT) {
-                       gchar buf[BUFFSIZE];
+               switch (ainfo->encoding) {
+
+               case ENC_7BIT:
+               case ENC_8BIT:
+                       /* if (ainfo->encoding == ENC_7BIT) { */
 
                        while (fgets(buf, sizeof(buf), attach_fp) != NULL) {
                                strcrchomp(buf);
                                fputs(buf, fp);
                        }
-               } else {
-                       gchar inbuf[B64_LINE_SIZE], outbuf[B64_BUFFSIZE];
+                       break;
+                       /* } else { */
+               case ENC_BASE64:
 
                        while ((len = fread(inbuf, sizeof(gchar),
                                            B64_LINE_SIZE, attach_fp))
@@ -3568,6 +3444,7 @@ static void compose_write_attach(Compose *compose, FILE *fp)
                                fputs(outbuf, fp);
                                fputc('\n', fp);
                        }
+                       break;
                }
 
                fclose(attach_fp);
@@ -4181,12 +4058,16 @@ static GtkWidget *compose_create_header(Compose *compose)
 
 GtkWidget *compose_create_attach(Compose *compose)
 {
-       gchar *titles[] = {_("MIME type"), _("Size"), _("Name")};
+       gchar *titles[N_ATTACH_COLS];
        gint i;
 
        GtkWidget *attach_scrwin;
        GtkWidget *attach_clist;
 
+       titles[COL_MIMETYPE] = _("MIME type");
+       titles[COL_SIZE]     = _("Size");
+       titles[COL_NAME]     = _("Name");
+
        /* attachment list */
        attach_scrwin = gtk_scrolled_window_new(NULL, NULL);
        gtk_scrolled_window_set_policy(GTK_SCROLLED_WINDOW(attach_scrwin),
@@ -4293,7 +4174,7 @@ static void compose_savemsg_select_cb(GtkWidget *widget, Compose *compose)
        FolderItem *dest;
        gchar * path;
 
-       dest = foldersel_folder_sel(NULL, NULL);
+       dest = foldersel_folder_sel(NULL, FOLDER_SEL_COPY, NULL);
        if (!dest) return;
 
        path = folder_item_get_identifier(dest);
@@ -4331,6 +4212,7 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode)
 
        UndoMain *undostruct;
 
+       gchar *titles[N_ATTACH_COLS];
        guint n_menu_entries;
        GtkStyle  *style, *new_style;
        GdkColormap *cmap;
@@ -4353,6 +4235,10 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode)
        debug_print(_("Creating compose window...\n"));
        compose = g_new0(Compose, 1);
 
+       titles[COL_MIMETYPE] = _("MIME type");
+       titles[COL_SIZE]     = _("Size");
+       titles[COL_NAME]     = _("Name");
+
        compose->account = account;
        compose->orig_account = account;
 
@@ -4364,10 +4250,7 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode)
                           GTK_SIGNAL_FUNC(compose_delete_cb), compose);
        gtk_signal_connect(GTK_OBJECT(window), "destroy",
                           GTK_SIGNAL_FUNC(compose_destroy_cb), compose);
-       gtk_signal_connect(GTK_OBJECT(window), "focus_in_event",
-                          GTK_SIGNAL_FUNC(manage_window_focus_in), NULL);
-       gtk_signal_connect(GTK_OBJECT(window), "focus_out_event",
-                          GTK_SIGNAL_FUNC(manage_window_focus_out), NULL);
+       MANAGE_WINDOW_SIGNALS_CONNECT(window);
        gtk_widget_realize(window);
 
        gtkut_widget_set_composer_icon(window);
@@ -4555,7 +4438,7 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode)
        menu_set_sensitive(ifactory, "/Edit/Undo", FALSE);
        menu_set_sensitive(ifactory, "/Edit/Redo", FALSE);
 
-       tmpl_menu = gtk_item_factory_get_item(ifactory, "/Tool/Template");
+       tmpl_menu = gtk_item_factory_get_item(ifactory, "/Tools/Template");
 #if 0 /* NEW COMPOSE GUI */
        gtk_widget_hide(bcc_hbox);
        gtk_widget_hide(bcc_entry);
@@ -4583,23 +4466,8 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode)
        }
 #endif
 
-       switch (prefs_common.toolbar_style) {
-       case TOOLBAR_NONE:
-               gtk_widget_hide(handlebox);
-               break;
-       case TOOLBAR_ICON:
-               gtk_toolbar_set_style(GTK_TOOLBAR(compose->toolbar),
-                                     GTK_TOOLBAR_ICONS);
-               break;
-       case TOOLBAR_TEXT:
-               gtk_toolbar_set_style(GTK_TOOLBAR(compose->toolbar),
-                                     GTK_TOOLBAR_TEXT);
-               break;
-       case TOOLBAR_BOTH:
-               gtk_toolbar_set_style(GTK_TOOLBAR(compose->toolbar),
-                                     GTK_TOOLBAR_BOTH);
-               break;
-       }
+       update_compose_actions_menu(ifactory, "/Tools/Actions", compose);
+
 
        undostruct = undo_init(text);
        undo_set_change_state_func(undostruct, &compose_undo_state_changed,
@@ -4673,6 +4541,7 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode)
                                          conv_get_current_charset_str(),
                                          prefs_common.misspelled_col,
                                          prefs_common.check_while_typing,
+                                         prefs_common.use_alternate,
                                          GTK_STEXT(text));
                if (!gtkpspell) {
                        alertpanel_error(_("Spell checker could not be started.\n%s"), gtkpspellcheckers->error_message);
@@ -4805,6 +4674,7 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode)
                gtk_widget_hide(ruler_hbox);
 
        select_account(compose, account);
+       set_toolbar_style(compose);
 
        return compose;
 }
@@ -4856,7 +4726,7 @@ static void compose_toolbar_create(Compose *compose, GtkWidget *container)
 
        gtk_toolbar_append_space(GTK_TOOLBAR(toolbar));
 
-       icon_wid = stock_pixmap_widget(container, STOCK_PIXMAP_PASTE);
+       icon_wid = stock_pixmap_widget(container, STOCK_PIXMAP_INSERT_FILE);
        insert_btn = gtk_toolbar_append_item(GTK_TOOLBAR(toolbar),
                                             _("Insert"),
                                             _("Insert file"),
@@ -4874,7 +4744,7 @@ static void compose_toolbar_create(Compose *compose, GtkWidget *container)
 
        gtk_toolbar_append_space(GTK_TOOLBAR(toolbar));
 
-       icon_wid = stock_pixmap_widget(container, STOCK_PIXMAP_MAIL);
+       icon_wid = stock_pixmap_widget(container, STOCK_PIXMAP_MAIL_SIGN);
        sig_btn = gtk_toolbar_append_item(GTK_TOOLBAR(toolbar),
                                          _("Signature"),
                                          _("Insert signature"),
@@ -4883,7 +4753,7 @@ static void compose_toolbar_create(Compose *compose, GtkWidget *container)
 
        gtk_toolbar_append_space(GTK_TOOLBAR(toolbar));
 
-       icon_wid = stock_pixmap_widget(container, STOCK_PIXMAP_MAIL_COMPOSE);
+       icon_wid = stock_pixmap_widget(container, STOCK_PIXMAP_EDIT_EXTERN);
        exteditor_btn = gtk_toolbar_append_item(GTK_TOOLBAR(toolbar),
                                                _("Editor"),
                                                _("Edit with external editor"),
@@ -5000,6 +4870,21 @@ void compose_reflect_prefs_all(void)
        }
 }
 
+void compose_reflect_prefs_pixmap_theme(void)
+{
+       GList *cur;
+       Compose *compose;
+
+       for (cur = compose_list; cur != NULL; cur = cur->next) {
+               compose = (Compose *)cur->data;
+               gtk_container_remove(GTK_CONTAINER(compose->handlebox), GTK_WIDGET(compose->toolbar));
+               compose->toolbar = NULL;
+               compose_toolbar_create(compose, compose->handlebox);
+               set_toolbar_style(compose);
+       }
+}
+
+
 static void compose_template_apply(Compose *compose, Template *tmpl)
 {
        gchar *qmark;
@@ -5022,7 +4907,7 @@ static void compose_template_apply(Compose *compose, Template *tmpl)
 
                memset(&dummyinfo, 0, sizeof(MsgInfo));
                parsed_str = compose_quote_fmt(compose, &dummyinfo,
-                                              tmpl->value, NULL);
+                                              tmpl->value, NULL, NULL);
        } else {
                if (prefs_common.quotemark && *prefs_common.quotemark)
                        qmark = prefs_common.quotemark;
@@ -5030,7 +4915,7 @@ static void compose_template_apply(Compose *compose, Template *tmpl)
                        qmark = "> ";
 
                parsed_str = compose_quote_fmt(compose, compose->replyinfo,
-                                              tmpl->value, qmark);
+                                              tmpl->value, qmark, NULL);
        }
 
        if (parsed_str && prefs_common.auto_sig)
@@ -5331,13 +5216,19 @@ static void compose_attach_property_create(gboolean *cancelled)
                         GTK_EXPAND|GTK_SHRINK|GTK_FILL, 0, 0, 0);
 
        optmenu = gtk_option_menu_new();
-       gtk_box_pack_start(GTK_BOX(hbox), optmenu, FALSE, FALSE, 0);
+       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);
+#if 0
        gtk_widget_set_sensitive(menuitem, FALSE);
+#endif
        MENUITEM_ADD(optmenu_menu, menuitem, "8bit", ENC_8BIT);
+       gtk_option_menu_set_menu(GTK_OPTION_MENU(optmenu), optmenu_menu);
+#if 0
        gtk_widget_set_sensitive(menuitem, FALSE);
+#endif
        MENUITEM_ADD(optmenu_menu, menuitem, "quoted-printable", ENC_QUOTED_PRINTABLE);
        gtk_widget_set_sensitive(menuitem, FALSE);
        MENUITEM_ADD(optmenu_menu, menuitem, "base64", ENC_BASE64);
@@ -5798,6 +5689,10 @@ static void toolbar_address_cb(GtkWidget *widget, gpointer data)
 
 static void select_account(Compose * compose, PrefsAccount * ac)
 {
+#if USE_GPGME
+       GtkItemFactory *ifactory;
+       GtkWidget *menuitem;
+#endif /* USE_GPGME */
        compose->account = ac;
        compose_set_title(compose);
 
@@ -5872,6 +5767,30 @@ static void select_account(Compose * compose, PrefsAccount * ac)
                }
                gtk_widget_queue_resize(compose->table_vbox);
 #endif
+#if USE_GPGME
+               ifactory = gtk_item_factory_from_widget(compose->menubar);
+                       menu_set_sensitive(ifactory,
+                                          "/Message/Sign", TRUE);
+                       menu_set_sensitive(ifactory,
+                                          "/Message/Encrypt", TRUE);
+
+                       menuitem = gtk_item_factory_get_item(ifactory, "/Message/Sign");
+               if (ac->default_sign)
+                       gtk_check_menu_item_set_active
+                               (GTK_CHECK_MENU_ITEM(menuitem), TRUE);
+               else
+                       gtk_check_menu_item_set_active
+                               (GTK_CHECK_MENU_ITEM(menuitem), FALSE);
+
+                       menuitem = gtk_item_factory_get_item(ifactory, "/Message/Encrypt");
+               if (ac->default_encrypt)
+                       gtk_check_menu_item_set_active
+                               (GTK_CHECK_MENU_ITEM(menuitem), TRUE);
+               else
+                       gtk_check_menu_item_set_active
+                               (GTK_CHECK_MENU_ITEM(menuitem), FALSE);
+#endif /* USE_GPGME */
+
 }
 
 static void account_activated(GtkMenuItem *menuitem, gpointer data)
@@ -5965,7 +5884,11 @@ static void compose_draft_cb(gpointer data, guint action, GtkWidget *widget)
 
        if (lock) return;
 
-       draft = folder_get_default_draft();
+       if (compose->account && compose->account->folder &&
+           FOLDER(compose->account->folder)->draft)
+               draft = FOLDER(compose->account->folder)->draft;
+       else
+               draft = folder_get_default_draft();
        g_return_if_fail(draft != NULL);
 
        lock = TRUE;
@@ -5980,7 +5903,7 @@ static void compose_draft_cb(gpointer data, guint action, GtkWidget *widget)
        }
 
        folder_item_scan(draft);
-       if ((msgnum = folder_item_add_msg(draft, tmp, TRUE)) <= 0) {
+       if ((msgnum = folder_item_add_msg(draft, tmp, TRUE)) < 0) {
                unlink(tmp);
                g_free(tmp);
                lock = FALSE;
@@ -6044,7 +5967,8 @@ static void compose_attach_cb(gpointer data, guint action, GtkWidget *widget)
 
                for ( tmp = file_list; tmp; tmp = tmp->next) {
                        gchar *file = (gchar *) tmp->data;
-                       compose_attach_append(compose, file, MIME_UNKNOWN);
+                       compose_attach_append(compose, file, file, NULL);
+                       compose_changed_cb(NULL, compose);
                        g_free(file);
                }
                g_list_free(file_list);
@@ -6467,8 +6391,10 @@ static void compose_attach_drag_received_cb (GtkWidget           *widget,
 
        list = uri_list_extract_filenames((const gchar *)data->data);
        for (tmp = list; tmp != NULL; tmp = tmp->next)
-               compose_attach_append(compose, (const gchar *)tmp->data,
-                                     MIME_UNKNOWN);
+               compose_attach_append
+                       (compose, (const gchar *)tmp->data,
+                        (const gchar *)tmp->data, NULL);
+       if (list) compose_changed_cb(NULL, compose);
        list_free_strings(list);
        g_list_free(list);
 }
@@ -6699,3 +6625,29 @@ static void compose_check_forwards_go(Compose *compose)
        }
 }
 #endif
+
+static void set_toolbar_style(Compose *compose)
+{
+       switch (prefs_common.toolbar_style) {
+       case TOOLBAR_NONE:
+               gtk_widget_hide(compose->handlebox);
+               break;
+       case TOOLBAR_ICON:
+               gtk_toolbar_set_style(GTK_TOOLBAR(compose->toolbar),
+                                     GTK_TOOLBAR_ICONS);
+               break;
+       case TOOLBAR_TEXT:
+               gtk_toolbar_set_style(GTK_TOOLBAR(compose->toolbar),
+                                     GTK_TOOLBAR_TEXT);
+               break;
+       case TOOLBAR_BOTH:
+               gtk_toolbar_set_style(GTK_TOOLBAR(compose->toolbar),
+                                     GTK_TOOLBAR_BOTH);
+               break;
+       }
+       
+       if (prefs_common.toolbar_style != TOOLBAR_NONE) {
+               gtk_widget_show(compose->handlebox);
+               gtk_widget_queue_resize(compose->handlebox);
+       }
+}