【rails】has_and_belongs_to_many関係で、関連先のモデルを関連元モデルのフォームからcollection_selectで編集したい。

Question

Artistモデルと、Genreモデルがあり、多対多の関係です。

Schemafileが、

create_table "artists_genres", force: :cascade do |t|
  t.integer  "genre_id",  limit: 4
  t.integer  "artist_id",  limit: 4
end

create_table "artists", force: :cascade do |t|
  t.string   "name", limit: 255
  t.string   "image"
end

create_table "genres", force: :cascade do |t|
  t.string  "genre", limit: 255
end

こうなっていて、モデルファイルはそれぞれ
artist.rb

  has_and_belongs_to_many :genres

genre.rb

  has_and_belongs_to_many :artists

このようになっています。

artistのeditページより、そのアーティストのジャンルを1つ~複数選んで変更の保存をしたいと考えています。

artists_controller.rbは

  def update
    @artist = Artist.find(params[:name])
    @artist.update_attributes(artist_params)
    redirect_to :controller => 'artists', action: 'show', name: @artist.name
  end

  private

  def artist_params
    params.require(:artist).permit(:image, :genre_ids => [])
  end

のようになっており、

editのviewファイルは、

= form_for @artist, :html=>{:multipart=>true} do |f|
			= f.file_field :image
			= f.collection_select(:genre_ids, Genre.all, :id, :genre, {prompt: true}, {multiple: true})
			= f.submit '編集を保存する'

というようになっています。

現状、editページの表示には問題ないのですが、そのフォームを送信したところでエラーがなにもおこらず、redirect_to :controller => 'artists', action: 'show', name: @artist.nameが実行されアーティストのshowページが表示されるのですが、選択したジャンルはそのアーティストのジャンルとして保存されていません。

関連づいたモデル間の操作方法がわからず、躓いています。

経験をお持ちの方、どうかお力添えください。よろしくお願いします。

Answer 1

アーティストのジャンルを編集する操作は、DBのテーブルのartists_ganresを変更することになりますよね?
ですから更新とはこのテーブルの行の削除or追加という事になります。

参考

こんな感じです。

artist.genres << ganre

更新は一度関連を全部消してから必要な関係を順に作ればいいと思います。

こんな感じかな?もっといいやり方があるかも知れません。

 artist.ganres.clear
 ganres = ジャンルのモデルの配列
 ganres.each do |ganre|
   artist.genres << ganre
 end

追記

以下の様な感じでできそうです。

 artist.ganres.clear
 artist.genres << ジャンルのモデルの配列

以下でもいいかも知れません

  artists.genre_ids = ジャンルのidの配列

Comments

[Taka624]

本当にありがとうございます！！！

具体的にわからない部分が、

def update
@artist = Artist.find(params[:name])
@artist.update_attributes(artist_params)
redirect_to :controller => 'artists', action: 'show', name: @artist.name
end

このupdateメソッドの内部に、どのようにお教えいただいた
artist.ganres.clear
artist.genres << ジャンルのモデルの配列
を組み込めばいいのか、という部分と、

params.require(:artist).permit(:image, :genre_ids => [])

permitの第二引数の部分の書き方(今:genre_ids => []となっている部分です。)
がわかりません。

その上で、
フォームのcollection_selectの第一引数(今:genre_idsとなっている部分)の書き方がわかりません。

= f.collection_select(:genre_ids, Genre.all, :id, :genre, {prompt: true}, {multiple: true})

もう少しお力を貸してください、どうぞよろしくお願いします！！！！！

[realizerS]

取り敢えずは送られてくるパラメータを確認しましょう。

サーバにアクセスした時、railsのサーバのログに

Parameters: {なんかハッシュ}

という情報があるはずですよね?

この情報がコントローラの`params`という変数へ格納されます。まずはそれを確認してください。

で、あとはparamsのキーartistに、imageとgenreのidの配列が入るようにビューを調整すればよいはずです。

つまりまずは、

params = {
...
artists: {image: ...,
artist_ids: [... ]
}
...
}

という感じになるようにビューを調整する必要があります。