accepts_nested_attributes_forを使った場合に、strong parameterからattributesの値を取り出せない

Question

事象

2つのモデルに同時登録をしたいため、accepts_nested_attributes_forを利用しているんですが、
strong parameterのparamsに渡したデータを取り出せず困っております。

約丸1日時間を使って格闘しておりますが解決出来ないので、
こちらで質問させて頂きました。

環境

Rails 5.0.7.1

Model
モデルはproductとitem_imageの2つを使用しております。

app/models/product.rb

class Product < ApplicationRecord
  has_many :item_images, :dependent => :destroy
  accepts_nested_attributes_for :item_images, allow_destroy: true
end

app/models/item_image.rb

class ItemImage < ApplicationRecord
  mount_uploader :name, ImageUploader
  belongs_to :product, optional: true
end

Contoroller

コントローラーではproductを中心にネストさせたitem_imageを保存する形です。

class ProductsController < ApplicationController

  def new
    @product = Product.new
    @product.item_images.build
  end

  def create
    binding.pry
    @product = Product.new(product_params)
    if @product.save
      redirect_to root_path, notice: '出品しました。'
    else
      render :new
    end
  end

  private
  def product_params
    params.require(:product).permit(:name, :description, :category_large, :category_middle, :category_small, :brand, :size, :shipping_charges_burden, :dispatch_area, :shipping_method, :number_of_the_days_to_ship, :price, :condition, item_images_attributes: [:name])
  end
end

View

= form_for @product, html: {class: "new-main__sell"} do |f|
  .new-main__image
    %h3.new-main__image-head
      出品画像
      %span.form-require
        必須
    %p
      最大10枚までアップロードできます
    .image-upload-dropbox__container.clearfix.state-image-number-10
      .image-upload-items__container
        .image-upload-items.have-item-0
          %ul
      %label.image-upload-dropbox.have-item-0
        = f.fields_for :item_images do |i|
          = i.file_field :name, multiple: true, class: 'image-upload-dropfile hidden', type: 'file'
          %pre.visible-pc
            ドラッグアンドドロップ<br />またはクリックしてファイルをアップロード
          %i.icon-camera

paramsの中身

ファイル名"1.png", "2.png", "3.png"をアップロードした時のparamsの中身です。

