diff --git a/addons/web/controllers/main.py b/addons/web/controllers/main.py index c775927a8cc..b6405873f80 100644 --- a/addons/web/controllers/main.py +++ b/addons/web/controllers/main.py @@ -1217,7 +1217,7 @@ class Binary(openerpweb.Controller): filename = res.get(filename_field, '') or filename return req.make_response(filecontent, [('Content-Type', 'application/octet-stream'), - ('Content-Disposition', 'attachment; filename=' + filename)]) + ('Content-Disposition', 'attachment; filename="%s"' % filename)]) @openerpweb.httprequest def upload(self, req, callback, ufile): diff --git a/addons/web/static/src/js/formats.js b/addons/web/static/src/js/formats.js index 4076c190a74..819301f9e55 100644 --- a/addons/web/static/src/js/formats.js +++ b/addons/web/static/src/js/formats.js @@ -289,11 +289,18 @@ openerp.web.format_cell = function (row_data, column, options) { return _.str.sprintf('', row_data[column.id].value ? 'checked="checked"' : ''); case "binary": + var text = _t("Download"), + download_url = _.str.sprintf('/web/binary/saveas?session_id=%s&model=%s&field=%s&id=%d', openerp.connection.session_id, options.model, column.id, options.id); + if (column.filename) { + download_url += '&filename_field=' + column.filename; + if (row_data[column.filename]) { + text = _.str.sprintf(_t("Download \"%s\""), openerp.web.format_value( + row_data[column.filename].value, {type: 'char'})); + } + } return _.str.sprintf('%(text)s (%(size)s)', { - text: _t("Download"), - href: _.str.sprintf('/web/binary/saveas?session_id=%s&model=%s&field=%s&id=%d', - openerp.connection.session_id, options.model, - column.id, options.id), + text: text, + href: download_url, size: row_data[column.id].value }); }