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 | def testcase_authorization |
|
40 | def testcase_authorization |
|
41 | #admin always has privileged |
|
41 | #admin always has privileged |
|
42 | - puts "haha" |
|
||
|
43 | if @current_user.admin? |
|
42 | if @current_user.admin? |
|
44 | return true |
|
43 | return true |
|
45 | end |
|
44 | end |
|
46 |
|
45 | ||
|
47 | - puts "hehe" |
|
||
|
48 | - puts GraderConfiguration["right.view_testcase"] |
|
||
|
49 | unauthorized_redirect unless GraderConfiguration["right.view_testcase"] |
|
46 | unauthorized_redirect unless GraderConfiguration["right.view_testcase"] |
|
50 | end |
|
47 | end |
|
51 |
|
48 | ||
@@ -61,27 +58,28 | |||||
|
61 | return false |
|
58 | return false |
|
62 | end |
|
59 | end |
|
63 |
|
60 | ||
|
|
61 | + | ||
|
64 | # check if run in single user mode |
|
62 | # check if run in single user mode |
|
65 | if GraderConfiguration[SINGLE_USER_MODE_CONF_KEY] |
|
63 | if GraderConfiguration[SINGLE_USER_MODE_CONF_KEY] |
|
66 | - user = User.find_by_id(session[:user_id]) |
|
64 | + if @current_user==nil or (not @current_user.admin?) |
|
67 | - if user==nil or (not user.admin?) |
|
||
|
68 | flash[:notice] = 'You cannot log in at this time' |
|
65 | flash[:notice] = 'You cannot log in at this time' |
|
69 | redirect_to :controller => 'main', :action => 'login' |
|
66 | redirect_to :controller => 'main', :action => 'login' |
|
70 | return false |
|
67 | return false |
|
71 | end |
|
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 | return true |
|
69 | return true |
|
78 | end |
|
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 | if GraderConfiguration.multicontests? |
|
79 | if GraderConfiguration.multicontests? |
|
81 | - user = User.find(session[:user_id]) |
|
80 | + return true if @current_user.admin? |
|
82 | - return true if user.admin? |
|
||
|
83 | begin |
|
81 | begin |
|
84 | - if user.contest_stat(true).forced_logout |
|
82 | + if @current_user.contest_stat(true).forced_logout |
|
85 | flash[:notice] = 'You have been automatically logged out.' |
|
83 | flash[:notice] = 'You have been automatically logged out.' |
|
86 | redirect_to :controller => 'main', :action => 'index' |
|
84 | redirect_to :controller => 'main', :action => 'index' |
|
87 | end |
|
85 | end |
@@ -195,6 +195,11 | |||||
|
195 | set_available(true) |
|
195 | set_available(true) |
|
196 | elsif params.has_key? 'disable_problem' |
|
196 | elsif params.has_key? 'disable_problem' |
|
197 | set_available(false) |
|
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 | end |
|
203 | end |
|
199 | redirect_to :action => 'manage' |
|
204 | redirect_to :action => 'manage' |
|
200 | end |
|
205 | end |
@@ -228,6 +228,7 | |||||
|
228 | end |
|
228 | end |
|
229 | end |
|
229 | end |
|
230 |
|
230 | ||
|
|
231 | + | ||
|
231 | # contest management |
|
232 | # contest management |
|
232 |
|
233 | ||
|
233 | def contests |
|
234 | def contests |
@@ -423,6 +424,8 | |||||
|
423 | @action[:set_enable] = params[:enabled] |
|
424 | @action[:set_enable] = params[:enabled] |
|
424 | @action[:enabled] = params[:enable] == "1" |
|
425 | @action[:enabled] = params[:enable] == "1" |
|
425 | @action[:gen_password] = params[:gen_password] |
|
426 | @action[:gen_password] = params[:gen_password] |
|
|
427 | + @action[:add_group] = params[:add_group] | ||
|
|
428 | + @action[:group_name] = params[:group_name] | ||
|
426 | end |
|
429 | end |
|
427 |
|
430 | ||
|
428 | if params[:commit] == "Perform" |
|
431 | if params[:commit] == "Perform" |
@@ -437,6 +440,10 | |||||
|
437 | u.save |
|
440 | u.save |
|
438 | end |
|
441 | end |
|
439 | end |
|
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 | end |
|
447 | end |
|
441 | end |
|
448 | end |
|
442 |
|
449 |
@@ -12,6 +12,7 | |||||
|
12 | MULTIPLE_IP_LOGIN_KEY = 'right.multiple_ip_login' |
|
12 | MULTIPLE_IP_LOGIN_KEY = 'right.multiple_ip_login' |
|
13 | VIEW_TESTCASE = 'right.view_testcase' |
|
13 | VIEW_TESTCASE = 'right.view_testcase' |
|
14 | SINGLE_USER_KEY = 'system.single_user_mode' |
|
14 | SINGLE_USER_KEY = 'system.single_user_mode' |
|
|
15 | + SYSTEM_USE_PROBLEM_GROUP = 'system.use_problem_group' | ||
|
15 |
|
16 | ||
|
16 | cattr_accessor :config_cache |
|
17 | cattr_accessor :config_cache |
|
17 | cattr_accessor :task_grading_info_cache |
|
18 | cattr_accessor :task_grading_info_cache |
@@ -120,6 +121,10 | |||||
|
120 | return get(SYSTEM_MODE_CONF_KEY) == 'analysis' |
|
121 | return get(SYSTEM_MODE_CONF_KEY) == 'analysis' |
|
121 | end |
|
122 | end |
|
122 |
|
123 | ||
|
|
124 | + def self.use_problem_group? | ||
|
|
125 | + return get(SYSTEM_USE_PROBLEM_GROUP) | ||
|
|
126 | + end | ||
|
|
127 | + | ||
|
123 | def self.contest_time_limit |
|
128 | def self.contest_time_limit |
|
124 | contest_time_str = GraderConfiguration[CONTEST_TIME_LIMIT_KEY] |
|
129 | contest_time_str = GraderConfiguration[CONTEST_TIME_LIMIT_KEY] |
|
125 |
|
130 |
@@ -2,6 +2,7 | |||||
|
2 |
|
2 | ||
|
3 | belongs_to :description |
|
3 | belongs_to :description |
|
4 | has_and_belongs_to_many :contests, :uniq => true |
|
4 | has_and_belongs_to_many :contests, :uniq => true |
|
|
5 | + has_and_belongs_to_many :groups | ||
|
5 | has_many :test_pairs, :dependent => :delete_all |
|
6 | has_many :test_pairs, :dependent => :delete_all |
|
6 | has_many :testcases, :dependent => :destroy |
|
7 | has_many :testcases, :dependent => :destroy |
|
7 |
|
8 |
@@ -7,6 +7,7 | |||||
|
7 | class User < ActiveRecord::Base |
|
7 | class User < ActiveRecord::Base |
|
8 |
|
8 | ||
|
9 | has_and_belongs_to_many :roles |
|
9 | has_and_belongs_to_many :roles |
|
|
10 | + has_and_belongs_to_many :groups | ||
|
10 |
|
11 | ||
|
11 | has_many :test_requests, -> {order(submitted_at: DESC)} |
|
12 | has_many :test_requests, -> {order(submitted_at: DESC)} |
|
12 |
|
13 | ||
@@ -290,7 +291,11 | |||||
|
290 |
|
291 | ||
|
291 | def available_problems |
|
292 | def available_problems |
|
292 | if not GraderConfiguration.multicontests? |
|
293 | if not GraderConfiguration.multicontests? |
|
|
294 | + if GraderConfiguration.use_problem_group? | ||
|
|
295 | + return available_problems_in_group | ||
|
|
296 | + else | ||
|
293 | return Problem.available_problems |
|
297 | return Problem.available_problems |
|
|
298 | + end | ||
|
294 | else |
|
299 | else |
|
295 | contest_problems = [] |
|
300 | contest_problems = [] |
|
296 | pin = {} |
|
301 | pin = {} |
@@ -307,6 +312,14 | |||||
|
307 | end |
|
312 | end |
|
308 | end |
|
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 | def can_view_problem?(problem) |
|
323 | def can_view_problem?(problem) |
|
311 | if not GraderConfiguration.multicontests? |
|
324 | if not GraderConfiguration.multicontests? |
|
312 | return problem.available |
|
325 | return problem.available |
@@ -47,6 +47,7 | |||||
|
47 | = add_menu( 'Announcements', 'announcements', 'index') |
|
47 | = add_menu( 'Announcements', 'announcements', 'index') |
|
48 | = add_menu( 'Problems', 'problems', 'index') |
|
48 | = add_menu( 'Problems', 'problems', 'index') |
|
49 | = add_menu( 'Users', 'user_admin', 'index') |
|
49 | = add_menu( 'Users', 'user_admin', 'index') |
|
|
50 | + = add_menu( 'User Groups', 'groups', 'index') | ||
|
50 | = add_menu( 'Graders', 'graders', 'list') |
|
51 | = add_menu( 'Graders', 'graders', 'list') |
|
51 | = add_menu( 'Message ', 'messages', 'console') |
|
52 | = add_menu( 'Message ', 'messages', 'console') |
|
52 | %li.divider{role: 'separator'} |
|
53 | %li.divider{role: 'separator'} |
@@ -39,7 +39,7 | |||||
|
39 | %p= link_to '[Back to problem list]', :action => 'list' |
|
39 | %p= link_to '[Back to problem list]', :action => 'list' |
|
40 |
|
40 | ||
|
41 | = form_tag :action=>'do_manage' do |
|
41 | = form_tag :action=>'do_manage' do |
|
42 | - .submitbox |
|
42 | + .submitbox.panel |
|
43 | What do you want to do to the selected problem? |
|
43 | What do you want to do to the selected problem? |
|
44 | %br/ |
|
44 | %br/ |
|
45 | (You can shift-click to select a range of problems) |
|
45 | (You can shift-click to select a range of problems) |
@@ -59,8 +59,13 | |||||
|
59 | Add to |
|
59 | Add to |
|
60 | = select("contest","id",Contest.all.collect {|c| [c.title, c.id]}) |
|
60 | = select("contest","id",Contest.all.collect {|c| [c.title, c.id]}) |
|
61 | = submit_tag 'Add', :name => 'add_to_contest' |
|
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 | %tr{style: "text-align: left;"} |
|
69 | %tr{style: "text-align: left;"} |
|
65 | %th= check_box_tag 'select_all' |
|
70 | %th= check_box_tag 'select_all' |
|
66 | %th Name |
|
71 | %th Name |
@@ -46,6 +46,15 | |||||
|
46 | %label.checkbox-inline |
|
46 | %label.checkbox-inline |
|
47 | = check_box_tag "gen_password", true, params[:gen_password] |
|
47 | = check_box_tag "gen_password", true, params[:gen_password] |
|
48 | Generate new random password |
|
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 | .row |
|
59 | .row |
|
51 | .col-md-12 |
|
60 | .col-md-12 |
@@ -1,9 +1,11 | |||||
|
1 | %h1 Editing user |
|
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 | = error_messages_for 'user' |
|
4 | = error_messages_for 'user' |
|
5 | = render partial: "form" |
|
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 | = link_to 'Show', :action => 'show', :id => @user |
|
11 | = link_to 'Show', :action => 'show', :id => @user |
@@ -29,7 +29,15 | |||||
|
29 | get 'import' |
|
29 | get 'import' |
|
30 | get 'manage' |
|
30 | get 'manage' |
|
31 | end |
|
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 | end |
|
41 | end |
|
34 |
|
42 | ||
|
35 | resources :testcases, only: [] do |
|
43 | resources :testcases, only: [] do |
@@ -3,7 +3,7 | |||||
|
3 | add_column :languages, :ext, :string, :limit => 10 |
|
3 | add_column :languages, :ext, :string, :limit => 10 |
|
4 |
|
4 | ||
|
5 | Language.reset_column_information |
|
5 | Language.reset_column_information |
|
6 |
- langs = Language. |
|
6 | + langs = Language.all |
|
7 | langs.each do |l| |
|
7 | langs.each do |l| |
|
8 | l.ext = l.name |
|
8 | l.ext = l.name |
|
9 | l.save |
|
9 | l.save |
@@ -4,7 +4,7 | |||||
|
4 | add_column :tasks, :updated_at, :datetime |
|
4 | add_column :tasks, :updated_at, :datetime |
|
5 |
|
5 | ||
|
6 | Task.reset_column_information |
|
6 | Task.reset_column_information |
|
7 |
- Task. |
|
7 | + Task.all.each do |task| |
|
8 | task.status_complete |
|
8 | task.status_complete |
|
9 | task.save |
|
9 | task.save |
|
10 | end |
|
10 | end |
@@ -9,8 +9,7 | |||||
|
9 | last_problem_id = nil |
|
9 | last_problem_id = nil |
|
10 | current_number = 0 |
|
10 | current_number = 0 |
|
11 |
|
11 | ||
|
12 | - Submission.find(:all, |
|
12 | + Submission.order('user_id, problem_id, submitted_at').each do |submission| |
|
13 | - :order => 'user_id, problem_id, submitted_at').each do |submission| |
|
||
|
14 | if submission.user_id==last_user_id and submission.problem_id==last_problem_id |
|
13 | if submission.user_id==last_user_id and submission.problem_id==last_problem_id |
|
15 | current_number += 1 |
|
14 | current_number += 1 |
|
16 | else |
|
15 | else |
@@ -7,7 +7,7 | |||||
|
7 | add_column :users, :site_id, :integer |
|
7 | add_column :users, :site_id, :integer |
|
8 | User.reset_column_information |
|
8 | User.reset_column_information |
|
9 |
|
9 | ||
|
10 |
- User. |
|
10 | + User.all.each do |user| |
|
11 |
|
11 | ||
|
12 | class << user |
|
12 | class << user |
|
13 | def valid? |
|
13 | def valid? |
@@ -3,7 +3,7 | |||||
|
3 | add_column :problems, :description_id, :integer |
|
3 | add_column :problems, :description_id, :integer |
|
4 | Problem.reset_column_information |
|
4 | Problem.reset_column_information |
|
5 |
|
5 | ||
|
6 |
- Problem. |
|
6 | + Problem.all.each do |problem| |
|
7 | if problem.body!=nil |
|
7 | if problem.body!=nil |
|
8 | description = Description.new |
|
8 | description = Description.new |
|
9 | description.body = problem.body |
|
9 | description.body = problem.body |
@@ -21,7 +21,7 | |||||
|
21 | add_column :problems, :body, :text |
|
21 | add_column :problems, :body, :text |
|
22 | Problem.reset_column_information |
|
22 | Problem.reset_column_information |
|
23 |
|
23 | ||
|
24 |
- Problem. |
|
24 | + Problem.all.each do |problem| |
|
25 | if problem.description_id != nil |
|
25 | if problem.description_id != nil |
|
26 | problem.body = Description.find(problem.description_id).body |
|
26 | problem.body = Description.find(problem.description_id).body |
|
27 | problem.save |
|
27 | problem.save |
@@ -3,7 +3,7 | |||||
|
3 | add_column :problems, :test_allowed, :boolean |
|
3 | add_column :problems, :test_allowed, :boolean |
|
4 | Problem.reset_column_information |
|
4 | Problem.reset_column_information |
|
5 |
|
5 | ||
|
6 |
- Problem. |
|
6 | + Problem.all.each do |problem| |
|
7 | problem.test_allowed = true |
|
7 | problem.test_allowed = true |
|
8 | problem.save |
|
8 | problem.save |
|
9 | end |
|
9 | end |
@@ -4,7 +4,7 | |||||
|
4 |
|
4 | ||
|
5 | User.reset_column_information |
|
5 | User.reset_column_information |
|
6 |
|
6 | ||
|
7 |
- User. |
|
7 | + User.all.each do |user| |
|
8 |
|
8 | ||
|
9 | # disable validation |
|
9 | # disable validation |
|
10 | class <<user |
|
10 | class <<user |
@@ -11,7 +11,7 | |||||
|
11 | 'cpp' => 'cpp,cc', |
|
11 | 'cpp' => 'cpp,cc', |
|
12 | 'pas' => 'pas' |
|
12 | 'pas' => 'pas' |
|
13 | } |
|
13 | } |
|
14 |
- Language. |
|
14 | + Language.all.each do |lang| |
|
15 | lang.common_ext = common_ext[lang.name] |
|
15 | lang.common_ext = common_ext[lang.name] |
|
16 | lang.save |
|
16 | lang.save |
|
17 | end |
|
17 | end |
@@ -11,7 +11,7 | |||||
|
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: 20170911091143) 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 |
@@ -79,6 +79,25 | |||||
|
79 |
|
79 | ||
|
80 | add_index "grader_processes", ["host", "pid"], name: "index_grader_processes_on_ip_and_pid", using: :btree |
|
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 | create_table "heart_beats", force: :cascade do |t| |
|
101 | create_table "heart_beats", force: :cascade do |t| |
|
83 | t.integer "user_id", limit: 4 |
|
102 | t.integer "user_id", limit: 4 |
|
84 | t.string "ip_address", limit: 255 |
|
103 | t.string "ip_address", limit: 255 |
@@ -163,7 +163,16 | |||||
|
163 | :value_type => 'string', |
|
163 | :value_type => 'string', |
|
164 | :default_value => 'none', |
|
164 | :default_value => 'none', |
|
165 | :description => "New user will be assigned to this contest automatically, if it exists. Set to 'none' if there is no default contest." |
|
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