From 8ffca196804fe235880b86d72456348752f5afa4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=A9rome=20Maes?= <jem@openerp.com>
Date: Fri, 25 Sep 2015 13:42:30 +0200
Subject: [PATCH] [FIX] web : attachment controller got unexpected args

On webclient, on 'Attachment(s)' dropdown (form view) when clicking on an attachment to download it, a request is sent to /web/content/ with a param 't' containing the current date. This param wasn't used neither before the new controller, but the old one accept additional kwargs. The new controller crashes, and since the t parameter is not used, removing it seems better.
---
 addons/web/static/src/js/widgets/sidebar.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/addons/web/static/src/js/widgets/sidebar.js b/addons/web/static/src/js/widgets/sidebar.js
index 9b7d223335fc..9ea15be128b2 100644
--- a/addons/web/static/src/js/widgets/sidebar.js
+++ b/addons/web/static/src/js/widgets/sidebar.js
@@ -179,7 +179,7 @@ var Sidebar = Widget.extend({
         _.each(attachments,function(a) {
             a.label = a.name;
             if(a.type === "binary") {
-                a.url = '/web/content/'  + a.id + '?download=true&t=' + (new Date().getTime());
+                a.url = '/web/content/'  + a.id + '?download=true';
             }
         });
         this.items.files = attachments;
-- 
GitLab