From: Tom Hughes <tom@compton.nu>
Date: Fri, 26 Feb 2016 16:30:32 +0000 (+0000)
Subject: Treeat user as logged in after a password reset
X-Git-Tag: live~3904
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/f0fa701563ee40723380f4f26a3eedc83b541cc9?hp=480c17be58ded999ecf4f2918777e1368e28932e

Treeat user as logged in after a password reset

Fixes #1160
---

diff --git a/app/controllers/user_controller.rb b/app/controllers/user_controller.rb
index 394f8676f..766ab37d1 100644
--- a/app/controllers/user_controller.rb
+++ b/app/controllers/user_controller.rb
@@ -182,7 +182,7 @@ class UserController < ApplicationController
           if @user.save
             token.destroy
             flash[:notice] = t "user.reset_password.flash changed"
-            redirect_to :action => "login"
+            successful_login(@user)
           end
         end
       else
diff --git a/test/controllers/user_controller_test.rb b/test/controllers/user_controller_test.rb
index 37b8a7f30..6a2769f17 100644
--- a/test/controllers/user_controller_test.rb
+++ b/test/controllers/user_controller_test.rb
@@ -702,7 +702,8 @@ class UserControllerTest < ActionController::TestCase
     # Test setting a new password
     post :reset_password, :token => token.token, :user => { :pass_crypt => "new_password", :pass_crypt_confirmation => "new_password" }
     assert_response :redirect
-    assert_redirected_to :action => :login
+    assert_redirected_to root_path
+    assert_equal users(:inactive_user).id, session[:user]
     user = User.find(users(:inactive_user).id)
     assert_equal "active", user.status
     assert_equal true, user.email_valid