質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

89.65%

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

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 1,758

t1gerkngd0m

score 24

事象

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されていないようです。
コンソール画像2

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • 退会済みユーザー

    退会済みユーザー

    2019/02/06 11:10

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

    キャンセル

  • t1gerkngd0m

    2019/02/06 12:12

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

    キャンセル

回答 3

check解決した方法

0

下記サイトを参考にして解消することができました。
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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

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

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

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


が正しいです。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/02/06 14:44 編集

    本来であれば

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

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

    キャンセル

  • 2019/02/06 16:13

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

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

    キャンセル

  • 2019/02/07 09:45

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

    キャンセル

0

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':のようなキーが入りこむ原因がわかりましたら回答を編集しておきます。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/02/07 18:41

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

    キャンセル

  • 2019/02/08 09:08

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

    キャンセル

  • 2019/02/09 10:30

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

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 89.65%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる