Description:
merged
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r685:101d67319b73 - - 43 files changed: 525 inserted, 159 deleted
@@ -0,0 +1,86 | |||
|
1 | + class GroupsController < ApplicationController | |
|
2 | + before_action :set_group, only: [:show, :edit, :update, :destroy, | |
|
3 | + :add_user, :remove_user, | |
|
4 | + :add_problem, :remove_problem, | |
|
5 | + ] | |
|
6 | + before_action :authenticate, :admin_authorization | |
|
7 | + | |
|
8 | + # GET /groups | |
|
9 | + def index | |
|
10 | + @groups = Group.all | |
|
11 | + end | |
|
12 | + | |
|
13 | + # GET /groups/1 | |
|
14 | + def show | |
|
15 | + end | |
|
16 | + | |
|
17 | + # GET /groups/new | |
|
18 | + def new | |
|
19 | + @group = Group.new | |
|
20 | + end | |
|
21 | + | |
|
22 | + # GET /groups/1/edit | |
|
23 | + def edit | |
|
24 | + end | |
|
25 | + | |
|
26 | + # POST /groups | |
|
27 | + def create | |
|
28 | + @group = Group.new(group_params) | |
|
29 | + | |
|
30 | + if @group.save | |
|
31 | + redirect_to @group, notice: 'Group was successfully created.' | |
|
32 | + else | |
|
33 | + render :new | |
|
34 | + end | |
|
35 | + end | |
|
36 | + | |
|
37 | + # PATCH/PUT /groups/1 | |
|
38 | + def update | |
|
39 | + if @group.update(group_params) | |
|
40 | + redirect_to @group, notice: 'Group was successfully updated.' | |
|
41 | + else | |
|
42 | + render :edit | |
|
43 | + end | |
|
44 | + end | |
|
45 | + | |
|
46 | + # DELETE /groups/1 | |
|
47 | + def destroy | |
|
48 | + @group.destroy | |
|
49 | + redirect_to groups_url, notice: 'Group was successfully destroyed.' | |
|
50 | + end | |
|
51 | + | |
|
52 | + def remove_user | |
|
53 | + user = User.find(params[:user_id]) | |
|
54 | + @group.users.delete(user) | |
|
55 | + redirect_to group_path(@group), notice: "User #{user.login} was removed from the group #{@group.name}" | |
|
56 | + end | |
|
57 | + | |
|
58 | + def add_user | |
|
59 | + user = User.find(params[:user_id]) | |
|
60 | + @group.users << user | |
|
61 | + redirect_to group_path(@group), notice: "User #{user.login} was add to the group #{@group.name}" | |
|
62 | + end | |
|
63 | + | |
|
64 | + def remove_problem | |
|
65 | + problem = Problem.find(params[:problem_id]) | |
|
66 | + @group.problems.delete(problem) | |
|
67 | + redirect_to group_path(@group), notice: "Problem #{problem.name} was removed from the group #{@group.name}" | |
|
68 | + end | |
|
69 | + | |
|
70 | + def add_problem | |
|
71 | + problem = Problem.find(params[:problem_id]) | |
|
72 | + @group.problems << problem | |
|
73 | + redirect_to group_path(@group), notice: "Problem #{problem.name} was add to the group #{@group.name}" | |
|
74 | + end | |
|
75 | + | |
|
76 | + private | |
|
77 | + # Use callbacks to share common setup or constraints between actions. | |
|
78 | + def set_group | |
|
79 | + @group = Group.find(params[:id]) | |
|
80 | + end | |
|
81 | + | |
|
82 | + # Only allow a trusted parameter "white list" through. | |
|
83 | + def group_params | |
|
84 | + params.require(:group).permit(:name, :description) | |
|
85 | + end | |
|
86 | + end |
@@ -0,0 +1,5 | |||
|
1 | + class Group < ActiveRecord::Base | |
|
2 | + has_and_belongs_to_many :problems | |
|
3 | + has_and_belongs_to_many :users | |
|
4 | + end | |
|
5 | + |
@@ -0,0 +1,20 | |||
|
1 | + %h1 Editing contest | |
|
2 | + = form_for(@contest) do |f| | |
|
3 | + = f.error_messages | |
|
4 | + %table | |
|
5 | + %tr | |
|
6 | + %td= f.label :name | |
|
7 | + %td= f.text_field :name | |
|
8 | + %tr | |
|
9 | + %td= f.label :title | |
|
10 | + %td= f.text_field :title | |
|
11 | + %tr | |
|
12 | + %td | |
|
13 | + %td | |
|
14 | + = f.check_box :enabled | |
|
15 | + = f.label :enabled | |
|
16 | + %p | |
|
17 | + = f.submit 'Update' | |
|
18 | + = link_to 'Show', @contest | |
|
19 | + | | |
|
20 | + = link_to 'Back', contests_path |
@@ -0,0 +1,27 | |||
|
1 | + %h1 Listing contests | |
|
2 | + .infobox | |
|
3 | + %b Go back to: | |
|
4 | + [#{link_to 'contest management', :controller => 'contest_management', :action => 'index'}] | |
|
5 | + %p= link_to 'New contest', new_contest_path, class: 'btn btn-success' | |
|
6 | + %table.table.table-striped | |
|
7 | + %tr | |
|
8 | + %th Name | |
|
9 | + %th Title | |
|
10 | + %th Enabled | |
|
11 | + %th | |
|
12 | + %th | |
|
13 | + %th | |
|
14 | + | |
|
15 | + - @contests.each do |contest| | |
|
16 | + - @contest = contest | |
|
17 | + %tr | |
|
18 | + -#%td= in_place_editor_field :contest, :name, {}, :rows => 1 | |
|
19 | + -#%td= in_place_editor_field :contest, :title, {}, :rows => 1 | |
|
20 | + -#%td= in_place_editor_field :contest, :enabled, {}, :rows => 1 | |
|
21 | + %td= best_in_place @contest, :name | |
|
22 | + %td= best_in_place @contest, :title | |
|
23 | + %td= best_in_place @contest, :enabled | |
|
24 | + %td= link_to 'Show', contest | |
|
25 | + %td= link_to 'Edit', edit_contest_path(contest) | |
|
26 | + %td= link_to 'Destroy', contest, :confirm => 'Are you sure?', :method => :delete | |
|
27 | + %br/ |
@@ -0,0 +1,18 | |||
|
1 | + %h1 New contest | |
|
2 | + = form_for(@contest) do |f| | |
|
3 | + = f.error_messages | |
|
4 | + %p | |
|
5 | + = f.label :name | |
|
6 | + %br/ | |
|
7 | + = f.text_field :name | |
|
8 | + %p | |
|
9 | + = f.label :title | |
|
10 | + %br/ | |
|
11 | + = f.text_field :title | |
|
12 | + %p | |
|
13 | + = f.label :enabled | |
|
14 | + %br/ | |
|
15 | + = f.check_box :enabled | |
|
16 | + %p | |
|
17 | + = f.submit 'Create' | |
|
18 | + = link_to 'Back', contests_path |
@@ -0,0 +1,11 | |||
|
1 | + %h1 | |
|
2 | + Contest: #{h @contest.title} | |
|
3 | + .infobox | |
|
4 | + %b Go back to: | |
|
5 | + [#{link_to 'contest management', :controller => 'contest_management', :action => 'index'}] | |
|
6 | + %p | |
|
7 | + %b Enabled: | |
|
8 | + = h @contest.enabled | |
|
9 | + = link_to 'Edit', edit_contest_path(@contest) | |
|
10 | + | | |
|
11 | + = link_to 'Back', contests_path |
@@ -0,0 +1,16 | |||
|
1 | + = form_for @group do |f| | |
|
2 | + - if @group.errors.any? | |
|
3 | + #error_explanation | |
|
4 | + %h2= "#{pluralize(@group.errors.count, "error")} prohibited this group from being saved:" | |
|
5 | + %ul | |
|
6 | + - @group.errors.full_messages.each do |msg| | |
|
7 | + %li= msg | |
|
8 | + | |
|
9 | + .form-group.field | |
|
10 | + = f.label :name | |
|
11 | + = f.text_field :name, class: 'form-control' | |
|
12 | + .form-group.field | |
|
13 | + = f.label :description | |
|
14 | + = f.text_field :description, class: 'form-control' | |
|
15 | + .form-group.actions | |
|
16 | + = f.submit 'Save', class: 'btn btn-primary' |
@@ -0,0 +1,7 | |||
|
1 | + %h1 Editing group | |
|
2 | + | |
|
3 | + = render 'form' | |
|
4 | + | |
|
5 | + = link_to 'Show', @group | |
|
6 | + \| | |
|
7 | + = link_to 'Back', groups_path |
@@ -0,0 +1,22 | |||
|
1 | + %h1 Groups | |
|
2 | + | |
|
3 | + %p | |
|
4 | + = link_to 'New Group', new_group_path, class: 'btn btn-primary' | |
|
5 | + %table.table.table-hover | |
|
6 | + %thead | |
|
7 | + %tr | |
|
8 | + %th Name | |
|
9 | + %th Description | |
|
10 | + %th | |
|
11 | + %th | |
|
12 | + | |
|
13 | + %tbody | |
|
14 | + - @groups.each do |group| | |
|
15 | + %tr | |
|
16 | + %td= group.name | |
|
17 | + %td= group.description | |
|
18 | + %td= link_to 'View', group, class: 'btn btn-default' | |
|
19 | + %td= link_to 'Destroy', group, :method => :delete, :data => { :confirm => 'Are you sure?' }, class: 'btn btn-danger' | |
|
20 | + | |
|
21 | + %br | |
|
22 | + |
@@ -0,0 +1,72 | |||
|
1 | + %p | |
|
2 | + %b Name: | |
|
3 | + = @group.name | |
|
4 | + %p | |
|
5 | + %b Description: | |
|
6 | + = @group.description | |
|
7 | + | |
|
8 | + %br | |
|
9 | + = link_to 'Edit', edit_group_path(@group) | |
|
10 | + \| | |
|
11 | + = link_to 'Back', groups_path | |
|
12 | + | |
|
13 | + .row | |
|
14 | + %h1 Group details | |
|
15 | + .row | |
|
16 | + .col-md-6 | |
|
17 | + .panel.panel-default | |
|
18 | + .panel-heading | |
|
19 | + .panel-title Users in this group | |
|
20 | + .panel-body | |
|
21 | + =form_tag add_user_group_path(@group), class: 'form-inline' do | |
|
22 | + .form-group | |
|
23 | + =label_tag :user_id, "User" | |
|
24 | + =select_tag :user_id, options_from_collection_for_select(User.all,'id','full_name'), class: 'select2' | |
|
25 | + =submit_tag "Add",class: 'btn btn-primary' | |
|
26 | + | |
|
27 | + | |
|
28 | + %table.table.table-hover | |
|
29 | + %thead | |
|
30 | + %tr | |
|
31 | + %th Login | |
|
32 | + %th Full name | |
|
33 | + %th Remark | |
|
34 | + %th | |
|
35 | + | |
|
36 | + %tbody | |
|
37 | + - @group.users.each do |user| | |
|
38 | + %tr | |
|
39 | + %td= user.login | |
|
40 | + %td= user.full_name | |
|
41 | + %td= user.remark | |
|
42 | + %td= link_to 'Remove', remove_user_group_path(@group,user), :method => :delete, :data => { :confirm => "Remove #{user.full_name}?" }, class: 'btn btn-danger' | |
|
43 | + .col-md-6 | |
|
44 | + .panel.panel-default | |
|
45 | + .panel-heading | |
|
46 | + .panel-title Problems | |
|
47 | + .panel-body | |
|
48 | + | |
|
49 | + =form_tag add_problem_group_path(@group), class: 'form-inline' do | |
|
50 | + .form-group | |
|
51 | + =label_tag :problem_id, "Problem" | |
|
52 | + =select_tag :problem_id, options_from_collection_for_select(Problem.all,'id','full_name'), class: 'select2' | |
|
53 | + =submit_tag "Add",class: 'btn btn-primary' | |
|
54 | + | |
|
55 | + | |
|
56 | + %table.table.table-hover | |
|
57 | + %thead | |
|
58 | + %tr | |
|
59 | + %th name | |
|
60 | + %th Full name | |
|
61 | + %th Full score | |
|
62 | + %th | |
|
63 | + | |
|
64 | + %tbody | |
|
65 | + - @group.problems.each do |problem| | |
|
66 | + %tr | |
|
67 | + %td= problem.name | |
|
68 | + %td= problem.full_name | |
|
69 | + %td= problem.full_score | |
|
70 | + %td= link_to 'Remove', remove_problem_group_path(@group,problem), :method => :delete, :data => { :confirm => "Remove #{problem.full_name}?" }, class: 'btn btn-danger' | |
|
71 | + | |
|
72 | + |
@@ -0,0 +1,31 | |||
|
1 | + = error_messages_for 'user' | |
|
2 | + / [form:user] | |
|
3 | + .form-group | |
|
4 | + %label.col-md-2.control-label{for: :login} Login | |
|
5 | + .col-md-4 | |
|
6 | + = text_field 'user', 'login', class: 'form-control' | |
|
7 | + .form-group | |
|
8 | + %label.col-md-2.control-label{for: :full_name} Full name | |
|
9 | + .col-md-4 | |
|
10 | + = text_field 'user', 'full_name', class: 'form-control' | |
|
11 | + .form-group | |
|
12 | + %label.col-md-2.control-label{for: :password} Password | |
|
13 | + .col-md-4 | |
|
14 | + = password_field 'user', 'password', class: 'form-control' | |
|
15 | + .form-group | |
|
16 | + %label.col-md-2.control-label{for: :password_confirmation} Password (confirm) | |
|
17 | + .col-md-4 | |
|
18 | + = password_field 'user', 'password_confirmation', class: 'form-control' | |
|
19 | + .form-group | |
|
20 | + %label.col-md-2.control-label{for: :email} E-mail | |
|
21 | + .col-md-4 | |
|
22 | + = email_field 'user', 'email', class: 'form-control' | |
|
23 | + .form-group | |
|
24 | + %label.col-md-2.control-label{for: :alias} Alias | |
|
25 | + .col-md-4 | |
|
26 | + = text_field 'user', 'alias', class: 'form-control' | |
|
27 | + .form-group | |
|
28 | + %label.col-md-2.control-label{for: :remark} Remark | |
|
29 | + .col-md-4 | |
|
30 | + = text_field 'user', 'remark', class: 'form-control' | |
|
31 | + / [eoform:user] |
@@ -0,0 +1,14 | |||
|
1 | + %h1 User information | |
|
2 | + - for column in User.content_columns | |
|
3 | + %p | |
|
4 | + %b | |
|
5 | + = column.human_name | |
|
6 | + \: | |
|
7 | + = h @user.send(column.name) | |
|
8 | + %p | |
|
9 | + %strong Group | |
|
10 | + \: | |
|
11 | + = @user.groups.map{ |x| link_to(x.name,group_path(x)).html_safe}.join(', ').html_safe | |
|
12 | + = link_to 'Edit', :action => 'edit', :id => @user | |
|
13 | + | | |
|
14 | + = link_to 'Back', :action => 'index' |
@@ -0,0 +1,30 | |||
|
1 | + class CreateGroups < ActiveRecord::Migration | |
|
2 | + | |
|
3 | + def change | |
|
4 | + create_table :groups do |t| | |
|
5 | + t.string :name | |
|
6 | + t.string :description | |
|
7 | + end | |
|
8 | + | |
|
9 | + create_join_table :groups, :users do |t| | |
|
10 | + # t.index [:group_id, :user_id] | |
|
11 | + t.index [:user_id, :group_id] | |
|
12 | + end | |
|
13 | + | |
|
14 | + create_join_table :problems, :groups do |t| | |
|
15 | + # t.index [:problem_id, :group_id] | |
|
16 | + t.index [:group_id, :problem_id] | |
|
17 | + end | |
|
18 | + | |
|
19 | + reversible do |change| | |
|
20 | + change.up do | |
|
21 | + GraderConfiguration.where(key: 'system.use_problem_group').first_or_create(value_type: 'boolean', value: 'false', | |
|
22 | + description: 'If true, available problem to the user will be only ones associated with the group of the user'); | |
|
23 | + end | |
|
24 | + | |
|
25 | + change.down do | |
|
26 | + GraderConfiguration.where(key: 'system.use_problem_group').destroy_all | |
|
27 | + end | |
|
28 | + end | |
|
29 | + end | |
|
30 | + end |
@@ -0,0 +1,49 | |||
|
1 | + require 'test_helper' | |
|
2 | + | |
|
3 | + class GroupsControllerTest < ActionController::TestCase | |
|
4 | + setup do | |
|
5 | + @group = groups(:one) | |
|
6 | + end | |
|
7 | + | |
|
8 | + test "should get index" do | |
|
9 | + get :index | |
|
10 | + assert_response :success | |
|
11 | + assert_not_nil assigns(:groups) | |
|
12 | + end | |
|
13 | + | |
|
14 | + test "should get new" do | |
|
15 | + get :new | |
|
16 | + assert_response :success | |
|
17 | + end | |
|
18 | + | |
|
19 | + test "should create group" do | |
|
20 | + assert_difference('Group.count') do | |
|
21 | + post :create, group: { description: @group.description, name: @group.name } | |
|
22 | + end | |
|
23 | + | |
|
24 | + assert_redirected_to group_path(assigns(:group)) | |
|
25 | + end | |
|
26 | + | |
|
27 | + test "should show group" do | |
|
28 | + get :show, id: @group | |
|
29 | + assert_response :success | |
|
30 | + end | |
|
31 | + | |
|
32 | + test "should get edit" do | |
|
33 | + get :edit, id: @group | |
|
34 | + assert_response :success | |
|
35 | + end | |
|
36 | + | |
|
37 | + test "should update group" do | |
|
38 | + patch :update, id: @group, group: { description: @group.description, name: @group.name } | |
|
39 | + assert_redirected_to group_path(assigns(:group)) | |
|
40 | + end | |
|
41 | + | |
|
42 | + test "should destroy group" do | |
|
43 | + assert_difference('Group.count', -1) do | |
|
44 | + delete :destroy, id: @group | |
|
45 | + end | |
|
46 | + | |
|
47 | + assert_redirected_to groups_path | |
|
48 | + end | |
|
49 | + end |
@@ -39,13 +39,10 | |||
|
39 | 39 | |
|
40 | 40 | def testcase_authorization |
|
41 | 41 | #admin always has privileged |
|
42 | - puts "haha" | |
|
43 | 42 | if @current_user.admin? |
|
44 | 43 | return true |
|
45 | 44 | end |
|
46 | 45 | |
|
47 | - puts "hehe" | |
|
48 | - puts GraderConfiguration["right.view_testcase"] | |
|
49 | 46 | unauthorized_redirect unless GraderConfiguration["right.view_testcase"] |
|
50 | 47 | end |
|
51 | 48 | |
@@ -61,27 +58,28 | |||
|
61 | 58 | return false |
|
62 | 59 | end |
|
63 | 60 | |
|
61 | + | |
|
64 | 62 | # check if run in single user mode |
|
65 | 63 | if GraderConfiguration[SINGLE_USER_MODE_CONF_KEY] |
|
66 | - user = User.find_by_id(session[:user_id]) | |
|
67 | - if user==nil or (not user.admin?) | |
|
64 | + if @current_user==nil or (not @current_user.admin?) | |
|
68 | 65 | flash[:notice] = 'You cannot log in at this time' |
|
69 | 66 | redirect_to :controller => 'main', :action => 'login' |
|
70 | 67 | return false |
|
71 | 68 | end |
|
72 | - unless user.enabled? | |
|
73 | - flash[:notice] = 'Your account is disabled' | |
|
74 | - redirect_to :controller => 'main', :action => 'login' | |
|
75 | - return false | |
|
76 | - end | |
|
77 | 69 | return true |
|
78 | 70 | end |
|
79 | 71 | |
|
72 | + # check if the user is enabled | |
|
73 | + unless @current_user.enabled? or @current_user.admin? | |
|
74 | + flash[:notice] = 'Your account is disabled' | |
|
75 | + redirect_to :controller => 'main', :action => 'login' | |
|
76 | + return false | |
|
77 | + end | |
|
78 | + | |
|
80 | 79 | if GraderConfiguration.multicontests? |
|
81 | - user = User.find(session[:user_id]) | |
|
82 | - return true if user.admin? | |
|
80 | + return true if @current_user.admin? | |
|
83 | 81 | begin |
|
84 | - if user.contest_stat(true).forced_logout | |
|
82 | + if @current_user.contest_stat(true).forced_logout | |
|
85 | 83 | flash[:notice] = 'You have been automatically logged out.' |
|
86 | 84 | redirect_to :controller => 'main', :action => 'index' |
|
87 | 85 | end |
@@ -195,6 +195,11 | |||
|
195 | 195 | set_available(true) |
|
196 | 196 | elsif params.has_key? 'disable_problem' |
|
197 | 197 | set_available(false) |
|
198 | + elsif params.has_key? 'add_group' | |
|
199 | + group = Group.find(params[:group_id]) | |
|
200 | + get_problems_from_params.each do |p| | |
|
201 | + group.problems << p | |
|
202 | + end | |
|
198 | 203 | end |
|
199 | 204 | redirect_to :action => 'manage' |
|
200 | 205 | end |
@@ -228,6 +228,7 | |||
|
228 | 228 | end |
|
229 | 229 | end |
|
230 | 230 | |
|
231 | + | |
|
231 | 232 | # contest management |
|
232 | 233 | |
|
233 | 234 | def contests |
@@ -423,6 +424,8 | |||
|
423 | 424 | @action[:set_enable] = params[:enabled] |
|
424 | 425 | @action[:enabled] = params[:enable] == "1" |
|
425 | 426 | @action[:gen_password] = params[:gen_password] |
|
427 | + @action[:add_group] = params[:add_group] | |
|
428 | + @action[:group_name] = params[:group_name] | |
|
426 | 429 | end |
|
427 | 430 | |
|
428 | 431 | if params[:commit] == "Perform" |
@@ -437,6 +440,10 | |||
|
437 | 440 | u.save |
|
438 | 441 | end |
|
439 | 442 | end |
|
443 | + if @action[:add_group] and @action[:group_name] | |
|
444 | + @group = Group.find(@action[:group_name]) | |
|
445 | + @users.each { |user| @group.users << user } | |
|
446 | + end | |
|
440 | 447 | end |
|
441 | 448 | end |
|
442 | 449 |
@@ -12,6 +12,7 | |||
|
12 | 12 | MULTIPLE_IP_LOGIN_KEY = 'right.multiple_ip_login' |
|
13 | 13 | VIEW_TESTCASE = 'right.view_testcase' |
|
14 | 14 | SINGLE_USER_KEY = 'system.single_user_mode' |
|
15 | + SYSTEM_USE_PROBLEM_GROUP = 'system.use_problem_group' | |
|
15 | 16 | |
|
16 | 17 | cattr_accessor :config_cache |
|
17 | 18 | cattr_accessor :task_grading_info_cache |
@@ -120,6 +121,10 | |||
|
120 | 121 | return get(SYSTEM_MODE_CONF_KEY) == 'analysis' |
|
121 | 122 | end |
|
122 | 123 | |
|
124 | + def self.use_problem_group? | |
|
125 | + return get(SYSTEM_USE_PROBLEM_GROUP) | |
|
126 | + end | |
|
127 | + | |
|
123 | 128 | def self.contest_time_limit |
|
124 | 129 | contest_time_str = GraderConfiguration[CONTEST_TIME_LIMIT_KEY] |
|
125 | 130 |
@@ -2,6 +2,7 | |||
|
2 | 2 | |
|
3 | 3 | belongs_to :description |
|
4 | 4 | has_and_belongs_to_many :contests, :uniq => true |
|
5 | + has_and_belongs_to_many :groups | |
|
5 | 6 | has_many :test_pairs, :dependent => :delete_all |
|
6 | 7 | has_many :testcases, :dependent => :destroy |
|
7 | 8 |
@@ -7,6 +7,7 | |||
|
7 | 7 | class User < ActiveRecord::Base |
|
8 | 8 | |
|
9 | 9 | has_and_belongs_to_many :roles |
|
10 | + has_and_belongs_to_many :groups | |
|
10 | 11 | |
|
11 | 12 | has_many :test_requests, -> {order(submitted_at: DESC)} |
|
12 | 13 | |
@@ -290,7 +291,11 | |||
|
290 | 291 | |
|
291 | 292 | def available_problems |
|
292 | 293 | if not GraderConfiguration.multicontests? |
|
294 | + if GraderConfiguration.use_problem_group? | |
|
295 | + return available_problems_in_group | |
|
296 | + else | |
|
293 | 297 | return Problem.available_problems |
|
298 | + end | |
|
294 | 299 | else |
|
295 | 300 | contest_problems = [] |
|
296 | 301 | pin = {} |
@@ -307,6 +312,14 | |||
|
307 | 312 | end |
|
308 | 313 | end |
|
309 | 314 | |
|
315 | + def available_problems_in_group | |
|
316 | + problem = [] | |
|
317 | + self.groups.each do |group| | |
|
318 | + group.problems.where(available: true).each { |p| problem << p } | |
|
319 | + end | |
|
320 | + return problem.uniq | |
|
321 | + end | |
|
322 | + | |
|
310 | 323 | def can_view_problem?(problem) |
|
311 | 324 | if not GraderConfiguration.multicontests? |
|
312 | 325 | return problem.available |
@@ -47,6 +47,7 | |||
|
47 | 47 | = add_menu( 'Announcements', 'announcements', 'index') |
|
48 | 48 | = add_menu( 'Problems', 'problems', 'index') |
|
49 | 49 | = add_menu( 'Users', 'user_admin', 'index') |
|
50 | + = add_menu( 'User Groups', 'groups', 'index') | |
|
50 | 51 | = add_menu( 'Graders', 'graders', 'list') |
|
51 | 52 | = add_menu( 'Message ', 'messages', 'console') |
|
52 | 53 | %li.divider{role: 'separator'} |
@@ -39,7 +39,7 | |||
|
39 | 39 | %p= link_to '[Back to problem list]', :action => 'list' |
|
40 | 40 | |
|
41 | 41 | = form_tag :action=>'do_manage' do |
|
42 | - .submitbox | |
|
42 | + .submitbox.panel | |
|
43 | 43 | What do you want to do to the selected problem? |
|
44 | 44 | %br/ |
|
45 | 45 | (You can shift-click to select a range of problems) |
@@ -59,8 +59,13 | |||
|
59 | 59 | Add to |
|
60 | 60 | = select("contest","id",Contest.all.collect {|c| [c.title, c.id]}) |
|
61 | 61 | = submit_tag 'Add', :name => 'add_to_contest' |
|
62 | + %li | |
|
63 | + Add problems to group | |
|
64 | + = select_tag "group_id", options_from_collection_for_select( Group.all, 'id','name',params[:group_name]), id: 'group_name',class: 'select2' | |
|
65 | + = submit_tag 'Add', name: 'add_group' | |
|
62 | 66 | |
|
63 | - %table | |
|
67 | + | |
|
68 | + %table.table.table-hover | |
|
64 | 69 | %tr{style: "text-align: left;"} |
|
65 | 70 | %th= check_box_tag 'select_all' |
|
66 | 71 | %th Name |
@@ -46,6 +46,15 | |||
|
46 | 46 | %label.checkbox-inline |
|
47 | 47 | = check_box_tag "gen_password", true, params[:gen_password] |
|
48 | 48 | Generate new random password |
|
49 | + .row.form-group | |
|
50 | + .col-md-4 | |
|
51 | + %label.checkbox-inline | |
|
52 | + = check_box_tag "add_group", true, params[:add_group] | |
|
53 | + Add users to group | |
|
54 | + %label.col-md-3.control-label.text-right Group name | |
|
55 | + .col-md-5 | |
|
56 | + = select_tag "group_name", options_from_collection_for_select( Group.all, 'id','name',params[:group_name]), id: 'group_name',class: 'form-control select2' | |
|
57 | + | |
|
49 | 58 | |
|
50 | 59 | .row |
|
51 | 60 | .col-md-12 |
@@ -1,9 +1,11 | |||
|
1 | 1 | %h1 Editing user |
|
2 | 2 | |
|
3 | - = form_tag :action => 'update', :id => @user do | |
|
3 | + = form_tag( {:action => 'update', :id => @user}, {class: 'form-horizontal'}) do | |
|
4 | 4 | = error_messages_for 'user' |
|
5 | 5 | = render partial: "form" |
|
6 | - = submit_tag "Edit" | |
|
6 | + .form-group | |
|
7 | + .col-md-offset-2.col-md-4 | |
|
8 | + = submit_tag "Edit", class: 'btn btn-primary' | |
|
7 | 9 | |
|
8 | 10 | |
|
9 | 11 | = link_to 'Show', :action => 'show', :id => @user |
@@ -29,7 +29,15 | |||
|
29 | 29 | get 'import' |
|
30 | 30 | get 'manage' |
|
31 | 31 | end |
|
32 | + end | |
|
32 | 33 | |
|
34 | + resources :groups do | |
|
35 | + member do | |
|
36 | + post 'add_user', to: 'groups#add_user', as: 'add_user' | |
|
37 | + delete 'remove_user/:user_id', to: 'groups#remove_user', as: 'remove_user' | |
|
38 | + post 'add_problem', to: 'groups#add_problem', as: 'add_problem' | |
|
39 | + delete 'remove_problem/:problem_id', to: 'groups#remove_problem', as: 'remove_problem' | |
|
40 | + end | |
|
33 | 41 | end |
|
34 | 42 | |
|
35 | 43 | resources :testcases, only: [] do |
@@ -3,7 +3,7 | |||
|
3 | 3 | add_column :languages, :ext, :string, :limit => 10 |
|
4 | 4 | |
|
5 | 5 | Language.reset_column_information |
|
6 |
- langs = Language. |
|
|
6 | + langs = Language.all | |
|
7 | 7 | langs.each do |l| |
|
8 | 8 | l.ext = l.name |
|
9 | 9 | l.save |
@@ -4,7 +4,7 | |||
|
4 | 4 | add_column :tasks, :updated_at, :datetime |
|
5 | 5 | |
|
6 | 6 | Task.reset_column_information |
|
7 |
- Task. |
|
|
7 | + Task.all.each do |task| | |
|
8 | 8 | task.status_complete |
|
9 | 9 | task.save |
|
10 | 10 | end |
@@ -9,8 +9,7 | |||
|
9 | 9 | last_problem_id = nil |
|
10 | 10 | current_number = 0 |
|
11 | 11 | |
|
12 | - Submission.find(:all, | |
|
13 | - :order => 'user_id, problem_id, submitted_at').each do |submission| | |
|
12 | + Submission.order('user_id, problem_id, submitted_at').each do |submission| | |
|
14 | 13 | if submission.user_id==last_user_id and submission.problem_id==last_problem_id |
|
15 | 14 | current_number += 1 |
|
16 | 15 | else |
@@ -7,7 +7,7 | |||
|
7 | 7 | add_column :users, :site_id, :integer |
|
8 | 8 | User.reset_column_information |
|
9 | 9 | |
|
10 |
- User. |
|
|
10 | + User.all.each do |user| | |
|
11 | 11 | |
|
12 | 12 | class << user |
|
13 | 13 | def valid? |
@@ -3,7 +3,7 | |||
|
3 | 3 | add_column :problems, :description_id, :integer |
|
4 | 4 | Problem.reset_column_information |
|
5 | 5 | |
|
6 |
- Problem. |
|
|
6 | + Problem.all.each do |problem| | |
|
7 | 7 | if problem.body!=nil |
|
8 | 8 | description = Description.new |
|
9 | 9 | description.body = problem.body |
@@ -21,7 +21,7 | |||
|
21 | 21 | add_column :problems, :body, :text |
|
22 | 22 | Problem.reset_column_information |
|
23 | 23 | |
|
24 |
- Problem. |
|
|
24 | + Problem.all.each do |problem| | |
|
25 | 25 | if problem.description_id != nil |
|
26 | 26 | problem.body = Description.find(problem.description_id).body |
|
27 | 27 | problem.save |
@@ -3,7 +3,7 | |||
|
3 | 3 | add_column :problems, :test_allowed, :boolean |
|
4 | 4 | Problem.reset_column_information |
|
5 | 5 | |
|
6 |
- Problem. |
|
|
6 | + Problem.all.each do |problem| | |
|
7 | 7 | problem.test_allowed = true |
|
8 | 8 | problem.save |
|
9 | 9 | end |
@@ -4,7 +4,7 | |||
|
4 | 4 | |
|
5 | 5 | User.reset_column_information |
|
6 | 6 | |
|
7 |
- User. |
|
|
7 | + User.all.each do |user| | |
|
8 | 8 | |
|
9 | 9 | # disable validation |
|
10 | 10 | class <<user |
@@ -11,7 +11,7 | |||
|
11 | 11 | 'cpp' => 'cpp,cc', |
|
12 | 12 | 'pas' => 'pas' |
|
13 | 13 | } |
|
14 |
- Language. |
|
|
14 | + Language.all.each do |lang| | |
|
15 | 15 | lang.common_ext = common_ext[lang.name] |
|
16 | 16 | lang.save |
|
17 | 17 | end |
@@ -11,7 +11,7 | |||
|
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: 20170911091143) do | |
|
15 | 15 | |
|
16 | 16 | create_table "announcements", force: :cascade do |t| |
|
17 | 17 | t.string "author", limit: 255 |
@@ -79,6 +79,25 | |||
|
79 | 79 | |
|
80 | 80 | add_index "grader_processes", ["host", "pid"], name: "index_grader_processes_on_ip_and_pid", using: :btree |
|
81 | 81 | |
|
82 | + create_table "groups", force: :cascade do |t| | |
|
83 | + t.string "name", limit: 255 | |
|
84 | + t.string "description", limit: 255 | |
|
85 | + end | |
|
86 | + | |
|
87 | + create_table "groups_problems", id: false, force: :cascade do |t| | |
|
88 | + t.integer "problem_id", limit: 4, null: false | |
|
89 | + t.integer "group_id", limit: 4, null: false | |
|
90 | + end | |
|
91 | + | |
|
92 | + add_index "groups_problems", ["group_id", "problem_id"], name: "index_groups_problems_on_group_id_and_problem_id", using: :btree | |
|
93 | + | |
|
94 | + create_table "groups_users", id: false, force: :cascade do |t| | |
|
95 | + t.integer "group_id", limit: 4, null: false | |
|
96 | + t.integer "user_id", limit: 4, null: false | |
|
97 | + end | |
|
98 | + | |
|
99 | + add_index "groups_users", ["user_id", "group_id"], name: "index_groups_users_on_user_id_and_group_id", using: :btree | |
|
100 | + | |
|
82 | 101 | create_table "heart_beats", force: :cascade do |t| |
|
83 | 102 | t.integer "user_id", limit: 4 |
|
84 | 103 | t.string "ip_address", limit: 255 |
@@ -163,7 +163,16 | |||
|
163 | 163 | :value_type => 'string', |
|
164 | 164 | :default_value => 'none', |
|
165 | 165 | :description => "New user will be assigned to this contest automatically, if it exists. Set to 'none' if there is no default contest." |
|
166 | - } | |
|
166 | + }, | |
|
167 | + | |
|
168 | + { | |
|
169 | + :key => 'system.use_problem_group', | |
|
170 | + :value_type => 'boolean', | |
|
171 | + :default_value => 'false', | |
|
172 | + :description => "If true, available problem to the user will be only ones associated with the group of the user." | |
|
173 | + }, | |
|
174 | + | |
|
175 | + | |
|
167 | 176 | |
|
168 | 177 | ] |
|
169 | 178 |
deleted file |
deleted file |
deleted file |
deleted file |
deleted file |
deleted file |
You need to be logged in to leave comments.
Login now