Description:
add ip to submission
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r446:d9ae0ff1e3ea - - 5 files changed: 31 inserted, 12 deleted
@@ -0,0 +1,5 | |||||
|
|
1 | + class AddIpToSubmissions < ActiveRecord::Migration | ||
|
|
2 | + def change | ||
|
|
3 | + add_column :submissions, :ip_address, :string | ||
|
|
4 | + end | ||
|
|
5 | + end |
@@ -64,12 +64,13 | |||||
|
64 | @submission.language_id = 0 |
|
64 | @submission.language_id = 0 |
|
65 | if (params['file']) and (params['file']!='') |
|
65 | if (params['file']) and (params['file']!='') |
|
66 | @submission.source = params['file'].read |
|
66 | @submission.source = params['file'].read |
|
67 | @submission.source_filename = params['file'].original_filename |
|
67 | @submission.source_filename = params['file'].original_filename |
|
68 | end |
|
68 | end |
|
69 | @submission.submitted_at = Time.new.gmtime |
|
69 | @submission.submitted_at = Time.new.gmtime |
|
|
70 | + @submission.ip_address = request.remote_ip | ||
|
70 |
|
71 | ||
|
71 | if GraderConfiguration.time_limit_mode? and user.contest_finished? |
|
72 | if GraderConfiguration.time_limit_mode? and user.contest_finished? |
|
72 | @submission.errors.add_to_base "The contest is over." |
|
73 | @submission.errors.add_to_base "The contest is over." |
|
73 | prepare_list_information |
|
74 | prepare_list_information |
|
74 | render :action => 'list' and return |
|
75 | render :action => 'list' and return |
|
75 | end |
|
76 | end |
@@ -47,11 +47,15 | |||||
|
47 | %tr{class: cycle('info-even','info-odd')} |
|
47 | %tr{class: cycle('info-even','info-odd')} |
|
48 | %td.field Runtime (s): |
|
48 | %td.field Runtime (s): |
|
49 | %td.value #{@submission.max_runtime} |
|
49 | %td.value #{@submission.max_runtime} |
|
50 | %tr{class: cycle('info-even','info-odd')} |
|
50 | %tr{class: cycle('info-even','info-odd')} |
|
51 | %td.field Memory (kb): |
|
51 | %td.field Memory (kb): |
|
52 | %td.value #{@submission.peak_memory} |
|
52 | %td.value #{@submission.peak_memory} |
|
|
53 | + - if session[:admin] | ||
|
|
54 | + %tr{class: cycle('info-even','info-odd')} | ||
|
|
55 | + %td.field IP: | ||
|
|
56 | + %td.value #{@submission.ip_address} | ||
|
53 |
|
57 | ||
|
54 | %h2 Source code |
|
58 | %h2 Source code |
|
55 | //%div.highlight{:style => "border: 1px solid black;"} |
|
59 | //%div.highlight{:style => "border: 1px solid black;"} |
|
56 | =@formatted_code.html_safe |
|
60 | =@formatted_code.html_safe |
|
57 |
|
61 |
@@ -28,19 +28,23 | |||||
|
28 | %th ID |
|
28 | %th ID |
|
29 | %th Problem code |
|
29 | %th Problem code |
|
30 | %th Problem name |
|
30 | %th Problem name |
|
31 | %th Language |
|
31 | %th Language |
|
32 | %th Result |
|
32 | %th Result |
|
33 | %th Score |
|
33 | %th Score |
|
|
34 | + - if session[:admin] | ||
|
|
35 | + %th IP | ||
|
34 | %tbody |
|
36 | %tbody |
|
35 | - @submission.each do |s| |
|
37 | - @submission.each do |s| |
|
36 | - next unless s.problem |
|
38 | - next unless s.problem |
|
37 | %tr |
|
39 | %tr |
|
38 | %td= link_to "#{s.id}", controller: "graders", action: "submission", id: s.id |
|
40 | %td= link_to "#{s.id}", controller: "graders", action: "submission", id: s.id |
|
39 | %td= s.problem.name |
|
41 | %td= s.problem.name |
|
40 | %td= s.problem.full_name |
|
42 | %td= s.problem.full_name |
|
41 | %td= s.language.pretty_name |
|
43 | %td= s.language.pretty_name |
|
42 | %td.fix-width= s.grader_comment |
|
44 | %td.fix-width= s.grader_comment |
|
43 | %td= (s.points*100)/s.problem.full_score |
|
45 | %td= (s.points*100)/s.problem.full_score |
|
|
46 | + - if session[:admin] | ||
|
|
47 | + %td= s.ip_address | ||
|
44 |
|
48 | ||
|
45 |
|
49 | ||
|
46 |
|
50 |
@@ -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 to check this file into your version control system. |
|
12 | # It's strongly recommended to check this file into your version control system. |
|
13 |
|
13 | ||
|
14 |
- ActiveRecord::Schema.define(:version => 20140 |
|
14 | + ActiveRecord::Schema.define(:version => 20140917150629) do |
|
15 |
|
15 | ||
|
16 | create_table "announcements", :force => true do |t| |
|
16 | create_table "announcements", :force => true do |t| |
|
17 | t.string "author" |
|
17 | t.string "author" |
|
18 |
- t.text "body" |
|
18 | + t.text "body" |
|
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" |
|
24 | t.string "title" |
@@ -47,25 +47,25 | |||||
|
47 | t.string "name" |
|
47 | t.string "name" |
|
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 => true do |t| |
|
52 | create_table "descriptions", :force => true do |t| |
|
53 |
- t.text "body" |
|
53 | + t.text "body" |
|
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 => true do |t| |
|
59 | create_table "grader_configurations", :force => true do |t| |
|
60 | t.string "key" |
|
60 | t.string "key" |
|
61 | t.string "value_type" |
|
61 | t.string "value_type" |
|
62 | t.string "value" |
|
62 | t.string "value" |
|
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" |
|
65 | + t.text "description" |
|
66 | end |
|
66 | end |
|
67 |
|
67 | ||
|
68 | create_table "grader_processes", :force => true do |t| |
|
68 | create_table "grader_processes", :force => true do |t| |
|
69 | t.string "host", :limit => 20 |
|
69 | t.string "host", :limit => 20 |
|
70 | t.integer "pid" |
|
70 | t.integer "pid" |
|
71 | t.string "mode" |
|
71 | t.string "mode" |
@@ -94,13 +94,13 | |||||
|
94 | end |
|
94 | end |
|
95 |
|
95 | ||
|
96 | create_table "messages", :force => true do |t| |
|
96 | create_table "messages", :force => true do |t| |
|
97 | t.integer "sender_id" |
|
97 | t.integer "sender_id" |
|
98 | t.integer "receiver_id" |
|
98 | t.integer "receiver_id" |
|
99 | t.integer "replying_message_id" |
|
99 | t.integer "replying_message_id" |
|
100 | - t.text "body", :limit => 16777215 |
|
100 | + t.text "body" |
|
101 | t.boolean "replied" |
|
101 | t.boolean "replied" |
|
102 |
t.datetime "created_at", |
|
102 | t.datetime "created_at", :null => false |
|
103 |
t.datetime "updated_at", |
|
103 | t.datetime "updated_at", :null => false |
|
104 | end |
|
104 | end |
|
105 |
|
105 | ||
|
106 | create_table "problems", :force => true do |t| |
|
106 | create_table "problems", :force => true do |t| |
@@ -139,13 +139,13 | |||||
|
139 | end |
|
139 | end |
|
140 |
|
140 | ||
|
141 | add_index "roles_users", ["user_id"], :name => "index_roles_users_on_user_id" |
|
141 | add_index "roles_users", ["user_id"], :name => "index_roles_users_on_user_id" |
|
142 |
|
142 | ||
|
143 | create_table "sessions", :force => true do |t| |
|
143 | create_table "sessions", :force => true do |t| |
|
144 | t.string "session_id" |
|
144 | t.string "session_id" |
|
145 |
- t.text "data" |
|
145 | + t.text "data" |
|
146 | t.datetime "updated_at" |
|
146 | t.datetime "updated_at" |
|
147 | end |
|
147 | end |
|
148 |
|
148 | ||
|
149 | add_index "sessions", ["session_id"], :name => "index_sessions_on_session_id" |
|
149 | add_index "sessions", ["session_id"], :name => "index_sessions_on_session_id" |
|
150 | add_index "sessions", ["updated_at"], :name => "index_sessions_on_updated_at" |
|
150 | add_index "sessions", ["updated_at"], :name => "index_sessions_on_updated_at" |
|
151 |
|
151 | ||
@@ -160,22 +160,26 | |||||
|
160 | end |
|
160 | end |
|
161 |
|
161 | ||
|
162 | create_table "submissions", :force => true do |t| |
|
162 | create_table "submissions", :force => true do |t| |
|
163 | t.integer "user_id" |
|
163 | t.integer "user_id" |
|
164 | t.integer "problem_id" |
|
164 | t.integer "problem_id" |
|
165 | t.integer "language_id" |
|
165 | t.integer "language_id" |
|
166 |
- t.text "source" |
|
166 | + t.text "source" |
|
167 | t.binary "binary" |
|
167 | t.binary "binary" |
|
168 | t.datetime "submitted_at" |
|
168 | t.datetime "submitted_at" |
|
169 | t.datetime "compiled_at" |
|
169 | t.datetime "compiled_at" |
|
170 |
- t.text "compiler_message" |
|
170 | + t.text "compiler_message" |
|
171 | t.datetime "graded_at" |
|
171 | t.datetime "graded_at" |
|
172 | t.integer "points" |
|
172 | t.integer "points" |
|
173 |
- t.text "grader_comment" |
|
173 | + t.text "grader_comment" |
|
174 | t.integer "number" |
|
174 | t.integer "number" |
|
175 | t.string "source_filename" |
|
175 | t.string "source_filename" |
|
|
176 | + t.float "max_runtime" | ||
|
|
177 | + t.integer "peak_memory" | ||
|
|
178 | + t.integer "effective_code_length" | ||
|
|
179 | + t.string "ip_address" | ||
|
176 | end |
|
180 | end |
|
177 |
|
181 | ||
|
178 | add_index "submissions", ["user_id", "problem_id", "number"], :name => "index_submissions_on_user_id_and_problem_id_and_number", :unique => true |
|
182 | add_index "submissions", ["user_id", "problem_id", "number"], :name => "index_submissions_on_user_id_and_problem_id_and_number", :unique => true |
|
179 | add_index "submissions", ["user_id", "problem_id"], :name => "index_submissions_on_user_id_and_problem_id" |
|
183 | add_index "submissions", ["user_id", "problem_id"], :name => "index_submissions_on_user_id_and_problem_id" |
|
180 |
|
184 | ||
|
181 | create_table "tasks", :force => true do |t| |
|
185 | create_table "tasks", :force => true do |t| |
@@ -184,14 +188,14 | |||||
|
184 | t.integer "status" |
|
188 | t.integer "status" |
|
185 | t.datetime "updated_at" |
|
189 | t.datetime "updated_at" |
|
186 | end |
|
190 | end |
|
187 |
|
191 | ||
|
188 | create_table "test_pairs", :force => true do |t| |
|
192 | create_table "test_pairs", :force => true do |t| |
|
189 | t.integer "problem_id" |
|
193 | t.integer "problem_id" |
|
190 |
- t.text "input", :limit => |
|
194 | + t.text "input", :limit => 16777215 |
|
191 |
- t.text "solution", :limit => |
|
195 | + t.text "solution", :limit => 16777215 |
|
192 |
t.datetime "created_at", |
|
196 | t.datetime "created_at", :null => false |
|
193 |
t.datetime "updated_at", |
|
197 | t.datetime "updated_at", :null => false |
|
194 | end |
|
198 | end |
|
195 |
|
199 | ||
|
196 | create_table "test_requests", :force => true do |t| |
|
200 | create_table "test_requests", :force => true do |t| |
|
197 | t.integer "user_id" |
|
201 | t.integer "user_id" |
@@ -201,13 +205,13 | |||||
|
201 | t.string "output_file_name" |
|
205 | t.string "output_file_name" |
|
202 | t.string "running_stat" |
|
206 | t.string "running_stat" |
|
203 | t.integer "status" |
|
207 | t.integer "status" |
|
204 |
t.datetime "updated_at", |
|
208 | t.datetime "updated_at", :null => false |
|
205 | t.datetime "submitted_at" |
|
209 | t.datetime "submitted_at" |
|
206 | t.datetime "compiled_at" |
|
210 | t.datetime "compiled_at" |
|
207 |
- t.text "compiler_message" |
|
211 | + t.text "compiler_message" |
|
208 | t.datetime "graded_at" |
|
212 | t.datetime "graded_at" |
|
209 | t.string "grader_comment" |
|
213 | t.string "grader_comment" |
|
210 |
t.datetime "created_at", |
|
214 | t.datetime "created_at", :null => false |
|
211 | t.float "running_time" |
|
215 | t.float "running_time" |
|
212 | t.string "exit_status" |
|
216 | t.string "exit_status" |
|
213 | t.integer "memory_usage" |
|
217 | t.integer "memory_usage" |
@@ -232,11 +236,12 | |||||
|
232 | t.string "email" |
|
236 | t.string "email" |
|
233 | t.integer "site_id" |
|
237 | t.integer "site_id" |
|
234 | t.integer "country_id" |
|
238 | t.integer "country_id" |
|
235 | t.boolean "activated", :default => false |
|
239 | t.boolean "activated", :default => false |
|
236 | t.datetime "created_at" |
|
240 | t.datetime "created_at" |
|
237 | t.datetime "updated_at" |
|
241 | t.datetime "updated_at" |
|
|
242 | + t.string "section" | ||
|
238 | end |
|
243 | end |
|
239 |
|
244 | ||
|
240 | add_index "users", ["login"], :name => "index_users_on_login", :unique => true |
|
245 | add_index "users", ["login"], :name => "index_users_on_login", :unique => true |
|
241 |
|
246 | ||
|
242 | end |
|
247 | end |
You need to be logged in to leave comments.
Login now