cafe-grader-web Commit - r745:75264caa616f

cafe-grader-web

Commit `r745:75264caa616f` public

parent | child

Description:

change depricated before_filter to before_action

Commit status:

[Not Reviewed]

References:

algo

Diff options:

| | | |

Comments:

0 Commit comments 0 Inline Comments

Unresolved TODOs:

There are no unresolved TODOs

Add another comment

r745:75264caa616f - Wed, 03 Jul 2019 10:25:07 - 20 files changed: 38 inserted, 31 deleted

Gemfile ¶ +1

               source 'https://rubygems.org'
               #rails
               gem 'rails', '~>5.0'
               gem 'activerecord-session_store'
               # Bundle edge Rails instead:
               # gem 'rails', :git => 'git://github.com/rails/rails.git'
               #---------------- database ---------------------
               #the database
               gem 'mysql2'
               #for testing
               gem 'sqlite3'
+            + gem 'rails-controller-testing'
               #for dumping database into yaml
               gem 'yaml_db'
               # Gems used only for assets and not required
               # in production environments by default.
               gem 'sass-rails'
               gem 'coffee-rails'
                 # See https://github.com/sstephenson/execjs#readme for more supported runtimes
                 # gem 'therubyracer', :platforms => :ruby
               gem 'uglifier'
               gem 'haml'
               gem 'haml-rails'
               # gem 'prototype-rails'
               # To use ActiveModel has_secure_password
               # gem 'bcrypt-ruby', '~> 3.0.0'
               # To use Jbuilder templates for JSON
               # gem 'jbuilder'
               # Use unicorn as the app server
               # gem 'unicorn'
               # Deploy with Capistrano
               # gem 'capistrano'
               # To use debugger
               # gem 'debugger'
               #
               #in-place editor
               gem 'best_in_place', '~> 3.0.1'
               # jquery addition
               gem 'jquery-rails'
               gem 'jquery-ui-rails'
               gem 'jquery-timepicker-addon-rails'
               gem 'jquery-tablesorter'
               gem 'jquery-countdown-rails'
               #syntax highlighter
               gem 'rouge'
               #bootstrap add-ons
               gem 'bootstrap-sass', '~> 3.4.1'

Gemfile.lock ¶ +5

                     railties (>= 3.1)
                     sass-rails
                   jquery-rails (4.3.3)
                     rails-dom-testing (>= 1, < 3)
                     railties (>= 4.2.0)
                     thor (>= 0.14, < 2.0)
                   jquery-tablesorter (1.26.1)
                     railties (>= 3.2, < 6)
                   jquery-timepicker-addon-rails (1.4.1)
                     railties (>= 3.1)
                   jquery-ui-rails (6.0.1)
                     railties (>= 3.2.16)
                   loofah (2.2.3)
                     crass (~> 1.0.2)
                     nokogiri (>= 1.5.9)
                   mail (2.7.1)
                     mini_mime (>= 0.1.1)
                   method_source (0.9.2)
                   mini_mime (1.0.1)
                   mini_portile2 (2.4.0)
                   minitest (5.11.3)
                   minitest-reporters (1.3.6)
                     ansi
                     builder
                     minitest (>= 5.0)
                     ruby-progressbar
                   momentjs-rails (2.20.1)
                     railties (>= 3.1)
                   multi_json (1.13.1)
                   mysql2 (0.5.2)
                   nio4r (2.3.1)
                   nokogiri (1.10.3)
                     mini_portile2 (~> 2.4.0)
                   rack (2.0.7)
                   rack-test (0.6.3)
                     rack (>= 1.0)
                   rails (5.0.7.2)
                     actioncable (= 5.0.7.2)
                     actionmailer (= 5.0.7.2)
                     actionpack (= 5.0.7.2)
                     actionview (= 5.0.7.2)
                     activejob (= 5.0.7.2)
                     activemodel (= 5.0.7.2)
                     activerecord (= 5.0.7.2)
                     activesupport (= 5.0.7.2)
                     bundler (>= 1.3.0)
                     railties (= 5.0.7.2)
                     sprockets-rails (>= 2.0.0)
