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

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

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

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

Q&A

解決済

2回答

654閲覧

rails カラム名の無いparametersがupdate時にunpermittedで弾かれてしまう

yamaoka_san

総合スコア22

Ruby on Rails

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

0グッド

0クリップ

投稿2020/02/22 06:17

初心者な質問で毎回申し訳ないです。

Parameters: { "visitors"=>{ "1"=>{"group_id"=>"1"}, "2"=>{"group_id"=>"1"}, "3"=>{"group_id"=>"1"}},"id"=>"2"}

という値をupdateする時、

Unpermitted parameters: :1, :2, :3

と怒られてしまい、更新できません。

この値にカラム名をつけるとか、idで紐付ける(?)なのか、何か更新できる手立てはありますか?
この値は、visitorモデルのidです。
こちらのサイトでもお世話になり、さらに下記のサイトを参考にしました。
Rails5.1で一括登録フォームを実装する
fields_for(record_name, record_object = nil, options = {}, &block) public
が、これ以上分かりませんでした。私には。。。

[controller]

def update p visitors = params[:visitors] @visitors = visitors.keys.each do |id| visitor = Visitor.find(id) visitor.update!(visitor_params) end rescue ActiveRecord::RecordInvalid => en p en.record.errors end private def visitor_params params.require(:visitors).permit(:group_id) end

試したこと。
値の取得や、フォーム上でカラム名を入れようとしたのですが、全てダメでした。
[controller]

params[:visitors][:id] #=>ダメでした。 params[:visitor][:id] #=>ダメでした。 params[:visitors]["1"] #=>一応拾えます。

申し訳ありませんが、教えていただけると大変助かります。どうぞよろしくお願いします。

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

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

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

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

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

guest

回答2

0

params.require(:visitors).permit(:group_id)と書くと、visitors[group_id]をPermitするという意味になります。

visitors[1][group_id]のようなものをpermitしたい場合は、params.require(:visitors).permit([:group_id])のように、group_idを配列としておく必要があります。

(もっとも、これでStrong Parametersは通りますが、そのままvisitor.updateに投げ込むことはおそらくできないかと思います)

投稿2020/02/22 09:46

maisumakun

総合スコア145121

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

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

yamaoka_san

2020/02/26 23:40

色々と試してみました!ajaxで送ることにしました。 自己解決のコーナーでまとめます。
guest

0

自己解決

form_forからのsubmitだと弾かれてしまう。
でもモデル使ってループしたいし、select要素を動的に生成したい。
なのでform_forのsubmitを使わず、jqueryで取り込んでParametersをイジリ、ajaxでupdateアクションに流す方法にしました。

[view] <%= form_for(@visitor, url: {controller: :visitors, action: :update}, method: :patch) do |f| %> <% @visitors.each do |visitor| %> <%= fields_for "visitors[]", visitor do |fi| %> <div class="rec"> <%= fi.select :group_id, options_from_collection_for_select( @groups, :id, :group_name), {prompt: '分類'} %> </div> <% end %> <% end %> <% f.submit "リストに加える" %> #=>一応作ってる。でも表示しない。 <% end %> <button class="submit">ボタン</button> #=>こっちで値を送る。

jsは、htmlのdiv要素から値を取得してます。動的なselectはidに番号があったので、それを取り出してます。

[js] $(".submit").click(function(){ var last_visitor_id = $(".edit_list_group_name:last").children().attr('id'); var last_visitor_id = last_visitor_id.split('_'); last_visitor_id = Number(last_visitor_id[1]) for (var i=1; i<=last_visitor_id; i++) { var group_id = $("#visitors_"+i+"_group_id").val();//group_idを取得 $.ajax({ url: '/visitors/:id', data: { visitors: {"id": i, "group_id": group_id}}, method: "PUT" }); }//for文オワリ });//clickオワリ

方針転換してからの目標は、ハッシュで一気に更新!など思っていましたが、ザセツしました。

[controller] def update p @visitor_id = params[:visitors][:id] p @group_id = params[:visitors][:group_id] visitor = Visitor.find(@visitor_id) visitor.update!(visitor_params) end private def visitor_params params.require(:visitors).permit(:group_id) end

これで無事に更新できました。
骨を折って頂いた方々、どうもありがとうございました!!
何気に見ていたフォームのページ、作るの大変なんですね。。。

投稿2020/02/27 00:05

yamaoka_san

総合スコア22

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問