Description:
returns correct content type for task description download
Commit status:
[Not Reviewed]
References:
Comments:
0 Commit comments 0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
Add another comment

r273:0e765be64abb - - 1 file changed: 7 inserted, 1 deleted

@@ -22,25 +22,31
22 22
23 23 if !check_user_viewability(base_name) or !FileTest.exists?(filename)
24 24 redirect_to :action => 'index' and return
25 25 end
26 26
27 27 if defined?(USE_APACHE_XSENDFILE) and USE_APACHE_XSENDFILE
28 28 response.headers['Content-Type'] = "application/force-download"
29 29 response.headers['Content-Disposition'] = "attachment; filename=\"#{File.basename(filename)}\""
30 30 response.headers["X-Sendfile"] = filename
31 31 response.headers['Content-length'] = File.size(filename)
32 32 render :nothing => true
33 33 else
34 - send_file filename, :stream => false, :filename => base_filename
34 + if params[:ext]=='pdf'
35 + content_type = 'application/pdf'
36 + else
37 + content_type = 'application/octet-stream'
38 + end
39 +
40 + send_file filename, :stream => false, :filename => base_filename, :type => content_type
35 41 end
36 42 end
37 43
38 44 protected
39 45
40 46 def check_viewability
41 47 @user = User.find(session[:user_id])
42 48 if @user==nil or !Configuration.show_tasks_to?(@user)
43 49 redirect_to :controller => 'main', :action => 'list'
44 50 return false
45 51 end
46 52 end
You need to be logged in to leave comments. Login now