+            +     rails-controller-testing (1.0.4)
+            +       actionpack (>= 5.0.1.x)
+            +       actionview (>= 5.0.1.x)
+            +       activesupport (>= 5.0.1.x)
                   rails-dom-testing (2.0.3)
                     activesupport (>= 4.2.0)
                     nokogiri (>= 1.6)
                   rails-html-sanitizer (1.0.4)
                     loofah (~> 2.2, >= 2.2.2)
                   rails_bootstrap_sortable (2.0.6)
                     momentjs-rails (>= 2.8.3)
                   railties (5.0.7.2)
                     actionpack (= 5.0.7.2)
                     activesupport (= 5.0.7.2)
                     method_source
                     rake (>= 0.8.7)
                     thor (>= 0.18.1, < 2.0)
                   rake (12.3.2)
                   rb-fsevent (0.10.3)
                   rb-inotify (0.10.0)
                     ffi (~> 1.0)
                   rdiscount (2.2.0.1)
                   rouge (3.3.0)
                   ruby-progressbar (1.10.0)
                   ruby_parser (3.13.1)
                     sexp_processor (~> 4.9)
                   sass (3.7.4)
                     sass-listen (~> 4.0.0)
                   sass-listen (4.0.0)
                     rb-fsevent (~> 0.9, >= 0.9.4)
                     rb-inotify (~> 0.9, >= 0.9.7)
                   sass-rails (5.0.7)
                     railties (>= 4.0.0, < 6)
                     sass (~> 3.1)
                     sprockets (>= 2.8, < 4.0)
                     sprockets-rails (>= 2.0, < 4.0)
                     tilt (>= 1.1, < 3)
                   sassc (2.0.1)
                     ffi (~> 1.9)
                     rake
                   sassc-rails (2.1.1)
                     railties (>= 4.0.0)
                     sassc (>= 2.0)
                     sprockets (> 3.0)
                     sprockets-rails
                     tilt
                   select2-rails (4.0.3)
                     thor (~> 0.14)
                   sexp_processor (4.12.0)
                   sprockets (3.7.2)
                     concurrent-ruby (~> 1.0)
                     rack (> 1, < 3)
                     sprockets (>= 3.0.0)
                   sqlite3 (1.4.1)
                   temple (0.8.1)
                   thor (0.20.3)
                   thread_safe (0.3.6)
                   tilt (2.0.9)
                   tzinfo (1.2.5)
                     thread_safe (~> 0.1)
                   uglifier (4.1.20)
                     execjs (>= 0.3.0, < 3)
                   websocket-driver (0.6.5)
                     websocket-extensions (>= 0.1.0)
                   websocket-extensions (0.1.4)
                   will_paginate (3.0.12)
                   yaml_db (0.7.0)
                     rails (>= 3.0)
                     rake (>= 0.8.7)
               PLATFORMS
                 ruby
               DEPENDENCIES
                 ace-rails-ap
                 activerecord-session_store
                 autoprefixer-rails
                 best_in_place (~> 3.0.1)
                 bootstrap-datepicker-rails
                 bootstrap-sass (~> 3.4.1)
                 bootstrap-switch-rails
                 bootstrap-toggle-rails
                 bootstrap3-datetimepicker-rails
                 coffee-rails
                 dynamic_form
                 fuzzy-string-match
                 haml
                 haml-rails
                 in_place_editing
                 jquery-countdown-rails
                 jquery-datatables-rails
                 jquery-rails
                 jquery-tablesorter
                 jquery-timepicker-addon-rails
                 jquery-ui-rails
                 mail
                 minitest-reporters
                 momentjs-rails
                 mysql2
                 rails (~> 5.0)
+            +   rails-controller-testing
                 rails_bootstrap_sortable
                 rdiscount
                 rouge
                 sass-rails
                 sassc-rails (>= 2.1.0)
                 select2-rails
                 sqlite3
                 uglifier
                 verification!
                 will_paginate (~> 3.0.7)
                 yaml_db
               BUNDLED WITH
 .17.2

app/controllers/announcements_controller.rb ¶ +1 -1

               class AnnouncementsController < ApplicationController
-            -   before_filter :admin_authorization
+            +   before_action :admin_authorization
                 in_place_edit_for :announcement, :published
                 # GET /announcements
                 # GET /announcements.xml
                 def index
                   @announcements = Announcement.order(created_at: :desc)
                   respond_to do |format|
                     format.html # index.html.erb
                     format.xml  { render :xml => @announcements }
                   end
                 end
                 # GET /announcements/1
                 # GET /announcements/1.xml
                 def show
                   @announcement = Announcement.find(params[:id])
                   respond_to do |format|
                     format.html # show.html.erb
                     format.xml  { render :xml => @announcement }
                   end
                 end
                 # GET /announcements/new
                 # GET /announcements/new.xml
                 def new
                   @announcement = Announcement.new
                   respond_to do |format|
                     format.html # new.html.erb
                     format.xml  { render :xml => @announcement }
                   end
                 end
                 # GET /announcements/1/edit
                 def edit
                   @announcement = Announcement.find(params[:id])
                 end
                 # POST /announcements
                 # POST /announcements.xml
                 def create
                   @announcement = Announcement.new(announcement_params)
                   respond_to do |format|
                     if @announcement.save

