【Ruby on Railsチュートリアル 第10章】フレンドリーフォワーディングが実装できない

Question

ちょうど1年前にもteratail内にて同質問がありましたが、回答が付いておりませんでした。
↓
Rails tutorial 10章のフレンドリーフォワーディングが実装できない

ログイン前に編集ページにアクセスし、ログインしたら編集ページにリダイレクトするはずが
ユーザー詳細ページに飛んでしまう状況です。

分かる方教えてください。
宜しくお願い致します。

app/controllers/users_controller.rb

class UsersController < ApplicationController
  
  before_action :logged_in_user, only: [:edit, :update]
  before_action :correct_user,   only: [:edit, :update]
  
  def show
    @user = User.find(params[:id])
    
  end
  
  def new
    @user = User.new
  end
  
  def create
    @user = User.new(user_params)
    if @user.save
      log_in @user
      flash[:success] = "Welcome to the Sample App!"
      redirect_to @user
    else
      render 'new'
    end
  end
  
  def edit
    @user = User.find(params[:id])
  end
  
  def update
    @user = User.find(params[:id])
    if @user.update_attributes(user_params)
       # 更新成功
       flash[:success] = "Profile updated"
       redirect_to @user
    else
      render 'edit'
    end
  end
  
  private

    def user_params
      params.require(:user).permit(:name, :email, :password,
                                   :password_confirmation)
    end
    
    # ログイン済みユーザーかどうか確認
    def logged_in_user
      unless logged_in?
        store_location
        flash[:danger] = "Please log in."
        redirect_to login_url
      end
    end
    
    # 正しいユーザーかどうか確認
    def correct_user
      @user = User.find(params[:id])
      redirect_to(root_url) unless current_user?(@user)
    end
    
    
    
    
    
end

app/helpers/sessions_helper.rb

module SessionsHelper
    
  #渡されたユーザーでログインする
  def log_in(user)
    session[:user_id] = user.id
  end
  
  #現在ログイン中のユーザーを返す (いる場合)
  def current_user
     if session[:user_id]
      @current_user ||= User.find_by(id: session[:user_id])
    end 
  end
  
  #ユーザーがログインしていればtrue、その他ならfalseを返す
  def logged_in?
    !current_user.nil?
  end
  
  # 現在のユーザーをログアウトする
  def log_out
    session.delete(:user_id)
    @current_user = nil
  end
  
  # 渡されたユーザーがログイン済みユーザーであればtrueを返す
  def current_user?(user)
    user == current_user
  end



  # 記憶したURL (もしくはデフォルト値) にリダイレクト
  def redirect_back_or(default)
    redirect_to(session[:forwarding_url] || default)
    session.delete(:forwarding_url)
  end

  # アクセスしようとしたURLを覚えておく
  def store_location
    session[:forwarding_url] = request.original_url if request.get?
  end

  def create
    user = User.find_by(email: params[:session][:email].downcase)
    if user && user.authenticate(params[:session][:password])
      log_in user
      params[:session][:remember_me] == '1' ? remember(user) : forget(user)
      redirect_back_or user
    else
      flash.now[:danger] = 'Invalid email/password combination'
      render 'new'
    end
  end


end

app/controllers/sessions_controller.rb

class SessionsController < ApplicationController
  
  def new
  end
  
  def create
    user = User.find_by(email: params[:session][:email].downcase)
    if user && user.authenticate(params[:session][:password])
      log_in user
      redirect_to user
    else
      flash.now[:danger] = "Invalid email/password combination"
      render 'new'
    end
  end
  
  def destroy
    log_out
    redirect_to root_path
  end
end

Answer 1

app/controllers/sessions_controller.rb
の

def create
    user = User.find_by(email: params[:session][:email].downcase)
    if user && user.authenticate(params[:session][:password])
      log_in user
      redirect_to user #ココ

でuserページにリダイレクトしているのでユーザー詳細ページに飛んでしまいます。

redirect_to userを
↓
redirect_back_or(user)
(メソッドはapp/helpers/sessions_helper.rbで定義済み）
に変更するとフレンドリーフォワーディングが実装できるはずです。

Comments

[pecchan]

有難う御座います。

redirect_back_or(user)
と変更しましたが、変わりませんでした。

詳細ページ(users/id)に飛ばされます。

[tttu]

session.helperはapplication.controllerで呼び出されていますか？？
(store_locationメソッドをuser.controller内で使うために必要です。)

[pecchan]

度々有難う御座います。

application.controller確認しました。
以下のようになってたので呼び出していると思うのですが・・。
class ApplicationController < ActionController::Base
protect_from_forgery with: :exception
include SessionsHelper
end