Fixes that user's last_login_on was not set when using registration with automatic activation.

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@2594 e93f8b46-1217-0410-a6f0-8f06a7374b81

Fixes that user's last_login_on was not set when using registration with automatic activation.
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@2594 e93f8b46-1217-0410-a6f0-8f06a7374b81
4181f859 · Jean-Philippe Lang · 175ac71b · 4181f859 · 4181f859
Commit 4181f859 authored Mar 15, 2009 by Jean-Philippe Lang
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 1 deletion

account_controller.rb app/controllers/account_controller.rb +1 -0

account_test.rb test/integration/account_test.rb +4 -1

No files found.
--- a/app/controllers/account_controller.rb
+++ b/app/controllers/account_controller.rb
@@ -248,6 +248,7 @@ class AccountController < ApplicationController
  def register_automatically(user, &block)
    # Automatic activation
    user.status = User::STATUS_ACTIVE
+    user.last_login_on = Time.now
    if user.save
      self.logged_user = user
      flash[:notice] = l(:notice_account_activated)

--- a/test/integration/account_test.rb
+++ b/test/integration/account_test.rb
@@ -110,7 +110,10 @@ class AccountTest < ActionController::IntegrationTest
    assert_response :success
    assert_template 'my/account'
    
-    assert User.find_by_login('newuser').active?
+    user = User.find_by_login('newuser')
+    assert_not_nil user
+    assert user.active?
+    assert_not_nil user.last_login_on
  end
  
  def test_register_with_manual_activation