Description:
add ip to submission
Commit status:
[Not Reviewed]
References:
Comments:
0 Commit comments 0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
Add another comment

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
@@ -58,24 +58,25
58 58 def submit
59 59 user = User.find(session[:user_id])
60 60
61 61 @submission = Submission.new
62 62 @submission.problem_id = params[:submission][:problem_id]
63 63 @submission.user = user
64 64 @submission.language_id = 0
65 65 if (params['file']) and (params['file']!='')
66 66 @submission.source = params['file'].read
67 67 @submission.source_filename = params['file'].original_filename
68 68 end
69 69 @submission.submitted_at = Time.new.gmtime
70 + @submission.ip_address = request.remote_ip
70 71
71 72 if GraderConfiguration.time_limit_mode? and user.contest_finished?
72 73 @submission.errors.add_to_base "The contest is over."
73 74 prepare_list_information
74 75 render :action => 'list' and return
75 76 end
76 77
77 78 if @submission.valid?
78 79 if @submission.save == false
79 80 flash[:notice] = 'Error saving your submission'
80 81 elsif Task.create(:submission_id => @submission.id,
81 82 :status => Task::STATUS_INQUEUE) == false
@@ -41,17 +41,21
41 41 %tr{class: cycle('info-even','info-odd')}
42 42 %td.field Points:
43 43 %td.value #{@submission.points}/#{@submission.problem.full_score}
44 44 %tr{class: cycle('info-even','info-odd')}
45 45 %td.field Comment:
46 46 %td.value #{@submission.grader_comment}
47 47 %tr{class: cycle('info-even','info-odd')}
48 48 %td.field Runtime (s):
49 49 %td.value #{@submission.max_runtime}
50 50 %tr{class: cycle('info-even','info-odd')}
51 51 %td.field Memory (kb):
52 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 58 %h2 Source code
55 59 //%div.highlight{:style => "border: 1px solid black;"}
56 60 =@formatted_code.html_safe
57 61
@@ -22,25 +22,29
22 22
23 23 %h2 Submissions
24 24
25 25 %table.tablesorter-cafe#submission_table
26 26 %thead
27 27 %tr
28 28 %th ID
29 29 %th Problem code
30 30 %th Problem name
31 31 %th Language
32 32 %th Result
33 33 %th Score
34 + - if session[:admin]
35 + %th IP
34 36 %tbody
35 37 - @submission.each do |s|
36 38 - next unless s.problem
37 39 %tr
38 40 %td= link_to "#{s.id}", controller: "graders", action: "submission", id: s.id
39 41 %td= s.problem.name
40 42 %td= s.problem.full_name
41 43 %td= s.language.pretty_name
42 44 %td.fix-width= s.grader_comment
43 45 %td= (s.points*100)/s.problem.full_score
46 + - if session[:admin]
47 + %td= s.ip_address
44 48
45 49
46 50
@@ -2,29 +2,29
2 2 # This file is auto-generated from the current state of the database. Instead
3 3 # of editing this file, please use the migrations feature of Active Record to
4 4 # incrementally modify your database, and then regenerate this schema definition.
5 5 #
6 6 # Note that this schema.rb definition is the authoritative source for your
7 7 # database schema. If you need to create the application database on another
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 to check this file into your version control system.
13 13
14 - ActiveRecord::Schema.define(:version => 20140826095949) do
14 + ActiveRecord::Schema.define(:version => 20140917150629) do
15 15
16 16 create_table "announcements", :force => true do |t|
17 17 t.string "author"
18 - t.text "body", :limit => 16777215
18 + t.text "body"
19 19 t.boolean "published"
20 20 t.datetime "created_at", :null => false
21 21 t.datetime "updated_at", :null => false
22 22 t.boolean "frontpage", :default => false
23 23 t.boolean "contest_only", :default => false
24 24 t.string "title"
25 25 t.string "notes"
26 26 end
27 27
28 28 create_table "contests", :force => true do |t|
29 29 t.string "title"
30 30 t.boolean "enabled"
@@ -41,37 +41,37
41 41 create_table "contests_users", :id => false, :force => true do |t|
42 42 t.integer "contest_id"
43 43 t.integer "user_id"
44 44 end
45 45
46 46 create_table "countries", :force => true do |t|
47 47 t.string "name"
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 => true do |t|
53 - t.text "body", :limit => 16777215
53 + t.text "body"
54 54 t.boolean "markdowned"
55 55 t.datetime "created_at", :null => false
56 56 t.datetime "updated_at", :null => false
57 57 end
58 58
59 59 create_table "grader_configurations", :force => true do |t|
60 60 t.string "key"
61 61 t.string "value_type"
62 62 t.string "value"
63 63 t.datetime "created_at", :null => false
64 64 t.datetime "updated_at", :null => false
65 - t.text "description", :limit => 16777215
65 + t.text "description"
66 66 end
67 67
68 68 create_table "grader_processes", :force => true do |t|
69 69 t.string "host", :limit => 20
70 70 t.integer "pid"
71 71 t.string "mode"
72 72 t.boolean "active"
73 73 t.datetime "created_at", :null => false
74 74 t.datetime "updated_at", :null => false
75 75 t.integer "task_id"
76 76 t.string "task_type"
77 77 t.boolean "terminated"
@@ -88,25 +88,25
88 88
89 89 create_table "logins", :force => true do |t|
90 90 t.string "user_id"
91 91 t.string "ip_address"
92 92 t.datetime "created_at", :null => false
93 93 t.datetime "updated_at", :null => false
94 94 end
95 95
96 96 create_table "messages", :force => true do |t|
97 97 t.integer "sender_id"
98 98 t.integer "receiver_id"
99 99 t.integer "replying_message_id"
100 - t.text "body", :limit => 16777215
100 + t.text "body"
101 101 t.boolean "replied"
102 102 t.datetime "created_at", :null => false
103 103 t.datetime "updated_at", :null => false
104 104 end
105 105
106 106 create_table "problems", :force => true do |t|
107 107 t.string "name", :limit => 30
108 108 t.string "full_name"
109 109 t.integer "full_score"
110 110 t.date "date_added"
111 111 t.boolean "available"
112 112 t.string "url"
@@ -133,87 +133,91
133 133 t.string "name"
134 134 end
135 135
136 136 create_table "roles_users", :id => false, :force => true do |t|
137 137 t.integer "role_id"
138 138 t.integer "user_id"
139 139 end
140 140
141 141 add_index "roles_users", ["user_id"], :name => "index_roles_users_on_user_id"
142 142
143 143 create_table "sessions", :force => true do |t|
144 144 t.string "session_id"
145 - t.text "data", :limit => 16777215
145 + t.text "data"
146 146 t.datetime "updated_at"
147 147 end
148 148
149 149 add_index "sessions", ["session_id"], :name => "index_sessions_on_session_id"
150 150 add_index "sessions", ["updated_at"], :name => "index_sessions_on_updated_at"
151 151
152 152 create_table "sites", :force => true do |t|
153 153 t.string "name"
154 154 t.boolean "started"
155 155 t.datetime "start_time"
156 156 t.datetime "created_at", :null => false
157 157 t.datetime "updated_at", :null => false
158 158 t.integer "country_id"
159 159 t.string "password"
160 160 end
161 161
162 162 create_table "submissions", :force => true do |t|
163 163 t.integer "user_id"
164 164 t.integer "problem_id"
165 165 t.integer "language_id"
166 - t.text "source", :limit => 16777215
166 + t.text "source"
167 167 t.binary "binary"
168 168 t.datetime "submitted_at"
169 169 t.datetime "compiled_at"
170 - t.text "compiler_message", :limit => 16777215
170 + t.text "compiler_message"
171 171 t.datetime "graded_at"
172 172 t.integer "points"
173 - t.text "grader_comment", :limit => 16777215
173 + t.text "grader_comment"
174 174 t.integer "number"
175 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 180 end
177 181
178 182 add_index "submissions", ["user_id", "problem_id", "number"], :name => "index_submissions_on_user_id_and_problem_id_and_number", :unique => true
179 183 add_index "submissions", ["user_id", "problem_id"], :name => "index_submissions_on_user_id_and_problem_id"
180 184
181 185 create_table "tasks", :force => true do |t|
182 186 t.integer "submission_id"
183 187 t.datetime "created_at"
184 188 t.integer "status"
185 189 t.datetime "updated_at"
186 190 end
187 191
188 192 create_table "test_pairs", :force => true do |t|
189 193 t.integer "problem_id"
190 - t.text "input", :limit => 2147483647
191 - t.text "solution", :limit => 2147483647
194 + t.text "input", :limit => 16777215
195 + t.text "solution", :limit => 16777215
192 196 t.datetime "created_at", :null => false
193 197 t.datetime "updated_at", :null => false
194 198 end
195 199
196 200 create_table "test_requests", :force => true do |t|
197 201 t.integer "user_id"
198 202 t.integer "problem_id"
199 203 t.integer "submission_id"
200 204 t.string "input_file_name"
201 205 t.string "output_file_name"
202 206 t.string "running_stat"
203 207 t.integer "status"
204 208 t.datetime "updated_at", :null => false
205 209 t.datetime "submitted_at"
206 210 t.datetime "compiled_at"
207 - t.text "compiler_message", :limit => 16777215
211 + t.text "compiler_message"
208 212 t.datetime "graded_at"
209 213 t.string "grader_comment"
210 214 t.datetime "created_at", :null => false
211 215 t.float "running_time"
212 216 t.string "exit_status"
213 217 t.integer "memory_usage"
214 218 end
215 219
216 220 add_index "test_requests", ["user_id", "problem_id"], :name => "index_test_requests_on_user_id_and_problem_id"
217 221
218 222 create_table "user_contest_stats", :force => true do |t|
219 223 t.integer "user_id"
@@ -226,17 +230,18
226 230 create_table "users", :force => true do |t|
227 231 t.string "login", :limit => 50
228 232 t.string "full_name"
229 233 t.string "hashed_password"
230 234 t.string "salt", :limit => 5
231 235 t.string "alias"
232 236 t.string "email"
233 237 t.integer "site_id"
234 238 t.integer "country_id"
235 239 t.boolean "activated", :default => false
236 240 t.datetime "created_at"
237 241 t.datetime "updated_at"
242 + t.string "section"
238 243 end
239 244
240 245 add_index "users", ["login"], :name => "index_users_on_login", :unique => true
241 246
242 247 end
You need to be logged in to leave comments. Login now