質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.62%

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

解決済

回答 1

投稿

  • 評価
  • クリップ 1
  • VIEW 444

furu0810

score 10

前提・実現したいこと

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

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

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

以下の画像になります。

イメージ説明

該当のソースコード

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

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/02/13 22:00

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

    キャンセル

  • 2020/02/13 22:15

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.62%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る