Description:
Merge branch 'master' into codejom (bug fixed)
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r250:6820e08d6a38 - - 1 file changed: 10 inserted, 0 deleted
@@ -1,55 +1,59 | |||
|
1 | 1 | class MainController < ApplicationController |
|
2 | 2 | |
|
3 | 3 | before_filter :authenticate, :except => [:index, :login] |
|
4 | 4 | before_filter :check_viewability, :except => [:index, :login] |
|
5 | 5 | |
|
6 | 6 | append_before_filter :update_user_start_time, :except => [:index, :login] |
|
7 | 7 | |
|
8 | + # to prevent log in box to be shown when user logged out of the | |
|
9 | + # system only in some tab | |
|
10 | + prepend_before_filter :reject_announcement_refresh_when_logged_out, :only => [:announcements] | |
|
11 | + | |
|
8 | 12 | # COMMENTED OUT: filter in each action instead |
|
9 | 13 | # before_filter :verify_time_limit, :only => [:submit] |
|
10 | 14 | |
|
11 | 15 | verify :method => :post, :only => [:submit, :download_input, :submit_solution], |
|
12 | 16 | :redirect_to => { :action => :index } |
|
13 | 17 | |
|
14 | 18 | # COMMENT OUT: only need when having high load |
|
15 | 19 | # caches_action :index, :login |
|
16 | 20 | |
|
17 | 21 | # NOTE: This method is not actually needed, 'config/routes.rb' has |
|
18 | 22 | # assigned action login as a default action. |
|
19 | 23 | def index |
|
20 | 24 | redirect_to :action => 'login' |
|
21 | 25 | end |
|
22 | 26 | |
|
23 | 27 | def login |
|
24 | 28 | saved_notice = flash[:notice] |
|
25 | 29 | reset_session |
|
26 | 30 | flash.now[:notice] = saved_notice |
|
27 | 31 | |
|
28 | 32 | # EXPERIMENT: |
|
29 | 33 | # Hide login if in single user mode and the url does not |
|
30 | 34 | # explicitly specify /login |
|
31 | 35 | # |
|
32 | 36 | # logger.info "PATH: #{request.path}" |
|
33 | 37 | # if Configuration['system.single_user_mode'] and |
|
34 | 38 | # request.path!='/main/login' |
|
35 | 39 | # @hidelogin = true |
|
36 | 40 | # end |
|
37 | 41 | |
|
38 | 42 | @announcements = Announcement.find_for_frontpage |
|
39 | 43 | render :action => 'login', :layout => 'empty' |
|
40 | 44 | end |
|
41 | 45 | |
|
42 | 46 | def list |
|
43 | 47 | prepare_list_information |
|
44 | 48 | end |
|
45 | 49 | |
|
46 | 50 | def help |
|
47 | 51 | @user = User.find(session[:user_id]) |
|
48 | 52 | end |
|
49 | 53 | |
|
50 | 54 | def submit |
|
51 | 55 | user = User.find(session[:user_id]) |
|
52 | 56 | |
|
53 | 57 | @submission = Submission.new(params[:submission]) |
|
54 | 58 | @submission.user = user |
|
55 | 59 | @submission.language_id = 0 |
@@ -417,50 +421,56 | |||
|
417 | 421 | :output_size => output_size |
|
418 | 422 | } |
|
419 | 423 | end |
|
420 | 424 | end |
|
421 | 425 | |
|
422 | 426 | # copied from grader/script/lib/test_request_helper.rb |
|
423 | 427 | def extract_running_stat(results) |
|
424 | 428 | running_stat_line = results[-1] |
|
425 | 429 | |
|
426 | 430 | # extract exit status line |
|
427 | 431 | run_stat = "" |
|
428 | 432 | if !(/[Cc]orrect/.match(results[0])) |
|
429 | 433 | run_stat = results[0].chomp |
|
430 | 434 | else |
|
431 | 435 | run_stat = 'Program exited normally' |
|
432 | 436 | end |
|
433 | 437 | |
|
434 | 438 | logger.info "Stat line: #{running_stat_line}" |
|
435 | 439 | |
|
436 | 440 | # extract running time |
|
437 | 441 | if res = /r(.*)u(.*)s/.match(running_stat_line) |
|
438 | 442 | seconds = (res[1].to_f + res[2].to_f) |
|
439 | 443 | time_stat = "Time used: #{seconds} sec." |
|
440 | 444 | else |
|
441 | 445 | seconds = nil |
|
442 | 446 | time_stat = "Time used: n/a sec." |
|
443 | 447 | end |
|
444 | 448 | |
|
445 | 449 | # extract memory usage |
|
446 | 450 | if res = /s(.*)m/.match(running_stat_line) |
|
447 | 451 | memory_used = res[1].to_i |
|
448 | 452 | else |
|
449 | 453 | memory_used = -1 |
|
450 | 454 | end |
|
451 | 455 | |
|
452 | 456 | return { |
|
453 | 457 | :msg => "#{run_stat}\n#{time_stat}", |
|
454 | 458 | :running_time => seconds, |
|
455 | 459 | :exit_status => run_stat, |
|
456 | 460 | :memory_usage => memory_used |
|
457 | 461 | } |
|
458 | 462 | end |
|
459 | 463 | |
|
460 | 464 | def update_user_start_time |
|
461 | 465 | user = User.find(session[:user_id]) |
|
462 | 466 | UserContestStat.update_user_start_time(user) |
|
463 | 467 | end |
|
464 | 468 | |
|
469 | + def reject_announcement_refresh_when_logged_out | |
|
470 | + if not session[:user_id] | |
|
471 | + render :text => 'Access forbidden', :status => 403 | |
|
472 | + end | |
|
473 | + end | |
|
474 | + | |
|
465 | 475 | end |
|
466 | 476 |
You need to be logged in to leave comments.
Login now