summaryrefslogtreecommitdiffstats
path: root/app/controllers
diff options
context:
space:
mode:
authorEric Davis <edavis@littlestreamsoftware.com>2010-08-03 15:26:50 +0000
committerEric Davis <edavis@littlestreamsoftware.com>2010-08-03 15:26:50 +0000
commit13234f85521d3d8a673fecda3e1a1f4abd1025b8 (patch)
treeaf69b26e6154a7410d1b8cc6f7e16e207cd736ca /app/controllers
parent25037b841bfd52ca05c6c1a179c06fd509f17443 (diff)
downloadredmine-13234f85521d3d8a673fecda3e1a1f4abd1025b8.tar.gz
redmine-13234f85521d3d8a673fecda3e1a1f4abd1025b8.zip
Refactor: Add methods to User to edit the encapsulate the status field.
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@3906 e93f8b46-1217-0410-a6f0-8f06a7374b81
Diffstat (limited to 'app/controllers')
-rw-r--r--app/controllers/account_controller.rb12
1 files changed, 6 insertions, 6 deletions
diff --git a/app/controllers/account_controller.rb b/app/controllers/account_controller.rb
index b0a5b42a7..5fa56b167 100644
--- a/app/controllers/account_controller.rb
+++ b/app/controllers/account_controller.rb
@@ -83,9 +83,9 @@ class AccountController < ApplicationController
else
@user = User.new(params[:user])
@user.admin = false
- @user.status = User::STATUS_REGISTERED
+ @user.register
if session[:auth_source_registration]
- @user.status = User::STATUS_ACTIVE
+ @user.activate
@user.login = session[:auth_source_registration][:login]
@user.auth_source_id = session[:auth_source_registration][:auth_source_id]
if @user.save
@@ -116,8 +116,8 @@ class AccountController < ApplicationController
token = Token.find_by_action_and_value('register', params[:token])
redirect_to(home_url) && return unless token and !token.expired?
user = token.user
- redirect_to(home_url) && return unless user.status == User::STATUS_REGISTERED
- user.status = User::STATUS_ACTIVE
+ redirect_to(home_url) && return unless user.registered?
+ user.activate
if user.save
token.destroy
flash[:notice] = l(:notice_account_activated)
@@ -170,7 +170,7 @@ class AccountController < ApplicationController
user.mail = registration['email'] unless registration['email'].nil?
user.firstname, user.lastname = registration['fullname'].split(' ') unless registration['fullname'].nil?
user.random_password
- user.status = User::STATUS_REGISTERED
+ user.register
case Setting.self_registration
when '1'
@@ -241,7 +241,7 @@ class AccountController < ApplicationController
# Pass a block for behavior when a user fails to save
def register_automatically(user, &block)
# Automatic activation
- user.status = User::STATUS_ACTIVE
+ user.activate
user.last_login_on = Time.now
if user.save
self.logged_user = user