Description:
added test assignment time out
Commit status:
[Not Reviewed]
References:
Comments:
0 Commit comments 0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
Add another comment

r222:2b6293a64dbb - - 8 files changed: 42 inserted, 2 deleted

@@ -194,24 +194,29
194 194 end
195 195 end
196 196
197 197 def submit_solution
198 198 problem = Problem.find(params[:id])
199 199 user = User.find(session[:user_id])
200 200 recent_assignment = user.get_recent_test_pair_assignment_for problem
201 201 if recent_assignment == nil
202 202 flash[:notice] = 'You have not requested for any input data for this problem. Please download an input first.'
203 203 redirect_to :action => 'list' and return
204 204 end
205 205
206 + if recent_assignment.expired?
207 + flash[:notice] = 'The current input is expired. Please download a new input data.'
208 + redirect_to :action => 'list' and return
209 + end
210 +
206 211 if recent_assignment.submitted
207 212 flash[:notice] = 'You have already submitted an incorrect solution for this input. Please download a new input data.'
208 213 redirect_to :action => 'list' and return
209 214 end
210 215
211 216 if params[:file] == nil
212 217 flash[:notice] = 'You have not submitted any output.'
213 218 redirect_to :action => 'list' and return
214 219 end
215 220
216 221 submitted_solution = params[:file].read
217 222 test_pair = recent_assignment.test_pair
@@ -1,5 +1,11
1 1 class TestPairAssignment < ActiveRecord::Base
2 +
2 3 belongs_to :user
3 4 belongs_to :test_pair
4 5 belongs_to :problem
6 +
7 + def expired?
8 + return created_at + TEST_ASSIGNMENT_EXPIRATION_DURATION < Time.new.gmtime
9 + end
10 +
5 11 end
@@ -87,25 +87,25
87 87 return nil
88 88 else
89 89 recent = assignments[0]
90 90 assignments.each do |a|
91 91 recent = a if a.request_number > recent.request_number
92 92 end
93 93 return recent
94 94 end
95 95 end
96 96
97 97 def can_request_new_test_pair_for?(problem)
98 98 recent = get_recent_test_pair_assignment_for problem
99 - return (recent == nil or recent.submitted)
99 + return (recent == nil or recent.submitted or recent.expired?)
100 100 end
101 101
102 102 def get_new_test_pair_assignment_for(problem)
103 103 previous_assignment_numbers =
104 104 get_test_pair_assignments_for(problem).collect {|a| a.test_pair_number }
105 105 test_pair = problem.random_test_pair(previous_assignment_numbers)
106 106 if test_pair
107 107 assignment = TestPairAssignment.new(:user => self,
108 108 :problem => problem,
109 109 :test_pair => test_pair,
110 110 :test_pair_number => test_pair.number,
111 111 :request_number =>
@@ -5,18 +5,18
5 5 <head>
6 6 <meta http-equiv="content-type" content="text/html;charset=UTF-8" />
7 7 <title><%= Configuration['contest.name'] %></title>
8 8 <%= stylesheet_link_tag 'application' %>
9 9 <%= yield :head %>
10 10 </head>
11 11 <body>
12 12
13 13 <div class="userbar">
14 14 <%= user_header %>
15 15 </div>
16 16
17 - <%= content_tag(:p,flash[:notice],:style => "color:green") if flash[:notice]!=nil %>
17 + <% if flash[:notice]!=nil %><div class="notice-bar"><span class="notice"><%= flash[:notice] %></span></div><% end %>
18 18
19 19 <%= yield %>
20 20
21 21 </body>
22 22 </html>
@@ -1,17 +1,18
1 1 .problem-panel{:id => "problem-panel-#{problem.id}", :style => "display:none"}
2 2 .problem-form{:id => "problem-form-#{problem.id}"}
3 3 - form_tag({ :action => 'download_input', :id => problem.id }, :method => :post) do
4 4 %b Input:
5 5 %input{:type => "submit", :value => "Download input"}
6 + = "After downloading, you have #{TEST_ASSIGNMENT_EXPIRATION_DURATION/60} minutes to submit."
6 7 - form_tag({ :action => 'submit_solution', :id => problem.id }, :method => :post, :multipart => true) do
7 8 %b Submit output:
8 9 %input{:type => "file", :name => "file"}
9 10 %input{:type => "submit", :value => "Submit solution"}
10 11
11 12 .problem-description
12 13 - if problem.description!=nil
13 14 - if problem.description.markdowned
14 15 = markdown(problem.description.body)
15 16 - else
16 17 = problem.description.body
17 18 - else
@@ -98,12 +98,13
98 98 # VALIDATE_USER_EMAILS = true
99 99
100 100 # Uncomment so that Apache X-Sendfile is used when delivering files
101 101 # (e.g., in /tasks/view).
102 102 # USE_APACHE_XSENDFILE = true
103 103
104 104 # Uncomment so that configuration is read only once when the server is loaded
105 105 # Configuration.enable_caching
106 106
107 107 # OPTIONS FOR CODE JOM
108 108 # --------------------
109 109 CODEJOM_MAX_ALIVE_LEVEL = 10
110 + TEST_ASSIGNMENT_EXPIRATION_DURATION = 5.minute
@@ -255,12 +255,25
255 255
256 256 .problem-content {
257 257 float: left;
258 258 margin-left: 10px;
259 259 width: 700px; }
260 260
261 261 .problem-panel {
262 262 border: 1px black solid;
263 263 padding: 5px; }
264 264 .problem-panel .problem-form {
265 265 border: 1px dotted #99aaee;
266 266 background: #eeeeff; }
267 +
268 + .notice-bar {
269 + margin-top: 3px;
270 + margin-bottom: 3px;
271 + text-align: center; }
272 + .notice-bar span.notice {
273 + color: white;
274 + font-weight: bold;
275 + background: #000070;
276 + padding: 3px 20px 3px 20px;
277 + -moz-border-radius: 2px;
278 + -webkit-border-radius: 5px;
279 + border-radius: 5px; }
@@ -301,12 +301,26
301 301 .problem-content
302 302 float: left
303 303 margin-left: 10px
304 304 width: 700px
305 305
306 306 .problem-panel
307 307 border: 1px black solid
308 308 padding: 5px
309 309
310 310 .problem-form
311 311 border: 1px dotted #99aaee
312 312 background: #eeeeff
313 +
314 + .notice-bar
315 + margin-top: 3px
316 + margin-bottom: 3px
317 + text-align: center
318 +
319 + span.notice
320 + color: white
321 + font-weight: bold
322 + background: #000070
323 + padding: 3px 20px 3px 20px
324 + -moz-border-radius: 2px
325 + -webkit-border-radius: 5px
326 + border-radius: 5px
You need to be logged in to leave comments. Login now