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

Merge pull request #635 from pdvyas/hotfix-attachment-url

Browse source 
fix bad attachment urls and encode them
This commit is contained in:
Anand Doshi 2014-06-24 16:26:17 +05:30
commit 53ff5daeb2
1 changed files with 8 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -61,7 +61,7 @@ frappe.ui.form.Attachments = Class.extend({
},
add_attachment: function(attachment) {
var file_name = attachment.file_name;
var file_url = attachment.file_url;
var file_url = this.get_file_url(attachment);
var fileid = attachment.name;
if (!file_name) {
file_name = file_url;
@ -96,6 +96,13 @@ frappe.ui.form.Attachments = Class.extend({
$close.remove();
}
},
get_file_url: function(attachment) {
var file_url = attachment.file_url;
if (!file_url) {
file_url = '/files/' + attachment.file_name;
}
return encodeURI(file_url);
},
remove_attachment_by_filename: function(filename, callback) {
this.remove_attachment(this.get_attachments()[filename], callback);
},