app/controllers/application_controller.rb ¶ +1 -1

               class ApplicationController < ActionController::Base
                 protect_from_forgery
-            -   before_filter :current_user
+            +   before_action :current_user
                 SINGLE_USER_MODE_CONF_KEY = 'system.single_user_mode'
                 MULTIPLE_IP_LOGIN_CONF_KEY = 'right.multiple_ip_login'
                 #report and redirect for unauthorized activities
                 def unauthorized_redirect
                   flash[:notice] = 'You are not authorized to view the page you requested'
                   redirect_to :controller => 'main', :action => 'login'
                 end
                 # Returns the current logged-in user (if any).
                 def current_user
                   return nil unless session[:user_id]
                   @current_user ||= User.find(session[:user_id])
                 end
                 def admin_authorization
                   return false unless authenticate
                   user = User.includes(:roles).find(session[:user_id])
                   unless user.admin?
                     unauthorized_redirect
                     return false
                   end
                   return true
                 end
                 def authorization_by_roles(allowed_roles)
                   return false unless authenticate
                   user = User.find(session[:user_id])
                   unless user.roles.detect { |role| allowed_roles.member?(role.name) }
                     unauthorized_redirect
                     return false
                   end
                 end
                 def testcase_authorization
                   #admin always has privileged
                   if @current_user.admin?
                     return true
                   end
                   unauthorized_redirect unless GraderConfiguration["right.view_testcase"]
                 end
                 protected
                 def authenticate
                   unless session[:user_id]

app/controllers/configurations_controller.rb ¶ +2 -2

               class ConfigurationsController < ApplicationController
-            -   before_filter :authenticate
+            +   before_action :authenticate
-            -   before_filter { |controller| controller.authorization_by_roles(['admin'])}
+            +   before_action { |controller| controller.authorization_by_roles(['admin'])}
                 def index
                   @configurations = GraderConfiguration.order(:key)
                   @group = GraderConfiguration.pluck("grader_configurations.key").map{ |x| x[0...(x.index('.'))] }.uniq.sort
                 end
                 def reload
                   GraderConfiguration.reload
                   redirect_to :action => 'index'
                 end
                 def update
                   @config = GraderConfiguration.find(params[:id])
                   User.clear_last_login if @config.key == GraderConfiguration::MULTIPLE_IP_LOGIN_KEY and @config.value == 'true' and params[:grader_configuration][:value] == 'false'
                   respond_to do |format|
                     if @config.update_attributes(configuration_params)
                       format.json { head :ok }
                     else
                       format.json { respond_with_bip(@config) }
                     end
                   end
                 end
               private
                 def configuration_params
                   params.require(:grader_configuration).permit(:key,:value_type,:value,:description)
                 end
               end

app/controllers/contest_management_controller.rb ¶ +1 -1

               class ContestManagementController < ApplicationController
-            -   before_filter :admin_authorization
+            +   before_action :admin_authorization
                 def index
                   @num_contests = Contest.count()
                 end
                 def user_stat
                   if not GraderConfiguration.indv_contest_mode?
                     redirect_to :action => 'index' and return
                   end
                   @users = User.all
                   @start_times = {}
                   UserContestStat.all.each do |stat|
                     @start_times[stat.user_id] = stat.started_at
                   end
                 end
                 def clear_stat
                   user = User.find(params[:id])
                   if user.contest_stat!=nil
                     user.contest_stat.destroy
                   end
                   redirect_to :action => 'user_stat'
                 end
                 def clear_all_stat
                   if not GraderConfiguration.indv_contest_mode?
                     redirect_to :action => 'index' and return
                   end
                   UserContestStat.delete_all()
                   flash[:notice] = 'All start time statistic cleared.'
                   redirect_to :action => 'index'
                 end
                 def change_contest_mode
                   if ['standard', 'contest', 'indv-contest', 'analysis'].include? params[:id]
                     config = GraderConfiguration.find_by_key('system.mode')
                     config.value = params[:id]
                     config.save
                   else
                     flash[:notice] = 'Wrong contest mode value'
                   end
                   redirect_to :action => 'index'
                 end
               end

app/controllers/contests_controller.rb ¶ +1 -1

               class ContestsController < ApplicationController
