タスク管理アプリにログイン後にログインページにリダイレクトされてしまう

Question

前提・実現したいこと

ログイン後にタスク管理アプリの一覧を表示させるようにしたいです。

発生している問題・エラーメッセージ

ログインしてもログインページにリダイレクトされてしまいます。

以下の画像になります。

該当のソースコード

該当する可能性のあるソースコードです。

sessions_controller.rb

class SessionsController < ApplicationController
  
  def new
  end

  def create
    email = params[:session][:email].downcase
    password = params[:session][:password]
    if login(email, password)
      flash[:success] = "ログインに成功しました。"
      redirect_to root_url
    else
      flash.now[:danger] = "ログインに失敗しました。"
      render :new
    end
  end

  def destroy
    session[:user_id] = nil
    flash[:success] = "ログアウトしました。"
    redirect_to root_url
  end
  
  private
  
  def login(email, password)
    @user = User.find_by(email: email)
    if @user && @user.authenticate(password)
      #　ログイン成功(sessionとcookieのログイン情報を照合)
      session[:user_id] = @user_id
      return true
    else
      #ログイン失敗
      return false
    end
  end
end

tasks_controller.rb

class TasksController < ApplicationController
  
  before_action :require_user_logged_in
  before_action :correct_user, only: [:destroy, :show, :update, :edit]
 # before_action :set_task, only: [:show, :edit, :update, :destroy]
  
   
  def index
        
    @tasks = current_user.tasks.order(id: :desc).page(params[:page]).per(10)
        
  end
    
  def create
        
    @task = Task.new(task_params)
        
      if @task.save
        flash[:success] = "タスクが正常に追加されました"
        redirect_to @task
                
      else 
        flash.now[:danger] = "タスクが正常に投稿されませんでした。"
        render :new
            
      end
        
  end
    
  def new
        
    @task = Task.new
        
  end
    
  def edit
        
  end
    
  def show
        
        
  end
    
  def update
        
        
    if @task.update(task_params)
      flash[:success] = "タスク内容は正常に更新されました"
      redirect_to @task
    else
      flash.now[:danger] = "タスク内容は正常に更新されませんでした"
      render :edit
                
    end 
        
  end
    
  def destroy
        
    @task.destroy
        
    flash[:success] = "タスクは正常に削除されました"
    redirect_to tasks_url
        
  end
    
  private
    
  # tasks_controller.rb内の@task = Task.find(params[:id])の部分テンプレート(パーション)
    
  #def set_task
        
   # @task = Task.find(params[:id])
    
  #end    
    
  #Strong parameter
    
  def task_params
        
    params.require(:task).permit(:content, :status)
        
  end
  
  def correct_user
    @task = current_user.tasks.find_by(id: params[:id])
    unless @task
      redirect_to root_url
    end
  end
end

application_controller.rb

class ApplicationController < ActionController::Base
  
  include SessionsHelper
  
  private
  
  def require_user_logged_in
    unless logged_in?
      redirect_to login_url
    end
  end
  
end

sessions_helper.rb

module SessionsHelper
  
  def current_user
    @current_user ||= User.find_by(id: session[:user_id])
  end
  
  def logged_in?
    !!current_user
  end
end

routes.rb

Rails.application.routes.draw do
 

  # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
    root to: "tasks#index"
    
    get "login", to: "sessions#new"
    post "login", to: "sessions#create"
    delete "logout", to: "sessions#destroy"
    
    get "signup", to: "users#new"
   
    resources :tasks
    resources :users, only: [:new, :create] 
end

users_controller.rb

class UsersController < ApplicationController
  
  def new
    @user = User.new
  end

  def create
    @user = User.new(user_params)
    
    if @user.save
      flash[:success] = "ユーザーを登録しました。"
    redirect_to login_url
    else
      flash.now[:danger] = "ユーザーを登録できませんでした。"
    render :new
    end
  end

  def show
    
  end
  
  def index
    
  end
  
  #Strong Parameter 
  
  private
  
  def user_params
    params.require(:user).permit(:name, :email, :password, :password_confirmation)
  end
  
end

試したこと

before_actionの定義しているコントローラーに誤りがないか確認をしました。
また、テキストの手順通り行っているか確認をしたのですが原因を見つけることが出来ませんでした。

補足情報（FW/ツールのバージョンなど）

Rails 5.2.2

Ruby 2.5.3

AWS Cloud9

Answer 1

SessionsControllerの@user_idが@user.idだと思いました。
@user_idはどこにも定義されていないので、session[user_id]にnilのような値が入っている用に見えます。
なので、require_user_logged_inメソッドのなかでログイン判定がfalseになってlogin_urlにリダイレクトされていると思います。

こちらに修正したコードを記載します。

class SessionsController < ApplicationController
  ...

  private

  def login(email, password)
    @user = User.find_by(email: email)
    if @user && @user.authenticate(password)
      #　ログイン成功(sessionとcookieのログイン情報を照合)
      session[:user_id] = @user.id # ここを修正しました。
      return true
    else
      #ログイン失敗
      return false
    end
  end
end

Comments

[furu0810]

ご指摘頂いたところを修正した結果、無事にタスク一覧画面に遷移しました。
ありがとうございました。

[shgtkshruch]

無事に解決できてよかったです〜。