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

Question

ログイン認証に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重リダイレクトが起こる為、上記のような書き方になっております。

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

Answer 1

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

Comments

[widget11]

ご回答ありがとうございます。

　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 でリダイレクトが繰り返し行われました”
とブラウザ上に表示され、リダイレクトループが永遠と起きてしまう事象が発生しまいます。
何故起きるかは調べても良く分からず。。。
情報が少なすぎて申し訳ないのですが何故リダイレクトループが起きるか分かりますでしょうか？
よろしくお願い致します。

[rhiroe]

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

[widget11]

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

[退会済みユーザー]

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

[widget11]

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