2005-10-12 [colin] 1.9.15cvs32
[claws.git] / src / compose.c
index 7e59990c0e945daa513f40510ae7c2dfec0e964c..d7e43829d937e75ea51bca7f2626b238254a674c 100644 (file)
@@ -469,7 +469,7 @@ static void text_inserted           (GtkTextBuffer  *buffer,
                                         const gchar    *text,
                                         gint            len,
                                         Compose        *compose);
-static void compose_generic_reply(MsgInfo *msginfo, gboolean quote,
+static Compose *compose_generic_reply(MsgInfo *msginfo, gboolean quote,
                                  gboolean to_all, gboolean to_ml,
                                  gboolean to_sender,
                                  gboolean followup_and_reply_to,
@@ -529,6 +529,7 @@ static GtkItemFactoryEntry compose_entries[] =
        {N_("/_Edit/Cu_t"),             "<control>X", compose_cut_cb,    0, NULL},
        {N_("/_Edit/_Copy"),            "<control>C", compose_copy_cb,   0, NULL},
        {N_("/_Edit/_Paste"),           "<control>V", compose_paste_cb,  0, NULL},
+       {N_("/_Edit/Special paste"),    NULL, NULL, 0, "<Branch>"},
        {N_("/_Edit/Special paste/as _quotation"),
                                        NULL, compose_paste_as_quote_cb, 0, NULL},
        {N_("/_Edit/Special paste/_wrapped"),
@@ -924,6 +925,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;
 }
 
@@ -932,6 +934,9 @@ static void compose_force_encryption(Compose *compose, PrefsAccount *account,
 {
        gchar *privacy = NULL;
 
+       g_return_if_fail(compose != NULL);
+       g_return_if_fail(account != NULL);
+
        if (override_pref == FALSE && account->default_encrypt_reply == FALSE)
                return;
 
@@ -971,109 +976,142 @@ static void compose_force_signing(Compose *compose, PrefsAccount *account)
        }
 }      
 
-void compose_reply_mode(ComposeMode mode, GSList *msginfo_list, gchar *body)
+Compose *compose_reply_mode(ComposeMode mode, GSList *msginfo_list, gchar *body)
 {
        MsgInfo *msginfo;
        guint list_len;
-
-       g_return_if_fail(msginfo_list != NULL);
+       Compose *compose = NULL;
+       g_return_val_if_fail(msginfo_list != NULL, NULL);
 
        msginfo = (MsgInfo*)g_slist_nth_data(msginfo_list, 0);
-       g_return_if_fail(msginfo != NULL);
+       g_return_val_if_fail(msginfo != NULL, NULL);
 
        list_len = g_slist_length(msginfo_list);
 
        switch (mode) {
        case COMPOSE_REPLY:
-               compose_reply(msginfo, prefs_common.reply_with_quote,
+               compose = compose_reply(msginfo, prefs_common.reply_with_quote,
                              FALSE, prefs_common.default_reply_list, FALSE, body);
                break;
        case COMPOSE_REPLY_WITH_QUOTE:
-               compose_reply(msginfo, TRUE, FALSE, prefs_common.default_reply_list, FALSE, body);
+               compose = compose_reply(msginfo, TRUE, 
+                       FALSE, prefs_common.default_reply_list, FALSE, body);
                break;
        case COMPOSE_REPLY_WITHOUT_QUOTE:
-               compose_reply(msginfo, FALSE, FALSE, prefs_common.default_reply_list, FALSE, NULL);
+               compose = compose_reply(msginfo, FALSE, 
+                       FALSE, prefs_common.default_reply_list, FALSE, NULL);
                break;
        case COMPOSE_REPLY_TO_SENDER:
-               compose_reply(msginfo, prefs_common.reply_with_quote,
+               compose = compose_reply(msginfo, prefs_common.reply_with_quote,
                              FALSE, FALSE, TRUE, body);
                break;
        case COMPOSE_FOLLOWUP_AND_REPLY_TO:
-               compose_followup_and_reply_to(msginfo,
+               compose = compose_followup_and_reply_to(msginfo,
                                              prefs_common.reply_with_quote,
                                              FALSE, FALSE, body);
                break;
        case COMPOSE_REPLY_TO_SENDER_WITH_QUOTE:
-               compose_reply(msginfo, TRUE, FALSE, FALSE, TRUE, body);
+               compose = compose_reply(msginfo, TRUE, 
+                       FALSE, FALSE, TRUE, body);
                break;
        case COMPOSE_REPLY_TO_SENDER_WITHOUT_QUOTE:
-               compose_reply(msginfo, FALSE, FALSE, FALSE, TRUE, NULL);
+               compose = compose_reply(msginfo, FALSE, 
+                       FALSE, FALSE, TRUE, NULL);
                break;
        case COMPOSE_REPLY_TO_ALL:
-               compose_reply(msginfo, prefs_common.reply_with_quote,
-                             TRUE, FALSE, FALSE, body);
+               compose = compose_reply(msginfo, prefs_common.reply_with_quote,
+                       TRUE, FALSE, FALSE, body);
                break;
        case COMPOSE_REPLY_TO_ALL_WITH_QUOTE:
-               compose_reply(msginfo, TRUE, TRUE, FALSE, FALSE, body);
+               compose = compose_reply(msginfo, TRUE, 
+                       TRUE, FALSE, FALSE, body);
                break;
        case COMPOSE_REPLY_TO_ALL_WITHOUT_QUOTE:
-               compose_reply(msginfo, FALSE, TRUE, FALSE, FALSE, NULL);
+               compose = compose_reply(msginfo, FALSE, 
+                       TRUE, FALSE, FALSE, NULL);
                break;
        case COMPOSE_REPLY_TO_LIST:
-               compose_reply(msginfo, prefs_common.reply_with_quote,
-                             FALSE, TRUE, FALSE, body);
+               compose = compose_reply(msginfo, prefs_common.reply_with_quote,
+                       FALSE, TRUE, FALSE, body);
                break;
        case COMPOSE_REPLY_TO_LIST_WITH_QUOTE:
-               compose_reply(msginfo, TRUE, FALSE, TRUE, FALSE, body);
+               compose = compose_reply(msginfo, TRUE, 
+                       FALSE, TRUE, FALSE, body);
                break;
        case COMPOSE_REPLY_TO_LIST_WITHOUT_QUOTE:
-               compose_reply(msginfo, FALSE, FALSE, TRUE, FALSE, NULL);
+               compose = compose_reply(msginfo, FALSE, 
+                       FALSE, TRUE, FALSE, NULL);
                break;
        case COMPOSE_FORWARD:
                if (prefs_common.forward_as_attachment) {
-                       compose_reply_mode(COMPOSE_FORWARD_AS_ATTACH, msginfo_list, body);
-                       return;
+                       compose = compose_reply_mode(COMPOSE_FORWARD_AS_ATTACH, msginfo_list, body);
+                       return compose;
                } else {
-                       compose_reply_mode(COMPOSE_FORWARD_INLINE, msginfo_list, body);
-                       return;
+                       compose = compose_reply_mode(COMPOSE_FORWARD_INLINE, msginfo_list, body);
+                       return compose;
                }
                break;
        case COMPOSE_FORWARD_INLINE:
                /* check if we reply to more than one Message */
                if (list_len == 1) {
-                       compose_forward(NULL, msginfo, FALSE, body, FALSE);
+                       compose = compose_forward(NULL, msginfo, FALSE, body, FALSE);
                        break;
                } 
                /* more messages FALL THROUGH */
        case COMPOSE_FORWARD_AS_ATTACH:
-               compose_forward_multiple(NULL, msginfo_list);
+               compose = compose_forward_multiple(NULL, msginfo_list);
                break;
        case COMPOSE_REDIRECT:
-               compose_redirect(NULL, msginfo);
+               compose = compose_redirect(NULL, msginfo);
                break;
        default:
                g_warning("compose_reply(): invalid Compose Mode: %d\n", mode);
        }
+       return compose;
 }
 
-void compose_reply(MsgInfo *msginfo, gboolean quote, gboolean to_all,
+Compose *compose_reply(MsgInfo *msginfo, gboolean quote, gboolean to_all,
                   gboolean to_ml, gboolean to_sender, 
                   const gchar *body)
 {
-       compose_generic_reply(msginfo, quote, to_all, to_ml, 
+       return compose_generic_reply(msginfo, quote, to_all, to_ml, 
                              to_sender, FALSE, body);
 }
 
-void compose_followup_and_reply_to(MsgInfo *msginfo, gboolean quote,
+Compose *compose_followup_and_reply_to(MsgInfo *msginfo, gboolean quote,
                                   gboolean to_all,
                                   gboolean to_sender,
                                   const gchar *body)
 {
-       compose_generic_reply(msginfo, quote, to_all, FALSE, 
+       return compose_generic_reply(msginfo, quote, to_all, FALSE, 
                              to_sender, TRUE, body);
 }
 
-static void compose_generic_reply(MsgInfo *msginfo, gboolean quote,
+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 Compose *compose_generic_reply(MsgInfo *msginfo, gboolean quote,
                                  gboolean to_all, gboolean to_ml,
                                  gboolean to_sender,
                                  gboolean followup_and_reply_to,
@@ -1088,12 +1126,12 @@ static void compose_generic_reply(MsgInfo *msginfo, gboolean quote,
        GtkTextIter iter;
        int cursor_pos;
 
-       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);
 
        account = account_get_reply_account(msginfo, prefs_common.reply_account_autosel);
        
-       g_return_if_fail(account != NULL);
+       g_return_val_if_fail(account != NULL, NULL);
 
        if (to_sender && account->protocol == A_NNTP &&
            !followup_and_reply_to) {
@@ -1102,7 +1140,7 @@ static void compose_generic_reply(MsgInfo *msginfo, gboolean quote,
                if (!reply_account)
                        reply_account = compose_current_mail_account();
                if (!reply_account)
-                       return;
+                       return NULL;
        } else
                reply_account = account;
 
@@ -1113,9 +1151,12 @@ static void compose_generic_reply(MsgInfo *msginfo, gboolean quote,
        menu_set_sensitive(ifactory, "/Options/Remove references", TRUE);
 
        compose->replyinfo = procmsg_msginfo_get_full_info(msginfo);
+
        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);
 
@@ -1129,7 +1170,7 @@ static void compose_generic_reply(MsgInfo *msginfo, gboolean quote,
                g_free(folderidentifier);
        }
 
-       if (compose_parse_header(compose, msginfo) < 0) return;
+       if (compose_parse_header(compose, msginfo) < 0) return NULL;
        compose_reply_set_entry(compose, msginfo, to_all, to_ml, 
                                to_sender, followup_and_reply_to);
        compose_show_first_last_header(compose, TRUE);
@@ -1180,6 +1221,9 @@ static void compose_generic_reply(MsgInfo *msginfo, gboolean quote,
 
        if (prefs_common.auto_exteditor)
                compose_exec_ext_editor(compose);
+               
+       compose_set_title(compose);
+       return compose;
 }
 
 #define INSERT_FW_HEADER(var, hdr) \
@@ -1212,6 +1256,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;
 
@@ -1286,6 +1332,7 @@ Compose *compose_forward(PrefsAccount *account, MsgInfo *msginfo,
                g_free(folderidentifier);
        }
 
+       compose_set_title(compose);
         return compose;
 }
 
@@ -1345,6 +1392,7 @@ Compose *compose_forward_multiple(PrefsAccount *account, GSList *msginfo_list)
 
        gtk_widget_grab_focus(compose->header_last->entry);
        
+       compose_set_title(compose);
        return compose;
 }
 
@@ -1480,6 +1528,8 @@ void compose_reedit(MsgInfo *msginfo)
        }
        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];
@@ -1542,6 +1592,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)
@@ -1620,6 +1671,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;
 }
 
@@ -2128,11 +2180,25 @@ static void compose_reply_set_entry(Compose *compose, MsgInfo *msginfo,
        g_return_if_fail(msginfo != NULL);
 
        if (compose->account->protocol != A_NNTP) {
-               if (!compose->replyto && to_ml && compose->ml_post
-                   && !(msginfo->folder && msginfo->folder->prefs->enable_default_reply_to))
+               if (to_ml && compose->ml_post
+                   && !(msginfo->folder && 
+                        msginfo->folder->prefs->enable_default_reply_to)) {
                        compose_entry_append(compose,
                                           compose->ml_post,
                                           COMPOSE_TO);
+                       if (compose->replyto) {
+                               gchar *tmp1 = NULL, *tmp2 = NULL;
+                               Xstrdup_a(tmp1, compose->replyto, return);
+                               if (compose->ml_post)
+                                       Xstrdup_a(tmp2, compose->ml_post, return);
+                               extract_address(tmp1);
+                               extract_address(tmp2);
+                               if (tmp1 && tmp2 && strcmp(tmp1, tmp2))
+                                       compose_entry_append(compose,
+                                                       compose->replyto,
+                                                       COMPOSE_CC);
+                       }
+               }
                else if (!(to_all || to_sender)
                         && msginfo->folder
                         && msginfo->folder->prefs->enable_default_reply_to) {
@@ -3283,15 +3349,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);
 }
 
 /**
@@ -3738,11 +3813,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;
@@ -3756,12 +3858,6 @@ 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));
 
@@ -3774,12 +3870,12 @@ static gint compose_write_to_file(Compose *compose, FILE *fp, gint action)
                                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);
@@ -3815,10 +3911,7 @@ static gint compose_write_to_file(Compose *compose, FILE *fp, gint action)
        /* protect trailing spaces when signing message */
        if (action == COMPOSE_WRITE_FOR_SEND && compose->use_signing && 
            privacy_system_can_sign(compose->privacy_system)) {
-               if (encoding == ENC_7BIT)
-                       encoding = ENC_QUOTED_PRINTABLE;
-               else if (encoding == ENC_8BIT)
-                       encoding = ENC_BASE64;
+               encoding = ENC_QUOTED_PRINTABLE;
        }
        
        /* check for line length limit */
@@ -4546,7 +4639,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);
 
@@ -4564,8 +4658,42 @@ static void compose_convert_header(Compose *compose, gchar *dest, gint len, gcha
                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);
 }
 
@@ -5459,7 +5587,7 @@ static void compose_update_privacy_system_menu_item(Compose * compose, gboolean
                        GList *alist = amenu->next;
 
                        systemid = g_object_get_data(G_OBJECT(amenu->data), "privacy_system");
-                       if (systemid != NULL)
+                       if (systemid != NULL) {
                                if (strcmp(systemid, compose->privacy_system) == 0) {
                                        menuitem = GTK_WIDGET(amenu->data);
 
@@ -5467,7 +5595,15 @@ static void compose_update_privacy_system_menu_item(Compose * compose, gboolean
                                        can_encrypt = privacy_system_can_encrypt(systemid);
                                        found = TRUE;
                                        break;
-                               }
+                               } 
+                       } else if (strlen(compose->privacy_system) == 0) {
+                                       menuitem = GTK_WIDGET(amenu->data);
+
+                                       can_sign = FALSE;
+                                       can_encrypt = FALSE;
+                                       found = TRUE;
+                                       break;
+                       }
 
                        amenu = alist;
                }
@@ -5482,7 +5618,7 @@ static void compose_update_privacy_system_menu_item(Compose * compose, gboolean
                        alertpanel_warning(tmp);
                        g_free(tmp);
                }
-       }
+       } 
 
        menu_set_sensitive(ifactory, "/Options/Sign", can_sign);
        menu_set_sensitive(ifactory, "/Options/Encrypt", can_encrypt);
@@ -5739,6 +5875,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);
 
@@ -6810,7 +6947,7 @@ 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"), _("Save to Drafts"), GTK_STOCK_CANCEL);
+                                _("_Discard"), _("_Save to Drafts"), GTK_STOCK_CANCEL);
 
                switch (val) {
                case G_ALERTDEFAULT:
@@ -7374,10 +7511,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)
@@ -7424,6 +7559,7 @@ static void compose_toggle_encrypt_cb(gpointer data, guint action,
 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, warn);
 }
