Show More
Commit Description:
add more error message to user list upgrade. Fix #9
Commit Description:
add more error message to user list upgrade. Fix #9
References:
File last commit:
Show/Diff file:
Action:
app/controllers/tasks_controller.rb
| 74 lines
| 2.1 KiB
| text/x-ruby
| RubyLexer
|
jittat
|
r88 | class TasksController < ApplicationController | ||
| r756 | before_action :check_valid_login, :check_viewability | |||
|
jittat
|
r88 | |||
| def index | ||||
| redirect_to :action => 'list' | ||||
| end | ||||
| def list | ||||
Jittat Fakcharoenphol
|
r288 | @problems = @user.available_problems | ||
|
jittat
|
r88 | end | ||
|
Jittat Fakcharoenphol
|
r282 | # this has contest-wide access control | ||
|
jittat
|
r130 | def view | ||
|
Jittat Fakcharoenphol
|
r271 | base_name = params[:file] | ||
|
Jittat Fakcharoenphol
|
r282 | base_filename = File.basename("#{base_name}.#{params[:ext]}") | ||
| filename = "#{Problem.download_file_basedir}/#{base_filename}" | ||||
| if !FileTest.exists?(filename) | ||||
|
Jittat Fakcharoenphol
|
r271 | redirect_to :action => 'index' and return | ||
| end | ||||
|
Jittat Fakcharoenphol
|
r282 | send_file_to_user(filename, base_filename) | ||
| end | ||||
|
Jittat Fakcharoenphol
|
r271 | |||
|
Jittat Fakcharoenphol
|
r282 | # this has problem-level access control | ||
| def download | ||||
| problem = Problem.find(params[:id]) | ||||
| r680 | unless @current_user.can_view_problem? problem | |||
| r762 | flash[:notice] = 'You are not authorized to access this file' | |||
|
jittat
|
r130 | redirect_to :action => 'index' and return | ||
| end | ||||
|
jittat
|
r133 | |||
|
Jittat Fakcharoenphol
|
r282 | base_name = params[:file] | ||
| base_filename = File.basename("#{base_name}.#{params[:ext]}") | ||||
| filename = "#{Problem.download_file_basedir}/#{params[:id]}/#{base_filename}" | ||||
| if !FileTest.exists?(filename) | ||||
| r762 | flash[:notice] = 'File does not exists' | |||
|
Jittat Fakcharoenphol
|
r282 | redirect_to :action => 'index' and return | ||
| end | ||||
| send_file_to_user(filename, base_filename) | ||||
| end | ||||
| protected | ||||
| def send_file_to_user(filename, base_filename) | ||||
|
jittat
|
r172 | if defined?(USE_APACHE_XSENDFILE) and USE_APACHE_XSENDFILE | ||
| response.headers['Content-Type'] = "application/force-download" | ||||
| response.headers['Content-Disposition'] = "attachment; filename=\"#{File.basename(filename)}\"" | ||||
| response.headers["X-Sendfile"] = filename | ||||
| response.headers['Content-length'] = File.size(filename) | ||||
| render :nothing => true | ||||
| else | ||||
|
Jittat Fakcharoenphol
|
r273 | if params[:ext]=='pdf' | ||
| content_type = 'application/pdf' | ||||
| else | ||||
| content_type = 'application/octet-stream' | ||||
| end | ||||
| r367 | send_file filename, :stream => false, :disposition => 'inline', :filename => base_filename, :type => content_type | |||
|
jittat
|
r172 | end | ||
|
jittat
|
r130 | end | ||
|
jittat
|
r122 | def check_viewability | ||
|
Jittat Fakcharoenphol
|
r271 | @user = User.find(session[:user_id]) | ||
|
Jittat Fakcharoenphol
|
r320 | if @user==nil or !GraderConfiguration.show_tasks_to?(@user) | ||
|
jittat
|
r122 | redirect_to :controller => 'main', :action => 'list' | ||
| return false | ||||
| end | ||||
| end | ||||
|
jittat
|
r88 | end | ||