Merge remote-tracking branch 'upstream/pull/3279'

diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb
index ec30eb4e7b14a3267b44dfcbd0386939c83009f9..727d8ee3faed139adce3edc3b0922bb81b94be0e 100644 (file)
--- a/app/controllers/users_controller.rb
+++ b/app/controllers/users_controller.rb
@@ -30,7 +30,7 @@ class UsersController < ApplicationController
         # Already agreed to terms, so just show settings
         redirect_to :action => :account, :display_name => current_user.display_name
       elsif current_user.nil? && session[:new_user].nil?
-        redirect_to :action => :login, :referer => request.fullpath
+        redirect_to login_path(:referer => request.fullpath)
       end
     end
   end

diff --git a/test/controllers/users_controller_test.rb b/test/controllers/users_controller_test.rb
index a73e48211f45dcc5a88533f8e43a19b3f8b8940a..10c5c3bee674e626585f7ba8632ff42455cc2afd 100644 (file)
--- a/test/controllers/users_controller_test.rb
+++ b/test/controllers/users_controller_test.rb
@@ -414,6 +414,12 @@ class UsersControllerTest < ActionDispatch::IntegrationTest
     assert_redirected_to :action => :terms, :referer => "/user/#{ERB::Util.u(user.display_name)}/account"
   end
 
+  def test_terms_not_logged_in
+    get user_terms_path
+
+    assert_redirected_to login_path(:referer => "/user/terms")
+  end
+
   def test_go_public
     user = create(:user, :data_public => false)
     session_for(user)