2005-09-27 [colin] 1.9.14cvs56
[claws.git] / src / compose.c
index ca353ed08a9ab8a7e9ff4a68882b870831990228..f4ae3cb1c8a4071bb58a1ab9b5dedf643dc9aa61 100644 (file)
@@ -14,7 +14,7 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
  */
 
 #ifdef HAVE_CONFIG_H
@@ -79,6 +79,7 @@
 #  include <wctype.h>
 #endif
 
+#include "sylpheed.h"
 #include "main.h"
 #include "mainwindow.h"
 #include "compose.h"
 #include "undo.h"
 #include "foldersel.h"
 #include "toolbar.h"
-
+#include "inc.h"
 enum
 {
        COL_MIMETYPE = 0,
@@ -419,9 +420,10 @@ static void compose_toggle_encrypt_cb      (gpointer        data,
                                         GtkWidget      *widget);
 static void compose_set_privacy_system_cb(GtkWidget      *widget,
                                          gpointer        data);
-static void compose_update_privacy_system_menu_item(Compose * compose);
+static void compose_update_privacy_system_menu_item(Compose * compose, gboolean warn);
 static void activate_privacy_system     (Compose *compose, 
-                                         PrefsAccount *account);
+                                         PrefsAccount *account,
+                                        gboolean warn);
 static void compose_use_signing(Compose *compose, gboolean use_signing);
 static void compose_use_encryption(Compose *compose, gboolean use_encryption);
 static void compose_toggle_return_receipt_cb(gpointer data, guint action,
@@ -493,7 +495,7 @@ 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 void compose_close      (Compose *compose);
+static gboolean compose_close  (Compose *compose);
 
 static GtkItemFactoryEntry compose_popup_entries[] =
 {
@@ -922,6 +924,7 @@ Compose *compose_generic_new(PrefsAccount *account, const gchar *mailto, FolderI
        if (prefs_common.auto_exteditor)
                compose_exec_ext_editor(compose);
 
+       compose_set_title(compose);
         return compose;
 }
 
@@ -944,7 +947,7 @@ static void compose_force_encryption(Compose *compose, PrefsAccount *account,
        }
        if (privacy != NULL) {
                compose->privacy_system = g_strdup(privacy);
-               compose_update_privacy_system_menu_item(compose);
+               compose_update_privacy_system_menu_item(compose, FALSE);
                compose_use_encryption(compose, TRUE);
        }
 }      
@@ -964,7 +967,7 @@ static void compose_force_signing(Compose *compose, PrefsAccount *account)
        }
        if (privacy != NULL) {
                compose->privacy_system = g_strdup(privacy);
-               compose_update_privacy_system_menu_item(compose);
+               compose_update_privacy_system_menu_item(compose, FALSE);
                compose_use_signing(compose, TRUE);
        }
 }      
@@ -1071,6 +1074,30 @@ void compose_followup_and_reply_to(MsgInfo *msginfo, gboolean quote,
                              to_sender, TRUE, body);
 }
 