-            -   before_filter :admin_authorization
+            +   before_action :admin_authorization
                 in_place_edit_for :contest, :title
                 in_place_edit_for :contest, :enabled
                 # GET /contests
                 # GET /contests.xml
                 def index
                   @contests = Contest.all
                   respond_to do |format|
                     format.html # index.html.erb
                     format.xml  { render :xml => @contests }
                   end
                 end
                 # GET /contests/1
                 # GET /contests/1.xml
                 def show
                   @contest = Contest.find(params[:id])
                   respond_to do |format|
                     format.html # show.html.erb
                     format.xml  { render :xml => @contest }
                   end
                 end
                 # GET /contests/new
                 # GET /contests/new.xml
                 def new
                   @contest = Contest.new
                   respond_to do |format|
                     format.html # new.html.erb
                     format.xml  { render :xml => @contest }
                   end
                 end
                 # GET /contests/1/edit
                 def edit
                   @contest = Contest.find(params[:id])
                 end
                 # POST /contests
                 # POST /contests.xml
                 def create
                   @contest = Contest.new(params[:contest])
                   respond_to do |format|

app/controllers/graders_controller.rb ¶ +1 -1

               class GradersController < ApplicationController
-            -   before_filter :admin_authorization
+            +   before_action :admin_authorization
                 verify :method => :post, :only => ['clear_all',
                                                    'start_exam',
                                                    'start_grading',
                                                    'stop_all',
                                                    'clear_terminated'],
                        :redirect_to => {:action => 'index'}
                 def index
                   redirect_to :action => 'list'
                 end
                 def list
                   @grader_processes = GraderProcess.find_running_graders
                   @stalled_processes = GraderProcess.find_stalled_process
                   @terminated_processes = GraderProcess.find_terminated_graders
                   @last_task = Task.last
                   @last_test_request = TestRequest.last
                   @submission = Submission.order("id desc").limit(20)
                   @backlog_submission = Submission.where('graded_at is null')
                 end
                 def clear
                   grader_proc = GraderProcess.find(params[:id])
                   grader_proc.destroy if grader_proc!=nil
                   redirect_to :action => 'list'
                 end
                 def clear_terminated
                   GraderProcess.find_terminated_graders.each do |p|
                     p.destroy
                   end
                   redirect_to :action => 'list'
                 end
                 def clear_all
                   GraderProcess.all.each do |p|
                     p.destroy
                   end
                   redirect_to :action => 'list'
                 end
                 def view
                   if params[:type]=='Task'
                     redirect_to :action => 'task', :id => params[:id]
                   else

app/controllers/heartbeat_controller.rb ¶ +1 -1

               class HeartbeatController < ApplicationController
-            -   before_filter :admin_authorization, :only => ['index']
+            +   before_action :admin_authorization, :only => ['index']
                 def edit
                   #@user = User.find_by_login(params[:id])
                   #unless @user
                   #  render text: "LOGIN_NOT_FOUND"
                   #  return
                   #end
                   #hb = HeartBeat.where(user_id: @user.id, ip_address: request.remote_ip).first
                   #puts "status = #{params[:status]}"
                   #if hb
                   #  if params[:status]
                   #    hb.status = params[:status]
                   #    hb.save
                   #  end
                   #  hb.touch
                   #else
                   #  HeartBeat.creae(user_id: @user.id, ip_address: request.remote_ip)
                   #end
                   #HeartBeat.create(user_id: @user.id, ip_address: request.remote_ip, status: params[:status])
                   res = GraderConfiguration['right.heartbeat_response']
                   res.strip! if res
                   full = GraderConfiguration['right.heartbeat_response_full']
                   full.strip! if full
                   if full and full != ''
                     l = Login.where(ip_address: request.remote_ip).last
                     @user = l.user
                     if @user.solve_all_available_problems?
                       render text: (full || 'OK')
                     else
                       render text: (res || 'OK')
                     end
                   else
                     render text: (GraderConfiguration['right.heartbeat_response'] || 'OK')
                   end
                 end
                 def index
                   @hb = HeartBeat.where("updated_at >= ?",Time.zone.now-2.hours).includes(:user).order(:user_id).all
                   @num = HeartBeat.where("updated_at >= ?",Time.zone.now-5.minutes).count(:user_id,distinct: true)
                 end
               end

app/controllers/main_controller.rb ¶ +5 -5

               class MainController < ApplicationController
-            -   before_filter :authenticate, :except => [:index, :login]
+            +   before_action :authenticate, :except => [:index, :login]
-            -   before_filter :check_viewability, :except => [:index, :login]
+            +   before_action :check_viewability, :except => [:index, :login]
-            -   append_before_filter :confirm_and_update_start_time,
+            +   append_before_action :confirm_and_update_start_time,
                                      :except => [:index,
                                                  :login,
                                                  :confirm_contest_start]
                 # to prevent log in box to be shown when user logged out of the
                 # system only in some tab
