diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb index d6efa4124..0e5f6ba54 100644 --- a/app/controllers/application_controller.rb +++ b/app/controllers/application_controller.rb @@ -300,7 +300,7 @@ class ApplicationController < ActionController::Base end def redirect_back_or_default(default) - back_url = CGI.unescape(params[:back_url].to_s) + back_url = params[:back_url].to_s if !back_url.blank? begin uri = URI.parse(back_url) diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb index a56371c5f..da6f6ae28 100644 --- a/app/controllers/users_controller.rb +++ b/app/controllers/users_controller.rb @@ -174,7 +174,7 @@ class UsersController < ApplicationController def destroy @user.destroy respond_to do |format| - format.html { redirect_to_referer_or(users_url) } + format.html { redirect_back_or_default(users_url) } format.api { render_api_ok } end end diff --git a/app/views/users/index.html.erb b/app/views/users/index.html.erb index a477a5c90..f88b5231e 100644 --- a/app/views/users/index.html.erb +++ b/app/views/users/index.html.erb @@ -46,7 +46,7 @@ <%= format_time(user.last_login_on) unless user.last_login_on.nil? %> <%= change_status_link(user) %> - <%= delete_link user_path(user) unless User.current == user %> + <%= delete_link user_path(user, :back_url => users_path(params)) unless User.current == user %> <% end -%>