+static void compose_extract_original_charset(Compose *compose)
+{
+       MsgInfo *info = NULL;
+       if (compose->replyinfo) {
+               info = compose->replyinfo;
+       } else if (compose->fwdinfo) {
+               info = compose->fwdinfo;
+       } else if (compose->targetinfo) {
+               info = compose->targetinfo;
+       }
+       if (info) {
+               MimeInfo *mimeinfo = procmime_scan_message(info);
+               MimeInfo *partinfo = mimeinfo;
+               while (partinfo && partinfo->type != MIMETYPE_TEXT)
+                       partinfo = procmime_mimeinfo_next(partinfo);
+               if (partinfo) {
+                       compose->orig_charset = 
+                               g_strdup(procmime_mimeinfo_get_parameter(
+                                               partinfo, "charset"));
+               }
+               procmime_mimeinfo_free_all(mimeinfo);
+       }
+}
+
 static void compose_generic_reply(MsgInfo *msginfo, gboolean quote,
                                  gboolean to_all, gboolean to_ml,
                                  gboolean to_sender,
@@ -1114,6 +1141,8 @@ static void compose_generic_reply(MsgInfo *msginfo, gboolean quote,
        if (!compose->replyinfo)
                compose->replyinfo = procmsg_msginfo_copy(msginfo);
 
+       compose_extract_original_charset(compose);
+       
        if (msginfo->folder && msginfo->folder->ret_rcpt)
                menu_set_active(ifactory, "/Options/Request Return Receipt", TRUE);
 
@@ -1178,6 +1207,8 @@ static void compose_generic_reply(MsgInfo *msginfo, gboolean quote,
 
        if (prefs_common.auto_exteditor)
                compose_exec_ext_editor(compose);
+               
+       compose_set_title(compose);
 }
 
 #define INSERT_FW_HEADER(var, hdr) \
@@ -1210,6 +1241,8 @@ Compose *compose_forward(PrefsAccount *account, MsgInfo *msginfo,
        if (!compose->fwdinfo)
                compose->fwdinfo = procmsg_msginfo_copy(msginfo);
 
+       compose_extract_original_charset(compose);
+
        if (msginfo->subject && *msginfo->subject) {
                gchar *buf, *buf2, *p;
 
@@ -1284,6 +1317,7 @@ Compose *compose_forward(PrefsAccount *account, MsgInfo *msginfo,
                g_free(folderidentifier);
        }
 
+       compose_set_title(compose);
         return compose;
 }
 
@@ -1343,6 +1377,7 @@ Compose *compose_forward_multiple(PrefsAccount *account, GSList *msginfo_list)
 
        gtk_widget_grab_focus(compose->header_last->entry);
        
+       compose_set_title(compose);
        return compose;
 }
 
@@ -1472,12 +1507,14 @@ void compose_reedit(MsgInfo *msginfo)
                compose->privacy_system = privacy_system;
                compose_use_signing(compose, use_signing);
                compose_use_encryption(compose, use_encryption);
-               compose_update_privacy_system_menu_item(compose);
+               compose_update_privacy_system_menu_item(compose, FALSE);
        } else {
-               activate_privacy_system(compose, account);
+               activate_privacy_system(compose, account, FALSE);
        }
        compose->targetinfo = procmsg_msginfo_copy(msginfo);
 
+       compose_extract_original_charset(compose);
+
         if (folder_has_parent_of_type(msginfo->folder, F_QUEUE) ||
            folder_has_parent_of_type(msginfo->folder, F_DRAFT)) {
                gchar queueheader_buf[BUFFSIZE];
@@ -1540,6 +1577,7 @@ void compose_reedit(MsgInfo *msginfo)
 
         if (prefs_common.auto_exteditor)
                compose_exec_ext_editor(compose);
+       compose_set_title(compose);
 }
 
 Compose *compose_redirect(PrefsAccount *account, MsgInfo *msginfo)
@@ -1618,6 +1656,7 @@ Compose *compose_redirect(PrefsAccount *account, MsgInfo *msginfo)
        gtk_widget_set_sensitive(compose->toolbar->linewrap_current_btn, FALSE);
        gtk_widget_set_sensitive(compose->toolbar->linewrap_all_btn, FALSE);
 
+       compose_set_title(compose);
         return compose;
 }
 
@@ -2270,7 +2309,7 @@ static void compose_reedit_set_entry(Compose *compose, MsgInfo *msginfo)
        SET_ADDRESS(COMPOSE_FOLLOWUPTO, compose->followup_to);
 
        compose_update_priority_menu_item(compose);
-       compose_update_privacy_system_menu_item(compose);
+       compose_update_privacy_system_menu_item(compose, FALSE);
        compose_show_first_last_header(compose, TRUE);
 }
 
