[balsa/69-resend-message: 9/16] Various: Refactor resending a message




commit 5b914676a31db108512b9d752dd0f87f02245139
Author: Peter Bloomfield <PeterBloomfield bellsouth net>
Date:   Tue Mar 1 14:29:33 2022 -0500

    Various: Refactor resending a message
    
    Move the GMimeMessage manipulation from src/sendmsg-window.c to libbalsa/send.c;
    see https://gitlab.gnome.org/GNOME/balsa/-/issues/69#note_1396270 and replies.
    
    Also fix saving and continuing a resend message.
    
    modified:   libbalsa/send.c
    modified:   src/balsa-index.c
    modified:   src/message-window.c
    modified:   src/sendmsg-window.c
    modified:   src/sendmsg-window.h

 libbalsa/send.c      | 100 +++++++++--
 src/balsa-index.c    |   2 +-
 src/message-window.c |   4 +-
 src/sendmsg-window.c | 470 +++++++++++++++++++++++----------------------------
 src/sendmsg-window.h |   4 +-
 5 files changed, 303 insertions(+), 277 deletions(-)
---
diff --git a/libbalsa/send.c b/libbalsa/send.c
index 842ec712c..fe65174aa 100644
--- a/libbalsa/send.c
+++ b/libbalsa/send.c
@@ -445,7 +445,8 @@ lbs_message_queue_real(LibBalsaMessage    *message,
                 g_mime_part_set_content_encoding(GMIME_PART
                                                      (mime_msgs[i]->mime_part),
                                                  GMIME_CONTENT_ENCODING_7BIT);
-                libbalsa_set_message_id(mime_msgs[i]);
+                if (i > 0)
+                    libbalsa_set_message_id(mime_msgs[i]);
             }
             if (rc) {
                 /* Temporarily modify message by changing its mime_msg: */
@@ -1576,6 +1577,7 @@ libbalsa_message_postpone(LibBalsaMessage *message,
                           GError         **error)
 {
     GMimeMessage *mime_message;
+    gboolean resending;
 
     mime_message = libbalsa_message_get_mime_message(message);
     if (mime_message == NULL) {
@@ -1585,6 +1587,17 @@ libbalsa_message_postpone(LibBalsaMessage *message,
             return FALSE;
     }
 
+    resending = libbalsa_message_get_user_header(message, "X-Balsa-Resend") != NULL;
+    if (resending) {
+        /* Cosmetics: copy the resent message's subject to the containing message,
+         * so that it displays helpfully in the draftbox */
+        GMimeObject *part = g_mime_message_get_mime_part(mime_message);
+        GMimeMessage *resend_mime_message =
+            g_mime_message_part_get_message(GMIME_MESSAGE_PART(part));
+        const char *subject = g_mime_message_get_subject(resend_mime_message);
+        g_mime_message_set_subject(mime_message, subject, NULL);
+    }
+
     if (extra_headers != NULL) {
         GMimeObject *object = GMIME_OBJECT(mime_message);
         gint i;
@@ -1628,7 +1641,6 @@ libbalsa_set_message_id(GMimeMessage *mime_message)
     gchar *message_id;
     guint8 *src;
     gchar *dst;
-    gboolean is_resend;
 
     g_mutex_lock(&mutex);
     if (rand == NULL) {
@@ -1678,19 +1690,76 @@ libbalsa_set_message_id(GMimeMessage *mime_message)
         }
     }
 
-    is_resend = g_mime_object_get_header(GMIME_OBJECT(mime_message), "Resent-From") != NULL;
-    if (is_resend) {
-        g_mime_object_prepend_header(GMIME_OBJECT(mime_message),
-                                     "Resent-Message-ID",
-                                     message_id,
-                                     NULL);
-    } else {
-        g_mime_message_set_message_id(mime_message, message_id);
-    }
+    g_mime_message_set_message_id(mime_message, message_id);
 
     g_free(message_id);
 }
 
+static void
+lbs_prepend_resent_header(GMimeObject         *object,
+                          InternetAddressList *list,
+                          const char          *field)
+{
+    if (ia_list_not_empty(list)) {
+        char *value = internet_address_list_to_string(list, NULL, FALSE);
+        g_mime_object_prepend_header(object, field, value, NULL);
+        g_free(value);
+    }
+}
+
+static void
+lbs_add_resent_headers(LibBalsaMessage *message, GMimeMessage *mime_message)
+{
+    LibBalsaMessageHeaders *headers = libbalsa_message_get_headers(message);
+    GMimeObject *object = GMIME_OBJECT(mime_message);
+    const char *mailbox = "";
+    char *value;
+    GDateTime *date_time;
+
+    lbs_prepend_resent_header(object, headers->bcc_list, "Resent-BCC");
+    lbs_prepend_resent_header(object, headers->cc_list, "Resent-CC");
+    lbs_prepend_resent_header(object, headers->to_list, "Resent-To");
+
+    if (ia_list_not_empty(headers->from)) {
+        InternetAddress *ia = internet_address_list_get_address(headers->from, 0);
+        while (ia != NULL && INTERNET_ADDRESS_IS_GROUP(ia))
+            ia = internet_address_list_get_address(INTERNET_ADDRESS_GROUP( ia)->members, 0);
+        if (ia != NULL)
+            mailbox = INTERNET_ADDRESS_MAILBOX(ia)->addr;
+    }
+    g_mime_object_prepend_header(object, "Resent-From", mailbox, NULL);
+
+    date_time = g_date_time_new_from_unix_local(headers->date);
+    value = g_mime_utils_header_format_date(date_time);
+    g_date_time_unref(date_time);
+    g_mime_object_prepend_header(object, "Resent-Date", value, NULL);
+    g_free(value);
+}
+
+static void
+create_mime_message_resend(LibBalsaMessage  *message,
+                           const char       *message_id,
+                           GMimeMessage    **return_message)
+{
+    GMimeObject *part;
+    GMimeMessage *mime_message;
+
+    part = g_mime_message_get_mime_part(*return_message);
+    mime_message = g_mime_message_part_get_message(GMIME_MESSAGE_PART(part));
+
+    g_mime_object_prepend_header(GMIME_OBJECT(mime_message),
+                                 "Resent-Message-ID",
+                                 message_id,
+                                 NULL);
+
+    lbs_add_resent_headers(message, mime_message);
+    libbalsa_message_set_subject(message, g_mime_message_get_subject(mime_message));
+
+    g_object_ref(*return_message);
+    libbalsa_message_set_mime_message(message, mime_message);
+    g_set_object(return_message, mime_message);
+    g_object_unref(*return_message);
+}
 
 /* balsa_create_msg:
    copies message to msg.
@@ -1701,6 +1770,7 @@ libbalsa_create_msg(LibBalsaMessage *message,
                     GError         **error)
 {
     GMimeMessage *mime_message;
+    gboolean resending;
 
     mime_message = libbalsa_message_get_mime_message(message);
     if (mime_message == NULL) {
@@ -1712,6 +1782,14 @@ libbalsa_create_msg(LibBalsaMessage *message,
 
     libbalsa_set_message_id(mime_message);
 
+    resending = libbalsa_message_get_user_header(message, "X-Balsa-Resend") != NULL;
+    if (resending) {
+        const char *message_id = g_mime_message_get_message_id(mime_message);
+        char *resent_message_id = g_strdup_printf("<%s>", message_id);
+        create_mime_message_resend(message, resent_message_id, &mime_message);
+        g_free(resent_message_id);
+    }
+
     return LIBBALSA_MESSAGE_CREATE_OK;
 }
 
diff --git a/src/balsa-index.c b/src/balsa-index.c
index 7297d9a21..3fe4b3b5a 100644
--- a/src/balsa-index.c
+++ b/src/balsa-index.c
@@ -1690,7 +1690,7 @@ bndx_compose_foreach(BalsaIndex * index, SendType send_type)
             sm = sendmsg_window_reply(mailbox, msgno, send_type);
             break;
         case SEND_RESEND:
-            sm = sendmsg_window_resend(mailbox, msgno);
+            sm = sendmsg_window_forward(mailbox, msgno, send_type);
             break;
         case SEND_CONTINUE:
             sm = sendmsg_window_continue(mailbox, msgno);
diff --git a/src/message-window.c b/src/message-window.c
index 933774c3c..e624fbd0f 100644
--- a/src/message-window.c
+++ b/src/message-window.c
@@ -449,7 +449,7 @@ mw_forward_attached_activated(GSimpleAction * action, GVariant * parameter,
 
     sendmsg_window_forward(libbalsa_message_get_mailbox(mw->message),
                            libbalsa_message_get_msgno(mw->message),
-                           TRUE);
+                           SEND_FORWARD_ATTACH);
 }
 
 static void
@@ -462,7 +462,7 @@ mw_forward_inline_activated(GSimpleAction * action, GVariant * parameter,
 
     sendmsg_window_forward(libbalsa_message_get_mailbox(mw->message),
                            libbalsa_message_get_msgno(mw->message),
-                           FALSE);
+                           SEND_FORWARD_INLINE);
 }
 
 #if 0
diff --git a/src/sendmsg-window.c b/src/sendmsg-window.c
index da04be6ee..063e5c1d7 100644
--- a/src/sendmsg-window.c
+++ b/src/sendmsg-window.c
@@ -2967,7 +2967,9 @@ continue_body(BalsaSendmsg * bsmsg, LibBalsaMessage * message)
 
     /* if the first part is of type text/plain with a NULL filename, it
        was the message... */
-    if (body != NULL && body->filename == NULL) {
+    if (body != NULL &&
+        body->filename == NULL &&
+        libbalsa_message_body_type(body) == LIBBALSA_MESSAGE_BODY_TYPE_TEXT) {
         GString *rbdy;
         gchar *body_type = libbalsa_message_body_get_mime_type(body);
         gint llen = -1;
@@ -4920,86 +4922,34 @@ sw_required_charset(BalsaSendmsg * bsmsg, const gchar * text)
     return charset;
 }
 
-static void
-bsmsg_add_resent_header(BalsaSendmsg *bsmsg,
-                        GMimeObject  *mime_object,
-                        const char   *address_view_type,
-                        const char   *field)
-{
-    InternetAddressList *list;
-    char *value;
-
-    list = libbalsa_address_view_get_list(bsmsg->recipient_view, address_view_type);
-    value = internet_address_list_to_string(list, NULL, FALSE);
-    g_mime_object_prepend_header(mime_object, field, value, NULL);
-    g_free(value);
-}
-
-static LibBalsaMessage *
-bsmsg2resent_message(BalsaSendmsg *bsmsg)
-{
-    LibBalsaMessage *message;
-    GMimeMessage *mime_message;
-    GMimeObject *mime_object;
-    InternetAddress *ia;
-    char *value;
-    GDateTime *date_time;
-
-    message = libbalsa_message_new();
-
-    mime_message = libbalsa_message_get_mime_message(bsmsg->parent_message);
-    libbalsa_message_set_mime_message(message, g_object_ref(mime_message));
-
-    mime_object = GMIME_OBJECT(mime_message);
-
-    ia = libbalsa_identity_get_address(bsmsg->ident);
-    value = internet_address_to_string(ia, NULL, FALSE);
-    g_mime_object_prepend_header(mime_object, "Resent-From", value, NULL);
-    g_free(value);
-
-    bsmsg_add_resent_header(bsmsg, mime_object, "To:", "Resent-To");
-    bsmsg_add_resent_header(bsmsg, mime_object, "CC:", "Resent-Cc");
-    bsmsg_add_resent_header(bsmsg, mime_object, "BCC:", "Resent-Bcc");
-
-    /* get the fcc-box from the option menu widget */
-    bsmsg->fcc_url =
-        g_strdup(balsa_mblist_mru_option_menu_get(bsmsg->fcc[1]));
-
-    date_time = g_date_time_new_now_local();
-    value = g_mime_utils_header_format_date(date_time);
-    g_date_time_unref(date_time);
-    g_mime_object_prepend_header(mime_object, "Resent-Date", value, NULL);
-    g_free(value);
-
-    return message;
-}
-
 static LibBalsaMessage *
 bsmsg2message(BalsaSendmsg * bsmsg)
 {
     LibBalsaMessage *message;
     LibBalsaMessageHeaders *headers;
     LibBalsaMessageBody *body;
-    gchar *tmp;
     GtkTextIter start, end;
     LibBalsaIdentity *ident = bsmsg->ident;
     InternetAddress *ia = libbalsa_identity_get_address(ident);
     GtkTextBuffer *buffer;
     GtkTextBuffer *new_buffer = NULL;
 
+    message = libbalsa_message_new();
+
     if (bsmsg->type == SEND_RESEND)
-        return bsmsg2resent_message(bsmsg);
+        libbalsa_message_set_user_header(message, "X-Balsa-Resend", "message is to be resent");
 
-    message = libbalsa_message_new();
     headers = libbalsa_message_get_headers(message);
 
     headers->from = internet_address_list_new ();
     internet_address_list_add(headers->from, ia);
 
-    tmp = gtk_editable_get_chars(GTK_EDITABLE(bsmsg->subject[1]), 0, -1);
-    strip_chars(tmp, "\r\n");
-    libbalsa_message_set_subject(message, tmp);
-    g_free(tmp);
+    if (bsmsg->subject[1] != NULL) {
+        char *subject = gtk_editable_get_chars(GTK_EDITABLE(bsmsg->subject[1]), 0, -1);
+        strip_chars(subject, "\r\n");
+        libbalsa_message_set_subject(message, subject);
+        g_free(subject);
+    }
 
     headers->to_list =
         libbalsa_address_view_get_list(bsmsg->recipient_view, "To:");
@@ -5015,92 +4965,97 @@ bsmsg2message(BalsaSendmsg * bsmsg)
     bsmsg->fcc_url =
         g_strdup(balsa_mblist_mru_option_menu_get(bsmsg->fcc[1]));
 
-    headers->reply_to =
-        libbalsa_address_view_get_list(bsmsg->replyto_view, "Reply To:");
-
-    if (bsmsg->req_mdn)
-       libbalsa_message_set_dispnotify(message, ia);
     libbalsa_message_set_request_dsn(message, bsmsg->req_dsn);
 
+    if (bsmsg->type != SEND_RESEND) {
+        headers->reply_to = libbalsa_address_view_get_list(bsmsg->replyto_view, "Reply To:");
 
-    sw_set_header_from_path(message, "Face", libbalsa_identity_get_face_path(ident),
-            /* Translators: please do not translate Face. */
-                            _("Could not load Face header file %s: %s"));
-    sw_set_header_from_path(message, "X-Face", libbalsa_identity_get_x_face_path(ident),
-            /* Translators: please do not translate Face. */
-                            _("Could not load X-Face header file %s: %s"));
+        if (bsmsg->req_mdn)
+            libbalsa_message_set_dispnotify(message, ia);
 
-    libbalsa_message_set_references(message, bsmsg->references);
-    bsmsg->references = NULL; /* steal it */
 
-    if (bsmsg->in_reply_to != NULL) {
-        libbalsa_message_set_in_reply_to(message,
-                                         g_list_prepend(NULL,
-                                                        g_strdup(bsmsg->in_reply_to)));
-    }
+        sw_set_header_from_path(message, "Face", libbalsa_identity_get_face_path(ident),
+                /* Translators: please do not translate Face. */
+                                _("Could not load Face header file %s: %s"));
+        sw_set_header_from_path(message, "X-Face", libbalsa_identity_get_x_face_path(ident),
+                /* Translators: please do not translate Face. */
+                                _("Could not load X-Face header file %s: %s"));
 
-    body = libbalsa_message_body_new(message);
+        libbalsa_message_set_references(message, bsmsg->references);
+        bsmsg->references = NULL; /* steal it */
 
-    buffer = gtk_text_view_get_buffer(GTK_TEXT_VIEW(bsmsg->text));
-    gtk_text_buffer_get_bounds(buffer, &start, &end);
+        if (bsmsg->in_reply_to != NULL) {
+            libbalsa_message_set_in_reply_to(message,
+                                             g_list_prepend(NULL,
+                                                            g_strdup(bsmsg->in_reply_to)));
+        }
 
-    if (bsmsg->flow) {
-        /* Copy the message text to a new buffer: */
-        GtkTextTagTable *table;
+        body = libbalsa_message_body_new(message);
 
-        table = gtk_text_buffer_get_tag_table(buffer);
-        new_buffer = gtk_text_buffer_new(table);
+        buffer = gtk_text_view_get_buffer(GTK_TEXT_VIEW(bsmsg->text));
+        gtk_text_buffer_get_bounds(buffer, &start, &end);
 
-        tmp = gtk_text_iter_get_text(&start, &end);
-        gtk_text_buffer_set_text(new_buffer, tmp, -1);
-        g_free(tmp);
+        if (bsmsg->flow) {
+            /* Copy the message text to a new buffer: */
+            GtkTextTagTable *table;
+            char *text;
 
-        /* Remove spaces before a newline: */
-        gtk_text_buffer_get_bounds(new_buffer, &start, &end);
-        libbalsa_unwrap_buffer(new_buffer, &start, -1);
-        gtk_text_buffer_get_bounds(new_buffer, &start, &end);
-    }
-
-    /* Copy the buffer text to the message: */
-    body->buffer = gtk_text_iter_get_text(&start, &end);
-    if (new_buffer)
-        g_object_unref(new_buffer);
-
-    if (bsmsg->send_mp_alt)
-        body->html_buffer =
-            libbalsa_text_to_html(LIBBALSA_MESSAGE_GET_SUBJECT(message), body->buffer,
-                                  bsmsg->spell_check_lang);
-    if (bsmsg->flow)
-       body->buffer =
-           libbalsa_wrap_rfc2646(body->buffer, balsa_app.wraplength,
-                                  TRUE, FALSE, TRUE);
-
-    /* Ildar reports that, when a message contains both text/plain and
-     * text/html parts, some broken MUAs use the charset from the
-     * text/plain part to display the text/html part; the latter is
-     * encoded as UTF-8 by add_mime_body_plain (send.c), so we'll use
-     * the same encoding for the text/plain part.
-     * https://bugzilla.gnome.org/show_bug.cgi?id=580704 */
-    body->charset =
-        g_strdup(bsmsg->send_mp_alt ?
-                 "UTF-8" : sw_required_charset(bsmsg, body->buffer));
-    libbalsa_message_append_part(message, body);
+            table = gtk_text_buffer_get_tag_table(buffer);
+            new_buffer = gtk_text_buffer_new(table);
+
+            text = gtk_text_iter_get_text(&start, &end);
+            gtk_text_buffer_set_text(new_buffer, text, -1);
+            g_free(text);
+
+            /* Remove spaces before a newline: */
+            gtk_text_buffer_get_bounds(new_buffer, &start, &end);
+            libbalsa_unwrap_buffer(new_buffer, &start, -1);
+            gtk_text_buffer_get_bounds(new_buffer, &start, &end);
+        }
+
+        /* Copy the buffer text to the message: */
+        body->buffer = gtk_text_iter_get_text(&start, &end);
+        if (new_buffer)
+            g_object_unref(new_buffer);
+
+        if (bsmsg->send_mp_alt)
+            body->html_buffer =
+                libbalsa_text_to_html(LIBBALSA_MESSAGE_GET_SUBJECT(message), body->buffer,
+                                      bsmsg->spell_check_lang);
+        if (bsmsg->flow)
+            body->buffer =
+                libbalsa_wrap_rfc2646(body->buffer, balsa_app.wraplength,
+                                      TRUE, FALSE, TRUE);
+
+        /* Ildar reports that, when a message contains both text/plain and
+         * text/html parts, some broken MUAs use the charset from the
+         * text/plain part to display the text/html part; the latter is
+         * encoded as UTF-8 by add_mime_body_plain (send.c), so we'll use
+         * the same encoding for the text/plain part.
+         * https://bugzilla.gnome.org/show_bug.cgi?id=580704 */
+        body->charset =
+            g_strdup(bsmsg->send_mp_alt ?
+                     "UTF-8" : sw_required_charset(bsmsg, body->buffer));
+        libbalsa_message_append_part(message, body);
+
+        if (balsa_app.has_openpgp || balsa_app.has_smime) {
+            libbalsa_message_set_gpg_mode(message,
+                (bsmsg->gpg_mode & LIBBALSA_PROTECT_MODE) != 0 ? bsmsg->gpg_mode : 0);
+            libbalsa_message_set_attach_pubkey(message, bsmsg->attach_pubkey);
+            libbalsa_message_set_identity(message, ident);
+        } else {
+            libbalsa_message_set_gpg_mode(message, 0);
+            libbalsa_message_set_attach_pubkey(message, FALSE);
+        }
+    }
 
     /* add attachments */
-    if (bsmsg->tree_view)
+    if (bsmsg->tree_view != NULL) {
         gtk_tree_model_foreach(BALSA_MSG_ATTACH_MODEL(bsmsg),
                                attachment2message, message);
+    }
 
     headers->date = time(NULL);
-    if (balsa_app.has_openpgp || balsa_app.has_smime) {
-        libbalsa_message_set_gpg_mode(message,
-            (bsmsg->gpg_mode & LIBBALSA_PROTECT_MODE) != 0 ? bsmsg->gpg_mode : 0);
-        libbalsa_message_set_attach_pubkey(message, bsmsg->attach_pubkey);
-        libbalsa_message_set_identity(message, ident);
-    } else {
-        libbalsa_message_set_gpg_mode(message, 0);
-        libbalsa_message_set_attach_pubkey(message, FALSE);
-    }
 
     /* remember the parent window */
     g_object_set_data(G_OBJECT(message), "parent-window",
@@ -5469,7 +5424,6 @@ send_message_handler(BalsaSendmsg * bsmsg, gboolean queue_only)
     LibBalsaMsgCreateResult result;
     LibBalsaMessage *message;
     LibBalsaMailbox *fcc;
-    LibBalsaMailbox *parent_mailbox = NULL;
     GtkTreeIter iter;
     GError * error = NULL;
 
@@ -5545,11 +5499,6 @@ send_message_handler(BalsaSendmsg * bsmsg, gboolean queue_only)
                                _("sending message with GPG mode %d"),
                                libbalsa_message_get_gpg_mode(message));
 
-    if (bsmsg->type == SEND_RESEND)
-        parent_mailbox = libbalsa_message_get_mailbox(bsmsg->parent_message);
-    if (parent_mailbox != NULL)
-        libbalsa_mailbox_lock_store(parent_mailbox);
-
     if(queue_only)
        result = libbalsa_message_queue(message, balsa_app.outbox, fcc,
                                        libbalsa_identity_get_smtp_server(bsmsg->ident),
@@ -5561,10 +5510,6 @@ send_message_handler(BalsaSendmsg * bsmsg, gboolean queue_only)
                                                                           balsa_app.send_progress_dialog,
                                        GTK_WINDOW(balsa_app.main_window),
                                        bsmsg->flow, &error);
-
-    if (parent_mailbox != NULL)
-        libbalsa_mailbox_unlock_store(parent_mailbox);
-
     if (result == LIBBALSA_MESSAGE_CREATE_OK) {
        if (bsmsg->parent_message != NULL) {
             LibBalsaMailbox *mailbox = libbalsa_message_get_mailbox(bsmsg->parent_message);
@@ -6538,8 +6483,7 @@ sendmsg_window_new_from_list(LibBalsaMailbox * mailbox,
     switch(type) {
     case SEND_FORWARD_ATTACH:
     case SEND_FORWARD_INLINE:
-        bsmsg = sendmsg_window_forward(mailbox, msgno,
-                                       type == SEND_FORWARD_ATTACH);
+        bsmsg = sendmsg_window_forward(mailbox, msgno, type);
         break;
     default:
         g_assert_not_reached(); /* since it hardly makes sense... */
@@ -6705,8 +6649,7 @@ sendmsg_window_set_title(BalsaSendmsg * bsmsg)
         break;
 
     case SEND_RESEND:
-        title_format = _("Resend message to %s: %s");
-        subject_string = LIBBALSA_MESSAGE_GET_SUBJECT(bsmsg->parent_message);
+        title_format = _("Resend message to %s");
         break;
 
     default:
@@ -6718,7 +6661,7 @@ sendmsg_window_set_title(BalsaSendmsg * bsmsg)
     to_string = internet_address_list_to_string(list, NULL, FALSE);
     g_object_unref(list);
 
-    if (subject_string == NULL)
+    if (bsmsg->subject[1] != NULL)
         subject_string = gtk_entry_get_text(GTK_ENTRY(bsmsg->subject[1]));
 
     title = g_strdup_printf(title_format, to_string ? to_string : "", subject_string);
@@ -7163,41 +7106,67 @@ sendmsg_window_reply_embedded(LibBalsaMessageBody *part,
 }
 
 BalsaSendmsg*
-sendmsg_window_forward(LibBalsaMailbox *mailbox, guint msgno,
-                       gboolean attach)
+sendmsg_window_forward(LibBalsaMailbox *mailbox, guint msgno, SendType send_type)
 {
-    LibBalsaMessage *message =
-        libbalsa_mailbox_get_message(mailbox, msgno);
-    SendType send_type = attach ? SEND_FORWARD_ATTACH : SEND_FORWARD_INLINE;
-    BalsaSendmsg *bsmsg = sendmsg_window_new(send_type);
+    LibBalsaMessage *message;
+    BalsaSendmsg *bsmsg;
 
-    g_assert(message);
+    g_return_val_if_fail(LIBBALSA_IS_MAILBOX(mailbox), NULL);
+    message = libbalsa_mailbox_get_message(mailbox, msgno);
+    g_return_val_if_fail(LIBBALSA_IS_MESSAGE(message), NULL);
 
-    if (attach) {
-       if(!attach_message(bsmsg, message))
-            balsa_information_parented(GTK_WINDOW(bsmsg->window),
-                                       LIBBALSA_INFORMATION_WARNING,
-                                       _("Attaching message failed.\n"
-                                         "Possible reason: not enough temporary space"));
-        bsmsg->state = SENDMSG_STATE_CLEAN;
-        bsmsg_set_subject_from_body(bsmsg, libbalsa_message_get_body_list(message), bsmsg->ident);
-    } else {
-        bsm_prepare_for_setup(message);
-        fill_body_from_message(bsmsg, message, QUOTE_NOPREFIX);
-        bsm_finish_setup(bsmsg, libbalsa_message_get_body_list(message));
+    bsmsg = sendmsg_window_new(send_type);
+
+    switch (send_type) {
+        case SEND_FORWARD_ATTACH:
+            if (!attach_message(bsmsg, message)) {
+                balsa_information_parented(GTK_WINDOW(bsmsg->window),
+                                           LIBBALSA_INFORMATION_WARNING,
+                                           _("Attaching message failed.\n"
+                                             "Possible reason: not enough temporary space"));
+            }
+            bsmsg->state = SENDMSG_STATE_CLEAN;
+            bsmsg_set_subject_from_body(bsmsg, libbalsa_message_get_body_list(message), bsmsg->ident);
+            if (libbalsa_identity_get_sig_whenforward(bsmsg->ident))
+                insert_initial_sig(bsmsg);
+
+            break;
+
+        case SEND_RESEND:
+            if (!attach_message(bsmsg, message)) {
+                balsa_information_parented(GTK_WINDOW(bsmsg->window),
+                                           LIBBALSA_INFORMATION_WARNING,
+                                           _("Attaching message failed.\n"
+                                             "Possible reason: not enough temporary space"));
+            }
+
+            break;
+
+        case SEND_FORWARD_INLINE:
+            {
+                GtkTextBuffer *buffer = gtk_text_view_get_buffer(GTK_TEXT_VIEW(bsmsg->text));
+                GtkTextIter pos;
+
+                bsm_prepare_for_setup(message);
+                fill_body_from_message(bsmsg, message, QUOTE_NOPREFIX);
+                bsm_finish_setup(bsmsg, libbalsa_message_get_body_list(message));
+
+                if (libbalsa_identity_get_sig_whenforward(bsmsg->ident))
+                    insert_initial_sig(bsmsg);
+
+                gtk_text_buffer_get_start_iter(buffer, &pos);
+                gtk_text_buffer_place_cursor(buffer, &pos);
+                gtk_text_buffer_insert_at_cursor(buffer, "\n", 1);
+                gtk_text_buffer_get_start_iter(buffer, &pos);
+                gtk_text_buffer_place_cursor(buffer, &pos);
+            }
+
+            break;
+
+        default:
+            g_assert_not_reached();
     }
-    if (libbalsa_identity_get_sig_whenforward(bsmsg->ident))
-        insert_initial_sig(bsmsg);
-    if(!attach) {
-        GtkTextBuffer *buffer =
-            gtk_text_view_get_buffer(GTK_TEXT_VIEW(bsmsg->text));
-        GtkTextIter pos;
-        gtk_text_buffer_get_start_iter(buffer, &pos);
-        gtk_text_buffer_place_cursor(buffer, &pos);
-        gtk_text_buffer_insert_at_cursor(buffer, "\n", 1);
-        gtk_text_buffer_get_start_iter(buffer, &pos);
-        gtk_text_buffer_place_cursor(buffer, &pos);
-     }
+
     return bsmsg;
 }
 
@@ -7206,6 +7175,7 @@ sendmsg_window_continue(LibBalsaMailbox * mailbox, guint msgno)
 {
     LibBalsaMessage *message =
         libbalsa_mailbox_get_message(mailbox, msgno);
+    gboolean resending;
     BalsaSendmsg *bsmsg;
     const gchar *postpone_hdr;
     GList *list, *refs = NULL;
@@ -7220,7 +7190,9 @@ sendmsg_window_continue(LibBalsaMailbox * mailbox, guint msgno)
         return NULL;
     }
 
-    bsmsg = sendmsg_window_new(SEND_CONTINUE);
+    resending = libbalsa_message_get_user_header(message, "X-Balsa-Resend") != NULL;
+    bsmsg = sendmsg_window_new(resending ? SEND_RESEND : SEND_CONTINUE);
+
     bsmsg->is_continue = TRUE;
     bsm_prepare_for_setup(message);
     bsmsg->draft_message = message;
@@ -7229,89 +7201,67 @@ sendmsg_window_continue(LibBalsaMailbox * mailbox, guint msgno)
     set_identity(bsmsg, message);
     setup_headers_from_message(bsmsg, message);
 
-    libbalsa_address_view_set_from_list(bsmsg->replyto_view,
-                                        "Reply To:",
-                                        libbalsa_message_get_headers(message)->reply_to);
-    in_reply_to = libbalsa_message_get_in_reply_to(message);
-    if (in_reply_to != NULL)
-        bsmsg->in_reply_to =
-            g_strconcat("<", in_reply_to->data, ">", NULL);
-
-    if ((postpone_hdr =
-         libbalsa_message_get_user_header(message, "X-Balsa-Crypto")))
-        bsmsg_setup_gpg_ui_by_mode(bsmsg, atoi(postpone_hdr));
-    postpone_hdr = libbalsa_message_get_user_header(message, "X-Balsa-Att-Pubkey");
-    if (postpone_hdr != NULL) {
-       sw_action_set_active(bsmsg, "attpubkey", atoi(postpone_hdr) != 0);
-    }
-    if ((postpone_hdr =
-         libbalsa_message_get_user_header(message, "X-Balsa-MDN")))
-        sw_action_set_active(bsmsg, "request-mdn", atoi(postpone_hdr) != 0);
-    if ((postpone_hdr =
-         libbalsa_message_get_user_header(message, "X-Balsa-DSN")))
-        sw_action_set_active(bsmsg, "request-dsn", atoi(postpone_hdr) != 0);
-    if ((postpone_hdr =
-         libbalsa_message_get_user_header(message, "X-Balsa-Lang"))) {
-        GtkWidget *langs =
-            gtk_menu_item_get_submenu(GTK_MENU_ITEM
-                                      (bsmsg->current_language_menu));
-        GList *children =
-            gtk_container_get_children(GTK_CONTAINER(langs));
-        set_locale(bsmsg, postpone_hdr);
-        for (list = children; list; list = list->next) {
-            GtkCheckMenuItem *menu_item = list->data;
-            const gchar *lang;
-
-            lang = g_object_get_data(G_OBJECT(menu_item),
-                                     BALSA_LANGUAGE_MENU_LANG);
-            if (strcmp(lang, postpone_hdr) == 0)
-                gtk_check_menu_item_set_active(menu_item, TRUE);
+    if (resending) {
+        continue_body(bsmsg, message);
+    } else {
+        libbalsa_address_view_set_from_list(bsmsg->replyto_view,
+                                            "Reply To:",
+                                            libbalsa_message_get_headers(message)->reply_to);
+        in_reply_to = libbalsa_message_get_in_reply_to(message);
+        if (in_reply_to != NULL)
+            bsmsg->in_reply_to =
+                g_strconcat("<", in_reply_to->data, ">", NULL);
+
+        if ((postpone_hdr =
+             libbalsa_message_get_user_header(message, "X-Balsa-Crypto")))
+            bsmsg_setup_gpg_ui_by_mode(bsmsg, atoi(postpone_hdr));
+        postpone_hdr = libbalsa_message_get_user_header(message, "X-Balsa-Att-Pubkey");
+        if (postpone_hdr != NULL) {
+            sw_action_set_active(bsmsg, "attpubkey", atoi(postpone_hdr) != 0);
         }
-        g_list_free(children);
-    }
-    if ((postpone_hdr =
-         libbalsa_message_get_user_header(message, "X-Balsa-Format")))
-        sw_action_set_active(bsmsg, "flowed", strcmp(postpone_hdr, "Fixed"));
-    if ((postpone_hdr =
-         libbalsa_message_get_user_header(message, "X-Balsa-MP-Alt")))
-        sw_action_set_active(bsmsg, "send-html", !strcmp(postpone_hdr, "yes"));
-    if ((postpone_hdr =
-         libbalsa_message_get_user_header(message, "X-Balsa-Send-Type")))
-        bsmsg->type = atoi(postpone_hdr);
-
-    for (list = libbalsa_message_get_references(message); list != NULL; list = list->next)
-        refs = g_list_prepend(refs, g_strdup(list->data));
-    bsmsg->references = g_list_reverse(refs);
-
-    continue_body(bsmsg, message);
-    bsm_finish_setup(bsmsg, libbalsa_message_get_body_list(message));
-    g_idle_add((GSourceFunc) sw_grab_focus_to_text,
-               g_object_ref(bsmsg->text));
-    return bsmsg;
-}
-
-/*
- * sendmsg_window_resend
- *
- * Opens a small compose window for resending the message to new
- * recipient(s).
- *
- * https://datatracker.ietf.org/doc/html/rfc5322#section-3.6.6
- */
-BalsaSendmsg *
-sendmsg_window_resend(LibBalsaMailbox *mailbox, guint msgno)
-{
-    LibBalsaMessage *message;
-    BalsaSendmsg *bsmsg;
-
-    g_return_val_if_fail(LIBBALSA_IS_MAILBOX(mailbox), NULL);
-    message = libbalsa_mailbox_get_message(mailbox, msgno);
-    g_return_val_if_fail(LIBBALSA_IS_MESSAGE(message), NULL);
-
-    bsmsg = sendmsg_window_new(SEND_RESEND);
-    bsmsg->parent_message = message;
-    libbalsa_mailbox_open(mailbox, NULL);
-    set_identity(bsmsg, message);
+        if ((postpone_hdr =
+             libbalsa_message_get_user_header(message, "X-Balsa-MDN")))
+            sw_action_set_active(bsmsg, "request-mdn", atoi(postpone_hdr) != 0);
+        if ((postpone_hdr =
+             libbalsa_message_get_user_header(message, "X-Balsa-DSN")))
+            sw_action_set_active(bsmsg, "request-dsn", atoi(postpone_hdr) != 0);
+        if ((postpone_hdr =
+             libbalsa_message_get_user_header(message, "X-Balsa-Lang"))) {
+            GtkWidget *langs =
+                gtk_menu_item_get_submenu(GTK_MENU_ITEM
+                                          (bsmsg->current_language_menu));
+            GList *children =
+                gtk_container_get_children(GTK_CONTAINER(langs));
+            set_locale(bsmsg, postpone_hdr);
+            for (list = children; list; list = list->next) {
+                GtkCheckMenuItem *menu_item = list->data;
+                const gchar *lang;
+
+                lang = g_object_get_data(G_OBJECT(menu_item),
+                                         BALSA_LANGUAGE_MENU_LANG);
+                if (strcmp(lang, postpone_hdr) == 0)
+                    gtk_check_menu_item_set_active(menu_item, TRUE);
+            }
+            g_list_free(children);
+        }
+        if ((postpone_hdr =
+             libbalsa_message_get_user_header(message, "X-Balsa-Format")))
+            sw_action_set_active(bsmsg, "flowed", strcmp(postpone_hdr, "Fixed"));
+        if ((postpone_hdr =
+             libbalsa_message_get_user_header(message, "X-Balsa-MP-Alt")))
+            sw_action_set_active(bsmsg, "send-html", !strcmp(postpone_hdr, "yes"));
+        if ((postpone_hdr =
+             libbalsa_message_get_user_header(message, "X-Balsa-Send-Type")))
+            bsmsg->type = atoi(postpone_hdr);
+
+        for (list = libbalsa_message_get_references(message); list != NULL; list = list->next)
+            refs = g_list_prepend(refs, g_strdup(list->data));
+        bsmsg->references = g_list_reverse(refs);
+
+        continue_body(bsmsg, message);
+        bsm_finish_setup(bsmsg, libbalsa_message_get_body_list(message));
+        g_idle_add((GSourceFunc) sw_grab_focus_to_text, g_object_ref(bsmsg->text));
+    }
 
     return bsmsg;
 }
diff --git a/src/sendmsg-window.h b/src/sendmsg-window.h
index 0ab21e7ae..d1125ec3b 100644
--- a/src/sendmsg-window.h
+++ b/src/sendmsg-window.h
@@ -114,10 +114,8 @@ G_BEGIN_DECLS
     BalsaSendmsg *sendmsg_window_reply_embedded(LibBalsaMessageBody *part,
                                                 SendType reply_type);
 
-    BalsaSendmsg *sendmsg_window_resend(LibBalsaMailbox *mailbox, guint msgno);
-
     BalsaSendmsg *sendmsg_window_forward(LibBalsaMailbox *,
-                                         guint msgno, gboolean attach);
+                                         guint msgno, SendType send_type);
     BalsaSendmsg *sendmsg_window_continue(LibBalsaMailbox *,
                                           guint msgno);
 


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]