Description:
fix bug for "stuck" and add authorization
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r464:310e993b7826 - - 1 file changed: 2 inserted, 1 deleted
@@ -1,6 +1,6 | |||||
|
1 | class ReportController < ApplicationController |
|
1 | class ReportController < ApplicationController |
|
2 |
|
2 | ||
|
3 | - before_filter :admin_authorization, only: [:login_stat,:submission_stat] |
|
3 | + before_filter :admin_authorization, only: [:login_stat,:submission_stat, :stuck] |
|
4 | before_filter(only: [:problem_hof]) { |c| |
|
4 | before_filter(only: [:problem_hof]) { |c| |
|
5 | return false unless authenticate |
|
5 | return false unless authenticate |
|
6 |
|
6 | ||
@@ -196,6 +196,7 | |||||
|
196 | @struggle = Array.new |
|
196 | @struggle = Array.new |
|
197 | record = {} |
|
197 | record = {} |
|
198 | Submission.includes(:problem,:user).order(:problem_id,:user_id).find_each do |sub| |
|
198 | Submission.includes(:problem,:user).order(:problem_id,:user_id).find_each do |sub| |
|
|
199 | + next unless sub.problem and sub.user | ||
|
199 | if user != sub.user_id or problem != sub.problem_id |
|
200 | if user != sub.user_id or problem != sub.problem_id |
|
200 | @struggle << { user: record[:user], problem: record[:problem], tries: tries } unless solve |
|
201 | @struggle << { user: record[:user], problem: record[:problem], tries: tries } unless solve |
|
201 | record = {user: sub.user, problem: sub.problem} |
|
202 | record = {user: sub.user, problem: sub.problem} |
You need to be logged in to leave comments.
Login now