[MERGE] forward port branch saas-14 up to 1985a49b92

addons/l10n_fr_fec/wizard/account_fr_fec.py

@@ -289,9 +289,9 @@ class AccountFrFec(models.TransientModel):
             END
             AS PieceRef,
             TO_CHAR(am.date, 'YYYYMMDD') AS PieceDate,
-            CASE WHEN aml.name IS NULL THEN '/'
+            CASE WHEN aml.name IS NULL OR aml.name = '' THEN '/'
                 WHEN aml.name SIMILAR TO '[\t|\s|\n]*' THEN '/'
-                ELSE replace(aml.name, '|', '/') END AS EcritureLib,
+                ELSE replace(replace(replace(aml.name, '|', '/'), '\t', ''), '\n', '') END AS EcritureLib,
             replace(CASE WHEN aml.debit = 0 THEN '0,00' ELSE to_char(aml.debit, '000000000000000D99') END, '.', ',') AS Debit,
             replace(CASE WHEN aml.credit = 0 THEN '0,00' ELSE to_char(aml.credit, '000000000000000D99') END, '.', ',') AS Credit,
             CASE WHEN rec.name IS NULL THEN '' ELSE rec.name END AS EcritureLet,

addons/web/static/src/js/views/form_widgets.js

@@ -1216,7 +1216,7 @@ var FieldBinary = common.AbstractField.extend(common.ReinitializeFieldMixin, {
                     'id': this.view.datarecord.id,
                     'field': this.name,
                     'filename_field': filename_fieldname,
-                    'filename': filename_field ? filename_field.get('value') : null,
+                    'filename': filename_field ? filename_field.get('value') : "",
                     'download': true,
                     'data': utils.is_bin_size(value) ? null : value,
                 },

odoo/addons/base/ir/ir_http.py

@@ -303,9 +303,9 @@ class IrHttp(models.AbstractModel):
         if not filename:
             if filename_field in obj:
                 filename = obj[filename_field]
-            elif module_resource_path:
+            if not filename and module_resource_path:
                 filename = os.path.basename(module_resource_path)
-            else:
+            if not filename:
                 filename = "%s-%s-%s" % (obj._name, obj.id, field)
 
         # mimetype
@@ -320,6 +320,11 @@ class IrHttp(models.AbstractModel):
             if not mimetype:
                 mimetype = guess_mimetype(base64.b64decode(content), default=default_mimetype)
 
+        # extension
+        extension = mimetypes.guess_extension(mimetype)
+        if extension and not filename.endswith(extension):
+            filename = "%s%s" % (filename, extension)
+
         headers += [('Content-Type', mimetype), ('X-Content-Type-Options', 'nosniff')]
 
         # cache