Q&A
ShopsControllerのshowを追記してください
#状況
食べログのようなアプリを作っています。Userモデル、Shopモデル、Reviewモデルと3つのモデルがあり、ユーザーがお店に対してレビューを投稿するフォームを作っています。
お店の詳細ページからレビュー投稿フォームに書いて投稿します。
例えば/shops/1/showからreviewsコントローラーのcreateアクションに情報をとばします。
#問題
フォームを送信すると情報を受け取った側のreviewsコントローラーでパラメーターがないというエラーが発生してしまいます。
画面に表示されるエラーは以下の通りです。
そこでフォームの画面をデベロッパーツールで確認して見ると、フォームに適切なname属性が入っていませんでした。本当はname = "review[title]"のようになって欲しいものがname = "title"となっており、strongparamsで設定した適切な形で情報を受け取れていませんでした。
おそらくform_withの書き方に問題があるのだと思うのですが、どのように修正すれば良いのでしょうか。
フォームがあるのはshopの詳細画面(/shops/1)で、そこから/users/1/reviews/にPOSTアクションをとばします（ログイン中のユーザーと投稿を紐付けるため）。
ルーティングの書き方も少し自信がないです...
フォームは以下の通りです。
show
1<%= form_with(model: @review, url: user_reviews_path(current_user), local: true) do |f| %> 2 <%= f.label :title, "タイトル" %> 3 <%= f.text_field :title, class: 'form-control' %> 4 5 <%= f.label :content, "本文" %> (300文字以内でお書きください) 6 <%= f.text_area :content, rows:10, class: 'form-control text_area' %> 7 8 <%= hidden_field_tag :shop_id, @shop.id %> 9 10 <%= f.submit '登録', class: "btn btn-dark " %> 11<% end %> 12
解決するのに他に必要な情報がありましたらご指摘ください。
よろしくお願いします。
ご指摘のShopsControllerのshowを確認したら@review = @shop.reviews.buildの記載がされていませんでした。これを追記することでうまく情報を流すことができました。気が付くきっかけを与えてくださりありがとうございました！
回答4件
0
■ルーティングの書き方にも少し自信がない
ShopモデルのルーティングがReviewにネストされていますか？
routes
1resources :shops, only: [:new, :create] do 2 resources :reviews, only: [:index, :create]
となっていれば（only以下のアクションは例です）、
erb
1<%= form_with model: [@shop, @review], local: true do |f| %>
上記のようにform_withの第1引数@shopにはどのお店のレビューとして保存したいのか、第2引数@reviewにはReviewモデルからのインスタンス（Review.new）を予めセットしておけば実装できるのではないかと思います。
投稿2020/11/21 21:50
総合スコア6
Shops Controllerを確認したところreviewインスタンスをbuildすべきところ記載が漏れており、それを記入したところうまくいきました！
一方でform_withでmodelを二つとる方法は知らなかったので勉強になりました。ありがとうございました！
0
自己解決
Shops Controllerでreviewのインスタンスを作っていなかったことが原因でした。@review = @shop.reviews.buildを追記したら解決できました。皆様お力を貸してくださりありがとうございました。
投稿2020/11/22 09:08
総合スコア7
0
恐らく以下で、formタグ内のアクションは変わらずに、name = "review[title]"になるかと思います。
<%= form_with url: user_reviews_path(current_user), scope: :review do |f| %>
もしmethodがgetになったら、
<%= form_with url: user_reviews_path(current_user), scope: :review, method: :post do |f| %>
を試してみて下さい。
如何でしょうか？
投稿2020/11/21 17:44
総合スコア563
0
その場しのぎの対処としてはscopeを設定するとよいでしょう。
erb
1<%= form_with(model: @review, scope: :review, url: user_reviews_path(current_user), local: true) do |f| %>
なお@reviewが、適切に設定されたReviewクラスのオブジェクトならば
erb
1<%= form_with(model: [current_user, @review], local: true) do |f| %>
が可能な筈です。
エラーになるという場合はコントローラかモデルに問題がありそうです。
投稿2020/11/21 17:41
総合スコア15149
ご指摘の通りコントローラーに問題があり、reviewインスタンスを作っていなかったことが原因でした。またこのような問題に当たった時はコントローラーとモデルをよく確認したうえでscopeの使い方もきちんと勉強しようと思います。ご助言ありがとうございました！
あなたの回答
tips
プレビュー
