Description:
fix allow admin to submit to any problem
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r682:405106fcdb33 - - 7 files changed: 17 inserted, 5 deleted
@@ -83,13 +83,13 | |||||
|
83 | if GraderConfiguration.time_limit_mode? and user.contest_finished? |
|
83 | if GraderConfiguration.time_limit_mode? and user.contest_finished? |
|
84 | @submission.errors.add(:base,"The contest is over.") |
|
84 | @submission.errors.add(:base,"The contest is over.") |
|
85 | prepare_list_information |
|
85 | prepare_list_information |
|
86 | render :action => 'list' and return |
|
86 | render :action => 'list' and return |
|
87 | end |
|
87 | end |
|
88 |
|
88 | ||
|
89 | - if @submission.valid? |
|
89 | + if @submission.valid?(@current_user) |
|
90 | if @submission.save == false |
|
90 | if @submission.save == false |
|
91 | flash[:notice] = 'Error saving your submission' |
|
91 | flash[:notice] = 'Error saving your submission' |
|
92 | elsif Task.create(:submission_id => @submission.id, |
|
92 | elsif Task.create(:submission_id => @submission.id, |
|
93 | :status => Task::STATUS_INQUEUE) == false |
|
93 | :status => Task::STATUS_INQUEUE) == false |
|
94 | flash[:notice] = 'Error adding your submission to task queue' |
|
94 | flash[:notice] = 'Error adding your submission to task queue' |
|
95 | end |
|
95 | end |
@@ -49,12 +49,17 | |||||
|
49 | end |
|
49 | end |
|
50 |
|
50 | ||
|
51 | #on-site new submission on specific problem |
|
51 | #on-site new submission on specific problem |
|
52 | def direct_edit_problem |
|
52 | def direct_edit_problem |
|
53 | @problem = Problem.find(params[:problem_id]) |
|
53 | @problem = Problem.find(params[:problem_id]) |
|
54 | @source = '' |
|
54 | @source = '' |
|
|
55 | + if (params[:user_id]) | ||
|
|
56 | + u = User.find(params[:user_id]) | ||
|
|
57 | + @submission = Submission.find_last_by_user_and_problem(u.id,@problem.id) | ||
|
|
58 | + @source = @submission.source.to_s if @submission and @submission.source | ||
|
|
59 | + end | ||
|
55 | render 'edit' |
|
60 | render 'edit' |
|
56 | end |
|
61 | end |
|
57 |
|
62 | ||
|
58 | # GET /submissions/1/edit |
|
63 | # GET /submissions/1/edit |
|
59 | def edit |
|
64 | def edit |
|
60 | @submission = Submission.find(params[:id]) |
|
65 | @submission = Submission.find(params[:id]) |
@@ -144,14 +144,18 | |||||
|
144 | end |
|
144 | end |
|
145 |
|
145 | ||
|
146 | def must_have_valid_problem |
|
146 | def must_have_valid_problem |
|
147 | return if self.source==nil |
|
147 | return if self.source==nil |
|
148 | if self.problem==nil |
|
148 | if self.problem==nil |
|
149 | errors.add('problem',"must be specified.") |
|
149 | errors.add('problem',"must be specified.") |
|
150 | - elsif (!self.problem.available) and (self.new_record?) |
|
150 | + else |
|
151 | - errors.add('problem',"must be valid.") |
|
151 | + #admin always have right |
|
|
152 | + return if self.user.admin? | ||
|
|
153 | + | ||
|
|
154 | + #check if user has the right to submit the problem | ||
|
|
155 | + errors.add('problem',"must be valid.") if (!self.user.available_problem.include?(self.problem)) and (self.new_record?) | ||
|
152 | end |
|
156 | end |
|
153 | end |
|
157 | end |
|
154 |
|
158 | ||
|
155 | # callbacks |
|
159 | # callbacks |
|
156 | def assign_latest_number_if_new_recond |
|
160 | def assign_latest_number_if_new_recond |
|
157 | return if !self.new_record? |
|
161 | return if !self.new_record? |
@@ -289,12 +289,13 | |||||
|
289 | sub = Submission.find_last_by_user_and_problem(u.id,p.id) |
|
289 | sub = Submission.find_last_by_user_and_problem(u.id,p.id) |
|
290 | return false if !p or !sub or sub.points < p.full_score |
|
290 | return false if !p or !sub or sub.points < p.full_score |
|
291 | end |
|
291 | end |
|
292 | return true |
|
292 | return true |
|
293 | end |
|
293 | end |
|
294 |
|
294 | ||
|
|
295 | + #get a list of available problem | ||
|
295 | def available_problems |
|
296 | def available_problems |
|
296 | if not GraderConfiguration.multicontests? |
|
297 | if not GraderConfiguration.multicontests? |
|
297 | if GraderConfiguration.use_problem_group? |
|
298 | if GraderConfiguration.use_problem_group? |
|
298 | return available_problems_in_group |
|
299 | return available_problems_in_group |
|
299 | else |
|
300 | else |
|
300 | return Problem.available_problems |
|
301 | return Problem.available_problems |
@@ -41,13 +41,13 | |||||
|
41 | - @problem=problem |
|
41 | - @problem=problem |
|
42 | %td= problem.name #in_place_editor_field :problem, :name, {}, :rows=>1 |
|
42 | %td= problem.name #in_place_editor_field :problem, :name, {}, :rows=>1 |
|
43 | %td |
|
43 | %td |
|
44 | = problem.full_name #in_place_editor_field :problem, :full_name, {}, :rows=>1 |
|
44 | = problem.full_name #in_place_editor_field :problem, :full_name, {}, :rows=>1 |
|
45 | = link_to_description_if_any "[#{t 'main.problem_desc'}] <span class='glyphicon glyphicon-file'></span>".html_safe, problem |
|
45 | = link_to_description_if_any "[#{t 'main.problem_desc'}] <span class='glyphicon glyphicon-file'></span>".html_safe, problem |
|
46 | %td.text-right= problem.full_score #in_place_editor_field :problem, :full_score, {}, :rows=>1 |
|
46 | %td.text-right= problem.full_score #in_place_editor_field :problem, :full_score, {}, :rows=>1 |
|
47 | - %td= link_to "Submit", direct_edit_problem_submissions_path(problem), class: 'btn btn-xs btn-primary' |
|
47 | + %td= link_to "Submit", direct_edit_problem_submissions_path(problem,@current_user.id), class: 'btn btn-xs btn-primary' |
|
48 | %td= problem.date_added |
|
48 | %td= problem.date_added |
|
49 | %td= toggle_button(@problem.available?, toggle_problem_path(@problem), "problem-avail-#{@problem.id}") |
|
49 | %td= toggle_button(@problem.available?, toggle_problem_path(@problem), "problem-avail-#{@problem.id}") |
|
50 | %td= toggle_button(@problem.view_testcase?, toggle_view_testcase_problem_path(@problem), "problem-view-testcase-#{@problem.id}") |
|
50 | %td= toggle_button(@problem.view_testcase?, toggle_view_testcase_problem_path(@problem), "problem-view-testcase-#{@problem.id}") |
|
51 | %td= toggle_button(@problem.test_allowed?, toggle_test_problem_path(@problem), "problem-test-#{@problem.id}") |
|
51 | %td= toggle_button(@problem.test_allowed?, toggle_test_problem_path(@problem), "problem-test-#{@problem.id}") |
|
52 | - if GraderConfiguration.multicontests? |
|
52 | - if GraderConfiguration.multicontests? |
|
53 | %td |
|
53 | %td |
@@ -8,12 +8,13 | |||||
|
8 | .alert.alert-info |
|
8 | .alert.alert-info |
|
9 | Write your code in the following box, choose language, and click submit button when finished |
|
9 | Write your code in the following box, choose language, and click submit button when finished |
|
10 | .row |
|
10 | .row |
|
11 | .col-md-8 |
|
11 | .col-md-8 |
|
12 | %div#editor{style: 'height: 500px; border-radius: 7px; font-size: 14px;'} |
|
12 | %div#editor{style: 'height: 500px; border-radius: 7px; font-size: 14px;'} |
|
13 | .col-md-4 |
|
13 | .col-md-4 |
|
|
14 | + - # submission form | ||
|
14 | = form_tag({controller: :main, :action => 'submit'}, :multipart => true, class: 'form') do |
|
15 | = form_tag({controller: :main, :action => 'submit'}, :multipart => true, class: 'form') do |
|
15 |
|
16 | ||
|
16 | = hidden_field_tag 'editor_text', @source |
|
17 | = hidden_field_tag 'editor_text', @source |
|
17 | = hidden_field_tag 'submission[problem_id]', @problem.id |
|
18 | = hidden_field_tag 'submission[problem_id]', @problem.id |
|
18 | .form-group |
|
19 | .form-group |
|
19 | = label_tag "Task:" |
|
20 | = label_tag "Task:" |
@@ -22,12 +23,13 | |||||
|
22 | .form-group |
|
23 | .form-group |
|
23 | = label_tag 'Language' |
|
24 | = label_tag 'Language' |
|
24 | = select_tag 'language_id', options_from_collection_for_select(Language.all, 'id', 'pretty_name', @lang_id || Language.find_by_pretty_name("Python").id || Language.first.id), class: 'form-control select', style: "width: 100px" |
|
25 | = select_tag 'language_id', options_from_collection_for_select(Language.all, 'id', 'pretty_name', @lang_id || Language.find_by_pretty_name("Python").id || Language.first.id), class: 'form-control select', style: "width: 100px" |
|
25 | .form-group |
|
26 | .form-group |
|
26 | = submit_tag 'Submit', class: 'btn btn-success', id: 'live_submit', |
|
27 | = submit_tag 'Submit', class: 'btn btn-success', id: 'live_submit', |
|
27 | data: {confirm: "Submitting this source code for task #{@problem.long_name}?"} |
|
28 | data: {confirm: "Submitting this source code for task #{@problem.long_name}?"} |
|
|
29 | + - # latest submission status | ||
|
28 | .panel.panel-info |
|
30 | .panel.panel-info |
|
29 | .panel-heading |
|
31 | .panel-heading |
|
30 | Latest Submission Status |
|
32 | Latest Submission Status |
|
31 | = link_to "Refresh",get_latest_submission_status_submissions_path(@submission.user,@problem), class: "btn btn-default btn-sm", remote: true if @submission |
|
33 | = link_to "Refresh",get_latest_submission_status_submissions_path(@submission.user,@problem), class: "btn btn-default btn-sm", remote: true if @submission |
|
32 | .panel-body |
|
34 | .panel-body |
|
33 | - if @submission |
|
35 | - if @submission |
@@ -70,13 +70,13 | |||||
|
70 | get 'download' |
|
70 | get 'download' |
|
71 | get 'compiler_msg' |
|
71 | get 'compiler_msg' |
|
72 | get 'rejudge' |
|
72 | get 'rejudge' |
|
73 | end |
|
73 | end |
|
74 | collection do |
|
74 | collection do |
|
75 | get 'prob/:problem_id', to: 'submissions#index', as: 'problem' |
|
75 | get 'prob/:problem_id', to: 'submissions#index', as: 'problem' |
|
76 | - get 'direct_edit_problem/:problem_id', to: 'submissions#direct_edit_problem', as: 'direct_edit_problem' |
|
76 | + get 'direct_edit_problem/:problem_id(/:user_id)', to: 'submissions#direct_edit_problem', as: 'direct_edit_problem' |
|
77 | get 'get_latest_submission_status/:uid/:pid', to: 'submissions#get_latest_submission_status', as: 'get_latest_submission_status' |
|
77 | get 'get_latest_submission_status/:uid/:pid', to: 'submissions#get_latest_submission_status', as: 'get_latest_submission_status' |
|
78 | end |
|
78 | end |
|
79 | end |
|
79 | end |
|
80 |
|
80 | ||
|
81 |
|
81 | ||
|
82 |
|
82 |
You need to be logged in to leave comments.
Login now