Description:
group enable
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
Author
-
r795:08d968fa05d5 - - 10 files changed: 45 inserted, 8 deleted
| @@ -0,0 +1,8 | |||
|
|
1 | + = render partial: 'toggle_button', | |
|
|
2 | + locals: {button_id: "#group-enabled-#{@group.id}",button_on: @group.enabled } | |
|
|
3 | + :plain | |
|
|
4 | + r = $("#group-#{@group.id}"); | |
|
|
5 | + r.removeClass('success'); | |
|
|
6 | + r.removeClass('danger'); | |
|
|
7 | + r.addClass("#{@group.enabled? ? 'success' : 'danger'}"); | |
|
|
8 | + | |
| @@ -0,0 +1,5 | |||
|
|
1 | + class AddEnabledToGroup < ActiveRecord::Migration[5.2] | |
|
|
2 | + def change | |
|
|
3 | + add_column :groups, :enabled, :boolean, default: true | |
|
|
4 | + end | |
|
|
5 | + end | |
| @@ -1,16 +1,17 | |||
|
|
1 | 1 | class GroupsController < ApplicationController |
|
|
2 | 2 | before_action :set_group, only: [:show, :edit, :update, :destroy, |
|
|
3 | 3 | :add_user, :remove_user,:remove_all_user, |
|
|
4 | 4 | :add_problem, :remove_problem,:remove_all_problem, |
|
|
5 | + :toggle, | |
|
|
5 | 6 | ] |
|
|
6 | 7 | before_action :admin_authorization |
|
|
7 | 8 | |
|
|
8 | 9 | # GET /groups |
|
|
9 | 10 | def index |
|
|
10 | 11 | @groups = Group.all |
|
|
11 | 12 | end |
|
|
12 | 13 | |
|
|
13 | 14 | # GET /groups/1 |
|
|
14 | 15 | def show |
|
|
15 | 16 | end |
|
|
16 | 17 | |
| @@ -40,24 +41,29 | |||
|
|
40 | 41 | redirect_to @group, notice: 'Group was successfully updated.' |
|
|
41 | 42 | else |
|
|
42 | 43 | render :edit |
|
|
43 | 44 | end |
|
|
44 | 45 | end |
|
|
45 | 46 | |
|
|
46 | 47 | # DELETE /groups/1 |
|
|
47 | 48 | def destroy |
|
|
48 | 49 | @group.destroy |
|
|
49 | 50 | redirect_to groups_url, notice: 'Group was successfully destroyed.' |
|
|
50 | 51 | end |
|
|
51 | 52 | |
|
|
53 | + def toggle | |
|
|
54 | + @group.enabled = @group.enabled? ? false : true | |
|
|
55 | + @group.save | |
|
|
56 | + end | |
|
|
57 | + | |
|
|
52 | 58 | def remove_user |
|
|
53 | 59 | user = User.find(params[:user_id]) |
|
|
54 | 60 | @group.users.delete(user) |
|
|
55 | 61 | redirect_to group_path(@group), flash: {success: "User #{user.login} was removed from the group #{@group.name}"} |
|
|
56 | 62 | end |
|
|
57 | 63 | |
|
|
58 | 64 | def remove_all_user |
|
|
59 | 65 | @group.users.clear |
|
|
60 | 66 | redirect_to group_path(@group), alert: 'All users removed' |
|
|
61 | 67 | end |
|
|
62 | 68 | |
|
|
63 | 69 | def remove_all_problem |
| @@ -90,15 +96,15 | |||
|
|
90 | 96 | redirect_to group_path(@group), alert: e.message |
|
|
91 | 97 | end |
|
|
92 | 98 | end |
|
|
93 | 99 | |
|
|
94 | 100 | private |
|
|
95 | 101 | # Use callbacks to share common setup or constraints between actions. |
|
|
96 | 102 | def set_group |
|
|
97 | 103 | @group = Group.find(params[:id]) |
|
|
98 | 104 | end |
|
|
99 | 105 | |
|
|
100 | 106 | # Only allow a trusted parameter "white list" through. |
|
|
101 | 107 | def group_params |
|
|
102 | - params.require(:group).permit(:name, :description) | |
|
|
108 | + params.require(:group).permit(:name, :description, :enabled) | |
|
|
103 | 109 | end |
|
|
104 | 110 | end |
| @@ -104,26 +104,27 | |||
|
|
104 | 104 | end |
|
|
105 | 105 | |
|
|
106 | 106 | def read_textfile(fname,max_size=2048) |
|
|
107 | 107 | begin |
|
|
108 | 108 | File.open(fname).read(max_size) |
|
|
109 | 109 | rescue |
|
|
110 | 110 | nil |
|
|
111 | 111 | end |
|
|
112 | 112 | end |
|
|
113 | 113 | |
|
|
114 | 114 | def toggle_button(on,toggle_url,id, option={}) |
|
|
115 | 115 | btn_size = option[:size] || 'btn-xs' |
|
|
116 | + btn_block = option[:block] || 'btn-block' | |
|
|
116 | 117 | link_to (on ? "Yes" : "No"), toggle_url, |
|
|
117 |
- {class: "btn |
|
|
|
118 | + {class: "btn #{btn_block} #{btn_size} btn-#{on ? 'success' : 'default'} ajax-toggle", | |
|
|
118 | 119 | id: id, |
|
|
119 | 120 | data: {remote: true, method: 'get'}} |
|
|
120 | 121 | end |
|
|
121 | 122 | |
|
|
122 | 123 | def get_ace_mode(language) |
|
|
123 | 124 | # return ace mode string from Language |
|
|
124 | 125 | |
|
|
125 | 126 | case language.pretty_name |
|
|
126 | 127 | when 'Pascal' |
|
|
127 | 128 | 'ace/mode/pascal' |
|
|
128 | 129 | when 'C++','C' |
|
|
129 | 130 | 'ace/mode/c_cpp' |
| @@ -264,47 +264,50 | |||
|
|
264 | 264 | contest.problems.available.each do |problem| |
|
|
265 | 265 | if not pin.has_key? problem.id |
|
|
266 | 266 | contest_problems << problem |
|
|
267 | 267 | end |
|
|
268 | 268 | pin[problem.id] = true |
|
|
269 | 269 | end |
|
|
270 | 270 | end |
|
|
271 | 271 | other_avaiable_problems = Problem.available.find_all {|p| pin[p.id]==nil and p.contests.length==0} |
|
|
272 | 272 | return contest_problems + other_avaiable_problems |
|
|
273 | 273 | end |
|
|
274 | 274 | end |
|
|
275 | 275 | |
|
|
276 | + # new feature, get list of available problem in all enabled group that the user belongs to | |
|
|
276 | 277 | def available_problems_in_group |
|
|
277 | 278 | problem = [] |
|
|
278 | - self.groups.each do |group| | |
|
|
279 | + self.groups.where(enabled: true).each do |group| | |
|
|
279 | 280 | group.problems.where(available: true).each { |p| problem << p } |
|
|
280 | 281 | end |
|
|
281 | 282 | problem.uniq! |
|
|
282 | 283 | if problem |
|
|
283 | 284 | problem.sort! do |a,b| |
|
|
284 | 285 | case |
|
|
285 | 286 | when a.date_added < b.date_added |
|
|
286 | 287 | 1 |
|
|
287 | 288 | when a.date_added > b.date_added |
|
|
288 | 289 | -1 |
|
|
289 | 290 | else |
|
|
290 | 291 | a.name <=> b.name |
|
|
291 | 292 | end |
|
|
292 | 293 | end |
|
|
293 | 294 | return problem |
|
|
294 | 295 | else |
|
|
295 | 296 | return [] |
|
|
296 | 297 | end |
|
|
297 | 298 | end |
|
|
298 | 299 | |
|
|
300 | + #check if the user has the right to view that problem | |
|
|
301 | + #this also consider group based problem policy | |
|
|
299 | 302 | def can_view_problem?(problem) |
|
|
300 | 303 | return true if admin? |
|
|
301 | 304 | return available_problems.include? problem |
|
|
302 | 305 | end |
|
|
303 | 306 | |
|
|
304 | 307 | def self.clear_last_login |
|
|
305 | 308 | User.update_all(:last_ip => nil) |
|
|
306 | 309 | end |
|
|
307 | 310 | |
|
|
308 | 311 | protected |
|
|
309 | 312 | def encrypt_new_password |
|
|
310 | 313 | return if password.blank? |
| @@ -1,25 +1,25 | |||
|
|
1 | 1 | |
|
|
2 | 2 | %tr |
|
|
3 | 3 | %td{:align => "center"} |
|
|
4 | 4 | = submission.number |
|
|
5 | 5 | %td.text-right |
|
|
6 | 6 | = link_to "##{submission.id}", submission_path(submission.id) |
|
|
7 | 7 | %td |
|
|
8 | 8 | = l submission.submitted_at, format: :long |
|
|
9 | 9 | = "( #{time_ago_in_words(submission.submitted_at)} ago)" |
|
|
10 | 10 | %td |
|
|
11 | 11 | = submission.source_filename |
|
|
12 | 12 | = " (#{submission.language.pretty_name}) " |
|
|
13 | - = link_to('[load]',{:action => 'source', :id => submission.id}) | |
|
|
13 | + = link_to '[load]', download_submission_path(submission) | |
|
|
14 | 14 | %td |
|
|
15 | 15 | - if submission.graded_at |
|
|
16 | 16 | = "Graded at #{format_short_time(submission.graded_at)}." |
|
|
17 | 17 | %br/ |
|
|
18 | 18 | = "Score: #{(submission.points*100/submission.problem.full_score).to_i} " if GraderConfiguration['ui.show_score'] |
|
|
19 | 19 | = " [" |
|
|
20 | 20 | %tt |
|
|
21 | 21 | = submission.grader_comment |
|
|
22 | 22 | = "]" |
|
|
23 | 23 | %td |
|
|
24 | 24 | = render :partial => 'compiler_message', :locals => {:compiler_message => submission.compiler_message } |
|
|
25 | 25 | %td |
| @@ -1,16 +1,27 | |||
|
|
1 | 1 | = form_for @group do |f| |
|
|
2 | 2 | - if @group.errors.any? |
|
|
3 | 3 | #error_explanation |
|
|
4 | 4 | %h2= "#{pluralize(@group.errors.count, "error")} prohibited this group from being saved:" |
|
|
5 | 5 | %ul |
|
|
6 | 6 | - @group.errors.full_messages.each do |msg| |
|
|
7 | 7 | %li= msg |
|
|
8 | - | |
|
|
8 | + .row | |
|
|
9 | + .col-md-6 | |
|
|
9 | 10 | .form-group.field |
|
|
10 | 11 | = f.label :name |
|
|
11 | 12 | = f.text_field :name, class: 'form-control' |
|
|
13 | + .row | |
|
|
14 | + .col-md-6 | |
|
|
12 | 15 | .form-group.field |
|
|
13 | 16 | = f.label :description |
|
|
14 | 17 | = f.text_field :description, class: 'form-control' |
|
|
18 | + .row | |
|
|
19 | + .col-md-6 | |
|
|
20 | + .checkbox | |
|
|
21 | + = f.label :enabled do | |
|
|
22 | + = f.check_box :enabled | |
|
|
23 | + Enabled | |
|
|
24 | + .row | |
|
|
25 | + .col-md-6 | |
|
|
15 | 26 | .form-group.actions |
|
|
16 | 27 | = f.submit 'Save', class: 'btn btn-primary' |
| @@ -1,22 +1,24 | |||
|
|
1 | 1 | %h1 Groups |
|
|
2 | 2 | |
|
|
3 | 3 | %p |
|
|
4 | 4 | = link_to 'New Group', new_group_path, class: 'btn btn-primary' |
|
|
5 | 5 | %table.table.table-hover |
|
|
6 | 6 | %thead |
|
|
7 | 7 | %tr |
|
|
8 | 8 | %th Name |
|
|
9 | 9 | %th Description |
|
|
10 | + %th Enabled? | |
|
|
10 | 11 | %th |
|
|
11 | 12 | %th |
|
|
12 | 13 | |
|
|
13 | 14 | %tbody |
|
|
14 | 15 | - @groups.each do |group| |
|
|
15 | - %tr | |
|
|
16 | + %tr{:class => "#{(group.enabled?) ? "success" : "danger"}", id: "group-#{group.id}"} | |
|
|
16 | 17 | %td= group.name |
|
|
17 | 18 | %td= group.description |
|
|
19 | + %td= toggle_button(group.enabled?, toggle_group_path(group), "group-enabled-#{group.id}", size: ' ', block: ' ') | |
|
|
18 | 20 | %td= link_to 'View', group, class: 'btn btn-default' |
|
|
19 | 21 | %td= link_to 'Destroy', group, :method => :delete, :data => { :confirm => 'Are you sure?' }, class: 'btn btn-danger' |
|
|
20 | 22 | |
|
|
21 | 23 | %br |
|
|
22 | 24 | |
| @@ -45,24 +45,25 | |||
|
|
45 | 45 | post 'do_import' |
|
|
46 | 46 | end |
|
|
47 | 47 | end |
|
|
48 | 48 | |
|
|
49 | 49 | resources :groups do |
|
|
50 | 50 | member do |
|
|
51 | 51 | post 'add_user', to: 'groups#add_user', as: 'add_user' |
|
|
52 | 52 | delete 'remove_user/:user_id', to: 'groups#remove_user', as: 'remove_user' |
|
|
53 | 53 | delete 'remove_all_user', to: 'groups#remove_all_user', as: 'remove_all_user' |
|
|
54 | 54 | post 'add_problem', to: 'groups#add_problem', as: 'add_problem' |
|
|
55 | 55 | delete 'remove_problem/:problem_id', to: 'groups#remove_problem', as: 'remove_problem' |
|
|
56 | 56 | delete 'remove_all_problem', to: 'groups#remove_all_problem', as: 'remove_all_problem' |
|
|
57 | + get 'toggle' | |
|
|
57 | 58 | end |
|
|
58 | 59 | collection do |
|
|
59 | 60 | |
|
|
60 | 61 | end |
|
|
61 | 62 | end |
|
|
62 | 63 | |
|
|
63 | 64 | resources :testcases, only: [] do |
|
|
64 | 65 | member do |
|
|
65 | 66 | get 'download_input' |
|
|
66 | 67 | get 'download_sol' |
|
|
67 | 68 | end |
|
|
68 | 69 | collection do |
| @@ -83,25 +84,24 | |||
|
|
83 | 84 | end |
|
|
84 | 85 | collection do |
|
|
85 | 86 | get 'profile' |
|
|
86 | 87 | post 'chg_passwd' |
|
|
87 | 88 | end |
|
|
88 | 89 | end |
|
|
89 | 90 | |
|
|
90 | 91 | resources :submissions do |
|
|
91 | 92 | member do |
|
|
92 | 93 | get 'download' |
|
|
93 | 94 | get 'compiler_msg' |
|
|
94 | 95 | get 'rejudge' |
|
|
95 | - get 'source' | |
|
|
96 | 96 | end |
|
|
97 | 97 | collection do |
|
|
98 | 98 | get 'prob/:problem_id', to: 'submissions#index', as: 'problem' |
|
|
99 | 99 | get 'direct_edit_problem/:problem_id(/:user_id)', to: 'submissions#direct_edit_problem', as: 'direct_edit_problem' |
|
|
100 | 100 | get 'get_latest_submission_status/:uid/:pid', to: 'submissions#get_latest_submission_status', as: 'get_latest_submission_status' |
|
|
101 | 101 | end |
|
|
102 | 102 | end |
|
|
103 | 103 | |
|
|
104 | 104 | |
|
|
105 | 105 | #user admin |
|
|
106 | 106 | resources :user_admin do |
|
|
107 | 107 | collection do |
| @@ -1,25 +1,25 | |||
|
|
1 | 1 | # This file is auto-generated from the current state of the database. Instead |
|
|
2 | 2 | # of editing this file, please use the migrations feature of Active Record to |
|
|
3 | 3 | # incrementally modify your database, and then regenerate this schema definition. |
|
|
4 | 4 | # |
|
|
5 | 5 | # Note that this schema.rb definition is the authoritative source for your |
|
|
6 | 6 | # database schema. If you need to create the application database on another |
|
|
7 | 7 | # system, you should be using db:schema:load, not running all the migrations |
|
|
8 | 8 | # from scratch. The latter is a flawed and unsustainable approach (the more migrations |
|
|
9 | 9 | # you'll amass, the slower it'll run and the greater likelihood for issues). |
|
|
10 | 10 | # |
|
|
11 | 11 | # It's strongly recommended that you check this file into your version control system. |
|
|
12 | 12 | |
|
|
13 |
- ActiveRecord::Schema.define(version: 20 |
|
|
|
13 | + ActiveRecord::Schema.define(version: 2020_08_13_083020) do | |
|
|
14 | 14 | |
|
|
15 | 15 | create_table "announcements", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=latin1", force: :cascade do |t| |
|
|
16 | 16 | t.string "author" |
|
|
17 | 17 | t.text "body" |
|
|
18 | 18 | t.boolean "published" |
|
|
19 | 19 | t.datetime "created_at" |
|
|
20 | 20 | t.datetime "updated_at" |
|
|
21 | 21 | t.boolean "frontpage", default: false |
|
|
22 | 22 | t.boolean "contest_only", default: false |
|
|
23 | 23 | t.string "title" |
|
|
24 | 24 | t.string "notes" |
|
|
25 | 25 | end |
| @@ -71,24 +71,25 | |||
|
|
71 | 71 | t.boolean "active" |
|
|
72 | 72 | t.datetime "created_at" |
|
|
73 | 73 | t.datetime "updated_at" |
|
|
74 | 74 | t.integer "task_id" |
|
|
75 | 75 | t.string "task_type" |
|
|
76 | 76 | t.boolean "terminated" |
|
|
77 | 77 | t.index ["host", "pid"], name: "index_grader_processes_on_host_and_pid" |
|
|
78 | 78 | end |
|
|
79 | 79 | |
|
|
80 | 80 | create_table "groups", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=latin1", force: :cascade do |t| |
|
|
81 | 81 | t.string "name" |
|
|
82 | 82 | t.string "description" |
|
|
83 | + t.boolean "enabled", default: true | |
|
|
83 | 84 | end |
|
|
84 | 85 | |
|
|
85 | 86 | create_table "groups_problems", id: false, options: "ENGINE=InnoDB DEFAULT CHARSET=latin1", force: :cascade do |t| |
|
|
86 | 87 | t.integer "problem_id", null: false |
|
|
87 | 88 | t.integer "group_id", null: false |
|
|
88 | 89 | t.index ["group_id", "problem_id"], name: "index_groups_problems_on_group_id_and_problem_id" |
|
|
89 | 90 | end |
|
|
90 | 91 | |
|
|
91 | 92 | create_table "groups_users", id: false, options: "ENGINE=InnoDB DEFAULT CHARSET=latin1", force: :cascade do |t| |
|
|
92 | 93 | t.integer "group_id", null: false |
|
|
93 | 94 | t.integer "user_id", null: false |
|
|
94 | 95 | t.index ["user_id", "group_id"], name: "index_groups_users_on_user_id_and_group_id" |
You need to be logged in to leave comments.
Login now