[MERGE] Forward port of saas-12.3 to saas-12.4 up to 87fc1554d6a1823

closes odoo/odoo#34820

Signed-off-by: Martin Trigaux (mat) <mat@odoo.com>

1 files changed, 24 insertions, 11 deletions

diff --git a/addons/mail/static/src/js/composers/basic_composer.js b/addons/mail/static/src/js/composers/basic_composer.js
index e84d7588b1b..2a496faf86d 100644
--- a/addons/mail/static/src/js/composers/basic_composer.js
+++ b/addons/mail/static/src/js/composers/basic_composer.js
@@ -438,10 +438,27 @@ var BasicComposer = Widget.extend({
      * @param {boolean} params.submitForm [optional]
      */
     _processAttachmentChange: function (params) {
-        var self = this,
-        attachments = this.get('attachment_ids'),
-        files = params.files,
-        submitForm = params.submitForm;
+        var self = this;
+        var attachments = this.get('attachment_ids');
+        var files = params.files;
+        var submitForm = params.submitForm;
+        var $form = this.$('form.o_form_binary_form');
+
+        /**
+         * makes a new formData as formData.delete() is not supported by IE or Safari Mobile.
+         *
+         * @return {FormData}
+         */
+        function makeFormDataWithoutUfile() {
+            var newFormData = new window.FormData();
+            $form.find('input').each(function (index, input) {
+                if (input.name !== 'ufile') {
+                    newFormData.append(input.name, input.value);
+                }
+            });
+            return newFormData;
+        }
+
         _.each(files, function (file) {
             var attachment = _.findWhere(attachments, {
                 name: file.name,
@@ -453,24 +470,20 @@ var BasicComposer = Widget.extend({
                 attachments = _.without(attachments, attachment);
             }
         });
-        var $form = this.$('form.o_form_binary_form');
         if (submitForm) {
             $form.submit();
             this._$attachmentButton.prop('disabled', true);
         } else {
-            var data = new FormData($form[0]);
             _.each(files, function (file) {
-                // removing existing key with blank data and appending again with file info
-                // In safari, existing key will not be updated when append with new file.
-                data.delete("ufile");
-                data.append("ufile", file, file.name);
+                var formData = makeFormDataWithoutUfile();
+                formData.append("ufile", file, file.name);
                 $.ajax({
                     url: $form.attr("action"),
                     type: "POST",
                     enctype: 'multipart/form-data',
                     processData: false,
                     contentType: false,
-                    data: data,
+                    data: formData,
                     success: function (result) {
                         var $el = $(result);
                         $.globalEval($el.contents().text());