Description:
add rejudge button
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r645:a2d233431d60 - - 8 files changed: 50 inserted, 18 deleted
@@ -0,0 +1,2 | |||||
|
|
1 | + :plain | ||
|
|
2 | + $("body").prepend("<div class=\"alert alert-info\"> Submission #{@submission.id}'s task status has been chaned to \"#{@task.status_str}\" </div>") |
@@ -0,0 +1,5 | |||||
|
|
1 | + class AddIndexToTask < ActiveRecord::Migration | ||
|
|
2 | + def change | ||
|
|
3 | + add_index :tasks, :submission_id | ||
|
|
4 | + end | ||
|
|
5 | + end |
@@ -1,9 +1,10 | |||||
|
1 | class SubmissionsController < ApplicationController |
|
1 | class SubmissionsController < ApplicationController |
|
2 |
- before_ |
|
2 | + before_action :authenticate |
|
3 |
- before_ |
|
3 | + before_action :submission_authorization, only: [:show, :direct_edit_submission, :download, :edit] |
|
|
4 | + before_action :admin_authorization, only: [:rejudge] | ||
|
4 |
|
5 | ||
|
5 | # GET /submissions |
|
6 | # GET /submissions |
|
6 | # GET /submissions.json |
|
7 | # GET /submissions.json |
|
7 | # Show problem selection and user's submission of that problem |
|
8 | # Show problem selection and user's submission of that problem |
|
8 | def index |
|
9 | def index |
|
9 | @user = @current_user |
|
10 | @user = @current_user |
@@ -28,12 +29,14 | |||||
|
28 | def show |
|
29 | def show |
|
29 | @submission = Submission.find(params[:id]) |
|
30 | @submission = Submission.find(params[:id]) |
|
30 |
|
31 | ||
|
31 | #log the viewing |
|
32 | #log the viewing |
|
32 | user = User.find(session[:user_id]) |
|
33 | user = User.find(session[:user_id]) |
|
33 | SubmissionViewLog.create(user_id: session[:user_id],submission_id: @submission.id) unless user.admin? |
|
34 | SubmissionViewLog.create(user_id: session[:user_id],submission_id: @submission.id) unless user.admin? |
|
|
35 | + | ||
|
|
36 | + @task = @submission.task | ||
|
34 | end |
|
37 | end |
|
35 |
|
38 | ||
|
36 | def download |
|
39 | def download |
|
37 | @submission = Submission.find(params[:id]) |
|
40 | @submission = Submission.find(params[:id]) |
|
38 | send_data(@submission.source, {:filename => @submission.download_filename, :type => 'text/plain'}) |
|
41 | send_data(@submission.source, {:filename => @submission.download_filename, :type => 'text/plain'}) |
|
39 | end |
|
42 | end |
@@ -69,12 +72,22 | |||||
|
69 | puts 'nil' unless @submission |
|
72 | puts 'nil' unless @submission |
|
70 | respond_to do |format| |
|
73 | respond_to do |format| |
|
71 | format.js |
|
74 | format.js |
|
72 | end |
|
75 | end |
|
73 | end |
|
76 | end |
|
74 |
|
77 | ||
|
|
78 | + # GET /submissions/:id/rejudge | ||
|
|
79 | + def rejudge | ||
|
|
80 | + @submission = Submission.find(params[:id]) | ||
|
|
81 | + @task = @submission.task | ||
|
|
82 | + @task.status_inqueue! if @task | ||
|
|
83 | + respond_to do |format| | ||
|
|
84 | + format.js | ||
|
|
85 | + end | ||
|
|
86 | + end | ||
|
|
87 | + | ||
|
75 |
|
88 | ||
|
76 | protected |
|
89 | protected |
|
77 |
|
90 | ||
|
78 | def submission_authorization |
|
91 | def submission_authorization |
|
79 | #admin always has privileged |
|
92 | #admin always has privileged |
|
80 | if @current_user.admin? |
|
93 | if @current_user.admin? |
@@ -206,13 +206,12 | |||||
|
206 |
|
206 | ||
|
207 | def bootstrap_class_for(flash_type) |
|
207 | def bootstrap_class_for(flash_type) |
|
208 | BOOTSTRAP_FLASH_MSG.fetch(flash_type.to_sym, flash_type.to_s) |
|
208 | BOOTSTRAP_FLASH_MSG.fetch(flash_type.to_sym, flash_type.to_s) |
|
209 | end |
|
209 | end |
|
210 |
|
210 | ||
|
211 | def flash_messages |
|
211 | def flash_messages |
|
212 | - puts "flahs size = #{flash.count}" |
|
||
|
213 | flash.each do |msg_type, message| |
|
212 | flash.each do |msg_type, message| |
|
214 | concat(content_tag(:div, message, class: "alert #{bootstrap_class_for(msg_type)} fade in") do |
|
213 | concat(content_tag(:div, message, class: "alert #{bootstrap_class_for(msg_type)} fade in") do |
|
215 | concat content_tag(:button, 'x', class: "close", data: { dismiss: 'alert' }) |
|
214 | concat content_tag(:button, 'x', class: "close", data: { dismiss: 'alert' }) |
|
216 | concat message |
|
215 | concat message |
|
217 | end) |
|
216 | end) |
|
218 | end |
|
217 | end |
@@ -10,12 +10,14 | |||||
|
10 | validates_presence_of :source |
|
10 | validates_presence_of :source |
|
11 | validates_length_of :source, :maximum => 100_000, :allow_blank => true, :message => 'too long' |
|
11 | validates_length_of :source, :maximum => 100_000, :allow_blank => true, :message => 'too long' |
|
12 | validates_length_of :source, :minimum => 1, :allow_blank => true, :message => 'too short' |
|
12 | validates_length_of :source, :minimum => 1, :allow_blank => true, :message => 'too short' |
|
13 | validate :must_have_valid_problem |
|
13 | validate :must_have_valid_problem |
|
14 | validate :must_specify_language |
|
14 | validate :must_specify_language |
|
15 |
|
15 | ||
|
|
16 | + has_one :task | ||
|
|
17 | + | ||
|
16 | before_save :assign_latest_number_if_new_recond |
|
18 | before_save :assign_latest_number_if_new_recond |
|
17 |
|
19 | ||
|
18 | def self.find_last_by_user_and_problem(user_id, problem_id) |
|
20 | def self.find_last_by_user_and_problem(user_id, problem_id) |
|
19 | where("user_id = ? AND problem_id = ?",user_id,problem_id).last |
|
21 | where("user_id = ? AND problem_id = ?",user_id,problem_id).last |
|
20 | end |
|
22 | end |
|
21 |
|
23 |
@@ -87,12 +87,20 | |||||
|
87 | view |
|
87 | view |
|
88 | - if session[:admin] |
|
88 | - if session[:admin] |
|
89 | %tr |
|
89 | %tr |
|
90 | %td.text-right |
|
90 | %td.text-right |
|
91 | %strong IP |
|
91 | %strong IP |
|
92 | %td #{@submission.ip_address} |
|
92 | %td #{@submission.ip_address} |
|
|
93 | + %tr | ||
|
|
94 | + %td.text-right | ||
|
|
95 | + %strong Grading Task Status | ||
|
|
96 | + %td | ||
|
|
97 | + = @task.status_str if @task | ||
|
|
98 | + - if session[:admin] | ||
|
|
99 | + = link_to "rejudge", rejudge_submission_path, data: {remote: true}, class: 'btn btn-info btn-xs' | ||
|
|
100 | + | ||
|
93 |
|
101 | ||
|
94 | .modal.fade#compiler{tabindex: -1,role: 'dialog'} |
|
102 | .modal.fade#compiler{tabindex: -1,role: 'dialog'} |
|
95 | .modal-dialog.modal-lg{role:'document'} |
|
103 | .modal-dialog.modal-lg{role:'document'} |
|
96 | .modal-content |
|
104 | .modal-content |
|
97 | .modal-header |
|
105 | .modal-header |
|
98 | %button.close{type: 'button', data: {dismissed: :modal}, aria: {label: 'close'}} |
|
106 | %button.close{type: 'button', data: {dismissed: :modal}, aria: {label: 'close'}} |
@@ -52,12 +52,13 | |||||
|
52 | end |
|
52 | end |
|
53 |
|
53 | ||
|
54 | resources :submissions do |
|
54 | resources :submissions do |
|
55 | member do |
|
55 | member do |
|
56 | get 'download' |
|
56 | get 'download' |
|
57 | get 'compiler_msg' |
|
57 | get 'compiler_msg' |
|
|
58 | + get 'rejudge' | ||
|
58 | end |
|
59 | end |
|
59 | collection do |
|
60 | collection do |
|
60 | get 'prob/:problem_id', to: 'submissions#index', as: 'problem' |
|
61 | get 'prob/:problem_id', to: 'submissions#index', as: 'problem' |
|
61 | get 'direct_edit_problem/:problem_id', to: 'submissions#direct_edit_problem', as: 'direct_edit_problem' |
|
62 | get 'direct_edit_problem/:problem_id', to: 'submissions#direct_edit_problem', as: 'direct_edit_problem' |
|
62 | get 'get_latest_submission_status/:uid/:pid', to: 'submissions#get_latest_submission_status', as: 'get_latest_submission_status' |
|
63 | get 'get_latest_submission_status/:uid/:pid', to: 'submissions#get_latest_submission_status', as: 'get_latest_submission_status' |
|
63 | end |
|
64 | end |
@@ -8,17 +8,17 | |||||
|
8 | # system, you should be using db:schema:load, not running all the migrations |
|
8 | # system, you should be using db:schema:load, not running all the migrations |
|
9 | # from scratch. The latter is a flawed and unsustainable approach (the more migrations |
|
9 | # from scratch. The latter is a flawed and unsustainable approach (the more migrations |
|
10 | # you'll amass, the slower it'll run and the greater likelihood for issues). |
|
10 | # you'll amass, the slower it'll run and the greater likelihood for issues). |
|
11 | # |
|
11 | # |
|
12 | # It's strongly recommended that you check this file into your version control system. |
|
12 | # It's strongly recommended that you check this file into your version control system. |
|
13 |
|
13 | ||
|
14 |
- ActiveRecord::Schema.define(version: 20170 |
|
14 | + ActiveRecord::Schema.define(version: 20170310110146) do |
|
15 |
|
15 | ||
|
16 | create_table "announcements", force: :cascade do |t| |
|
16 | create_table "announcements", force: :cascade do |t| |
|
17 | t.string "author", limit: 255 |
|
17 | t.string "author", limit: 255 |
|
18 |
- t.text "body", limit: |
|
18 | + t.text "body", limit: 65535 |
|
19 | t.boolean "published" |
|
19 | t.boolean "published" |
|
20 |
t.datetime "created_at", |
|
20 | t.datetime "created_at", null: false |
|
21 |
t.datetime "updated_at", |
|
21 | t.datetime "updated_at", null: false |
|
22 |
t.boolean "frontpage", |
|
22 | t.boolean "frontpage", default: false |
|
23 |
t.boolean "contest_only", |
|
23 | t.boolean "contest_only", default: false |
|
24 | t.string "title", limit: 255 |
|
24 | t.string "title", limit: 255 |
@@ -47,25 +47,25 | |||||
|
47 | t.string "name", limit: 255 |
|
47 | t.string "name", limit: 255 |
|
48 | t.datetime "created_at", null: false |
|
48 | t.datetime "created_at", null: false |
|
49 | t.datetime "updated_at", null: false |
|
49 | t.datetime "updated_at", null: false |
|
50 | end |
|
50 | end |
|
51 |
|
51 | ||
|
52 | create_table "descriptions", force: :cascade do |t| |
|
52 | create_table "descriptions", force: :cascade do |t| |
|
53 |
- t.text "body", limit: |
|
53 | + t.text "body", limit: 65535 |
|
54 | t.boolean "markdowned" |
|
54 | t.boolean "markdowned" |
|
55 |
t.datetime "created_at", |
|
55 | t.datetime "created_at", null: false |
|
56 |
t.datetime "updated_at", |
|
56 | t.datetime "updated_at", null: false |
|
57 | end |
|
57 | end |
|
58 |
|
58 | ||
|
59 | create_table "grader_configurations", force: :cascade do |t| |
|
59 | create_table "grader_configurations", force: :cascade do |t| |
|
60 | t.string "key", limit: 255 |
|
60 | t.string "key", limit: 255 |
|
61 | t.string "value_type", limit: 255 |
|
61 | t.string "value_type", limit: 255 |
|
62 | t.string "value", limit: 255 |
|
62 | t.string "value", limit: 255 |
|
63 |
t.datetime "created_at", |
|
63 | t.datetime "created_at", null: false |
|
64 |
t.datetime "updated_at", |
|
64 | t.datetime "updated_at", null: false |
|
65 |
- t.text "description", limit: |
|
65 | + t.text "description", limit: 65535 |
|
66 | end |
|
66 | end |
|
67 |
|
67 | ||
|
68 | create_table "grader_processes", force: :cascade do |t| |
|
68 | create_table "grader_processes", force: :cascade do |t| |
|
69 | t.string "host", limit: 255 |
|
69 | t.string "host", limit: 255 |
|
70 | t.integer "pid", limit: 4 |
|
70 | t.integer "pid", limit: 4 |
|
71 | t.string "mode", limit: 255 |
|
71 | t.string "mode", limit: 255 |
@@ -104,13 +104,13 | |||||
|
104 | end |
|
104 | end |
|
105 |
|
105 | ||
|
106 | create_table "messages", force: :cascade do |t| |
|
106 | create_table "messages", force: :cascade do |t| |
|
107 | t.integer "sender_id", limit: 4 |
|
107 | t.integer "sender_id", limit: 4 |
|
108 | t.integer "receiver_id", limit: 4 |
|
108 | t.integer "receiver_id", limit: 4 |
|
109 | t.integer "replying_message_id", limit: 4 |
|
109 | t.integer "replying_message_id", limit: 4 |
|
110 |
- t.text "body", limit: |
|
110 | + t.text "body", limit: 65535 |
|
111 | t.boolean "replied" |
|
111 | t.boolean "replied" |
|
112 |
t.datetime "created_at", |
|
112 | t.datetime "created_at", null: false |
|
113 |
t.datetime "updated_at", |
|
113 | t.datetime "updated_at", null: false |
|
114 | end |
|
114 | end |
|
115 |
|
115 | ||
|
116 | create_table "problems", force: :cascade do |t| |
|
116 | create_table "problems", force: :cascade do |t| |
@@ -150,13 +150,13 | |||||
|
150 | end |
|
150 | end |
|
151 |
|
151 | ||
|
152 | add_index "roles_users", ["user_id"], name: "index_roles_users_on_user_id", using: :btree |
|
152 | add_index "roles_users", ["user_id"], name: "index_roles_users_on_user_id", using: :btree |
|
153 |
|
153 | ||
|
154 | create_table "sessions", force: :cascade do |t| |
|
154 | create_table "sessions", force: :cascade do |t| |
|
155 | t.string "session_id", limit: 255 |
|
155 | t.string "session_id", limit: 255 |
|
156 |
- t.text "data", limit: |
|
156 | + t.text "data", limit: 65535 |
|
157 | t.datetime "updated_at" |
|
157 | t.datetime "updated_at" |
|
158 | end |
|
158 | end |
|
159 |
|
159 | ||
|
160 | add_index "sessions", ["session_id"], name: "index_sessions_on_session_id", using: :btree |
|
160 | add_index "sessions", ["session_id"], name: "index_sessions_on_session_id", using: :btree |
|
161 | add_index "sessions", ["updated_at"], name: "index_sessions_on_updated_at", using: :btree |
|
161 | add_index "sessions", ["updated_at"], name: "index_sessions_on_updated_at", using: :btree |
|
162 |
|
162 | ||
@@ -178,20 +178,20 | |||||
|
178 | end |
|
178 | end |
|
179 |
|
179 | ||
|
180 | create_table "submissions", force: :cascade do |t| |
|
180 | create_table "submissions", force: :cascade do |t| |
|
181 | t.integer "user_id", limit: 4 |
|
181 | t.integer "user_id", limit: 4 |
|
182 | t.integer "problem_id", limit: 4 |
|
182 | t.integer "problem_id", limit: 4 |
|
183 | t.integer "language_id", limit: 4 |
|
183 | t.integer "language_id", limit: 4 |
|
184 |
- t.text "source", limit: |
|
184 | + t.text "source", limit: 65535 |
|
185 | t.binary "binary", limit: 65535 |
|
185 | t.binary "binary", limit: 65535 |
|
186 | t.datetime "submitted_at" |
|
186 | t.datetime "submitted_at" |
|
187 | t.datetime "compiled_at" |
|
187 | t.datetime "compiled_at" |
|
188 |
- t.text "compiler_message", limit: |
|
188 | + t.text "compiler_message", limit: 65535 |
|
189 | t.datetime "graded_at" |
|
189 | t.datetime "graded_at" |
|
190 | t.integer "points", limit: 4 |
|
190 | t.integer "points", limit: 4 |
|
191 |
- t.text "grader_comment", limit: |
|
191 | + t.text "grader_comment", limit: 65535 |
|
192 | t.integer "number", limit: 4 |
|
192 | t.integer "number", limit: 4 |
|
193 | t.string "source_filename", limit: 255 |
|
193 | t.string "source_filename", limit: 255 |
|
194 | t.float "max_runtime", limit: 24 |
|
194 | t.float "max_runtime", limit: 24 |
|
195 | t.integer "peak_memory", limit: 4 |
|
195 | t.integer "peak_memory", limit: 4 |
|
196 | t.integer "effective_code_length", limit: 4 |
|
196 | t.integer "effective_code_length", limit: 4 |
|
197 | t.string "ip_address", limit: 255 |
|
197 | t.string "ip_address", limit: 255 |
@@ -204,16 +204,18 | |||||
|
204 | t.integer "submission_id", limit: 4 |
|
204 | t.integer "submission_id", limit: 4 |
|
205 | t.datetime "created_at" |
|
205 | t.datetime "created_at" |
|
206 | t.integer "status", limit: 4 |
|
206 | t.integer "status", limit: 4 |
|
207 | t.datetime "updated_at" |
|
207 | t.datetime "updated_at" |
|
208 | end |
|
208 | end |
|
209 |
|
209 | ||
|
|
210 | + add_index "tasks", ["submission_id"], name: "index_tasks_on_submission_id", using: :btree | ||
|
|
211 | + | ||
|
210 | create_table "test_pairs", force: :cascade do |t| |
|
212 | create_table "test_pairs", force: :cascade do |t| |
|
211 | t.integer "problem_id", limit: 4 |
|
213 | t.integer "problem_id", limit: 4 |
|
212 |
- t.text "input", limit: |
|
214 | + t.text "input", limit: 16777215 |
|
213 |
- t.text "solution", limit: |
|
215 | + t.text "solution", limit: 16777215 |
|
214 |
t.datetime "created_at", |
|
216 | t.datetime "created_at", null: false |
|
215 |
t.datetime "updated_at", |
|
217 | t.datetime "updated_at", null: false |
|
216 | end |
|
218 | end |
|
217 |
|
219 | ||
|
218 | create_table "test_requests", force: :cascade do |t| |
|
220 | create_table "test_requests", force: :cascade do |t| |
|
219 | t.integer "user_id", limit: 4 |
|
221 | t.integer "user_id", limit: 4 |
@@ -223,13 +225,13 | |||||
|
223 | t.string "output_file_name", limit: 255 |
|
225 | t.string "output_file_name", limit: 255 |
|
224 | t.string "running_stat", limit: 255 |
|
226 | t.string "running_stat", limit: 255 |
|
225 | t.integer "status", limit: 4 |
|
227 | t.integer "status", limit: 4 |
|
226 |
t.datetime "updated_at", |
|
228 | t.datetime "updated_at", null: false |
|
227 | t.datetime "submitted_at" |
|
229 | t.datetime "submitted_at" |
|
228 | t.datetime "compiled_at" |
|
230 | t.datetime "compiled_at" |
|
229 |
- t.text "compiler_message", limit: |
|
231 | + t.text "compiler_message", limit: 65535 |
|
230 | t.datetime "graded_at" |
|
232 | t.datetime "graded_at" |
|
231 | t.string "grader_comment", limit: 255 |
|
233 | t.string "grader_comment", limit: 255 |
|
232 |
t.datetime "created_at", |
|
234 | t.datetime "created_at", null: false |
|
233 | t.float "running_time", limit: 24 |
|
235 | t.float "running_time", limit: 24 |
|
234 | t.string "exit_status", limit: 255 |
|
236 | t.string "exit_status", limit: 255 |
|
235 | t.integer "memory_usage", limit: 4 |
|
237 | t.integer "memory_usage", limit: 4 |
@@ -241,14 +243,14 | |||||
|
241 | t.integer "problem_id", limit: 4 |
|
243 | t.integer "problem_id", limit: 4 |
|
242 | t.integer "num", limit: 4 |
|
244 | t.integer "num", limit: 4 |
|
243 | t.integer "group", limit: 4 |
|
245 | t.integer "group", limit: 4 |
|
244 | t.integer "score", limit: 4 |
|
246 | t.integer "score", limit: 4 |
|
245 | t.text "input", limit: 4294967295 |
|
247 | t.text "input", limit: 4294967295 |
|
246 | t.text "sol", limit: 4294967295 |
|
248 | t.text "sol", limit: 4294967295 |
|
247 |
- t.datetime "created_at" |
|
249 | + t.datetime "created_at" |
|
248 |
- t.datetime "updated_at" |
|
250 | + t.datetime "updated_at" |
|
249 | end |
|
251 | end |
|
250 |
|
252 | ||
|
251 | add_index "testcases", ["problem_id"], name: "index_testcases_on_problem_id", using: :btree |
|
253 | add_index "testcases", ["problem_id"], name: "index_testcases_on_problem_id", using: :btree |
|
252 |
|
254 | ||
|
253 | create_table "user_contest_stats", force: :cascade do |t| |
|
255 | create_table "user_contest_stats", force: :cascade do |t| |
|
254 | t.integer "user_id", limit: 4 |
|
256 | t.integer "user_id", limit: 4 |
@@ -267,15 +269,15 | |||||
|
267 | t.string "email", limit: 255 |
|
269 | t.string "email", limit: 255 |
|
268 | t.integer "site_id", limit: 4 |
|
270 | t.integer "site_id", limit: 4 |
|
269 | t.integer "country_id", limit: 4 |
|
271 | t.integer "country_id", limit: 4 |
|
270 | t.boolean "activated", default: false |
|
272 | t.boolean "activated", default: false |
|
271 | t.datetime "created_at" |
|
273 | t.datetime "created_at" |
|
272 | t.datetime "updated_at" |
|
274 | t.datetime "updated_at" |
|
273 | - t.string "section", limit: 255 |
|
||
|
274 | t.boolean "enabled", default: true |
|
275 | t.boolean "enabled", default: true |
|
275 | t.string "remark", limit: 255 |
|
276 | t.string "remark", limit: 255 |
|
276 | t.string "last_ip", limit: 255 |
|
277 | t.string "last_ip", limit: 255 |
|
|
278 | + t.string "section", limit: 255 | ||
|
277 | end |
|
279 | end |
|
278 |
|
280 | ||
|
279 | add_index "users", ["login"], name: "index_users_on_login", unique: true, using: :btree |
|
281 | add_index "users", ["login"], name: "index_users_on_login", unique: true, using: :btree |
|
280 |
|
282 | ||
|
281 | end |
|
283 | end |
You need to be logged in to leave comments.
Login now