@@ -7834,7 +7970,9 @@ void compose_reply_from_messageview(MessageView *msgview, GSList *msginfo_list,
        gchar *body;
        GSList *new_msglist = NULL;
        MsgInfo *tmp_msginfo = NULL;
-       
+       gboolean originally_enc = FALSE;
+       Compose *compose = NULL;
+
        g_return_if_fail(msgview != NULL);
 
        g_return_if_fail(msginfo_list != NULL);
@@ -7845,11 +7983,13 @@ void compose_reply_from_messageview(MessageView *msgview, GSList *msginfo_list,
                
                if (mimeinfo != NULL && mimeinfo->type == MIMETYPE_MESSAGE && 
                    !g_ascii_strcasecmp(mimeinfo->subtype, "rfc822")) {
-                       
                        tmp_msginfo = procmsg_msginfo_new_from_mimeinfo(
                                                orig_msginfo, mimeinfo);
                        if (tmp_msginfo != NULL) {
                                new_msglist = g_slist_append(NULL, tmp_msginfo);
+                               if (procmime_msginfo_is_encrypted(orig_msginfo)) {
+                                       originally_enc = TRUE;
+                               }
                        } 
                }
        }
@@ -7857,11 +7997,15 @@ void compose_reply_from_messageview(MessageView *msgview, GSList *msginfo_list,
        body = messageview_get_selection(msgview);
 
        if (new_msglist) {
-               compose_reply_mode((ComposeMode)action, new_msglist, body);
+               compose = compose_reply_mode((ComposeMode)action, new_msglist, body);
                procmsg_msginfo_free(tmp_msginfo);
                g_slist_free(new_msglist);
        } else
-               compose_reply_mode((ComposeMode)action, msginfo_list, body);
+               compose = compose_reply_mode((ComposeMode)action, msginfo_list, body);
+
+       if (originally_enc) {
+               compose_force_encryption(compose, compose->account, FALSE);
+       }
 
        g_free(body);
 }