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

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

新規登録して質問してみよう
ただいま回答率
85.48%
Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

解決済

1回答

5171閲覧

Filter機能でBefore process_actionがdefinedされてしまう

rieru

総合スコア6

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

0クリップ

投稿2019/12/01 08:27

前提・実現したいこと

<前提>
「速習実践ガイド」で学習中です。現在、Chapte4のタスク管理アプリケーションを作成しています。
<実現したいこと>
ログインしていなければタスク管理を利用できないようにしたい。

タスク管理機能の書くアクションの前に処理されるフィルタでユーザーがログインしているかどうか調べ、ログインしていなければログイン画面にリダイレクトさせ、タスク管理機能を実質的に利用できなくさせたい。

loginがdefineされた理由がわからずに困っております。

app/controllers/application_controller.rbbefore_action :login_requiedを追記

app/controllers/sessions_controller.rbskip_before_action :login_requiredを追記したところ下記のエラーに遭遇しました。

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

ArgumentError Before process_action callback :login_required has not been defined class SessionsController < ApplicationController skip_before_action :login_required skip_before_action :method_name def new end

該当のソースコード

application_controller.rb

application_controller.rb

1class ApplicationController < ActionController::Base 2 helper_method :current_user 3 before_action :login_requied 4 private 5 6 def current_user 7 @current_user ||= User.find_by(id: session[:user_id]) if session[:user_id] 8 end 9 10 def login_requied 11 redirect_to login_path unless current_user 12 end 13end

app/controllers/sessions_controller.rb

session_controller.rb

1Class SessionsController < ApplicationController 2 skip_before_action :login_required 3 skip_before_action :method_name 4 def new 5 end 6 7 def create 8 user = User.find_by(email: session_params[:email]) 9 10 if user&.authenticate(session_params[:email]) 11 session[:user_id] = user.user_id 12 redirect_to root_path, notice: 'ログインしました。' 13 else 14 render :new 15 end 16 end 17 18 def destroy 19 reset_session 20 redirect_to root_path, notice: 'ログアウトしました。' 21 end 22 23private 24 25 def session_params 26 params.require(:session).permit(:email, :password) 27 end 28end 29

試したこと

skip_before_action :login_required
skip_before_action :login_required, raise: falseに変えてみてもredirectループになってしまいました。

ここに問題に対して試したことを記載してください。

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

ruby 2.6.3 rails 2.5.3
ここにより詳細な情報を記載してください。

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

loginがdefineされた理由がわからずに困っております。
とありますが、
:login_required has not been defined
は  :login_required が未だ define されていない
というエラーです。

def login_required を叱るべく定義して下さい。

投稿2019/12/01 11:03

winterboum

総合スコア23347

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

rieru

2019/12/01 13:21

winterboum様。ご回答頂きありがとうございます。`def login_required`の定義について、全機能に共通の制限を設ける為`application_controller.rb`の末尾に記述していたのですが、:login_required が未だ define されていないという旨のエラーが返ってきた為困っておりました。 追加での質問になり申し訳ないのですが、定義する場所が違っていたということなのでしょうか。
winterboum

2019/12/01 22:38

それですと、質問のタイトルも文章も逆になってますね。 protected => クラス内、同一パッケージ、サブクラスからアクセス可 private => クラス内のみアクセス可能 です
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問