[PATCH conductor] BZ 788117 - Don't set session[:return_to] in UserSessions#new

Matt Wagner matt.wagner at redhat.com
Tue Feb 7 21:53:07 UTC 2012


This is already set up by require_user, so this was redundant.
Additionally, there was a bug where if you logged out, you were sent to /login
which would set return_to to request.request_uri... which was /login.

Resolves https://bugzilla.redhat.com/show_bug.cgi?id=788117
---
 src/app/controllers/user_sessions_controller.rb |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/src/app/controllers/user_sessions_controller.rb b/src/app/controllers/user_sessions_controller.rb
index d791d8d..492ebb1 100644
--- a/src/app/controllers/user_sessions_controller.rb
+++ b/src/app/controllers/user_sessions_controller.rb
@@ -23,7 +23,6 @@ class UserSessionsController < ApplicationController
   layout 'login'
 
   def new
-    session[:return_to] ||= request.get? ? request.request_uri : request.referer
   end
 
   def create
-- 
1.7.6.5




More information about the aeolus-devel mailing list