バリデーションのonオプションをupdate内でうまく発動したい。

Question

validatesメソッドのonオプションの発動タイミングで悩んでいます。

package.rb
ｄ
validates :haitatsu_kiboubi, presence: true, on: :step20

これをupdateで

packages_controller

  def update

    if @package.valid?(:step20) && @package.update(package_params)
      flash[:success] = "更新されました。"
      redirect_to root_path
    else
      render "edit"
    endd
  end

と記述すれば、入力フォームにhaitatsu_kiboubiの値が入っていれば更新されると思っていました。
が、実際は値なしと判断されて、editに差し戻されます。

if文の前後を入れ替えて

packages_controller

if @package.update(package_params) && @package.valid?(:step20)

と記述すると、バリデーションは効くのですが、先にupdateが働いてしまい、バリデーション対象外の項目はDBで更新されてしまいます。

そもそも最初から

package.rb

validates :haitatsu_kiboubi, presence: true, on: :update

と記述すれば、問題ないのはわかっているのですが、update内で条件によってバリデーション対象を切り替えたいため、上記のような書き方をしています。

書き方を修正すれば動くのか、別のアプローチを取った方がいいのか、お教え下さい。

Answer 1

先にassign_attributesで値の変更だけしてからバリデーション、保存するという形ではどうでしょうか。

@package.assign_attributes(package_params)
if @package.valid?(:step20) && @package.save

Comments

[maisumakun]

元の「if @package.valid?(:step20) && @package.update(package_params)」では、valid?の判定時点でまだpackage_paramsからの値が代入されていません。

[kohdohg]

教えていただいたとおり修正したら、動きました。完璧です!
assign_attributesで、先に値の変更がかけられるのですね。
ありがとうございました!

Answer 2

コールバックが役に立ちませんか？

Active Record コールバック - Railsガイド