@@ -3281,15 +3320,24 @@ static void compose_set_title(Compose *compose)
 {
        gchar *str;
        gchar *edited;
-
+       gchar *subject;
+       
        edited = compose->modified ? _(" [Edited]") : "";
-       if (compose->account && compose->account->address)
+       
+       subject = gtk_editable_get_chars(
+                       GTK_EDITABLE(compose->subject_entry), 0, -1);
+
+       if (subject && strlen(subject))
+               str = g_strdup_printf(_("%s - Compose message%s"),
+                                     subject, edited); 
+       else if (compose->account && compose->account->address)
                str = g_strdup_printf(_("%s - Compose message%s"),
                                      compose->account->address, edited);
        else
                str = g_strdup_printf(_("Compose message%s"), edited);
        gtk_window_set_title(GTK_WINDOW(compose->window), str);
        g_free(str);
+       g_free(subject);
 }
 
 /**
@@ -3341,7 +3389,7 @@ static void compose_select_account(Compose *compose, PrefsAccount *account,
        else
                menu_set_active(ifactory, "/Options/Encrypt", FALSE);
                                       
-       activate_privacy_system(compose, account);
+       activate_privacy_system(compose, account, FALSE);
 
        if (!init && compose->mode != COMPOSE_REDIRECT)
                compose_insert_sig(compose, TRUE);
@@ -3429,7 +3477,10 @@ gint compose_send(Compose *compose)
        val = compose_queue(compose, &msgnum, &folder);
 
        if (val) {
-               if (val == -3) {
+               if (val == -4) {
+                       alertpanel_error(_("Could not queue message for sending:\n\n"
+                                          "Charset conversion failed."));
+               } else if (val == -3) {
                        alertpanel_error(_("Could not queue message for sending:\n\n"
                                           "Signature failed."));
                } else if (val == -2) {
@@ -3733,11 +3784,38 @@ static gint compose_write_to_file(Compose *compose, FILE *fp, gint action)
                out_codeset = CS_US_ASCII;
                encoding = ENC_7BIT;
        } else {
-               const gchar *src_codeset;
+               const gchar *src_codeset = CS_INTERNAL;
 
                out_codeset = conv_get_charset_str(compose->out_encoding);
-               if (!out_codeset)
+
+               if (!out_codeset) {
+                       gchar *test_conv_global_out = NULL;
+                       gchar *test_conv_reply = NULL;
+
+                       /* automatic mode. be automatic. */
+                       codeconv_set_strict(TRUE);
+                       
                        out_codeset = conv_get_outgoing_charset_str();
+                       if (out_codeset) {
+                               debug_print("trying to convert to %s\n", out_codeset);
+                               test_conv_global_out = conv_codeset_strdup(chars, src_codeset, out_codeset);
+                       }
+                       
+                       if (!test_conv_global_out && compose->orig_charset) {
+                               out_codeset = compose->orig_charset;
+                               debug_print("failure; trying to convert to %s\n", out_codeset);
+                               test_conv_reply = conv_codeset_strdup(chars, src_codeset, out_codeset);
+                       }
+                       
+                       if (!test_conv_global_out && !test_conv_reply) {
+                               /* we're lost */
+                               out_codeset = CS_INTERNAL;
+                               debug_print("finally using %s\n", out_codeset);
+                       }
+                       g_free(test_conv_global_out);
+                       g_free(test_conv_reply);
+                       codeconv_set_strict(FALSE);
+               }
 
                if (!g_ascii_strcasecmp(out_codeset, CS_US_ASCII))
                        out_codeset = CS_ISO_8859_1;
@@ -3751,32 +3829,28 @@ static gint compose_write_to_file(Compose *compose, FILE *fp, gint action)
                else
                        encoding = procmime_get_encoding_for_charset(out_codeset);
 
