Description:
heartbeat response full
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r649:3cb38436f7f6 - - 6 files changed: 41 inserted, 3 deleted
@@ -0,0 +1,9 | |||||
|
|
1 | + class AddHeartBeatFull < ActiveRecord::Migration | ||
|
|
2 | + def up | ||
|
|
3 | + GraderConfiguration.create key: 'right.heartbeat_response_full', value_type: 'string', value:'RESTART', description:'Heart beat response text when user got full score (set this value to the empty string to disable this feature)' | ||
|
|
4 | + end | ||
|
|
5 | + | ||
|
|
6 | + def down | ||
|
|
7 | + | ||
|
|
8 | + end | ||
|
|
9 | + end |
@@ -12,20 +12,35 | |||||
|
12 | #puts "status = #{params[:status]}" |
|
12 | #puts "status = #{params[:status]}" |
|
13 | #if hb |
|
13 | #if hb |
|
14 | # if params[:status] |
|
14 | # if params[:status] |
|
15 | # hb.status = params[:status] |
|
15 | # hb.status = params[:status] |
|
16 | # hb.save |
|
16 | # hb.save |
|
17 | # end |
|
17 | # end |
|
18 | # hb.touch |
|
18 | # hb.touch |
|
19 | #else |
|
19 | #else |
|
20 | # HeartBeat.creae(user_id: @user.id, ip_address: request.remote_ip) |
|
20 | # HeartBeat.creae(user_id: @user.id, ip_address: request.remote_ip) |
|
21 | #end |
|
21 | #end |
|
22 | #HeartBeat.create(user_id: @user.id, ip_address: request.remote_ip, status: params[:status]) |
|
22 | #HeartBeat.create(user_id: @user.id, ip_address: request.remote_ip, status: params[:status]) |
|
23 |
|
23 | ||
|
|
24 | + res = GraderConfiguration['right.heartbeat_response'] | ||
|
|
25 | + res.strip! if res | ||
|
|
26 | + full = GraderConfiguration['right.heartbeat_response_full'] | ||
|
|
27 | + full.strip! if full | ||
|
|
28 | + | ||
|
|
29 | + if full and full != '' | ||
|
|
30 | + l = Login.where(ip_address: request.remote_ip).last | ||
|
|
31 | + @user = l.user | ||
|
|
32 | + if @user.solve_all_available_problems? | ||
|
|
33 | + render text: (full || 'OK') | ||
|
|
34 | + else | ||
|
|
35 | + render text: (res || 'OK') | ||
|
|
36 | + end | ||
|
|
37 | + else | ||
|
24 | render text: (GraderConfiguration['right.heartbeat_response'] || 'OK') |
|
38 | render text: (GraderConfiguration['right.heartbeat_response'] || 'OK') |
|
25 | end |
|
39 | end |
|
|
40 | + end | ||
|
26 |
|
41 | ||
|
27 | def index |
|
42 | def index |
|
28 | @hb = HeartBeat.where("updated_at >= ?",Time.zone.now-2.hours).includes(:user).order(:user_id).all |
|
43 | @hb = HeartBeat.where("updated_at >= ?",Time.zone.now-2.hours).includes(:user).order(:user_id).all |
|
29 | @num = HeartBeat.where("updated_at >= ?",Time.zone.now-5.minutes).count(:user_id,distinct: true) |
|
44 | @num = HeartBeat.where("updated_at >= ?",Time.zone.now-5.minutes).count(:user_id,distinct: true) |
|
30 | end |
|
45 | end |
|
31 | end |
|
46 | end |
@@ -270,24 +270,33 | |||||
|
270 | :problems => available_problems |
|
270 | :problems => available_problems |
|
271 | } |
|
271 | } |
|
272 | available_problems.each {|p| pin[p.id] = true} |
|
272 | available_problems.each {|p| pin[p.id] = true} |
|
273 | end |
|
273 | end |
|
274 | other_avaiable_problems = Problem.available.find_all {|p| pin[p.id]==nil and p.contests.length==0} |
|
274 | other_avaiable_problems = Problem.available.find_all {|p| pin[p.id]==nil and p.contests.length==0} |
|
275 | contest_problems << { |
|
275 | contest_problems << { |
|
276 | :contest => nil, |
|
276 | :contest => nil, |
|
277 | :problems => other_avaiable_problems |
|
277 | :problems => other_avaiable_problems |
|
278 | } |
|
278 | } |
|
279 | return contest_problems |
|
279 | return contest_problems |
|
280 | end |
|
280 | end |
|
281 |
|
281 | ||
|
|
282 | + def solve_all_available_problems? | ||
|
|
283 | + available_problems.each do |p| | ||
|
|
284 | + u = self | ||
|
|
285 | + sub = Submission.find_last_by_user_and_problem(u.id,p.id) | ||
|
|
286 | + return false if !p or !sub or sub.points < p.full_score | ||
|
|
287 | + end | ||
|
|
288 | + return true | ||
|
|
289 | + end | ||
|
|
290 | + | ||
|
282 | def available_problems |
|
291 | def available_problems |
|
283 | if not GraderConfiguration.multicontests? |
|
292 | if not GraderConfiguration.multicontests? |
|
284 | return Problem.available_problems |
|
293 | return Problem.available_problems |
|
285 | else |
|
294 | else |
|
286 | contest_problems = [] |
|
295 | contest_problems = [] |
|
287 | pin = {} |
|
296 | pin = {} |
|
288 | contests.enabled.each do |contest| |
|
297 | contests.enabled.each do |contest| |
|
289 | contest.problems.available.each do |problem| |
|
298 | contest.problems.available.each do |problem| |
|
290 | if not pin.has_key? problem.id |
|
299 | if not pin.has_key? problem.id |
|
291 | contest_problems << problem |
|
300 | contest_problems << problem |
|
292 | end |
|
301 | end |
|
293 | pin[problem.id] = true |
|
302 | pin[problem.id] = true |
@@ -81,20 +81,18 | |||||
|
81 | post 'report/show_max_score', to: 'report#show_max_score', as: 'report_show_max_score' |
|
81 | post 'report/show_max_score', to: 'report#show_max_score', as: 'report_show_max_score' |
|
82 |
|
82 | ||
|
83 |
|
83 | ||
|
84 | # |
|
84 | # |
|
85 | get 'tasks/view/:file.:ext' => 'tasks#view' |
|
85 | get 'tasks/view/:file.:ext' => 'tasks#view' |
|
86 | get 'tasks/download/:id/:file.:ext' => 'tasks#download' |
|
86 | get 'tasks/download/:id/:file.:ext' => 'tasks#download' |
|
87 | get 'heartbeat/:id/edit' => 'heartbeat#edit' |
|
87 | get 'heartbeat/:id/edit' => 'heartbeat#edit' |
|
88 |
|
88 | ||
|
89 | #grader |
|
89 | #grader |
|
90 | get 'graders/list', to: 'graders#list', as: 'grader_list' |
|
90 | get 'graders/list', to: 'graders#list', as: 'grader_list' |
|
91 |
|
91 | ||
|
92 |
|
92 | ||
|
93 | - get 'heartbeat/:id/edit' => 'heartbeat#edit' |
|
||
|
94 | - |
|
||
|
95 | # See how all your routes lay out with "rake routes" |
|
93 | # See how all your routes lay out with "rake routes" |
|
96 |
|
94 | ||
|
97 | # This is a legacy wild controller route that's not recommended for RESTful applications. |
|
95 | # This is a legacy wild controller route that's not recommended for RESTful applications. |
|
98 | # Note: This route will make all actions in every controller accessible via GET requests. |
|
96 | # Note: This route will make all actions in every controller accessible via GET requests. |
|
99 | match ':controller(/:action(/:id))(.:format)', via: [:get, :post] |
|
97 | match ':controller(/:action(/:id))(.:format)', via: [:get, :post] |
|
100 | end |
|
98 | end |
@@ -2,25 +2,25 | |||||
|
2 | # This file is auto-generated from the current state of the database. Instead |
|
2 | # This file is auto-generated from the current state of the database. Instead |
|
3 | # of editing this file, please use the migrations feature of Active Record to |
|
3 | # of editing this file, please use the migrations feature of Active Record to |
|
4 | # incrementally modify your database, and then regenerate this schema definition. |
|
4 | # incrementally modify your database, and then regenerate this schema definition. |
|
5 | # |
|
5 | # |
|
6 | # Note that this schema.rb definition is the authoritative source for your |
|
6 | # Note that this schema.rb definition is the authoritative source for your |
|
7 | # database schema. If you need to create the application database on another |
|
7 | # database schema. If you need to create the application database on another |
|
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: 20170427070345) 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: 65535 |
|
18 | t.text "body", limit: 65535 |
|
19 | t.boolean "published" |
|
19 | t.boolean "published" |
|
20 | t.datetime "created_at", null: false |
|
20 | t.datetime "created_at", null: false |
|
21 | t.datetime "updated_at", null: false |
|
21 | t.datetime "updated_at", null: false |
|
22 | t.boolean "frontpage", default: false |
|
22 | t.boolean "frontpage", default: false |
|
23 | t.boolean "contest_only", default: false |
|
23 | t.boolean "contest_only", default: false |
|
24 | t.string "title", limit: 255 |
|
24 | t.string "title", limit: 255 |
|
25 | t.string "notes", limit: 255 |
|
25 | t.string "notes", limit: 255 |
|
26 | end |
|
26 | end |
@@ -81,24 +81,31 | |||||
|
81 | :default_value => 'true', |
|
81 | :default_value => 'true', |
|
82 | :description => 'When false, a user must accept usage agreement before login' |
|
82 | :description => 'When false, a user must accept usage agreement before login' |
|
83 | }, |
|
83 | }, |
|
84 |
|
84 | ||
|
85 | { |
|
85 | { |
|
86 | :key => 'right.heartbeat_response', |
|
86 | :key => 'right.heartbeat_response', |
|
87 | :value_type => 'string', |
|
87 | :value_type => 'string', |
|
88 | :default_value => 'OK', |
|
88 | :default_value => 'OK', |
|
89 | :description => 'Heart beat response text' |
|
89 | :description => 'Heart beat response text' |
|
90 | }, |
|
90 | }, |
|
91 |
|
91 | ||
|
92 | { |
|
92 | { |
|
|
93 | + :key => 'right.heartbeat_response_full', | ||
|
|
94 | + :value_type => 'string', | ||
|
|
95 | + :default_value => 'OK', | ||
|
|
96 | + :description => 'Heart beat response text when user got full score (set this value to the empty string to disable this feature)' | ||
|
|
97 | + }, | ||
|
|
98 | + | ||
|
|
99 | + { | ||
|
93 | :key => 'right.view_testcase', |
|
100 | :key => 'right.view_testcase', |
|
94 | :value_type => 'boolean', |
|
101 | :value_type => 'boolean', |
|
95 | :default_value => 'false', |
|
102 | :default_value => 'false', |
|
96 | :description => 'When true, any user can view/download test data' |
|
103 | :description => 'When true, any user can view/download test data' |
|
97 | }, |
|
104 | }, |
|
98 | # If Configuration['system.online_registration'] is true, the |
|
105 | # If Configuration['system.online_registration'] is true, the |
|
99 | # system allows online registration, and will use these |
|
106 | # system allows online registration, and will use these |
|
100 | # information for sending confirmation emails. |
|
107 | # information for sending confirmation emails. |
|
101 | { |
|
108 | { |
|
102 | :key => 'system.online_registration.smtp', |
|
109 | :key => 'system.online_registration.smtp', |
|
103 | :value_type => 'string', |
|
110 | :value_type => 'string', |
|
104 | :default_value => 'smtp.somehost.com' |
|
111 | :default_value => 'smtp.somehost.com' |
You need to be logged in to leave comments.
Login now