Description:
shows recent problem after wrong submissions
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r223:1586858ff9d5 - - 4 files changed: 32 inserted, 12 deleted
@@ -198,23 +198,28 | |||
|
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 | 202 | if recent_assignment == nil |
|
202 | 203 | flash[:notice] = 'You have not requested for any input data for this problem. Please download an input first.' |
|
204 | + session[:current_problem_id] = problem.id | |
|
203 | 205 | redirect_to :action => 'list' and return |
|
204 | 206 | end |
|
205 | 207 | |
|
206 | 208 | if recent_assignment.expired? |
|
207 | 209 | flash[:notice] = 'The current input is expired. Please download a new input data.' |
|
210 | + session[:current_problem_id] = problem.id | |
|
208 | 211 | redirect_to :action => 'list' and return |
|
209 | 212 | end |
|
210 | 213 | |
|
211 | 214 | if recent_assignment.submitted |
|
212 | 215 | flash[:notice] = 'You have already submitted an incorrect solution for this input. Please download a new input data.' |
|
216 | + session[:current_problem_id] = problem.id | |
|
213 | 217 | redirect_to :action => 'list' and return |
|
214 | 218 | end |
|
215 | 219 | |
|
216 | 220 | if params[:file] == nil |
|
217 | 221 | flash[:notice] = 'You have not submitted any output.' |
|
222 | + session[:current_problem_id] = problem.id | |
|
218 | 223 | redirect_to :action => 'list' and return |
|
219 | 224 | end |
|
220 | 225 | |
@@ -247,6 +252,7 | |||
|
247 | 252 | flash[:notice] = 'Correct solution.' |
|
248 | 253 | user.update_codejom_status |
|
249 | 254 | else |
|
255 | + session[:current_problem_id] = problem.id | |
|
250 | 256 | flash[:notice] = 'Incorrect solution.' |
|
251 | 257 | end |
|
252 | 258 | redirect_to :action => 'list' |
@@ -280,6 +286,13 | |||
|
280 | 286 | sub_count[status.problem_id] = status.submission_count |
|
281 | 287 | end |
|
282 | 288 | |
|
289 | + if session.has_key? :current_problem_id | |
|
290 | + @current_problem_id = session[:current_problem_id] | |
|
291 | + session.delete(:current_problem_id) | |
|
292 | + else | |
|
293 | + @current_problem_id = nil | |
|
294 | + end | |
|
295 | + | |
|
283 | 296 | @problems = all_problems.reject { |problem| passed.has_key? problem.id } |
|
284 | 297 | |
|
285 | 298 | @prob_submissions = Array.new |
@@ -19,9 +19,13 | |||
|
19 | 19 | end |
|
20 | 20 | |
|
21 | 21 | def random_test_pair(forbidden_numbers=nil) |
|
22 | - begin | |
|
22 | + if forbidden_numbers.length < test_pair_count | |
|
23 | + begin | |
|
24 | + test_num = 1 + rand(test_pair_count) | |
|
25 | + end while forbidden_numbers!=nil and forbidden_numbers.include? test_num | |
|
26 | + else | |
|
23 | 27 | test_num = 1 + rand(test_pair_count) |
|
24 | - end while forbidden_numbers!=nil and forbidden_numbers.include? test_num | |
|
28 | + end | |
|
25 | 29 | test_pairs.find_by_number test_num |
|
26 | 30 | end |
|
27 | 31 |
@@ -1,13 +1,15 | |||
|
1 | - .problem-panel{:id => "problem-panel-#{problem.id}", :style => "display:none"} | |
|
1 | + .problem-panel{:id => "problem-panel-#{problem.id}", :style => "#{(problem.id != @current_problem_id) ? "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." | |
|
7 | - - form_tag({ :action => 'submit_solution', :id => problem.id }, :method => :post, :multipart => true) do | |
|
8 | - %b Submit output: | |
|
9 | - %input{:type => "file", :name => "file"} | |
|
10 | - %input{:type => "submit", :value => "Submit solution"} | |
|
6 | + %span{:id => "problem-timing-message-#{problem.id}"} | |
|
7 | + = "After downloading, you have #{TEST_ASSIGNMENT_EXPIRATION_DURATION/60} minutes to submit." | |
|
8 | + %div{:id => "problem-submission-form-#{problem.id}"} | |
|
9 | + - form_tag({ :action => 'submit_solution', :id => problem.id }, :method => :post, :multipart => true) do | |
|
10 | + %b Submit output: | |
|
11 | + %input{:type => "file", :name => "file"} | |
|
12 | + %input{:type => "submit", :value => "Submit solution"} | |
|
11 | 13 | |
|
12 | 14 | .problem-description |
|
13 | 15 | - if problem.description!=nil |
@@ -20,10 +20,11 | |||
|
20 | 20 | .problem-list |
|
21 | 21 | = render :partial => 'problem_title', :collection => @problems, :as => :problem |
|
22 | 22 | .problem-content |
|
23 | - %span{:id => "problem-panel-filler"} | |
|
24 | - %b Welcome to Code Jom | |
|
25 | - %br/ | |
|
26 | - Choose problems from the list on the right. | |
|
23 | + - if @current_problem_id==nil | |
|
24 | + %span{:id => "problem-panel-filler"} | |
|
25 | + %b Welcome to Code Jom | |
|
26 | + %br/ | |
|
27 | + Choose problems from the list on the right. | |
|
27 | 28 | = render :partial => 'problem', :collection => @problems |
|
28 | 29 | |
|
29 | 30 | %br{:clear=>'both'}/ |
You need to be logged in to leave comments.
Login now