vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 4

Escape hash in attachment url, fix frappe/erpnext#1961

Browse source
This commit is contained in:
Pratik Vyas 2014-07-20 19:39:15 +05:30
parent 4d6fd109bc
commit b878bb42e6
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
frappe/public/js/frappe/form/attachments.js
View file

@ -106,7 +106,8 @@ frappe.ui.form.Attachments = Class.extend({
file_url = '/files/' + attachment.file_name;
}
}
return encodeURI(file_url);
// hash is not escaped, https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/encodeURI
return encodeURI(file_url).replace('#', '%23');
},
remove_attachment_by_filename: function(filename, callback) {
this.remove_attachment(this.get_attachments()[filename], callback);