Ruby on Railsのフォームオブジェクトのエラーについて

Question

Ruby on Railsのフォームオブジェクトの関連付けについて質問があります。
https://teratail.com/questions/250421
上記のURLの続きの内容となります。

post_article_controller.rb

def new
    @post_article_form = PostArticleForm.new
end

def create
    @post_article_form = PostArticleForm.new
    @post_article_form.assign_attributes(params[:form])
    # ここでタグの文字列をパラメータでFormに送信しています
    @post_article_form.assign_tags(params[:tag_list])

    @post_article_form.post_article.poster = current_user
    if @post_article_form.save
      redirect_to :root, notice: "投稿しました。"
    else
      render 'new'
    end
end

post_article.rb
def save_tags(tags)
    current_tags = self.tags.pluck(:name) unless self.tags.nil?
    old_tags = current_tags - tags
    new_tags = tags - current_tags

    # タグの削除
    old_tags.each do |old_name|
      self.tags.delete(Tag.find_by(name: old_name))
    end

    # タグの追加
    new_tags.each do |new_name|
      tagging = Tag.find_or_create_by(name: new_name)
      self.tags << tagging
    end
  end

class PostArticleForm

  delegate :save, to: :post_article

  def initialize(post_article = nil)
    @post_article = post_article
    @post_article ||= PostArticle.new
    (5 - @post_article.photos.size).times do
      @post_article.photos.build
    end
    # ここでタグを生成しています
    @post_article.tags.build unless @post_article.tags.present?
  end

def assign_tags(tag_list)
    tag_list = tag_list.split(",")
    post_article.save_tags(tag_list)
    # binding.pryでここで止めると以下のように目的のタグ以外にidがnilのタグが作成されています。
    # コンソール
    => [#<Tag:0x007ffd242a34c8 id: nil, name: nil>, #<Tag:0x007ffd25ed34e0 id: 10, name: "入力したタグ名">]
end

上記の、[#<Tag:0x007ffd242a34c8 id: nil, name: nil>, #<Tag:0x007ffd25ed34e0 id: 10, name: "入力したタグ名">]の部分にnilが入ってしまっているため「Field 'name' doesn't have a default value」というエラーが出てしまいタグの保存ができません。

何故でしょうか？

大変お手数をお掛けしますが、どなたかご教授の方をよろしくお願い致します

Answer 1

params[:tag_list]の値を確認してみましょう。
この場合は「入力したタグ名,」等の入力をしたように思えます。
post_article_form.rbのassign_tagsで、上記の値に対してsplit(",")を実行した結果が["", "入力したタグ名"]になった結果、nameが空のpost_article_tagsを生成しようとしています。
おそらくpost_article_tagsのテーブルにはnull: falseな制約がついており保存が失敗した結果、nilのTagが生成されたように思えます。

モデル内でself.tags.nil?でnilチェックを行っているようですが、実は空の文字列でnil?をやってもfalseが帰ります。からの文字列も弾きたい場合はself.tags.blank?を使うといいでしょう。

Comments

[退会済みユーザー]

ご回答ありがとうございます！
params[:tag_list]の中身は"入力した文字列"になっていまして""の空文字にはならないです。
split(",")を実行した結果も["入力したタグ名"]になっております。
また、モデル内でself.tags.blank？を使っても同じエラーが発生致します。

[退会済みユーザー]

モデルのsave_tagsメソッドに飛んでいる引数も["入力したタグ名"]になっております。
idがnilのtagはそのまま残っている状態です。save_tagsメソッドの処理が完了しても
[#<Tag:0x007ffd242a34c8 id: nil, name: nil>, #<Tag:0x007ffd25ed34e0 id: 10, name: "入力したタグ名">]とidがnilのtagはそのまま残っている状態です。何故でしょうか？

[退会済みユーザー]

ちなみに、assign_tagsのメソッドを以下のように修正し、一回tagを全て削除するという方法にすると、idがnilのtagが消えて上手くいくのですが、少し考え方が無理やりな感じがします。
def assign_tags(tag_list)
tag_list = tag_list.split(",")
post_article.save_tags(tag_list)

post_article.tags.each do |tag|
post_article.tags.delete(tag)
end
end

[退会済みユーザー]

しかし、上記のようにすると
1つタグを入力した場合には上手く動きますが、
2つ以上タグを入力した場合は最初の1つ目のタグしか登録されないため
やはり上手く動きません。

[mottox2]

返信が遅れました。
改めて見ると`@post_article.tags.build`を実行していてその時点で空のTagが生成されている気がします。
このあたりが原因ではないでしょうか？