-               src_codeset = CS_INTERNAL;
-               /* if current encoding is US-ASCII, set it the same as
-                  outgoing one to prevent code conversion failure */
-               if (!g_ascii_strcasecmp(src_codeset, CS_US_ASCII))
-                       src_codeset = out_codeset;
-
                debug_print("src encoding = %s, out encoding = %s, transfer encoding = %s\n",
                            src_codeset, out_codeset, procmime_get_encoding_str(encoding));
 
                if (action == COMPOSE_WRITE_FOR_SEND) {
+                       codeconv_set_strict(TRUE);
                        buf = conv_codeset_strdup(chars, src_codeset, out_codeset);
-                       
+                       codeconv_set_strict(FALSE);
+
                        if (!buf) {
                                AlertValue aval;
                                gchar *msg;
 
-                               msg = g_strdup_printf(_("Can't convert the character encoding of the message from\n"
-                                                       "%s to %s.\n"
-                                                       "Send it anyway?"), src_codeset, out_codeset);
+                               msg = g_strdup_printf(_("Can't convert the character encoding of the message \n"
+                                                       "to the specified %s charset.\n"
+                                                       "Send it as %s?"), out_codeset, src_codeset);
                                aval = alertpanel_full(_("Error"), msg, GTK_STOCK_YES, GTK_STOCK_NO, NULL, FALSE,
                                                      NULL, ALERT_ERROR, G_ALERTALTERNATE);
-       g_free(msg);
+                               g_free(msg);
 
                                if (aval != G_ALERTDEFAULT) {
                                        g_free(chars);
-                                       return -1;
+                                       return -3;
                                } else {
                                        buf = chars;
                                        out_codeset = src_codeset;
@@ -4133,7 +4207,7 @@ static gint compose_queue_sub(Compose *compose, gint *msgnum, FolderItem **item,
                        fclose(fp);
                        g_unlink(tmp);
                        g_free(tmp);
-                       return result - 1; /* -2 for a generic error, -3 for signing error */
+                       return result - 1; /* -2 for a generic error, -3 for signing error, -4 for encoding */
                }
        }
 
@@ -4370,6 +4444,7 @@ static gchar *compose_get_header(Compose *compose)
 
        /* Subject */
        str = gtk_editable_get_chars(GTK_EDITABLE(compose->subject_entry), 0, -1);
+
        if (*str != '\0' && !IS_IN_CUSTOM_HEADER("Subject")) {
                g_strstrip(str);
                if (*str != '\0') {
@@ -4538,7 +4613,8 @@ static void compose_convert_header(Compose *compose, gchar *dest, gint len, gcha
                                   gint header_len, gboolean addr_field)
 {
        gchar *tmpstr = NULL;
-       
+       const gchar *out_codeset = NULL;
+
        g_return_if_fail(src != NULL);
        g_return_if_fail(dest != NULL);
 
@@ -4550,8 +4626,48 @@ static void compose_convert_header(Compose *compose, gchar *dest, gint len, gcha
        subst_char(tmpstr, '\r', ' ');
        g_strchomp(tmpstr);
 
+       if (!g_utf8_validate(tmpstr, -1, NULL)) {
+               gchar *mybuf = g_malloc(strlen(tmpstr)*2 +1);
+               conv_localetodisp(mybuf, strlen(tmpstr)*2 +1, tmpstr);
+               g_free(tmpstr);
+               tmpstr = mybuf;
+       }
+
+       codeconv_set_strict(TRUE);
        conv_encode_header_full(dest, len, tmpstr, header_len, addr_field, 
                conv_get_charset_str(compose->out_encoding));
+       codeconv_set_strict(FALSE);
+       
+       if (!dest || *dest == '\0') {
+               gchar *test_conv_global_out = NULL;
+               gchar *test_conv_reply = NULL;
+
+               /* automatic mode. be automatic. */
+               codeconv_set_strict(TRUE);
+
+               out_codeset = conv_get_outgoing_charset_str();
+               if (out_codeset) {
+                       debug_print("trying to convert to %s\n", out_codeset);
+                       test_conv_global_out = conv_codeset_strdup(src, CS_INTERNAL, out_codeset);
+               }
+
+               if (!test_conv_global_out && compose->orig_charset) {
+                       out_codeset = compose->orig_charset;
+                       debug_print("failure; trying to convert to %s\n", out_codeset);
+                       test_conv_reply = conv_codeset_strdup(src, CS_INTERNAL, out_codeset);
+               }
+
+               if (!test_conv_global_out && !test_conv_reply) {
+                       /* we're lost */
+                       out_codeset = CS_INTERNAL;
+                       debug_print("finally using %s\n", out_codeset);
+               }
+               g_free(test_conv_global_out);
+               g_free(test_conv_reply);
+               conv_encode_header_full(dest, len, tmpstr, header_len, addr_field, 
+                                       out_codeset);
+               codeconv_set_strict(FALSE);
+       }
        g_free(tmpstr);
 }
 
@@ -4856,13 +4972,45 @@ static void compose_savemsg_select_cb(GtkWidget *widget, Compose *compose)
 
 static void entry_paste_clipboard(Compose *compose, GtkWidget *entry, gboolean wrap,
                                  GdkAtom clip);
+
+#define BLOCK_WRAP() {                                                 \
+       prev_autowrap = compose->autowrap;                              \
+       buffer = gtk_text_view_get_buffer(                              \
+                                       GTK_TEXT_VIEW(compose->text));  \
+       compose->autowrap = FALSE;                                      \
+                                                                       \
+       g_signal_handlers_block_by_func(G_OBJECT(buffer),               \
+                               G_CALLBACK(compose_changed_cb),         \
+                               compose);                               \
+       g_signal_handlers_block_by_func(G_OBJECT(buffer),               \
+                               G_CALLBACK(text_inserted),              \
+                               compose);                               \
+}
+#define UNBLOCK_WRAP() {                                               \
+       compose->autowrap = prev_autowrap;                              \
+       if (compose->autowrap)                                          \
+               compose_wrap_all(compose);                              \
+                                                                       \
+       g_signal_handlers_unblock_by_func(G_OBJECT(buffer),             \
+                               G_CALLBACK(compose_changed_cb),         \
+                               compose);                               \
+       g_signal_handlers_unblock_by_func(G_OBJECT(buffer),             \
+                               G_CALLBACK(text_inserted),              \
+                               compose);                               \
+}
+
+
 static gboolean text_clicked(GtkWidget *text, GdkEventButton *event,
                                        Compose *compose)
 {
+       gint prev_autowrap;
+       GtkTextBuffer *buffer;
        if (event->button == 2) {
+               BLOCK_WRAP();
                entry_paste_clipboard(compose, compose->focused_editable, 
                                prefs_common.linewrap_pastes,
                                GDK_SELECTION_PRIMARY);
+               UNBLOCK_WRAP();
                return TRUE;
        }
        return FALSE;
@@ -4925,7 +5073,9 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode)
        titles[COL_NAME]     = _("Name");
 
        compose->account = account;
-
+       
+       compose->mutex = g_mutex_new();
+       
        window = gtk_window_new(GTK_WINDOW_TOPLEVEL);
        gtk_window_set_resizable(GTK_WINDOW(window), TRUE);
        gtk_widget_set_size_request(window, -1, prefs_common.compose_height);
@@ -4934,6 +5084,7 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode)
                geometry.max_width = gdk_screen_width();
                geometry.max_height = gdk_screen_height();
        }
+
        gtk_window_set_geometry_hints(GTK_WINDOW(window), NULL,
                                      &geometry, GDK_HINT_MAX_SIZE);
        if (!geometry.min_width) {
@@ -5268,7 +5419,7 @@ static Compose *compose_create(PrefsAccount *account, ComposeMode mode)
        /* Privacy Systems menu */
        compose_update_privacy_systems_menu(compose);
 
-       activate_privacy_system(compose, account);
+       activate_privacy_system(compose, account, TRUE);
        toolbar_set_style(compose->toolbar->toolbar, compose->handlebox, prefs_common.toolbar_style);
        gtk_widget_show(window);
        
@@ -5387,13 +5538,14 @@ static void compose_set_privacy_system_cb(GtkWidget *widget, gpointer data)
        menu_set_sensitive(ifactory, "/Options/Encrypt", can_encrypt);
 }
 
-static void compose_update_privacy_system_menu_item(Compose * compose)
+static void compose_update_privacy_system_menu_item(Compose * compose, gboolean warn)
 {
        static gchar *branch_path = "/Options/Privacy System";
        GtkItemFactory *ifactory;
        GtkWidget *menuitem = NULL;
        GList *amenu;
        gboolean can_sign = FALSE, can_encrypt = FALSE;
+       gboolean found = FALSE;
 
        ifactory = gtk_item_factory_from_widget(compose->menubar);
 
@@ -5415,7 +5567,7 @@ static void compose_update_privacy_system_menu_item(Compose * compose)
 
                                        can_sign = privacy_system_can_sign(systemid);
                                        can_encrypt = privacy_system_can_encrypt(systemid);
-
+                                       found = TRUE;
                                        break;
                                }
 
@@ -5423,6 +5575,15 @@ static void compose_update_privacy_system_menu_item(Compose * compose)
                }
                if (menuitem != NULL)
                        gtk_check_menu_item_set_active(GTK_CHECK_MENU_ITEM(menuitem), TRUE);
