[evolution/wip/webkit2] Bug 766713 - Remove attachment does not work on multiple level message structure

[Tomas Popela <tpopela src gnome org>]

commit aabe624a5ab45de22a04223711353688f9f79d5f
Author: Milan Crha <mcrha redhat com>
Date:   Wed May 25 22:58:35 2016 +0200

    Bug 766713 - Remove attachment does not work on multiple level message structure

 libemail-engine/e-mail-folder-utils.c |   39 ++++++++++++++++++++++++--------
 1 files changed, 29 insertions(+), 10 deletions(-)
---
diff --git a/libemail-engine/e-mail-folder-utils.c b/libemail-engine/e-mail-folder-utils.c
index b02b802..89d773c 100644
--- a/libemail-engine/e-mail-folder-utils.c
+++ b/libemail-engine/e-mail-folder-utils.c
@@ -1278,31 +1278,32 @@ mail_folder_remove_attachments_thread (GSimpleAsyncResult *simple,
                g_simple_async_result_take_error (simple, error);
 }
 
-/* Helper for e_mail_folder_remove_attachments_sync() */
 static gboolean
-mail_folder_strip_message (CamelFolder *folder,
-                           CamelMimeMessage *message,
-                           const gchar *message_uid,
-                           GCancellable *cancellable,
-                           GError **error)
+mail_folder_strip_message_level (CamelMimePart *in_part,
+                                GCancellable *cancellable)
 {
        CamelDataWrapper *content;
        CamelMultipart *multipart;
        gboolean modified = FALSE;
-       gboolean success = TRUE;
        guint ii, n_parts;
 
-       content = camel_medium_get_content (CAMEL_MEDIUM (message));
+       g_return_val_if_fail (CAMEL_IS_MIME_PART (in_part), FALSE);
+
+       content = camel_medium_get_content (CAMEL_MEDIUM (in_part));
+
+       if (CAMEL_IS_MIME_MESSAGE (content)) {
+               return mail_folder_strip_message_level (CAMEL_MIME_PART (content), cancellable);
+       }
 
        if (!CAMEL_IS_MULTIPART (content))
-               return TRUE;
+               return FALSE;
 
        multipart = CAMEL_MULTIPART (content);
        n_parts = camel_multipart_get_number (multipart);
 
        /* Replace MIME parts with "attachment" or "inline" dispositions
         * with a small "text/plain" part saying the file was removed. */
-       for (ii = 0; ii < n_parts; ii++) {
+       for (ii = 0; ii < n_parts && !g_cancellable_is_cancelled (cancellable); ii++) {
                CamelMimePart *mime_part;
                const gchar *disposition;
                gboolean is_attachment;
@@ -1340,9 +1341,27 @@ mail_folder_strip_message (CamelFolder *folder,
                                mime_part, disposition);
 
                        modified = TRUE;
+               } else {
+                       modified = mail_folder_strip_message_level (mime_part, cancellable) || modified;
                }
        }
 
+       return modified;
+}
+
+/* Helper for e_mail_folder_remove_attachments_sync() */
+static gboolean
+mail_folder_strip_message (CamelFolder *folder,
+                           CamelMimeMessage *message,
+                           const gchar *message_uid,
+                           GCancellable *cancellable,
+                           GError **error)
+{
+       gboolean modified;
+       gboolean success = TRUE;
+
+       modified = mail_folder_strip_message_level (CAMEL_MIME_PART (message), cancellable);
+
        /* Append the modified message with removed attachments to
         * the folder and mark the original message for deletion. */
        if (modified) {