【Rails】【devise関連？】一定条件でかかるリダイレクト処理を一定画面だけ除外したい

Question

今、以下のような新規登録フォームを作成しています。

①アカウント登録（メアド/Twitter）
②プロフィール設定
③登録完了

■仕様
①が完了した時点でUsersテーブルにデータが追加され実質ログイン状態となります。
①完了後、画面から離脱→再方→②から再開という場合に備えたい。

そのために、①が完了していて、②が完了していないユーザーがログインしてきた場合は、自動的に②の画面に遷移させています。

それが、ApplicationController内に以下のようなメソッドになります。

before_action :fix_user_sign_up_detail

▼ApplicationController

class ApplicationController < ActionController::Base
  protect_from_forgery with: :exception

  before_action :fix_user_sign_up_detail
  before_action :store_current_location
  before_action :configure_permitted_parameters, if: :devise_controller?

  private

  def fix_user_sign_up_detail
    redirect_to edit_user_registration_detail_path if user_signed_in? && current_user.handle_name.blank?
  end

  def store_current_location
    store_location_for(:user, request.url)
  end

  protected
    def configure_permitted_parameters
      devise_parameter_sanitizer.permit(:sign_up, keys:[:email])
      devise_parameter_sanitizer.permit(:sign_in, keys:[:login, :handle_name])
      devise_parameter_sanitizer.permit(:account_update, keys:[:handle_name, :email, :password, :password_confirmation, :current_password])
    end
end

ただ、これだと、②の画面に遷移した時にもリダイレクトがかかり、②の画面の無限ループに陥ります。

それを回避する施策で、Users::RegistrationsControllerにて以下のメソッドを追加しています。

prepend_before_action :authenticate_scope!, only: [:edit_detail, :update_detail]

class Users::RegistrationsController < Devise::RegistrationsController

  prepend_before_action :authenticate_scope!, only: [:edit_detail, :update_detail]
  
 # Twitter/メアドの選択画面
  def select
  end

  # GET /resource/sign_up
  def new
    super
  end
 
 # ②の画面。名前やプロフィールを設定する。
  def edit_detail
  end

  def update_detail
    resource_updated = resource.update(detail_params)
    if resource_updated
      @modal = {name: 'update-detail-modal'}
    end
    render :edit_detail
  end



  protected

    def detail_params
      params.require(:user).permit(:icon, :handle_name, :profile)
    end

    # The path used after sign up.
    def after_sign_up_path_for(resource)
      edit_user_registration_detail_path
    end

end

しかしながら、edit_user_registration_detail_path（②の画面。edit_detailアクションのViewです。）に遷移するとリダイレクトの無限ループになります。

これってそもそも、prepend_before_action :authenticate_scope!, only: [:edit_detail, :update_detail]の解釈が誤っていますか？？つまり、リダイレクト（before_action :fix_user_sign_up_detail）を適用外にする効力は持っていない？？

もし、その場合、どのような方法で、edit_detailとupdate_detailで無限ループに陥るのを回避出来ますでしょうか？？

※before_action :fix_user_sign_up_detail,except:%w( edit_detail update_detail )でも回避出来る？と思いますが、あまり良い方法に思えず、これ以外の方法が知りたいのです。

Answer 1

before_action の if: オプションを使う手もありますね。

class ApplicationController < ActionController::Base
  before_action :fix_user_sign_up_detail, if: :needed

  private

  def needed
    true
  end
end

class Users::RegistrationsController < Devise::RegistrationsController
  private 

  def needed
    false
  end
end

(メソッド名はもうちょっとマシな名前にしてあげてください)

---

①この方法でコントローラー毎ではなく、アクション毎のスキップは出来ないでしょうか？

できますよ。 #needed をそのように実装してあげればいいだけです。

def needed
  params[:action] != 'kino_action_ha_skip'
end

②結局のところ、prepend_before_action :authenticate_scope!, only: [:edit_detail, :update_detail]　というメソッドでは、アクション毎にbefore_actionをskip出来ないのでしょうか？？

before_action は通常書く度に後ろに追加されていきますが prepend_before_action は先頭に追加されていくというのが違いという風に思っています。

なぜskipするのに使えると思ったのかがわからないので、なんとも言えないですが、prepend_before_action で登録済の action をキャンセルする方法は僕はわかりません。ごめんなさい。

Comments

[gogoackman3]

ありがとうございます！２点ありまして、
①この方法でコントローラー毎ではなく、アクション毎のスキップは出来ないでしょうか？
調べたところ、コントローラー毎で制御出来るのは理解出来ました。
https://blog.hello-world.jp.net/ruby/2637/

②結局のところ、prepend_before_action :authenticate_scope!, only: [:edit_detail, :update_detail]　というメソッドでは、アクション毎にbefore_actionをskip出来ないのでしょうか？？

[gogoackman3]

再度、回答ありがとうございます！！頂いた方法で実装してみたいと思います！

②のprepend_before_action :authenticate_scope!でskip出来るのかな？と思ったのは、以下の記事を見たからという安直なものでした・・・！

http://qiita.com/MasatoYoshioka@github/items/7de96ac05bc0c17b7535