Description:
added default contest
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r308:c400f7405eee - - 3 files changed: 23 inserted, 1 deleted
@@ -51,13 +51,13 | |||||
|
51 | @user.activated = true |
|
51 | @user.activated = true |
|
52 | if @user.save |
|
52 | if @user.save |
|
53 | flash[:notice] = 'User was successfully created.' |
|
53 | flash[:notice] = 'User was successfully created.' |
|
54 | redirect_to :action => 'list' |
|
54 | redirect_to :action => 'list' |
|
55 | else |
|
55 | else |
|
56 | render :action => 'new' |
|
56 | render :action => 'new' |
|
57 | - end |
|
57 | + end |
|
58 | end |
|
58 | end |
|
59 |
|
59 | ||
|
60 | def create_from_list |
|
60 | def create_from_list |
|
61 | lines = params[:user_list] |
|
61 | lines = params[:user_list] |
|
62 |
|
62 | ||
|
63 | note = [] |
|
63 | note = [] |
@@ -50,12 +50,13 | |||||
|
50 | #validates_presence_of :province |
|
50 | #validates_presence_of :province |
|
51 |
|
51 | ||
|
52 | attr_accessor :password |
|
52 | attr_accessor :password |
|
53 |
|
53 | ||
|
54 | before_save :encrypt_new_password |
|
54 | before_save :encrypt_new_password |
|
55 | before_save :assign_default_site |
|
55 | before_save :assign_default_site |
|
|
56 | + before_save :assign_default_contest | ||
|
56 |
|
57 | ||
|
57 | # this is for will_paginate |
|
58 | # this is for will_paginate |
|
58 | cattr_reader :per_page |
|
59 | cattr_reader :per_page |
|
59 | @@per_page = 50 |
|
60 | @@per_page = 50 |
|
60 |
|
61 | ||
|
61 | def self.authenticate(login, password) |
|
62 | def self.authenticate(login, password) |
@@ -270,12 +271,25 | |||||
|
270 | end |
|
271 | end |
|
271 | end |
|
272 | end |
|
272 | rescue |
|
273 | rescue |
|
273 | end |
|
274 | end |
|
274 | end |
|
275 | end |
|
275 |
|
276 | ||
|
|
277 | + def assign_default_contest | ||
|
|
278 | + # have to catch error when migrating (because self.site is not available). | ||
|
|
279 | + begin | ||
|
|
280 | + if self.contests.length == 0 | ||
|
|
281 | + default_contest = Contest.find_by_name(Configuration['contest.default_contest_name']) | ||
|
|
282 | + if default_contest | ||
|
|
283 | + self.contests = [default_contest] | ||
|
|
284 | + end | ||
|
|
285 | + end | ||
|
|
286 | + rescue | ||
|
|
287 | + end | ||
|
|
288 | + end | ||
|
|
289 | + | ||
|
276 | def password_required? |
|
290 | def password_required? |
|
277 | self.hashed_password.blank? || !self.password.blank? |
|
291 | self.hashed_password.blank? || !self.password.blank? |
|
278 | end |
|
292 | end |
|
279 |
|
293 | ||
|
280 | def self.encrypt(string,salt) |
|
294 | def self.encrypt(string,salt) |
|
281 | Digest::SHA1.hexdigest(salt + string) |
|
295 | Digest::SHA1.hexdigest(salt + string) |
@@ -104,13 +104,21 | |||||
|
104 | }, |
|
104 | }, |
|
105 |
|
105 | ||
|
106 | { |
|
106 | { |
|
107 | :key => 'contest.confirm_indv_contest_start', |
|
107 | :key => 'contest.confirm_indv_contest_start', |
|
108 | :value_type => 'boolean', |
|
108 | :value_type => 'boolean', |
|
109 | :default_value => 'false' |
|
109 | :default_value => 'false' |
|
|
110 | + }, | ||
|
|
111 | + | ||
|
|
112 | + { | ||
|
|
113 | + :key => 'contest.default_contest_name', | ||
|
|
114 | + :value_type => 'string', | ||
|
|
115 | + :default_value => 'none', | ||
|
|
116 | + :description => "New user will be assigned to this contest automatically, if it exists. Set to 'none' if there is no default contest." | ||
|
110 | } |
|
117 | } |
|
|
118 | + | ||
|
111 | ] |
|
119 | ] |
|
112 |
|
120 | ||
|
113 |
|
121 | ||
|
114 | def create_configuration_key(key, |
|
122 | def create_configuration_key(key, |
|
115 | value_type, |
|
123 | value_type, |
|
116 | default_value, |
|
124 | default_value, |
You need to be logged in to leave comments.
Login now