-            -   prepend_before_filter :reject_announcement_refresh_when_logged_out,
+            +   prepend_before_action :reject_announcement_refresh_when_logged_out,
                                       :only => [:announcements]
-            -   before_filter :authenticate_by_ip_address, :only => [:list]
+            +   before_action :authenticate_by_ip_address, :only => [:list]
                 # COMMENTED OUT: filter in each action instead
                 # before_filter :verify_time_limit, :only => [:submit]
                 verify :method => :post, :only => [:submit],
                        :redirect_to => { :action => :index }
                 # COMMENT OUT: only need when having high load
                 # caches_action :index, :login
                 # NOTE: This method is not actually needed, 'config/routes.rb' has
                 # assigned action login as a default action.
                 def index
                   redirect_to :action => 'login'
                 end
                 def login
                   saved_notice = flash[:notice]
                   reset_session
                   flash.now[:notice] = saved_notice
                   # EXPERIMENT:
                   # Hide login if in single user mode and the url does not
                   # explicitly specify /login
                   #
                   # logger.info "PATH: #{request.path}"
                   # if GraderConfiguration['system.single_user_mode'] and
                   #     request.path!='/main/login'
                   #   @hidelogin = true
                   # end
                   @announcements = Announcement.frontpage
                   render :action => 'login', :layout => 'empty'
                 end
                 def list
                   prepare_list_information
                 end
                 def help
                   @user = User.find(session[:user_id])
                 end
                 def submit
                   user = User.find(session[:user_id])
                   @submission = Submission.new
                   @submission.problem_id = params[:submission][:problem_id]

app/controllers/messages_controller.rb ¶ +1 -1

               class MessagesController < ApplicationController
-            -   before_filter :authenticate
+            +   before_action :authenticate
                 verify :method => :post, :only => ['create'],
                        :redirect_to => { :action => 'list' }
                 before_filter :admin_authorization, :only => ['console','show',
                                                               'reply','hide','list_all']
                 def list
                   @user = User.find(session[:user_id])
                   @messages = Message.find_all_sent_by_user(@user)
                 end
                 def console
                   @user = User.find(session[:user_id])
                   @messages = Message.find_all_system_unreplied_messages
                 end
                 def show
                   @message = Message.find(params[:id])
                 end
                 def list_all
                   @user = User.find(session[:user_id])
                   @messages = Message.where(receiver_id: nil).order(:created_at)
                 end
                 def create
                   user = User.find(session[:user_id])
                   @message = Message.new(params[:message])
                   @message.sender = user
                   if @message.body == '' or !@message.save
                     flash[:notice] = 'An error occurred'
                   else
                     flash[:notice] = 'New message posted'
                   end
                   redirect_to :action => 'list'
                 end
                 def reply
                   user = User.find(session[:user_id])
                   @message = Message.new(params[:r_message])
                   @message.sender = user
                   if @message.body == '' or !@message.save
                     flash[:notice] = 'An error occurred'
                     redirect_to :action => 'show', :id => @message.replying_message_id
                   else
                     flash[:notice] = 'Message replied'
                     rep_msg = @message.replying_message

app/controllers/report_controller.rb ¶ +3 -3

               require 'csv'
               class ReportController < ApplicationController
