Ruby on rails: deviseのvalidateについて

Question

###validateに引っかかった時にエラーメッセージを表示させたい。
deviseで利用規約に同意するカラムを追加し、登録時にチェックボックスが空の場合、そのまま送信せずにエラーメッセージを表示させたい。

###現状について
このようにvalidateをかけて、チェックボックスが空の場合rollbackされるようになっているのが、これだとユーザー側が登録できたと思ってしまうので、送信せずにチェックボックスがチェックされるまで送信せずにエラーメッセージを表示させたい

#ApplicationController
  before_action :configure_permitted_parameters, if: :devise_controller?
    protect_from_forgery with: :exception

    def configure_permitted_parameters
        devise_parameter_sanitizer.permit(:sign_up, keys:[:user_type_id, :accepted])
    end

#User.rb
before_validation :before_validations

  def before_validations
    if accepted == false
      #ここに何を書けばいいか分からない
      #flashをここに書いたところエラーが起きた
　　　 flash[:alert] = "利用規約に同意のほどお願いします。"
    end
  end

class AddAcceptedToUsers < ActiveRecord::Migration[5.2]
  def change
    add_column :users, :accepted, :boolean, default: false, null: false
  end
end

Answer 1

javascriptでやるほうが適切な感じがしますね。

checkboxのON, OFFでsubmitにdisabledをセットしたり外したりすればよいかと思います。

Comments

[rails_ruby]

JavaScriptで一応、空の場合送信出来ないようにはしているのですが、念のためにバリデーションをかけて同意されてない状態で送信が万が一された時にリダイレクトさせたいなって考えてるのですがやはり不必要な処理なんでしょうか？

[odyu]

全然不必要ではないです。考え方は人それぞれですが、

js周り → UI/UXを利便性高める
controller周り → UI/UXを安全性を高める

そう考えれば良いと思います。

[rails_ruby]

分かりました。
稚拙な質問なんですが、user.rbでfalseの時にリダイレクトする処理を書いてみたのですがエラーが出てしまったのですが、モデルの方では基本的にリダイレクト処理は書けないのでしょうか？

[odyu]

モデルには書けないですね。

大雑把な説明になりますが、モデルでバリデーション、コントローラーでレンダリングするか、リダイレクトするかを判定することになります。

これが参考になるかと思います

https://qiita.com/dawn_628/items/99cf5b0b8e8b38ba73b2

処理が意外と少ないように見えますが、deviseとrailsが裏で色々やってくれています