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

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

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

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

Devise

Deviseとは、Ruby-on-Railsの認証機能を追加するプラグインです。

Ruby on Rails

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

Q&A

解決済

1回答

1060閲覧

application_controllerに定義したどのコントローラーのどのアクションでもbefore_actionが作動しない

widget11

総合スコア221

Ruby on Rails 5

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

Devise

Deviseとは、Ruby-on-Railsの認証機能を追加するプラグインです。

Ruby on Rails

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

0グッド

0クリップ

投稿2019/10/02 09:49

ログイン認証にdeviseを用いたアプリケーションを作っています。

最近のウェブサービスだとよくあるようにまず初回登録とメールで認証を終えた後はログイン後にウィザード形式の登録画面に飛ばしたいです。
また業務上必要なプロフィール等は絶対に入力させたいので、
ログインしたユーザーエンティティが持つfirst_login_flag(bool)というカラムがfalseである限り、
どのコントローラーのどのアクションを踏んでもウィザード形式の登録画面を飛ばすようにしたいです。
ウィザード登録できたらこのフラグをtrueにするようにします。

しかし現在書いている処理ですとログイン後は上手くウィザード画面に飛んでくれるのですが、
例えば直接/user/show/1とurlを叩いても、普通にその画面に遷移ができてしまいます。

処理は以下のようにしております

#application_controller.rb class ApplicationController < ActionController::Base before_action :set_input, except: :after_sign_in_path_for  def after_sign_in_path_for(resource)  set_input  end   def set_input   if user_signed_in?    if user.first_login_flag == false    first_user_steps_path    end else root_path end  end end

一応説明ですが、
共通コントローラーであるapplication_controllerにset_inputというdeviseの組込メソッドを用いたメソッドを用意してます、
ifがネストしてて見辛いかもしれませんが、処理としては
・ログインしているかを確認
・ログインしていたらログインユーザーのモデルからfirst_login_flagの値を確認
・falseだったらfirst_user_steps_path(ウィザード画面のアクション)に飛ばすようにしており、
これを全コントローラーの共通処理としたいのでapplication_controllerのbefore_actionとしています。
ちなみにafter_sign_in_path_forをexceptしないと2重リダイレクトが起こる為、上記のような書き方になっております。

この書き方の何がいけないのでしょうか?
長くなりましたがよろしくお願い致します。

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

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

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

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

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

guest

回答1

0

ベストアンサー

xxx_path が routes.rb で定義したものであれば、パスの文字列が返ってくるだけですから redirect_to を使ってみてください

投稿2019/10/02 10:27

退会済みユーザー

退会済みユーザー

総合スコア0

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

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

widget11

2019/10/03 01:58 編集

ご回答ありがとうございます。  def set_input   if user_signed_in?    if user.first_login_flag == false    redirect_to first_user_steps_url end end 書きそびれてもうしわけないのですが、以上のようにset_input内でredirect_toをしてしまうと ”localhost でリダイレクトが繰り返し行われました” とブラウザ上に表示され、リダイレクトループが永遠と起きてしまう事象が発生しまいます。 何故起きるかは調べても良く分からず。。。 情報が少なすぎて申し訳ないのですが何故リダイレクトループが起きるか分かりますでしょうか? よろしくお願い致します。
Mugheart

2019/10/03 02:01

ログインしていないのでbefore_actionでrootにリダイレクトされる、リダイレクトされた先でまたbefore_actionが実行され、ログインしてないのでrootにリダイレクトされる、リダイレクトされた先でまた...と無限ループしているからです。
widget11

2019/10/03 02:38

なるほど。。盲点でした。。。 そうなると単純にelse句含め、それ以降の処理を消し ただuserがsign_inしてるだけかをみるようにすれば問題は解決しそうですかね?
退会済みユーザー

退会済みユーザー

2019/10/03 03:41

現時点の set_input の内容がどのようになっているのか判然としないので正確なアドバイスが難しいですが、「ログインしていないならログインさせる」「ログインしているかつプロフィール未入力ならプロフィール入力に飛ぶ」をそれぞれ書けばよいように見えます。 あと、何がおきているのかわからなくなったら log/development.log を見ると良いです。
widget11

2019/10/04 01:09

ありがとうございます。無事解決しました!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問