+               
+               if (warn && !found && strlen(compose->privacy_system)) {
+                       gchar *tmp = g_strdup_printf(
+                               _("The privacy system '%s' cannot be loaded. You "
+                                 "will not be able to sign or encrypt this message."),
+                                 compose->privacy_system);
+                       alertpanel_warning(tmp);
+                       g_free(tmp);
+               }
        }
 
        menu_set_sensitive(ifactory, "/Options/Sign", can_sign);
@@ -5680,6 +5841,7 @@ static void compose_destroy(Compose *compose)
 
        g_free(compose->exteditor_file);
 
+       g_free(compose->orig_charset);
        if (addressbook_get_target_compose() == compose)
                addressbook_set_target_compose(NULL);
 
@@ -5700,6 +5862,7 @@ static void compose_destroy(Compose *compose)
        gtk_widget_destroy(compose->window);
        toolbar_destroy(compose->toolbar);
        g_free(compose->toolbar);
+       g_mutex_free(compose->mutex);
        g_free(compose);
 }
 
@@ -5889,7 +6052,7 @@ static void compose_attach_property(Compose *compose)
                gtk_list_store_set(GTK_LIST_STORE(model), &iter,
                                   COL_MIMETYPE, ainfo->content_type,
                                   COL_SIZE, text,
-                                  COL_NAME, ainfo->file,
+                                  COL_NAME, ainfo->name,
                                   -1);
                
                break;
