Railsのupdateアクションでデータの更新がされない

Question

前提・実現したいこと

閲覧いただきありがとうございます。

Railsでレビューサイトを作っています。
productにひもづくreviewの編集をしたいのですがうまくいきません。

発生している問題・エラーメッセージ

エラーは出ておらず、editアクション、updateアクション共に呼ばれ、ビューも呼ばれています。
編集フォームに入力しトップに戻ることができますが、カラムの値が更新されていません。
createとdestroyは正常に動きます。

該当のソースコード

reviews_controller.rb

　def edit
    @review = Review.find(params[:id])
  end
  
  def update
    review = Review.find(params[:id])
    review.update(review_params) if current_user.id == review.user_id
  end

  private
  def review_params
    params.require(:review).permit(:text).merge(product_id: params[:product_id], user_id: current_user.id)
  end

edit.html.haml

  = form_for (@review) do |f|
    .form__text
    = f.text_area :text, placeholder: @review.text
    .form__buttom
    = f.submit "更新する"

routes.rb

Rails.application.routes.draw do
  devise_for :users
  resources :users, only: :show
  resources :reviews, except:[:new, :create]
  resources :products, only:[:index, :show] do
    resources :reviews, only:[:new, :create]
  end
  root 'reviews#index'
end

↑newとcreateのみproductにネストさせています（これがいけない？）
はじめは全部ネストさせていたのですが、editもネストさせるとidが二つ(/products/:product_id/reviews/:id)のようになり
link_toなどでprefixに引数を渡しても先頭の:idにしか反映されなかったので、
editはそもそもproductにネストさせる必要ないかなーと思いこのようにしました。

↓ちなみに、new.html.haml

= form_for [@product, @review] do |f|
    .form__text
    = f.text_area :text
    .form__buttom
    = f.submit "投稿する"

newとcreateはこれでうまくいっています。

試したこと

updateアクションでbinding.pry

{"utf8"=>"✓", "_method"=>"patch", "authenticity_token"=>"pyTs75BIVj8bu7t9lyDJd/6n6pZOFb93dRUADR0+/2KGlJD5wKxagKMtFr0T3eU32/YX/7rNolaHzBokCyqRNw==", "review"=><ActionController::Parameters {"text"=>"初投稿〜\r\n編集後"} permitted: false>, "commit"=>"更新する", "controller"=>"reviews", "action"=>"update", "id"=>"12"} permitted: false>

ruby 2.5.1

paramsはきちんと渡っていると思うので、なぜうまくいかないか分かりません。
宜しくお願い致します。

Comments

[rhiroe]

> はじめは全部ネストさせていたのですが、editもネストさせるとidが二つ(product/:id/review/:id)のようになり
/products/:product_id/reviews/:id
のようになりませんか...？普通はこうなるんですが...。

Rails.application.routes.draw do
resources :products, only:[:index, :show] do
resources :reviews
end
end

$ rails routes | grep reviews
product_reviews GET /products/:product_id/reviews(.:format) reviews#index
POST /products/:product_id/reviews(.:format) reviews#create
new_product_review GET /products/:product_id/reviews/new(.:format) reviews#new
edit_product_review GET /products/:product_id/reviews/:id/edit(.:format) reviews#edit
product_review GET /products/:product_id/reviews/:id(.:format) reviews#show
PATCH /products/:product_id/reviews/:id(.:format) reviews#update
PUT /products/:product_id/reviews/:id(.:format) reviews#update
DELETE /products/:product_id/reviews/:id(.:format) reviews#destroy

[退会済みユーザー]

失礼致しました、仰る通りです。
質問を修正しました。
ご指摘頂きありがとうございます！

[rhiroe]

直接reviewのIDでeditにアクセスしているなら問題ないはずなので
もしかするとproductのidでreviewを検索していませんか？

[退会済みユーザー]

ご回答ありがとうございます。
editアクションで編集画面に推移したときに、http://localhost:3000/reviews/12/editとなっており、SequelProでも確認しましたが該当レビューのidがきちんと反映されていました、、

[rhiroe]

if current_user.id == review.user_id の部分が true になってないのかもしれないですね
デバッグして確認してみてください

[退会済みユーザー]

デバックしてみましたが、こちらも問題なくidが渡っていました。
editアクションもネストさせてやってみようかと思います。

Answer 1

結局editもネストさせ、/products/:product_id/reviews/:idの状態でidを双方に渡したらupdateできました。

色々とご教授頂いて有難うございました！