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