nested_form_forでuniquenessのようなバリデーションをかけたい

Question

nested_formでuniquenessのようなバリデーションをかけたいです。

モデル
Artice(記事)
Tag(タグ)

以上のようなモデルがあり、１つの記事にいくつかのタグを付けます。
記事を作成する際に、タグも作成したく、nested_form_forで実現させようと考えています。

article.rbを以下のように実装しました。バリデーション部分は省略しています。

class Article < ApplicationRecord
  has_many :tags
  accepts_nested_attributes_for :tags, allow_destroy: true

  validate :validate_uniqueness

  def validate_uniqueness
   #self.tags間でのユニークのバリデーション 
　　errors[:base] << "タグが重複しています" 
unless self.tags.pluck(:name).uniq.size == self.tags.count 
  end
end

新規作成時は上記でうまくいくのですが、
更新時には
self.tags = すでにDBに保存されているタグたち ≠ 更新したいタグたち
となっており、モデル内でバリデーションをかけられないでいます。

nested_form_forの子モデル間でバリデーション をかけたい場合どのようにすれば良いでしょうか？

Comments

[winterboum]

Tagとは1:多で多:多ではないですね？
どんな validationを掛けたいのですか？

[a10]

ご質問ありがとうございます。
情報に不足があってすみません。
ArticeとTagは1:多の関係となっています。
バリデーションとしては、
1つのArticleに紐づくtag同士が重複していないこととしたいです。
質問文の通り新規作成時はうまくいくのですが、更新時にはうまくいきません。

[winterboum]

ごめん、質問のしかたがまずかった
重複しないように　というのは判るのです
「重複の判定」のしかた、です。
いまの validate_uniquenessは載せられない？

[a10]

バリデーション部分追記いたしました。
更新時、self.tagsの部分がフォーム上で更新したものではなく、DB上のタグたちを見てしまうためバリデーションできていない状況です。

Answer 1

ArticleでvalidationせずにTagでvalidationさせるという方法があります。
validates :name, presence: true, uniqueness: { scope: :user }
これをArticleのsaveの時に実行させるには
validates_associated :tags
ただしこのvalidationのエラーは「タグで問題があった」ということしかわからないですが。

Comments

[a10]

ありがとうございます。
ただこちら試してみたのですがうまくいきませんでした。。
uniqueness: {scope: :article}は現在DBに保存されているtagたちと比較しているようで、
tag1 tag2 tag3 tag4
=>(更新)
tag4 tag3 tag2 tag1
のように並び変えたりして更新すると
（DB上には別のIDでtag4が保存されているため, nameが重複されているとみなされて）
バリデーション にかかってしまいます。。

[winterboum]

？？？
それが望ましい動きでは無いのですか？
更新とは、総入れ替え？

[a10]

現在
Article (id:1)に対して
id:1 name: tag_1
id:2 name: tag_2
id:3 name: tag_3
が紐づいているとします。

この状態で、例えば
id:1 name: tag_1
id:2 name: tag_3
id:3 name: tag_2
と変更したとします。

その場合、本来はarticleに対してタグがuniqueになっているためバリデーションはクリア
すべきなのですが、DBとの比較によって tag_2 tag_3はバリデーション にかかってしまいます。
（nested_fields_forなので、article:1に対するタグが総更新されます）

[winterboum]

？？？
id2 と id3 のnameを入れ替える意味がわかりません
何方にしても ,tag1, tag2, tag3 という３つが付くことに変わりはないので、、、
何をしたいのか、が良くわかりません

[a10]

確かにtag1, tag2, tag3の入れ替えは例としては悪かったですね...
tag1, tag2, tag3 という３つが付くことに変わりはないですが、
使用するユーザの動作として、入れ替えたり、削除してまた付け直したりするということは十分あり得ると思います。
その際、DB上のものと比較するバリデーションは希望する要件を満たせないと感じています。

[winterboum]

「DB上のものと比較するバリデーションは希望する要件を満たせない」
とすると、
「今画面にあるtagの中で重複が無いことを確認する」
ということでしょうか？
次の場合どう振る舞うのが良いのですか？
DBに tag1,tag2,tag3 があり、画面で tag1、tag4、tag5 を入れた場合
1) validationは通る
2-1) DBには tag1,tag2,tag3,tag1,tag4,tag5が出来る（tag1は２つ出来る）
2-2) DBには tag2,tag3,tag1,tag4,tag5が出来る（tag1は１つにする）
2-3) DBは ｔag1,tag4,tag5 に置き換わる

[a10]

回答ありがとうございます。
nested_field_forでの一括更新を想定しているため、
1) →　2-3)の動作が正しいです。

> 画面で tag1、tag4、tag5 を入れた場合
とのことですが、更新時なので
ユーザーの行う動作としては、
タグ更新画面からtag2, tag3を削除　&　tag4, tag5を追加となります。

[winterboum]

いまのupdate関連のcodeを載せていただけますか。それを元に修正案を考えます