Rails 投稿内容が保存されない

Question

解決したいこと

インスタライクなアプリを制作しており、投稿内容がデータベースに保存されません。
投稿内容は、tag、title、image、textになります。
tagは、予めデータベース保存している６項目をformのcollection_selectで選択するようにしています。

発生している問題・エラーメッセージ

エラーは発生せず、createアクション内の条件分岐でelseにより、元のページに戻されてしまいます。
データベースには何も保存されていません。

new.html.erb

<div class="contents row">
  <div class="container">
    <%= form_with(model: @post, local: true) do |form| %>
      <h2>投稿する</h2>
      <%= form.collection_select :tags_ids, Tag.all, :id, :name, :prompt => "カテゴリーを選択して下さい" %>
      <%= form.text_field :title, placeholder: "タイトル" %>
      <%= form.file_field :image, placeholder: "写真のURL" %>
      <%= form.text_area :text, placeholder: "投稿内容" , rows: "10" %>
      <%= form.submit "送 信" %>
    <% end %>
  </div>
</div>

seed.rb

~省略~

Tag.create([
  { name: '風景' },
  { name: 'バイク' },
  { name: 'カスタム' },
  { name: 'マップ' },
  { name: 'グルメ' },
  { name: '宿、ホテル' }
])

posts_controller.rb

  def new
    @post = Post.new
  end

  def create
    @post = Post.new(post_params)
    if @post.save
      redirect_to root_path
    else
      render :new
    end
  end

~省略~

  private
  def post_params
    params.require(:post).permit(:tag, :title, :text, :image).merge(user_id: current_user.id)
  end

post.rb

class Post < ApplicationRecord
  validates :tag, :title, :text, :image, presence: true
  belongs_to :user
  has_many :post_tags, dependent: :destroy
  has_many :tags, through: :post_tags

  mount_uploader :image, ImageUploader
end

tag.rb

class Tag < ApplicationRecord
  has_many :post_tags, dependent: :destroy
  has_many :posts, through: :post_tags
end

migrate/_create_posts.rb

class CreatePosts < ActiveRecord::Migration[5.2]
  def change
    create_table :posts do |t|
      t.string :title
      t.text :text
      t.string :image
      t.timestamps
    end
  end
end

migrate/_create_tags.rb

class CreateTags < ActiveRecord::Migration[5.2]
  def change
    create_table :tags do |t|
      t.string :name
      t.timestamps
    end
  end
end

migrate/_add_tag_to_posts.rb

class AddTagToPosts < ActiveRecord::Migration[5.2]
  def change
    add_column :posts, :tag, :string
  end
end

binding.pry後のパラメーター

<ActionController::Parameters {"utf8"=>"✓", "authenticity_token"=>"OA+BMUWJpLSbpkhO7wlyNeBmqXodoWTyqQYd809VUuKTu6hMm6hSKynBj1aE7mOUYu3UmOKMt3M+yA6lCYK3kg==", "post"=><ActionController::Parameters {"tags_ids"=>"1", "title"=>"テスト", "image"=>#<ActionDispatch::Http::UploadedFile:0x00007fef92f6f7f0 @tempfile=#<Tempfile:/var/folders/b9/nlkpx4jd5plg9fbf9dpp648h0000gn/T/RackMultipart20200516-9745-19ak7j0.jpeg>, @original_filename="3A866F5C-61CC-443A-9643-466184FDCF23_1_105_c.jpeg", @content_type="image/jpeg", @headers="Content-Disposition: form-data; name=\"post[image]\"; filename=\"3A866F5C-61CC-443A-9643-466184FDCF23_1_105_c.jpeg\"\r\nContent-Type: image/jpeg\r\n">, "text"=>"test"} permitted: false>, "commit"=>"送 信", "controller"=>"posts", "action"=>"create"} permitted: false>

試したこと

save!メソッドにより、エラーの原因を探りました。
Validation failed: Tag can't be blank
エラー画面によるとtagが空であると出ます。
パラメーターでは取得できていると思うのですが。
解決法やエラーの原因がわからず困っています。
ご助力よろしくお願いします。

save!メソッドの結果

https://gyazo.com/b6a6372279e585d91a186fff839accc5

Answer 1

構成に混乱と矛盾があります。
0. tagは一つですか多数ですか？

class Post < ApplicationRecord
  validates :tag, :title, :text, :image, presence: true
  has_many :post_tags, dependent: :destroy
  has_many :tags, through: :post_tags

とtagに関して二つの記述があります。
validatesからはtagはPostの項目であるかのように見えますが、残り２行はmodel Tagとの多対多に見えます。

1. tagは一つですか多数ですか？

form.collection_select :tags_ids, とあるので、名前的には複数に取れますが複数選択可能なoptionがありません。
一つですか？

Tagは一つしかつけられないのか、複数つけられるのかによって回答が変わります。
どちらでしょう？

＃＃＃ひとつだけなのでしたら
0. Postのtagは　tag_id(integer)に替える
0. class Post のvalidates,has_many ３行は削除しbelongs_to :tagにする
ただし、tag無しでも良しとするならbelongs_to :tag, optional: true
0. form.collection_select :tags_ids, は form.collection_select :tags_id,
0. permit(:tag, :title, :text, :image) は permit(:tag_id, :title, :text, :image)

で行けるかな

Comments

[TaaShii]

回答ありがとうございます。
1つの投稿に対して付けられるtagは1つ選択できるようにしています。

[TaaShii]

解決し、無事に投稿内容が保存できるようになりました。
原因は、大まかにいうと２つあリました。
１つ目は、投稿が一つに対してtagは一つしかつけられないのにtagsと複数形で登録や取得をしようとしていたこと。
・postモデルのバリデーションのtags_idsをtag_idに修正
・postとtagの多対多の関係になっていたアソシエーションを修正
２つ目は、tagは一つしかつけられないので、投稿postとのアソシエーションは多対一の関係であるのに中間テーブルを作ってしまっていたことにあった。
・中間テーブルを削除

Answer 2

以下の記事が参考になりませんでしょうか？

• 【週末Rails勉強会】1つのformで親子孫関係のテーブルデータを登録する方法 | サブカルエンジニアのブログ
• Rails 複数の子レコードの作成・更新を自在に扱う (accepts_nested_attributes_for) - Qiita
• Railsで1つのフォームで複数のモデルを登録・更新させる - Qiita

@post = Post.new

の後、

@post.tags.build

などが必要かと思われます。

また、Post モデルに accepts_nested_attributes_for を指定する必要があると思います。

View 作成時に子テーブルに入れる情報は、fields_for で指定する必要がありそうです。

まだ、不足事項があるかもしれませんが、一度、リンク先等を読んでみてください。

Comments

[TaaShii]

回答ありがとうございます。
参考記事を読んで確認してみます。