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

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

新規登録して質問してみよう
ただいま回答率
85.50%
Ruby on Rails 5

Ruby on Rails 5は、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

解決済

3回答

8787閲覧

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

t1gerkngd0m

総合スコア26

Ruby on Rails 5

Ruby on Rails 5は、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

4クリップ

投稿2019/02/06 01:32

編集2019/02/07 09:40

事象

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

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

環境

Rails 5.0.7.1

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

Ruby

1app/models/product.rb 2 3class Product < ApplicationRecord 4 has_many :item_images, :dependent => :destroy 5 accepts_nested_attributes_for :item_images, allow_destroy: true 6end 7 8app/models/item_image.rb 9 10class ItemImage < ApplicationRecord 11 mount_uploader :name, ImageUploader 12 belongs_to :product, optional: true 13end

Contoroller

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

ruby

1class ProductsController < ApplicationController 2 3 def new 4 @product = Product.new 5 @product.item_images.build 6 end 7 8 def create 9 binding.pry 10 @product = Product.new(product_params) 11 if @product.save 12 redirect_to root_path, notice: '出品しました。' 13 else 14 render :new 15 end 16 end 17 18 private 19 def product_params 20 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]) 21 end 22end

View

haml

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

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2019/02/06 02:10

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

2019/02/06 03:12

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

回答3

0

自己解決

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

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

view

haml

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

controller

ruby

1def new 2 @product = Product.new 3 @item_image = @product.item_images.build 4end 5 6def create 7 @product = Product.new(product_params) 8 if @product.save 9 binding.pry 10 params[:item_images]['name'].each do |a| 11 @item_image = @product.item_images.create!(name: a) 12 end 13 redirect_to root_path, notice: '出品しました。' 14 else 15 render :new 16 end 17end 18 19private 20def product_params 21 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]) 22end

投稿2019/02/09 01:36

t1gerkngd0m

総合スコア26

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

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

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

slim

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

slim

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

ruby

1# byebug 2params 3#=> <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> 4 5parent_params = params.require(:user).permit(absence: { images: [] }) 6#=> <ActionController::Parameters {"child"=><ActionController::Parameters {"images"=>[#<ActionDispatch::Http::UploadedFile:0x00007fa5b370a138 ...>, #<ActionDispatch::Http::UploadedFile:0x00007fa5b370a0e8 ...>]} permitted: true>} permitted: true> 7 8parent_params[:child][:images] 9#=> [#<ActionDispatch::Http::UploadedFile:0x00007fa5b370a138 ...">, #<ActionDispatch::Http::UploadedFile:0x00007fa5b370a0e8 ...">]

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

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

投稿2019/02/07 00:44

編集2019/02/08 01:40
退会済みユーザー

退会済みユーザー

総合スコア0

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

t1gerkngd0m

2019/02/07 01:29

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

2019/02/07 09:29

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

退会済みユーザー

2019/02/07 09:41

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

退会済みユーザー

2019/02/08 00:08

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

2019/02/09 01:30

ありがとうございます! こちら下記サイトを参考にしたところ、無事にエラーを解消することができました! https://kolosek.com/carrierwave-upload-multiple-images/ やはりparamsから正常に値を取り出すことができていなかったことと、multiple: trueにした時にparamsに渡される配列がitem_images_attributes[name][0][]だったので、view側でname属性の指定をitem_images[name][]にしてあげないといけないようです。
guest

0

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

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

ruby

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

が正しいです。

投稿2019/02/06 04:26

退会済みユーザー

退会済みユーザー

総合スコア0

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

t1gerkngd0m

2019/02/06 05:16

ありがとうございます。コンソールで上記のコマンドを試したところ、下記エラーが出ました。 これは値が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'
退会済みユーザー

退会済みユーザー

2019/02/06 06:00 編集

あまりこういったフォームを作らないのでちょっとよくわかりませんが 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>
退会済みユーザー

退会済みユーザー

2019/02/06 05:45 編集

本来であれば { product: { ..., item_images_attributes: [ { name: '1.png' }, { name: '2.png' }, { name: '3.png' } ] }} みたいな形のパラメータが正しいような気がしますが...
t1gerkngd0m

2019/02/06 07:13

僕もそういう形が正しいと思うのですが。 逆にそういうparamsを得るためにはフォームでどのように記述すべきでしょうか。 ちなみに、multiple: trueの記述を削除すると画像の複数アップロードは出来ませんがDB保存出来ます。 フォームの形はこちらの記事を参考にしました。 https://qiita.com/shinichiro81/items/4edb8af4a64991897d5a
退会済みユーザー

退会済みユーザー

2019/02/07 00:45

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問