-            -   before_filter :authenticate
+            +   before_action :authenticate
-            -   before_filter :admin_authorization, only: [:login_stat,:submission_stat, :stuck, :cheat_report, :cheat_scruntinize, :show_max_score, :current_score]
+            +   before_action :admin_authorization, only: [:login_stat,:submission_stat, :stuck, :cheat_report, :cheat_scruntinize, :show_max_score, :current_score]
-            -   before_filter(only: [:problem_hof]) { |c|
+            +   before_action(only: [:problem_hof]) { |c|
                   return false unless authenticate
                   admin_authorization unless GraderConfiguration["right.user_view_submission"]
                 }
                 def max_score
                 end
                 def current_score
                   @problems = Problem.available_problems
                   @users = User.includes(:contests).includes(:contest_stat).where(enabled: true)
                   @scorearray = calculate_max_score(@problems, @users,0,0,true)
                   #rencer accordingly
                   if params[:button] == 'download' then
                     csv = gen_csv_from_scorearray(@scorearray,@problems)
                     send_data csv, filename: 'max_score.csv'
                   else
                     #render template: 'user_admin/user_stat'
                     render 'current_score'
                   end
                 end
                 def show_max_score
                   #process parameters
                   #problems
                   @problems = []
                   if params[:problem_id]
                     params[:problem_id].each do |id|
                       next unless id.strip != ""
                       pid = Problem.find_by_id(id.to_i)
                       @problems << pid if pid
                     end
                   end
                   #users
                   @users = if params[:users] == "all" then
                              User.includes(:contests).includes(:contest_stat)
                            else
                              User.includes(:contests).includes(:contest_stat).where(enabled: true)
                            end
                   #set up range from param
                   @since_id = params.fetch(:from_id, 0).to_i
                   @until_id = params.fetch(:to_id, 0).to_i
                   @since_id = nil if @since_id == 0
                   @until_id = nil if @until_id == 0

app/controllers/site_controller.rb ¶ +1 -1

               class SiteController < ApplicationController
-            -   before_filter :site_admin_authorization, :except => 'login'
+            +   before_action :site_admin_authorization, :except => 'login'
                 def login
                   # Site administrator login
                   @countries = Country.includes(:sites).all
                   @country_select = @countries.collect { |c| [c.name, c.id] }
                   @country_select_with_all = [['Any',0]]
                   @countries.each do |country|
                     @country_select_with_all << [country.name, country.id]
                   end
                   @site_select = []
                   @countries.each do |country|
                     country.sites.each do |site|
                       @site_select << ["#{site.name}, #{country.name}", site.id]
                     end
                   end
                   @default_site = Site.first if !GraderConfiguration['contest.multisites']
                   render :action => 'login', :layout => 'empty'
                 end
                 def index
                   if @site.started
                     render :action => 'started', :layout => 'empty'
                   else
                     render :action => 'prompt', :layout => 'empty'
                   end
                 end
                 def start
                   @site.started = true
                   @site.start_time = Time.new.gmtime
                   @site.save
                   redirect_to :action => 'index'
                 end
                 def logout
                   reset_session
                   redirect_to :controller => 'main', :action => 'login'
                 end
                 protected
                 def site_admin_authorization
                   if session[:site_id]==nil
                     redirect_to :controller => 'site', :action => 'login' and return
                   end

app/controllers/sites_controller.rb ¶ +1 -1

               class SitesController < ApplicationController
-            -   before_filter :admin_authorization
+            +   before_action :admin_authorization
                 # GET /sites
                 # GET /sites.xml
                 def index
                   @sites = Site.order(:country_id)
                   respond_to do |format|
                     format.html # index.html.erb
                     format.xml  { render :xml => @sites }
                   end
                 end
                 # GET /sites/1
                 # GET /sites/1.xml
                 def show
                   @site = Site.find(params[:id])
                   respond_to do |format|
                     format.html # show.html.erb
                     format.xml  { render :xml => @site }
                   end
                 end
                 # GET /sites/new
                 # GET /sites/new.xml
                 def new
                   @site = Site.new
                   respond_to do |format|
                     format.html # new.html.erb
                     format.xml  { render :xml => @site }
                   end
                 end
                 # GET /sites/1/edit
                 def edit
                   @site = Site.find(params[:id])
                 end
                 # POST /sites
                 # POST /sites.xml
                 def create
                   @site = Site.new(params[:site])
                   @site.clear_start_time_if_not_started
                   respond_to do |format|
                     if @site.save
                       flash[:notice] = 'Site was successfully created.'

app/controllers/tasks_controller.rb ¶ +1 -1

               class TasksController < ApplicationController
-            -   before_filter :authenticate, :check_viewability
+            +   before_action :authenticate, :check_viewability
                 def index
                   redirect_to :action => 'list'
                 end
                 def list
                   @problems = @user.available_problems
                 end
                 # this has contest-wide access control
                 def view
                   base_name = params[:file]
                   base_filename = File.basename("#{base_name}.#{params[:ext]}")
                   filename = "#{Problem.download_file_basedir}/#{base_filename}"
                   if !FileTest.exists?(filename)
                     redirect_to :action => 'index' and return
                   end
                   send_file_to_user(filename, base_filename)
                 end
                 # this has problem-level access control
                 def download
                   problem = Problem.find(params[:id])
                   unless @current_user.can_view_problem? problem
                     redirect_to :action => 'index' and return
                   end
                   base_name = params[:file]
                   base_filename = File.basename("#{base_name}.#{params[:ext]}")
                   filename = "#{Problem.download_file_basedir}/#{params[:id]}/#{base_filename}"
                   puts "SENDING: #{filename}"
                   if !FileTest.exists?(filename)
                     redirect_to :action => 'index' and return
                   end
                   puts "SENDING: #{filename}"
                   send_file_to_user(filename, base_filename)
                 end
                 protected
                 def send_file_to_user(filename, base_filename)
                   if defined?(USE_APACHE_XSENDFILE) and USE_APACHE_XSENDFILE
                     response.headers['Content-Type'] = "application/force-download"

app/controllers/test_controller.rb ¶ +1 -1

               class TestController < ApplicationController
-            -   before_filter :authenticate, :check_viewability
+            +   before_action :authenticate, :check_viewability
               #
               #  COMMENT OUT: filter in each action instead
               #
               #  before_filter :verify_time_limit, :only => [:submit]
                 verify :method => :post, :only => [:submit],
                        :redirect_to => { :action => :index }
                 def index
                   prepare_index_information
                 end
                 def submit
                   @user = User.find(session[:user_id])
                   @submitted_test_request = TestRequest.new_from_form_params(@user,params[:test_request])
                   if ! @submitted_test_request.errors.empty?
                     prepare_index_information
                     render :action => 'index' and return
                   end
                   if GraderConfiguration.time_limit_mode?
                     if @user.contest_finished?
                       @submitted_test_request.errors.add(:base,'Contest is over.')
                       prepare_index_information
                       render :action => 'index' and return
                     end
                     if !GraderConfiguration.allow_test_request(@user)
                       prepare_index_information
                       flash[:notice] = 'Test request is not allowed during the last 30 minutes'
                       redirect_to :action => 'index' and return
                     end
                   end
                   if @submitted_test_request.save
                     redirect_to :action => 'index'
                   else
                     prepare_index_information
                     render :action => 'index'
                   end
                 end
                 def read
                   user = User.find(session[:user_id])
                   begin

app/controllers/user_admin_controller.rb ¶ +1 -1

               require 'csv'
               class UserAdminController < ApplicationController
                 include MailHelperMethods
-            -   before_filter :admin_authorization
+            +   before_action :admin_authorization
                 # GETs should be safe (see http://www.w3.org/2001/tag/doc/whenToUseGet.html)
                 verify :method => :post, :only => [
                                                     :create, :create_from_list,
                                                     :update,
                                                     :manage_contest,
                                                     :bulk_mail
                                                   ],
                        :redirect_to => { :action => :list }
                 def index
                   @user_count = User.count
                   if params[:page] == 'all'
                     @users = User.all
                     @paginated = false
                   else
                     @users = User.paginate :page => params[:page]
                     @paginated = true
                   end
                   @users = User.all
                   @hidden_columns = ['hashed_password', 'salt', 'created_at', 'updated_at']
                   @contests = Contest.enabled
                 end
                 def active
                   sessions = ActiveRecord::SessionStore::Session.where("updated_at >= ?", 60.minutes.ago)
                   @users = []
                   sessions.each do |session|
                     if session.data[:user_id]
                       @users << User.find(session.data[:user_id])
                     end
                   end
                 end
                 def show
                   @user = User.find(params[:id])
                 end
                 def new
                   @user = User.new
                 end
                 def create
                   @user = User.new(user_params)
                   @user.activated = true
                   if @user.save
                     flash[:notice] = 'User was successfully created.'
                     redirect_to :action => 'index'

app/controllers/users_controller.rb ¶ +4 -4

               require 'net/smtp'
               class UsersController < ApplicationController
                 include MailHelperMethods
-            -   before_filter :authenticate, :except => [:new,
+            +   before_action :authenticate, :except => [:new,
                                                          :register,
                                                          :confirm,
                                                          :forget,
                                                          :retrieve_password]
-            -   before_filter :verify_online_registration, :only => [:new,
+            +   before_action :verify_online_registration, :only => [:new,
                                                                      :register,
                                                                      :forget,
                                                                      :retrieve_password]
-            -   before_filter :authenticate, :profile_authorization, only: [:profile]
+            +   before_action :authenticate, :profile_authorization, only: [:profile]
-            -   before_filter :admin_authorization, only: [:stat, :toggle_activate, :toggle_enable]
+            +   before_action :admin_authorization, only: [:stat, :toggle_activate, :toggle_enable]
                 verify :method => :post, :only => [:chg_passwd],
                        :redirect_to => { :action => :index }
                 #in_place_edit_for :user, :alias_for_editing
                 #in_place_edit_for :user, :email_for_editing
                 def index
                   if !GraderConfiguration['system.user_setting_enabled']
                     redirect_to :controller => 'main', :action => 'list'
                   else
                     @user = User.find(session[:user_id])
                   end
                 end
                 def chg_passwd
                   user = User.find(session[:user_id])
                   user.password = params[:passwd]
                   user.password_confirmation = params[:passwd_verify]
                   if user.save
                     flash[:notice] = 'password changed'
                   else
                     flash[:notice] = 'Error: password changing failed'
                   end
                   redirect_to :action => 'index'
                 end
                 def new
                   @user = User.new
                   render :action => 'new', :layout => 'empty'
                 end
                 def register
                   if(params[:cancel])
                     redirect_to :controller => 'main', :action => 'login'
                     return
                   end
                   @user = User.new(user_params)
                   @user.password_confirmation = @user.password = User.random_password
                   @user.activated = false
                   if (@user.valid?) and (@user.save)
                     if send_confirmation_email(@user)
                       render :action => 'new_splash', :layout => 'empty'
                     else
                       @admin_email = GraderConfiguration['system.admin_email']
                       render :action => 'email_error', :layout => 'empty'
                     end

config/routes.rb ¶ +2 -1

               CafeGrader::Application.routes.draw do
                 resources :tags
                 get "sources/direct_edit"
                 root :to => 'main#login'
                 #logins
-            -   get 'login/login',  to: 'login#login'
+            +   match 'login/login',  to: 'login#login', via: [:get,:post]
+            +
                 resources :contests
                 resources :sites
                 resources :announcements do
                   member do
                     get 'toggle','toggle_front'
                   end
                 end
                 resources :problems do
                   member do
                     get 'toggle'
                     get 'toggle_test'
                     get 'toggle_view_testcase'
                     get 'stat'
                   end
                   collection do
                     get 'turn_all_off'
                     get 'turn_all_on'
                     get 'import'
                     get 'manage'
                   end
                 end
                 resources :groups do
                   member do
                     post 'add_user', to: 'groups#add_user', as: 'add_user'
                     delete 'remove_user/:user_id', to: 'groups#remove_user', as: 'remove_user'
                     delete 'remove_all_user', to: 'groups#remove_all_user', as: 'remove_all_user'
                     post 'add_problem', to: 'groups#add_problem', as: 'add_problem'
                     delete 'remove_problem/:problem_id', to: 'groups#remove_problem', as: 'remove_problem'
                     delete 'remove_all_problem', to: 'groups#remove_all_problem', as: 'remove_all_problem'
                   end
                   collection do
                   end
                 end
                 resources :testcases, only: [] do
                   member do
                     get 'download_input'
                     get 'download_sol'
                   end
                   collection do
                     get 'show_problem/:problem_id(/:test_num)' => 'testcases#show_problem', as: 'show_problem'
                   end

test/integration/login_test.rb ¶ +4 -4

               require 'test_helper'
               class LoginTest < ActionDispatch::IntegrationTest
                 # test "the truth" do
                 #   assert true
                 # end
                 test "login with invalid information" do
                   get root_path
                   assert_response :success
-            -     post login_login_path, login: "root", password: "hahaha"
+            +     post login_login_path, params: {login: "root", password: "hahaha"}
                   assert_redirected_to root_path
                 end
                 test "normal user login" do
                   get root_path
                   assert_response :success
-            -     post login_login_path, {login: "john", password: "hello" }
+            +     post login_login_path, params: {login: "john", password: "hello" }
                   assert_redirected_to main_list_path
                 end
                 test "normal user login in single_user mode" do
                   GraderConfiguration.find_by(key: GraderConfiguration::SINGLE_USER_KEY).update_attributes(value: 'true')
                   GraderConfiguration.reload
                   get root_path
                   assert_response :success
-            -     post login_login_path, {login: "john", password: "hello" }
+            +     post login_login_path, params: {login: "john", password: "hello" }
                   follow_redirect!
                   assert_redirected_to root_path
                 end
                 test "root login in in single_user mode" do
                   GraderConfiguration.find_by(key: GraderConfiguration::SINGLE_USER_KEY).update_attributes(value: 'true')
                   GraderConfiguration.reload
                   get root_path
                   assert_response :success
-            -     post login_login_path, {login: "admin", password: "admin" }
+            +     post login_login_path, params: {login: "admin", password: "admin" }
                   assert_redirected_to main_list_path
                 end
               end

Write
Preview

You need to be logged in to leave comments. Login now

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository permissions settings

Sign in to your account

Author

r745:75264caa616f - Wed, 03 Jul 2019 10:25:07 - 20 files changed: 38 inserted, 31 deleted

Sign in to your account

Commit r745:75264caa616f public

Author

r745:75264caa616f - Wed, 03 Jul 2019 10:25:07 - 20 files changed: 38 inserted, 31 deleted

Commit `r745:75264caa616f` public