<ActionController::Parameters {"utf8"=>"✓", "authenticity_token"=>"UrXgrCUx9A7MeBZ6XBIOGr9dPB+DFl3TEFyCItecugR9VUwMpJICHVJW2kKkPZqoEQOM9o5jrxCj6mU/zdL+yQ==", "product"=>{"item_images_attributes"=>{"0"=>{"name"=>[#<ActionDispatch::Http::UploadedFile:0x007fcdb3cd8a60 @tempfile=#Tempfile:/var/folders/8w/0jszx71s4999fhh3qryy0lyh0000gn/T/RackMultipart20190206-8800-1vlw5r9.png, @original_filename="3.png", @content_type="image/png", @headers="Content-Disposition: form-data; name="product[item_images_attributes][0][name][]"; filename="3.png"\r\nContent-Type: image/png\r\n">, #<ActionDispatch::Http::UploadedFile:0x007fcdb3cd8a10 @tempfile=#Tempfile:/var/folders/8w/0jszx71s4999fhh3qryy0lyh0000gn/T/RackMultipart20190206-8800-1h3qyj3.png, @original_filename="2.png", @content_type="image/png", @headers="Content-Disposition: form-data; name="product[item_images_attributes][0][name][]"; filename="2.png"\r\nContent-Type: image/png\r\n">, #<ActionDispatch::Http::UploadedFile:0x007fcdb3cd89c0 @tempfile=#Tempfile:/var/folders/8w/0jszx71s4999fhh3qryy0lyh0000gn/T/RackMultipart20190206-8800-zi8ilf.png, @original_filename="1.png", @content_type="image/png", @headers="Content-Disposition: form-data; name="product[item_images_attributes][0][name][]"; filename="1.png"\r\nContent-Type: image/png\r\n">]}}, "name"=>"", "description"=>"", "category_large"=>"---", "category_middle"=>"---", "category_small"=>"---", "size"=>"---", "brand"=>"", "condition"=>"---", "shipping_charges_burden"=>"---", "shipping_method"=>"---", "dispatch_area"=>"---", "number_of_the_days_to_ship"=>"---", "price"=>"1000"}, "button"=>"", "controller"=>"products", "action"=>"create"} permitted: false>

困っていること

paramsにはitem_imageの情報が渡されているのに、そこから取り出すことが出来ず、結果的にDBにはproductテーブルに1レコード保存され、item_imageテーブルにもnameカラムが空の状態で1レコードだけ保存されるような状態です。
どうすればproduct 1レコードに対してitem_image が正しく複数レコード保存されるようになるでしょうか？
アドバイスを頂きたいです。

追記

paramsから値が取り出せないとはどういうことか、というご質問を頂きましたので、コンソールの画像を添付します。

追記#2

Image_itemモデルのnameカラムにバリデーションをかけたところ、コンソールに下記が表示されました。
やはりparamsの値がpermitされていないようです。

Comments

[退会済みユーザー]

取り出せないというのは product_params でエラーが起きているということですか？
それとも取り出した値が期待していたものと違うという意味ですか？

[t1gerkngd0m]

binding.pryで止めてコンソールでparams、params.require(:product)と打った時は上記のように画像データが入っているように見えるのですが、params.require(:product).require(:item_images_attributes)と打つとitem_images_attributesがUnpermittedで出てくる、という意味です。

Answer 1

下記サイトを参考にして解消することができました。
https://kolosek.com/carrierwave-upload-multiple-images/

multiple: tureを指定するとparamsに配列で渡されるようなので、item_imagesのインスタンスは@productとは別で保存される処理が必要なようです。

view

= f.fields_for :item_images do |i|
  = i.file_field :name, multiple: true, class: 'image-upload-dropfile hidden', type: 'file', name: "item_images[name][]"

controller

def new
  @product = Product.new
  @item_image = @product.item_images.build
end

def create
  @product = Product.new(product_params)
  if @product.save
    binding.pry
    params[:item_images]['name'].each do |a|
      @item_image = @product.item_images.create!(name: a)
    end
    redirect_to root_path, notice: '出品しました。'
  else
    render :new
  end
end

private
def product_params
  params.require(:product).permit(:name, :description, :category_large, :category_middle, :category_small, :brand, :size, :shipping_charges_burden, :dispatch_area, :shipping_method, :number_of_the_days_to_ship, :price, :condition, item_images_attributes: [:name])
end

Answer 2

multiple: trueを指定した場合、配列のパラメータが送られてくるようなので
multiple: trueを使うのであれば、パラメータを受け取ったあと、別々に保存する処理が必要そうです。
今回の質問内容とは異なるので割愛します。

試しにフォームを作って見ました。

= form_with model: @parent, local: true do |f|
  = f.fields_for @child do |a|
    = a.file_field :images, multiple: true, type: :file
  = f.submit

= form_for @parent do |f|
  = f.fields_for @child do |a|
    = a.file_field :images, multiple: true, type: :file
  = f.submit

# byebug
params
#=> <ActionController::Parameters {"utf8"=>"✓", "authenticity_token"=>"...", "parent"=>{"child"=>{"images"=>[#<ActionDispatch::Http::UploadedFile:0x00007fa5b370a138 ...>, #<ActionDispatch::Http::UploadedFile:0x00007fa5b370a0e8 ...>]}}, "commit"=>"登録する", "controller"=>"parent", "action"=>"create", "parent_id"=>"1"} permitted: false>

parent_params = params.require(:user).permit(absence: { images: [] })
#=> <ActionController::Parameters {"child"=><ActionController::Parameters {"images"=>[#<ActionDispatch::Http::UploadedFile:0x00007fa5b370a138 ...>, #<ActionDispatch::Http::UploadedFile:0x00007fa5b370a0e8 ...>]} permitted: true>} permitted: true>

parent_params[:child][:images]
#=> [#<ActionDispatch::Http::UploadedFile:0x00007fa5b370a138 ...">, #<ActionDispatch::Http::UploadedFile:0x00007fa5b370a0e8 ...">]

'0':のようなキーは入り込みませんね。
どちらにせよ@product = Product.new(product_params)な形で登録するのは難しいと思います。
(配列で渡されるimageを分解して別々に登録する処理が必要なため)

'0':のようなキーが入りこむ原因がわかりましたら回答を編集しておきます。

Comments

[t1gerkngd0m]

すみません、こちら頂きました回答が表示されていないようでして。。
お手数ですが同じ回答をもう一度投稿して頂けませんでしょうか。

[t1gerkngd0m]

ありがとうございます！これはform_forで書いていたところをform_withで書くことによって、"0"が入らなくなったということでしょうか？
>multiple: trueを指定した場合、配列のパラメータが送られてくるようなので
>multiple: trueを使うのであれば、パラメータを受け取ったあと、別々に保存する処理が必要そうです。
>今回の質問内容とは異なるので割愛します。
質問のタイトルは「strong parameterからattributesの値を取り出せない」ですが、そもそもDBに保存出来れば良いので、出来ればこちらもご教授いただきたいです。

[退会済みユーザー]

form_forとform_withでそういった差異があるのは聞いたことがないですが
form_forで書いておくべきでしたね。
フォームにクラスがたくさんあるみたいですが
一旦そういったものをすべて取っ払って極力シンプルなフォームを作ってみるといいかもしれません。

[退会済みユーザー]

form_forにしてみましたが結果は同じでした。

[t1gerkngd0m]

ありがとうございます！
こちら下記サイトを参考にしたところ、無事にエラーを解消することができました！
https://kolosek.com/carrierwave-upload-multiple-images/

やはりparamsから正常に値を取り出すことができていなかったことと、multiple: trueにした時にparamsに渡される配列がitem_images_attributes[name][0][]だったので、view側でname属性の指定をitem_images[name][]にしてあげないといけないようです。

Answer 3

params.require(:product).require(:item_images_attributes)と打つとitem_images_attributesがUnpermittedで出てくる

item_images_attributesはnameがネストしており、nameは配列であるので

params.require(:product).require(item_images_attributes: { name: [] })

が正しいです。

Comments

[t1gerkngd0m]

ありがとうございます。コンソールで上記のコマンドを試したところ、下記エラーが出ました。
これは値がemptyということでしょうか？

ActionController::ParameterMissing: param is missing or the value is empty: {:item_images_attributes=>{:name=>[]}}
from /Users/miwataiga/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/actionpack-5.0.7.1/lib/action_controller/metal/strong_parameters.rb:462:in `require'

[退会済みユーザー]

あまりこういったフォームを作らないのでちょっとよくわかりませんが
item_images_attributesとnameの間にある:"0"とか気になりますね。
これはなぜあるんでしょうか。
フォームの書き方は正しいですか？

ちなみに手元で実行したものは以下です

params
=> <ActionController::Parameters {"product"=><ActionController::Parameters {"item_images_attributes"=>{"0"=>{"name"=>["<3.png添付ファイル>", "<2.png添付ファイル>", "<1.png添付ファイル>"]}}} permitted: false>} permitted: false>

params.require(:product).permit(item_images_attributes: { name: [] })
=> <ActionController::Parameters {"item_images_attributes"=><ActionController::Parameters {"0"=><ActionController::Parameters {"name"=>["<3.png添付ファイル>", "<2.png添付ファイル>", "<1.png添付ファイル>"]} permitted: true>} permitted: true>} permitted: true>

[退会済みユーザー]

本来であれば

{ product: {
...,
item_images_attributes: [
{ name: '1.png' },
{ name: '2.png' },
{ name: '3.png' }
]
}}

みたいな形のパラメータが正しいような気がしますが...

[t1gerkngd0m]

僕もそういう形が正しいと思うのですが。
逆にそういうparamsを得るためにはフォームでどのように記述すべきでしょうか。
ちなみに、multiple: trueの記述を削除すると画像の複数アップロードは出来ませんがDB保存出来ます。

フォームの形はこちらの記事を参考にしました。
https://qiita.com/shinichiro81/items/4edb8af4a64991897d5a

[退会済みユーザー]

ちょっと検証して見て別回答にします