@@ -6494,11 +6657,8 @@ static void compose_send_cb(gpointer data, guint action, GtkWidget *widget)
 {
        Compose *compose = (Compose *)data;
        
-       if (prefs_common.work_offline)
-               if (alertpanel(_("Offline warning"), 
-                              _("You're working offline. Override?"),
-                              GTK_STOCK_YES, GTK_STOCK_NO, NULL) != G_ALERTDEFAULT)
-                       return;
+       if (prefs_common.work_offline && !inc_offline_should_override())
+               return;
        
        if (compose->draft_timeout_tag != -1) { /* CLAWS: disable draft timeout */
                gtk_timeout_remove(compose->draft_timeout_tag);
@@ -6542,14 +6702,23 @@ static void compose_draft_cb(gpointer data, guint action, GtkWidget *widget)
 
        draft = account_get_special_folder(compose->account, F_DRAFT);
        g_return_if_fail(draft != NULL);
-
+       
+       if (!g_mutex_trylock(compose->mutex)) {
+               /* we don't want to lock the mutex once it's available,
+                * because as the only other part of compose.c locking
+                * it is compose_close - which means once unlocked,
+                * the compose struct will be freed */
+               debug_print("couldn't lock mutex, probably sending\n");
+               return;
+       }
+       
        lock = TRUE;
 
        tmp = g_strdup_printf("%s%cdraft.%p", get_tmp_dir(),
                              G_DIR_SEPARATOR, compose);
        if ((fp = g_fopen(tmp, "wb")) == NULL) {
                FILE_OP_ERROR(tmp, "fopen");
-               return;
+               goto unlock;
        }
 
        /* chmod for security */
@@ -6579,8 +6748,7 @@ static void compose_draft_cb(gpointer data, guint action, GtkWidget *widget)
                fclose(fp);
                g_unlink(tmp);
                g_free(tmp);
-               lock = FALSE;
-               return;
+               goto unlock;
        }
        fclose(fp);
 
@@ -6588,10 +6756,9 @@ static void compose_draft_cb(gpointer data, guint action, GtkWidget *widget)
        if ((msgnum = folder_item_add_msg(draft, tmp, &flag, TRUE)) < 0) {
                g_unlink(tmp);
                g_free(tmp);
-               lock = FALSE;
                if (action != COMPOSE_AUTO_SAVE)
                        alertpanel_error(_("Could not save draft."));
-               return;
+               goto unlock;
        }
        g_free(tmp);
        draft->mtime = 0;       /* force updating */
@@ -6613,11 +6780,12 @@ static void compose_draft_cb(gpointer data, guint action, GtkWidget *widget)
        
        folder_item_scan(draft);
        
-       lock = FALSE;
-
-       if (action == COMPOSE_QUIT_EDITING)
+       if (action == COMPOSE_QUIT_EDITING) {
+               lock = FALSE;
+               g_mutex_unlock(compose->mutex); /* must be done before closing */
                compose_close(compose);
-       else {
+               return;
+       } else {
                struct stat s;
                gchar *path;
 
@@ -6626,8 +6794,7 @@ static void compose_draft_cb(gpointer data, guint action, GtkWidget *widget)
                if (g_stat(path, &s) < 0) {
                        FILE_OP_ERROR(path, "stat");
                        g_free(path);
-                       lock = FALSE;
-                       return;
+                       goto unlock;
                }
                g_free(path);
 
@@ -6645,6 +6812,9 @@ static void compose_draft_cb(gpointer data, guint action, GtkWidget *widget)
                compose->modified = FALSE;
                compose_set_title(compose);
        }
+unlock:
+       lock = FALSE;
+       g_mutex_unlock(compose->mutex);
 }
 
 static void compose_attach_cb(gpointer data, guint action, GtkWidget *widget)
@@ -6742,8 +6912,8 @@ static void compose_close_cb(gpointer data, guint action, GtkWidget *widget)
 
        if (compose->modified) {
                val = alertpanel(_("Discard message"),
-                                _("This message has been modified. discard it?"),
-                                _("Discard"), _("to Draft"), GTK_STOCK_CANCEL);
+                                _("This message has been modified. Discard it?"),
+                                _("Discard"), _("Save to Drafts"), GTK_STOCK_CANCEL);
 
                switch (val) {
                case G_ALERTDEFAULT:
@@ -6917,32 +7087,6 @@ static void compose_copy_cb(Compose *compose)
                entry_copy_clipboard(compose->focused_editable);
 }
 
-#define BLOCK_WRAP() {                                                 \
-       prev_autowrap = compose->autowrap;                              \
-       buffer = gtk_text_view_get_buffer(                              \
-                                       GTK_TEXT_VIEW(compose->text));  \
-       compose->autowrap = FALSE;                                      \
-                                                                       \
-       g_signal_handlers_block_by_func(G_OBJECT(buffer),               \
-                               G_CALLBACK(compose_changed_cb),         \
-                               compose);                               \
-       g_signal_handlers_block_by_func(G_OBJECT(buffer),               \
-                               G_CALLBACK(text_inserted),              \
-                               compose);                               \
-}
-#define UNBLOCK_WRAP() {                                               \
-       compose->autowrap = prev_autowrap;                              \
-       if (compose->autowrap)                                          \
-               compose_wrap_all(compose);                              \
-                                                                       \
-       g_signal_handlers_unblock_by_func(G_OBJECT(buffer),             \
-                               G_CALLBACK(compose_changed_cb),         \
-                               compose);                               \
-       g_signal_handlers_unblock_by_func(G_OBJECT(buffer),             \
-                               G_CALLBACK(text_inserted),              \
-                               compose);                               \
-}
-
 static void compose_paste_cb(Compose *compose)
 {
        gint prev_autowrap;
@@ -7333,10 +7477,8 @@ static void compose_grab_focus_cb(GtkWidget *widget, Compose *compose)
 
 static void compose_changed_cb(GtkTextBuffer *textbuf, Compose *compose)
 {
-       if (compose->modified == FALSE) {
-               compose->modified = TRUE;
-               compose_set_title(compose);
-       }
+       compose->modified = TRUE;
+       compose_set_title(compose);
 }
 
 static void compose_wrap_cb(gpointer data, guint action, GtkWidget *widget)
@@ -7380,11 +7522,11 @@ static void compose_toggle_encrypt_cb(gpointer data, guint action,
                compose->use_encryption = FALSE;
 }
 
-static void activate_privacy_system(Compose *compose, PrefsAccount *account) 
+static void activate_privacy_system(Compose *compose, PrefsAccount *account, gboolean warn
 {
        g_free(compose->privacy_system);
        compose->privacy_system = g_strdup(account->default_privacy_system);
-       compose_update_privacy_system_menu_item(compose);
+       compose_update_privacy_system_menu_item(compose, warn);
 }
 
 static void compose_toggle_ruler_cb(gpointer data, guint action,
@@ -7413,8 +7555,9 @@ static void compose_attach_drag_received_cb (GtkWidget            *widget,
 {
        Compose *compose = (Compose *)user_data;
        GList *list, *tmp;
-       
-       if (gdk_atom_name(data->type) && !strcmp(gdk_atom_name(data->type), "text/uri-list")) {
+
+       if (gdk_atom_name(data->type) && 
+           !strcmp(gdk_atom_name(data->type), "text/uri-list")) {
                list = uri_list_extract_filenames((const gchar *)data->data);
                for (tmp = list; tmp != NULL; tmp = tmp->next)
                        compose_attach_append
@@ -7423,6 +7566,32 @@ static void compose_attach_drag_received_cb (GtkWidget           *widget,
                if (list) compose_changed_cb(NULL, compose);
                list_free_strings(list);
                g_list_free(list);
+       } else if (gdk_atom_name(data->type) && 
+                  !strcmp(gdk_atom_name(data->type), "text/plain") &&
+                  data->data && !strcmp(data->data, "Dummy-Summaryview")) {
+               /* comes from our summaryview */
+               SummaryView * summaryview = NULL;
+               GSList * list = NULL, *cur = NULL;
+               
+               if (mainwindow_get_mainwindow())
+                       summaryview = mainwindow_get_mainwindow()->summaryview;
+               
+               if (summaryview)
+                       list = summary_get_selected_msg_list(summaryview);
+               
+               for (cur = list; cur; cur = cur->next) {
+                       MsgInfo *msginfo = (MsgInfo *)cur->data;
+                       gchar *file = NULL;
+                       if (msginfo)
+                               file = procmsg_get_message_file_full(msginfo, 
+                                       TRUE, TRUE);
+                       if (file) {
+                               compose_attach_append(compose, (const gchar *)file, 
+                                       (const gchar *)file, "message/rfc822");
+                               g_free(file);
+                       }
+               }
+               g_slist_free(list);
        }
 }
 
@@ -7723,15 +7892,25 @@ static PrefsAccount *compose_guess_forward_account_from_msginfo(MsgInfo *msginfo
        return account;
 }
 
-static void compose_close(Compose *compose)
+static gboolean compose_close(Compose *compose)
 {
        gint x, y;
 
-       g_return_if_fail(compose);
+       if (!g_mutex_trylock(compose->mutex)) {
+               /* we have to wait for the (possibly deferred by auto-save)
+                * drafting to be done, before destroying the compose under
+                * it. */
+               debug_print("waiting for drafting to finish...\n");
+               g_timeout_add (500, (GSourceFunc) compose_close, compose);
+               return FALSE;
+       }
+       g_return_val_if_fail(compose, FALSE);
        gtkut_widget_get_uposition(compose->window, &x, &y);
        prefs_common.compose_x = x;
        prefs_common.compose_y = y;
+       g_mutex_unlock(compose->mutex);
        compose_destroy(compose);
+       return FALSE;
 }
 
 /**