Description:
better user import
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r798:46ce575fc051 - - 6 files changed: 29 inserted, 18 deleted
@@ -44,79 +44,89 | |||||
|
44 | if @user.save |
|
44 | if @user.save |
|
45 | flash[:notice] = 'User was successfully created.' |
|
45 | flash[:notice] = 'User was successfully created.' |
|
46 | redirect_to :action => 'index' |
|
46 | redirect_to :action => 'index' |
|
47 | else |
|
47 | else |
|
48 | render :action => 'new' |
|
48 | render :action => 'new' |
|
49 | end |
|
49 | end |
|
50 | end |
|
50 | end |
|
51 |
|
51 | ||
|
52 | def clear_last_ip |
|
52 | def clear_last_ip |
|
53 | @user = User.find(params[:id]) |
|
53 | @user = User.find(params[:id]) |
|
54 | @user.last_ip = nil |
|
54 | @user.last_ip = nil |
|
55 | @user.save |
|
55 | @user.save |
|
56 | redirect_to action: 'index', page: params[:page] |
|
56 | redirect_to action: 'index', page: params[:page] |
|
57 | end |
|
57 | end |
|
58 |
|
58 | ||
|
59 | def create_from_list |
|
59 | def create_from_list |
|
60 | lines = params[:user_list] |
|
60 | lines = params[:user_list] |
|
61 |
|
61 | ||
|
62 | note = [] |
|
62 | note = [] |
|
63 | error_note = [] |
|
63 | error_note = [] |
|
64 | error_msg = nil |
|
64 | error_msg = nil |
|
65 | ok_user = [] |
|
65 | ok_user = [] |
|
66 |
|
66 | ||
|
67 | lines.split("\n").each do |line| |
|
67 | lines.split("\n").each do |line| |
|
68 | - items = line.chomp.split(',') |
|
68 | + #split with large limit, this will cause consecutive ',' to be result in a blank |
|
|
69 | + items = line.chomp.split(',',1000) | ||
|
69 | if items.length>=2 |
|
70 | if items.length>=2 |
|
70 | login = items[0] |
|
71 | login = items[0] |
|
71 | full_name = items[1] |
|
72 | full_name = items[1] |
|
72 | remark ='' |
|
73 | remark ='' |
|
73 | user_alias = '' |
|
74 | user_alias = '' |
|
74 |
|
75 | ||
|
75 | added_random_password = false |
|
76 | added_random_password = false |
|
76 | - if items.length >= 3 and items[2].chomp(" ").length > 0; |
|
77 | + added_password = false |
|
77 | - password = items[2].chomp(" ") |
|
78 | + if items.length >= 3 |
|
|
79 | + if items[2].chomp(" ").length > 0 | ||
|
|
80 | + password = items[2].chomp(" ") | ||
|
|
81 | + added_password = true | ||
|
|
82 | + end | ||
|
78 | else |
|
83 | else |
|
79 | password = random_password |
|
84 | password = random_password |
|
80 | added_random_password=true; |
|
85 | added_random_password=true; |
|
81 | end |
|
86 | end |
|
82 |
|
87 | ||
|
83 | if items.length>= 4 and items[3].chomp(" ").length > 0; |
|
88 | if items.length>= 4 and items[3].chomp(" ").length > 0; |
|
84 | user_alias = items[3].chomp(" ") |
|
89 | user_alias = items[3].chomp(" ") |
|
85 | else |
|
90 | else |
|
86 | user_alias = login |
|
91 | user_alias = login |
|
87 | end |
|
92 | end |
|
88 |
|
93 | ||
|
|
94 | + | ||
|
|
95 | + has_remark = false | ||
|
89 | if items.length>=5 |
|
96 | if items.length>=5 |
|
90 | remark = items[4].strip; |
|
97 | remark = items[4].strip; |
|
|
98 | + has_remark = true | ||
|
91 | end |
|
99 | end |
|
92 |
|
100 | ||
|
93 | user = User.find_by_login(login) |
|
101 | user = User.find_by_login(login) |
|
94 | if (user) |
|
102 | if (user) |
|
95 | user.full_name = full_name |
|
103 | user.full_name = full_name |
|
96 | - user.password = password |
|
104 | + user.remark = remark if has_remark |
|
97 | - user.remark = remark |
|
105 | + user.password = password if added_password || added_random_password |
|
98 | else |
|
106 | else |
|
|
107 | + #create a random password if none are given | ||
|
|
108 | + password = random_password unless password | ||
|
99 | user = User.new({:login => login, |
|
109 | user = User.new({:login => login, |
|
100 | :full_name => full_name, |
|
110 | :full_name => full_name, |
|
101 | :password => password, |
|
111 | :password => password, |
|
102 | :password_confirmation => password, |
|
112 | :password_confirmation => password, |
|
103 | :alias => user_alias, |
|
113 | :alias => user_alias, |
|
104 | :remark => remark}) |
|
114 | :remark => remark}) |
|
105 | end |
|
115 | end |
|
106 | user.activated = true |
|
116 | user.activated = true |
|
107 |
|
117 | ||
|
108 | if user.save |
|
118 | if user.save |
|
109 | if added_random_password |
|
119 | if added_random_password |
|
110 | note << "'#{login}' (+)" |
|
120 | note << "'#{login}' (+)" |
|
111 | else |
|
121 | else |
|
112 | note << login |
|
122 | note << login |
|
113 | end |
|
123 | end |
|
114 | ok_user << user |
|
124 | ok_user << user |
|
115 | else |
|
125 | else |
|
116 | error_note << "'#{login}'" |
|
126 | error_note << "'#{login}'" |
|
117 | error_msg = user.errors.full_messages.to_sentence unless error_msg |
|
127 | error_msg = user.errors.full_messages.to_sentence unless error_msg |
|
118 | end |
|
128 | end |
|
119 |
|
129 | ||
|
120 | end |
|
130 | end |
|
121 | end |
|
131 | end |
|
122 |
|
132 |
@@ -161,51 +161,48 | |||||
|
161 | " #{format_short_duration(user.contest_time_left)}" |
|
161 | " #{format_short_duration(user.contest_time_left)}" |
|
162 | end |
|
162 | end |
|
163 | end |
|
163 | end |
|
164 |
|
164 | ||
|
165 | # |
|
165 | # |
|
166 | # if the contest is in the anaysis mode |
|
166 | # if the contest is in the anaysis mode |
|
167 | if GraderConfiguration.analysis_mode? |
|
167 | if GraderConfiguration.analysis_mode? |
|
168 | header = <<ANALYSISMODE |
|
168 | header = <<ANALYSISMODE |
|
169 | <tr><td colspan="2" align="center"> |
|
169 | <tr><td colspan="2" align="center"> |
|
170 | <span class="contest-over-msg">ANALYSIS MODE</span> |
|
170 | <span class="contest-over-msg">ANALYSIS MODE</span> |
|
171 | </td></tr> |
|
171 | </td></tr> |
|
172 | ANALYSISMODE |
|
172 | ANALYSISMODE |
|
173 | end |
|
173 | end |
|
174 |
|
174 | ||
|
175 | contest_name = GraderConfiguration['contest.name'] |
|
175 | contest_name = GraderConfiguration['contest.name'] |
|
176 |
|
176 | ||
|
177 | # |
|
177 | # |
|
178 | # build real title bar |
|
178 | # build real title bar |
|
179 | result = <<TITLEBAR |
|
179 | result = <<TITLEBAR |
|
180 | <div class="title"> |
|
180 | <div class="title"> |
|
181 | <table> |
|
181 | <table> |
|
182 | #{header} |
|
182 | #{header} |
|
183 | <tr> |
|
183 | <tr> |
|
184 | <td class="left-col"> |
|
184 | <td class="left-col"> |
|
185 | - #{user.full_name}<br/> |
|
||
|
186 | - #{t 'title_bar.current_time'} #{format_short_time(Time.zone.now)} |
|
||
|
187 | - #{time_left} |
|
||
|
188 | <br/> |
|
185 | <br/> |
|
189 | </td> |
|
186 | </td> |
|
190 | <td class="right-col">#{contest_name}</td> |
|
187 | <td class="right-col">#{contest_name}</td> |
|
191 | </tr> |
|
188 | </tr> |
|
192 | </table> |
|
189 | </table> |
|
193 | </div> |
|
190 | </div> |
|
194 | TITLEBAR |
|
191 | TITLEBAR |
|
195 | result.html_safe |
|
192 | result.html_safe |
|
196 | end |
|
193 | end |
|
197 |
|
194 | ||
|
198 | def markdown(text) |
|
195 | def markdown(text) |
|
199 | markdown = RDiscount.new(text) |
|
196 | markdown = RDiscount.new(text) |
|
200 | markdown.to_html.html_safe |
|
197 | markdown.to_html.html_safe |
|
201 | end |
|
198 | end |
|
202 |
|
199 | ||
|
203 |
|
200 | ||
|
204 | BOOTSTRAP_FLASH_MSG = { |
|
201 | BOOTSTRAP_FLASH_MSG = { |
|
205 | success: 'alert-success', |
|
202 | success: 'alert-success', |
|
206 | error: 'alert-danger', |
|
203 | error: 'alert-danger', |
|
207 | alert: 'alert-danger', |
|
204 | alert: 'alert-danger', |
|
208 | notice: 'alert-info' |
|
205 | notice: 'alert-info' |
|
209 | } |
|
206 | } |
|
210 |
|
207 | ||
|
211 | def bootstrap_class_for(flash_type) |
|
208 | def bootstrap_class_for(flash_type) |
@@ -1,20 +1,14 | |||||
|
1 | module MainHelper |
|
1 | module MainHelper |
|
2 |
|
2 | ||
|
3 |
- def link_to_description_if_any(name, problem |
|
3 | + def link_to_description_if_any(name, problem) |
|
4 | if !problem.url.blank? |
|
4 | if !problem.url.blank? |
|
5 |
- return link_to name, problem.url |
|
5 | + return link_to name, problem.url |
|
6 | elsif !problem.description_filename.blank? |
|
6 | elsif !problem.description_filename.blank? |
|
7 | - #build a link to a problem (via task controller) |
|
||
|
8 | basename, ext = problem.description_filename.split('.') |
|
7 | basename, ext = problem.description_filename.split('.') |
|
9 | - options[:controller] = 'tasks' |
|
8 | + return link_to name, download_task_path(problem.id,basename,ext), target: '_blank' |
|
10 | - options[:action] = 'download' |
|
||
|
11 | - options[:id] = problem.id |
|
||
|
12 | - options[:file] = basename |
|
||
|
13 | - options[:ext] = ext |
|
||
|
14 | - return link_to name, options |
|
||
|
15 | else |
|
9 | else |
|
16 | return '' |
|
10 | return '' |
|
17 | end |
|
11 | end |
|
18 | end |
|
12 | end |
|
19 |
|
13 | ||
|
20 | end |
|
14 | end |
@@ -21,48 +21,49 | |||||
|
21 | e = ace.edit("editor") |
|
21 | e = ace.edit("editor") |
|
22 | e.setOptions({ maxLines: Infinity }) |
|
22 | e.setOptions({ maxLines: Infinity }) |
|
23 | e.setValue($("#data").text()) |
|
23 | e.setValue($("#data").text()) |
|
24 | e.gotoLine(1) |
|
24 | e.gotoLine(1) |
|
25 | e.getSession().setMode("#{get_ace_mode(@submission.language)}") |
|
25 | e.getSession().setMode("#{get_ace_mode(@submission.language)}") |
|
26 | e.setReadOnly(true) |
|
26 | e.setReadOnly(true) |
|
27 | .col-md-5 |
|
27 | .col-md-5 |
|
28 | %table.table.table-striped |
|
28 | %table.table.table-striped |
|
29 | %tr |
|
29 | %tr |
|
30 | %td.text-right |
|
30 | %td.text-right |
|
31 | %strong User |
|
31 | %strong User |
|
32 | %td |
|
32 | %td |
|
33 | - if @submission.user |
|
33 | - if @submission.user |
|
34 | = link_to "#{@submission.user.login}", stat_user_path(@submission.user) |
|
34 | = link_to "#{@submission.user.login}", stat_user_path(@submission.user) |
|
35 | = @submission.user.full_name |
|
35 | = @submission.user.full_name |
|
36 | - else |
|
36 | - else |
|
37 | = "(n/a)" |
|
37 | = "(n/a)" |
|
38 | %tr |
|
38 | %tr |
|
39 | %td.text-right |
|
39 | %td.text-right |
|
40 | %strong Task |
|
40 | %strong Task |
|
41 | %td |
|
41 | %td |
|
42 | - if @submission.problem!=nil |
|
42 | - if @submission.problem!=nil |
|
43 | = link_to "[#{@submission.problem.name}]", stat_problem_path(@submission.problem) |
|
43 | = link_to "[#{@submission.problem.name}]", stat_problem_path(@submission.problem) |
|
44 | = @submission.problem.full_name |
|
44 | = @submission.problem.full_name |
|
|
45 | + = link_to_description_if_any "[download] <span class='glyphicon glyphicon-file'></span>".html_safe, @submission.problem | ||
|
45 | - else |
|
46 | - else |
|
46 | = "(n/a)" |
|
47 | = "(n/a)" |
|
47 | %tr |
|
48 | %tr |
|
48 | %td.text-right |
|
49 | %td.text-right |
|
49 | %strong Tries |
|
50 | %strong Tries |
|
50 | %td= @submission.number |
|
51 | %td= @submission.number |
|
51 | %tr |
|
52 | %tr |
|
52 | %td.text-right |
|
53 | %td.text-right |
|
53 | %strong Language |
|
54 | %strong Language |
|
54 | %td= @submission.language.pretty_name |
|
55 | %td= @submission.language.pretty_name |
|
55 | %tr |
|
56 | %tr |
|
56 | %td.text-right |
|
57 | %td.text-right |
|
57 | %strong Submitted |
|
58 | %strong Submitted |
|
58 | %td #{time_ago_in_words(@submission.submitted_at)} ago (at #{@submission.submitted_at.to_formatted_s(:long)}) |
|
59 | %td #{time_ago_in_words(@submission.submitted_at)} ago (at #{@submission.submitted_at.to_formatted_s(:long)}) |
|
59 | %tr |
|
60 | %tr |
|
60 | %td.text-right |
|
61 | %td.text-right |
|
61 | %strong Graded |
|
62 | %strong Graded |
|
62 | - if @submission.graded_at |
|
63 | - if @submission.graded_at |
|
63 | %td #{time_ago_in_words(@submission.graded_at)} ago (at #{@submission.graded_at.to_formatted_s(:long)}) |
|
64 | %td #{time_ago_in_words(@submission.graded_at)} ago (at #{@submission.graded_at.to_formatted_s(:long)}) |
|
64 | - else |
|
65 | - else |
|
65 | %td - |
|
66 | %td - |
|
66 | %tr |
|
67 | %tr |
|
67 | %td.text-right |
|
68 | %td.text-right |
|
68 | %strong Points |
|
69 | %strong Points |
@@ -5,41 +5,50 | |||||
|
5 | .row |
|
5 | .row |
|
6 | .col-md-6 |
|
6 | .col-md-6 |
|
7 | .panel.panel-default |
|
7 | .panel.panel-default |
|
8 | .panel-heading |
|
8 | .panel-heading |
|
9 | .panel-title Info |
|
9 | .panel-title Info |
|
10 | .panel-body |
|
10 | .panel-body |
|
11 | %ul |
|
11 | %ul |
|
12 | %li |
|
12 | %li |
|
13 | List of user information in this format: |
|
13 | List of user information in this format: |
|
14 | %tt user_id,name(,passwd(,alias(,remark))) |
|
14 | %tt user_id,name(,passwd(,alias(,remark))) |
|
15 | %li |
|
15 | %li |
|
16 | Note that |
|
16 | Note that |
|
17 | %tt passwd, alias |
|
17 | %tt passwd, alias |
|
18 | and |
|
18 | and |
|
19 | %tt remark |
|
19 | %tt remark |
|
20 | is optional. |
|
20 | is optional. |
|
21 | %li |
|
21 | %li |
|
22 | When |
|
22 | When |
|
23 | %tt passwd |
|
23 | %tt passwd |
|
24 | or |
|
24 | or |
|
25 | %tt alias |
|
25 | %tt alias |
|
26 | is empty, the original value will be used instead. |
|
26 | is empty, the original value will be used instead. |
|
27 | %li |
|
27 | %li |
|
28 | If the users with the same user_id already exists, existing information will be overwritten. |
|
28 | If the users with the same user_id already exists, existing information will be overwritten. |
|
|
29 | + Example: | ||
|
|
30 | + %ol | ||
|
|
31 | + %li | ||
|
|
32 | + %pre user1,Somchai Jaidee | ||
|
|
33 | + will create (or update) a user with login "user1" and setting the fullname to "Somchai Jaidee", also setting a random password. | ||
|
|
34 | + %li | ||
|
|
35 | + %pre user1,Somchai Jaidee, | ||
|
|
36 | + will create (or update) a user with login "user1" and and setting the fullname "Somchai Jaidee". No change is made to the password unless this is a new user. If this is a new user, a random password will be generated. | ||
|
|
37 | + | ||
|
29 |
|
38 | ||
|
30 | .row |
|
39 | .row |
|
31 | .col-md-6 |
|
40 | .col-md-6 |
|
32 | = form_tag :action => 'create_from_list' do |
|
41 | = form_tag :action => 'create_from_list' do |
|
33 | .form-group |
|
42 | .form-group |
|
34 | = submit_tag 'Create following users',class: 'btn btn-success' |
|
43 | = submit_tag 'Create following users',class: 'btn btn-success' |
|
35 | .form-group |
|
44 | .form-group |
|
36 | .div.checkbox |
|
45 | .div.checkbox |
|
37 | %label |
|
46 | %label |
|
38 | = check_box_tag :add_to_group |
|
47 | = check_box_tag :add_to_group |
|
39 | Also add these users to the following group |
|
48 | Also add these users to the following group |
|
40 | = select_tag "group_id", options_from_collection_for_select( Group.all, 'id','name',params[:group_name]), id: 'group_name',class: 'select2' |
|
49 | = select_tag "group_id", options_from_collection_for_select( Group.all, 'id','name',params[:group_name]), id: 'group_name',class: 'select2' |
|
41 | .form-group |
|
50 | .form-group |
|
42 | = text_area_tag 'user_list', nil, :rows => 50, :cols => 80 |
|
51 | = text_area_tag 'user_list', nil, :rows => 50, :cols => 80 |
|
43 | .col-md-6 |
|
52 | .col-md-6 |
|
44 |
|
53 | ||
|
45 |
|
54 |
@@ -156,49 +156,49 | |||||
|
156 | #get 'report/current_score', to: 'report#current_score', as: 'report_current_score' |
|
156 | #get 'report/current_score', to: 'report#current_score', as: 'report_current_score' |
|
157 | #get 'report/problem_hof(/:id)', to: 'report#problem_hof', as: 'report_problem_hof' |
|
157 | #get 'report/problem_hof(/:id)', to: 'report#problem_hof', as: 'report_problem_hof' |
|
158 | #get "report/login" |
|
158 | #get "report/login" |
|
159 | #get 'report/max_score', to: 'report#max_score', as: 'report_max_score' |
|
159 | #get 'report/max_score', to: 'report#max_score', as: 'report_max_score' |
|
160 | #post 'report/show_max_score', to: 'report#show_max_score', as: 'report_show_max_score' |
|
160 | #post 'report/show_max_score', to: 'report#show_max_score', as: 'report_show_max_score' |
|
161 |
|
161 | ||
|
162 | resource :main, only: [], controller: 'main' do |
|
162 | resource :main, only: [], controller: 'main' do |
|
163 | get 'login' |
|
163 | get 'login' |
|
164 | get 'logout' |
|
164 | get 'logout' |
|
165 | get 'list' |
|
165 | get 'list' |
|
166 | get 'submission(/:id)', action: 'submission', as: 'main_submission' |
|
166 | get 'submission(/:id)', action: 'submission', as: 'main_submission' |
|
167 | get 'announcements' |
|
167 | get 'announcements' |
|
168 | get 'help' |
|
168 | get 'help' |
|
169 | post 'submit' |
|
169 | post 'submit' |
|
170 | end |
|
170 | end |
|
171 | #main |
|
171 | #main |
|
172 | #get "main/list" |
|
172 | #get "main/list" |
|
173 | #get 'main/submission(/:id)', to: 'main#submission', as: 'main_submission' |
|
173 | #get 'main/submission(/:id)', to: 'main#submission', as: 'main_submission' |
|
174 | #post 'main/submit', to: 'main#submit' |
|
174 | #post 'main/submit', to: 'main#submit' |
|
175 | #get 'main/announcements', to: 'main#announcements' |
|
175 | #get 'main/announcements', to: 'main#announcements' |
|
176 |
|
176 | ||
|
177 |
|
177 | ||
|
178 | # |
|
178 | # |
|
179 | get 'tasks/view/:file.:ext' => 'tasks#view' |
|
179 | get 'tasks/view/:file.:ext' => 'tasks#view' |
|
180 | - get 'tasks/download/:id/:file.:ext' => 'tasks#download' |
|
180 | + get 'tasks/download/:id/:file.:ext' => 'tasks#download', as: 'download_task' |
|
181 | get 'heartbeat/:id/edit' => 'heartbeat#edit' |
|
181 | get 'heartbeat/:id/edit' => 'heartbeat#edit' |
|
182 |
|
182 | ||
|
183 | #grader |
|
183 | #grader |
|
184 | get 'graders/list', to: 'graders#list', as: 'grader_list' |
|
184 | get 'graders/list', to: 'graders#list', as: 'grader_list' |
|
185 | namespace :graders do |
|
185 | namespace :graders do |
|
186 | get 'task/:id/:type', action: 'task', as: 'task' |
|
186 | get 'task/:id/:type', action: 'task', as: 'task' |
|
187 | get 'view/:id/:type', action: 'view', as: 'view' |
|
187 | get 'view/:id/:type', action: 'view', as: 'view' |
|
188 | get 'clear/:id', action: 'clear', as: 'clear' |
|
188 | get 'clear/:id', action: 'clear', as: 'clear' |
|
189 | get 'stop' |
|
189 | get 'stop' |
|
190 | get 'stop_all' |
|
190 | get 'stop_all' |
|
191 | get 'clear_all' |
|
191 | get 'clear_all' |
|
192 | get 'clear_terminated' |
|
192 | get 'clear_terminated' |
|
193 | get 'start_grading' |
|
193 | get 'start_grading' |
|
194 | get 'start_exam' |
|
194 | get 'start_exam' |
|
195 |
|
195 | ||
|
196 | end |
|
196 | end |
|
197 |
|
197 | ||
|
198 |
|
198 | ||
|
199 | # See how all your routes lay out with "rake routes" |
|
199 | # See how all your routes lay out with "rake routes" |
|
200 |
|
200 | ||
|
201 | # This is a legacy wild controller route that's not recommended for RESTful applications. |
|
201 | # This is a legacy wild controller route that's not recommended for RESTful applications. |
|
202 | # Note: This route will make all actions in every controller accessible via GET requests. |
|
202 | # Note: This route will make all actions in every controller accessible via GET requests. |
|
203 | # match ':controller(/:action(/:id))(.:format)', via: [:get, :post] |
|
203 | # match ':controller(/:action(/:id))(.:format)', via: [:get, :post] |
|
204 | end |
|
204 | end |
You need to be logged in to leave comments.
Login now