Description:
[web] added mode + access control, when sites started/finished
git-svn-id: http://theory.cpe.ku.ac.th/grader/web/trunk@250 6386c4cd-e34a-4fa8-8920-d93eb39b512e
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r122:dfa0387efb1e - - 9 files changed: 121 inserted, 17 deleted
@@ -0,0 +1,17 | |||
|
1 | + class AddModeToConfigurations < ActiveRecord::Migration | |
|
2 | + def self.up | |
|
3 | + | |
|
4 | + # Configuration['system.mode']: | |
|
5 | + # * 'standard' mode | |
|
6 | + # * 'contest' mode (check site start time/stop time) | |
|
7 | + # * 'analysis' mode (show results, no new submissions) | |
|
8 | + | |
|
9 | + Configuration.create(:key => 'system.mode', | |
|
10 | + :value_type => 'string', | |
|
11 | + :value => 'standard') | |
|
12 | + end | |
|
13 | + | |
|
14 | + def self.down | |
|
15 | + Configuration.find_by_key('system.mode').destroy | |
|
16 | + end | |
|
17 | + end |
@@ -1,9 +1,12 | |||
|
1 | 1 | class MainController < ApplicationController |
|
2 | 2 | |
|
3 | + SYSTEM_MODE_CONF_KEY = 'system.mode' | |
|
4 | + | |
|
3 | 5 | before_filter :authenticate, :except => [:index, :login] |
|
6 | + before_filter :check_viewability | |
|
4 | 7 | |
|
5 | 8 | # |
|
6 | 9 | # COMMENT OUT: filter in each action instead |
|
7 | 10 | # |
|
8 | 11 | # before_filter :verify_time_limit, :only => [:submit] |
|
9 | 12 | |
@@ -40,13 +43,14 | |||
|
40 | 43 | if params['file']!='' |
|
41 | 44 | @submission.source = params['file'].read |
|
42 | 45 | @submission.source_filename = params['file'].original_filename |
|
43 | 46 | end |
|
44 | 47 | @submission.submitted_at = Time.new.gmtime |
|
45 | 48 | |
|
46 | - if user.site!=nil and user.site.finished? | |
|
49 | + if Configuration[SYSTEM_MODE_CONF_KEY]=='contest' and | |
|
50 | + user.site!=nil and user.site.finished? | |
|
47 | 51 | @submission.errors.add_to_base "The contest is over." |
|
48 | 52 | prepare_list_information |
|
49 | 53 | render :action => 'list' and return |
|
50 | 54 | end |
|
51 | 55 | |
|
52 | 56 | if @submission.valid? |
@@ -122,8 +126,16 | |||
|
122 | 126 | |
|
123 | 127 | @announcements = Announcement.find(:all, |
|
124 | 128 | :conditions => "published = 1", |
|
125 | 129 | :order => "created_at DESC") |
|
126 | 130 | end |
|
127 | 131 | |
|
132 | + def check_viewability | |
|
133 | + user = User.find(session[:user_id]) | |
|
134 | + if (!Configuration.show_tasks_to?(user)) and | |
|
135 | + ((action_name=='submission') or (action_name=='submit')) | |
|
136 | + redirect_to :action => 'list' and return | |
|
137 | + end | |
|
128 | 138 | end |
|
129 | 139 | |
|
140 | + end | |
|
141 | + |
@@ -1,15 +1,24 | |||
|
1 | 1 | class TasksController < ApplicationController |
|
2 | 2 | |
|
3 | - before_filter :authenticate | |
|
4 | - | |
|
3 | + before_filter :authenticate, :check_viewability | |
|
5 | 4 | |
|
6 | 5 | def index |
|
7 | 6 | redirect_to :action => 'list' |
|
8 | 7 | end |
|
9 | 8 | |
|
10 | 9 | def list |
|
11 | 10 | @problems = Problem.find_available_problems |
|
12 | 11 | @user = User.find(session[:user_id]) |
|
13 | 12 | end |
|
14 | 13 | |
|
14 | + protected | |
|
15 | + | |
|
16 | + def check_viewability | |
|
17 | + user = User.find(session[:user_id]) | |
|
18 | + if user==nil or !Configuration.show_tasks_to?(user) | |
|
19 | + redirect_to :controller => 'main', :action => 'list' | |
|
20 | + return false | |
|
15 | 21 | end |
|
22 | + end | |
|
23 | + | |
|
24 | + end |
@@ -1,9 +1,11 | |||
|
1 | 1 | class TestController < ApplicationController |
|
2 | 2 | |
|
3 | - before_filter :authenticate | |
|
3 | + SYSTEM_MODE_CONF_KEY = 'system.mode' | |
|
4 | + | |
|
5 | + before_filter :authenticate, :check_viewability | |
|
4 | 6 | |
|
5 | 7 | # |
|
6 | 8 | # COMMENT OUT: filter in each action instead |
|
7 | 9 | # |
|
8 | 10 | # before_filter :verify_time_limit, :only => [:submit] |
|
9 | 11 | |
@@ -21,13 +23,14 | |||
|
21 | 23 | |
|
22 | 24 | if @submitted_test_request.errors.length != 0 |
|
23 | 25 | prepare_index_information |
|
24 | 26 | render :action => 'index' and return |
|
25 | 27 | end |
|
26 | 28 | |
|
27 | - if @user.site!=nil and @user.site.finished? | |
|
29 | + if Configuration[SYSTEM_MODE_CONF_KEY]=='contest' and | |
|
30 | + @user.site!=nil and @user.site.finished? | |
|
28 | 31 | @submitted_test_request.errors.add_to_base('Contest is over.') |
|
29 | 32 | prepare_index_information |
|
30 | 33 | render :action => 'index' and return |
|
31 | 34 | end |
|
32 | 35 | |
|
33 | 36 | if @submitted_test_request.save |
@@ -89,7 +92,17 | |||
|
89 | 92 | @problems << problem |
|
90 | 93 | end |
|
91 | 94 | end |
|
92 | 95 | @test_requests = @user.test_requests |
|
93 | 96 | end |
|
94 | 97 | |
|
98 | + def check_viewability | |
|
99 | + user = User.find(session[:user_id]) | |
|
100 | + if !Configuration.show_tasks_to?(user) | |
|
101 | + redirect_to :controller => 'main', :action => 'list' | |
|
95 | 102 | end |
|
103 | + if (!Configuration.show_submitbox_to?(user)) and (action_name=='submit') | |
|
104 | + redirect_to :controller => 'test', :action => 'index' | |
|
105 | + end | |
|
106 | + end | |
|
107 | + | |
|
108 | + end |
@@ -1,9 +1,11 | |||
|
1 | 1 | # Methods added to this helper will be available to all templates in the application. |
|
2 | 2 | module ApplicationHelper |
|
3 | 3 | |
|
4 | + SYSTEM_MODE_CONF_KEY = 'system.mode' | |
|
5 | + | |
|
4 | 6 | def user_header |
|
5 | 7 | menu_items = '' |
|
6 | 8 | user = User.find(session[:user_id]) |
|
7 | 9 | |
|
8 | 10 | if (user!=nil) and (session[:admin]) |
|
9 | 11 | # admin menu |
@@ -18,15 +20,18 | |||
|
18 | 20 | menu_items << "<br/>" |
|
19 | 21 | end |
|
20 | 22 | |
|
21 | 23 | # main page |
|
22 | 24 | append_to menu_items, '[Main]', 'main', 'list' |
|
23 | 25 | append_to menu_items, '[Messages]', 'messages', 'list' |
|
26 | + | |
|
27 | + if (user!=nil) and (Configuration.show_tasks_to?(user)) | |
|
24 | 28 | append_to menu_items, '[Tasks]', 'tasks', 'list' |
|
25 | 29 | append_to menu_items, '[Submissions]', 'main', 'submission' |
|
26 | 30 | append_to menu_items, '[Test]', 'test', 'index' |
|
31 | + end | |
|
27 | 32 | append_to menu_items, '[Help]', 'main', 'help' |
|
28 | 33 | #append_to menu_items, '[Settings]', 'users', 'index' |
|
29 | 34 | append_to menu_items, '[Log out]', 'main', 'login' |
|
30 | 35 | |
|
31 | 36 | menu_items |
|
32 | 37 | end |
@@ -45,40 +50,57 | |||
|
45 | 50 | (time.year != now.year) |
|
46 | 51 | st = time.strftime("%x ") |
|
47 | 52 | end |
|
48 | 53 | st + time.strftime("%X") |
|
49 | 54 | end |
|
50 | 55 | |
|
56 | + def read_textfile(fname,max_size=2048) | |
|
57 | + begin | |
|
58 | + File.open(fname).read(max_size) | |
|
59 | + rescue | |
|
60 | + nil | |
|
61 | + end | |
|
62 | + end | |
|
51 | 63 | |
|
52 | 64 | def user_title_bar(user) |
|
53 | - if user.site!=nil and user.site.finished? | |
|
54 | - contest_over_string = <<CONTEST_OVER | |
|
65 | + header = '' | |
|
66 | + | |
|
67 | + # | |
|
68 | + # if the contest is over | |
|
69 | + if Configuration[SYSTEM_MODE_CONF_KEY]=='contest' and | |
|
70 | + user.site!=nil and user.site.finished? | |
|
71 | + header = <<CONTEST_OVER | |
|
55 | 72 | <tr><td colspan="2" align="center"> |
|
56 | 73 | <span class="contest-over-msg">THE CONTEST IS OVER</span> |
|
57 | 74 | </td></tr> |
|
58 | 75 | CONTEST_OVER |
|
59 | 76 | end |
|
77 | + | |
|
78 | + # | |
|
79 | + # if the contest is in the anaysis mode | |
|
80 | + if Configuration[SYSTEM_MODE_CONF_KEY]=='analysis' | |
|
81 | + header = <<ANALYSISMODE | |
|
82 | + <tr><td colspan="2" align="center"> | |
|
83 | + <span class="contest-over-msg">ANALYSIS MODE</span> | |
|
84 | + </td></tr> | |
|
85 | + ANALYSISMODE | |
|
86 | + end | |
|
87 | + | |
|
88 | + # | |
|
89 | + # build real title bar | |
|
60 | 90 | <<TITLEBAR |
|
61 | 91 | <div class="title"> |
|
62 | 92 | <table> |
|
63 | - #{contest_over_string} | |
|
93 | + #{header} | |
|
64 | 94 | <tr> |
|
65 | 95 | <td class="left-col"> |
|
66 | 96 | #{user.full_name}<br/> |
|
67 | 97 | Current time is #{format_short_time(Time.new.gmtime)} UTC<br/> |
|
68 | 98 | </td> |
|
69 | 99 | <td class="right-col">APIO'08</td> |
|
70 | 100 | </tr> |
|
71 | 101 | </table> |
|
72 | 102 | </div> |
|
73 | 103 | TITLEBAR |
|
74 | 104 | end |
|
75 | 105 | |
|
76 | - def read_textfile(fname,max_size=2048) | |
|
77 | - begin | |
|
78 | - File.open(fname).read(max_size) | |
|
79 | - rescue | |
|
80 | - nil | |
|
81 | 106 |
|
|
82 | - end | |
|
83 | - | |
|
84 | - end |
@@ -1,8 +1,13 | |||
|
1 | + # | |
|
2 | + # This class also contains various login of the system. | |
|
3 | + # | |
|
1 | 4 | class Configuration < ActiveRecord::Base |
|
2 | 5 | |
|
6 | + SYSTEM_MODE_CONF_KEY = 'system.mode' | |
|
7 | + | |
|
3 | 8 | @@configurations = nil |
|
4 | 9 | |
|
5 | 10 | def self.get(key) |
|
6 | 11 | if @@configurations == nil |
|
7 | 12 | self.read_config |
|
8 | 13 | end |
@@ -18,12 +23,34 | |||
|
18 | 23 | end |
|
19 | 24 | |
|
20 | 25 | def self.clear |
|
21 | 26 | @@configurations = nil |
|
22 | 27 | end |
|
23 | 28 | |
|
29 | + # | |
|
30 | + # View decision | |
|
31 | + # | |
|
32 | + def self.show_submitbox_to?(user) | |
|
33 | + mode = get(SYSTEM_MODE_CONF_KEY) | |
|
34 | + return false if mode=='analysis' | |
|
35 | + if (mode=='contest') | |
|
36 | + return false if (user.site!=nil) and | |
|
37 | + ((user.site.started==false) or (user.site.finished?)) | |
|
38 | + end | |
|
39 | + return true | |
|
40 | + end | |
|
41 | + | |
|
42 | + def self.show_tasks_to?(user) | |
|
43 | + mode = get(SYSTEM_MODE_CONF_KEY) | |
|
44 | + if (mode=='contest') | |
|
45 | + return false if (user.site!=nil) and (user.site.started==false) | |
|
46 | + end | |
|
47 | + return true | |
|
48 | + end | |
|
49 | + | |
|
50 | + | |
|
24 | 51 | protected |
|
25 | 52 | def self.read_config |
|
26 | 53 | @@configurations = {} |
|
27 | 54 | Configuration.find(:all).each do |conf| |
|
28 | 55 | key = conf.key |
|
29 | 56 | val = conf.value |
@@ -3,19 +3,21 | |||
|
3 | 3 | - if @announcements.length!=0 |
|
4 | 4 | .announcementbox |
|
5 | 5 | %span{:class => 'title'} |
|
6 | 6 | Announcements |
|
7 | 7 | = render :partial => 'announcement', :collection => @announcements |
|
8 | 8 | |
|
9 | + - if Configuration.show_submitbox_to?(@user) | |
|
9 | 10 | .submitbox |
|
10 | 11 | = error_messages_for 'submission' |
|
11 | 12 | = render :partial => 'submission_box' |
|
12 | 13 | |
|
13 | 14 | |
|
14 | 15 | %hr/ |
|
15 | 16 | |
|
17 | + - if Configuration.show_tasks_to?(@user) | |
|
16 | 18 | %table.info |
|
17 | 19 | %tr.info-head |
|
18 | 20 | %th |
|
19 | 21 | %th Tasks |
|
20 | 22 | %th # of sub(s) |
|
21 | 23 | %th Results |
@@ -27,12 +27,13 | |||
|
27 | 27 | submissionSelect.add(new Option(""+i,""+i,false,false)); |
|
28 | 28 | } |
|
29 | 29 | } |
|
30 | 30 | } |
|
31 | 31 | </script> |
|
32 | 32 | |
|
33 | + <% if Configuration.show_submitbox_to?(@user) %> | |
|
33 | 34 | <div class="submitbox"> |
|
34 | 35 | <%= error_messages_for 'submitted_test_request' %> |
|
35 | 36 | <% form_for :test_request, nil, |
|
36 | 37 | :url => { :action => 'submit'}, |
|
37 | 38 | :html => { :multipart => true } do |f| %> |
|
38 | 39 | <table> |
@@ -82,12 +83,13 | |||
|
82 | 83 | <%= submit_tag 'submit' %> |
|
83 | 84 | </td> |
|
84 | 85 | </tr> |
|
85 | 86 | </table> |
|
86 | 87 | <% end %> |
|
87 | 88 | </div> |
|
89 | + <% end %> | |
|
88 | 90 | |
|
89 | 91 | <h3>Previous requests</h3> |
|
90 | 92 | |
|
91 | 93 | <table class="info"> |
|
92 | 94 | <tr class="info-head"> |
|
93 | 95 | <th>at</th> |
@@ -6,13 +6,13 | |||
|
6 | 6 | # to create the application database on another system, you should be using db:schema:load, not running |
|
7 | 7 | # all the migrations from scratch. The latter is a flawed and unsustainable approach (the more migrations |
|
8 | 8 | # you'll amass, the slower it'll run and the greater likelihood for issues). |
|
9 | 9 | # |
|
10 | 10 | # It's strongly recommended to check this file into your version control system. |
|
11 | 11 | |
|
12 |
- ActiveRecord::Schema.define(:version => 3 |
|
|
12 | + ActiveRecord::Schema.define(:version => 36) do | |
|
13 | 13 | |
|
14 | 14 | create_table "announcements", :force => true do |t| |
|
15 | 15 | t.string "author" |
|
16 | 16 | t.text "body" |
|
17 | 17 | t.boolean "published" |
|
18 | 18 | t.datetime "created_at" |
You need to be logged in to leave comments.
Login now