UnpermittedParametersに苦しんでいます。（rails）

Question

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

Ruby on Rails 5.0.0.1で口コミサイトを開発しています。
ユーザーに紐付いた口コミが更新されずに困っています。
「パラメーターがない！」とエラーで指摘を受けているのですが、実際、createしているときと同じものをいれているので間違いないはずなのですが・・・

app[web.1]: Completed 500 Internal Server Error in 7ms (ActiveRecord: 1.7ms)
app[web.1]: ActionController::UnpermittedParameters (found unpermitted parameter: id):
app[web.1]: app/controllers/reviews_controller.rb:54:in `create_params'
app[web.1]: app/controllers/reviews_controller.rb:20:in `update'

##ソースコード

reviewモデル

class Review < ApplicationRecord
  belongs_to :user
  belongs_to :product
  has_one :review_image
  accepts_nested_attributes_for :review_image

  default_scope -> { order(created_at: :desc) }
  validates :user_id, presence: true
  validates :product_id, presence: true
  validates :content, presence: true, length: { minimum: 200 }
end

すみませんが、どうぞ宜しくお願いします。

##追記

        <div class="review-edit">
            <%= form_for(@review) do |f| %>
            <div class="post-rate">
              <div class="form-group">
                <label for="rate">評価</label>
                <%= f.select :rate, {'1' => 1, '2' => 2, '3' => 3, '4' => 4, '5' => 5}, class: "form-control" %>
              </div>
            </div>
            <div class="post-contents">
              <div class="form-group">
                <label for="textarea">口コミを入力</label>
                <%= f.text_area :content, class: "form-control", placeholder: "口コミ内容", rows: "10", onKeyUp: "countLength(value, 'textlength')" %>
                <p>現在<span id="textlength">0文字</span></p>
              </div>
            </div>
            <div class="post-photo">
              <div class="form-group">
                <label for="review_image">画像を投稿</label>
                <%= f.fields_for :review_image do |review_image| %>
                  <%= review_image.file_field :image %>
                <% end %>
              </div>
            </div>
            <%= f.submit "更新する", class: "btn" %>
            <% end %>
          </div>

##追記エラー

heroku[router]: at=info method=POST path="/reviews/114" dyno=web.1 connect=1ms service=32ms status=500 bytes=1827 protocol=http
app[web.1]: Started PATCH "/reviews/114" for 108.162.246.164 at 2017-08-15 08:12:08 +0000
app[web.1]: Processing by ReviewsController#update as HTML
app[web.1]: Parameters: {"utf8"=>"✓", "authenticity_token"=>"R+GNVKUKE/9SPW4ULsVptV7C6bVw==", "review"=>{"rate"=>"5", "content"=>"口コミが入ります。", "review_image_attributes"=>{"id"=>"20"}}, "commit"=>"編集する", "id"=>"114"}
app[web.1]: Review Load (1.9ms)  SELECT  "reviews".* FROM "reviews" WHERE "reviews"."id" = $1 ORDER BY "reviews"."created_at" DESC LIMIT $2  [["id", 114], ["LIMIT", 1]]
app[web.1]: Completed 500 Internal Server Error in 6ms (ActiveRecord: 1.9ms)
app[web.1]: ActionController::UnpermittedParameters (found unpermitted parameter: id):
app[web.1]: app/controllers/reviews_controller.rb:55:in `create_params'
app[web.1]: app/controllers/reviews_controller.rb:33:in `update'

reviewコントローラー

class ReviewsController < ApplicationController
  before_action :set_review, only:[:show, :edit, :update, :destroy]
  before_action :correct_user, only: [:edit, :destroy]

  def create
   @review = current_user.reviews.create(create_params)
   if @review.save
     flash[:success] = "口コミが投稿されました！"
     redirect_to root_path
   else
     flash[:alert] = "口コミの投稿に失敗しました。"
     redirect_to root_path
   end
 end

 def new
   @product = Peroduct.find(params[:id])
   if signed_in?
     @review = current_user.reviews.build
     @review.build_review_image
   else
     redirect_to new_user_registration_path
   end
 end

 def show
 end

 def edit
 end

 def update
   if @review.update_attributes(create_params)
     flash[:success] = "口コミが更新されました"
     redirect_to @review
   else
     flash[:alert] = "口コミの編集に失敗しました。"
     render 'edit'
   end
 end

 def destroy
   @review.destroy
   flash[:success] = "口コミが削除されました"
   redirect_to request.referrer || root_url
 end

 private

 def set_review
   @review = Review.find(params[:id])
 end

 def create_params
   params.require(:review).permit(:id, :content, :rate, :title, :product_id, :user_id, review_image_attributes: [:image])
 end

 def correct_user
   @review = current_user.reviews.find_by(id: params[:id])
   redirect_to root_url if @review.nil?
 end
end

Comments

[gouf]

たとえば pry を用いてブレークポイントを仕掛け、プログラムの動作を追ってみましたか?(特にparams の内容など) : [今更聞けないpryの使い方と便利プラグイン集 - Qiita](http://qiita.com/k0kubun/items/b118e9ccaef8707c4d9f)

[yamady]

ありがとうございます！やってみます！pryマスターできてなかったので、勉強し直すようにします。。。

Answer 1

   params.require(:review).permit(:id, :content, :rate, :title, :product_id, :user_id, review_image_attributes: [:image])

:idはURLに含まれているはずですので、フォーム経由のパラメータとして渡されるべきではありません。

---

コメントだと追いかけられないのでこちらにも書いておきますが、ネストされたreview_image_attributesの方に:idがないのでここで引っかかってます。作成の時はなくても動くのですが、更新の時は:idが必要になります。

Comments

[yamady]

suzukisさま、ご回答いただきましてありがとうございます。
ローカル環境ではうまくいっているのですが、本番環境で同じ文言でやはりエラーになてしまいます。

def create_params
params.require(:review).permit(:content, :rate, :title, :product_id, :user_id, review_image_attributes: [:image])
end

found unpermitted parameter: idと出てしまうのですが、これは
product_idやuser_idもはずさければならないということなのでしょうかね。。

[chelsy7110]

UnpermittedParametersのエラーですが、permitに余計なものが入っているかどうかではありません。
formから送られてきたparamsにpermitに入っていないものが入っていた場合に出るエラーです。
formがどうなっているかコードを追記してもらえますでしょうか？

[suzukis]

`config.action_controller.action_on_unpermitted_parameters = :raise`を設定していないと不正なパラメータは無視されます。ローカルと本番でこの差がないでしょうか。いずれにせよ、idがパラメータとして飛んできている時点でおかしいので、フォームを修正してください。

[yamady]

みなさま、ありがとうございます。
なるほど、ビューでいらないidが送信されてしまっていたということですね(*_*)
このあたり、変なものは渡されていないはずなんですが。。。涙

[suzukis]

あれ。おかしなものなさそうですね。そうなるとちょっとややこしいです。
・コントローラのソースとエラーメッセージの行数が一致してません。
・コンソールまたはログに送信されたパラメータが書いてあるはずです
このあたり質問の追記修正をお願いします。

[yamady]

エラーメッセージとコントローラ、およびパラメータを追記いたしました。。汗

[chelsy7110]

あああああああ！
review_image_attributesの方に:idがないですね。strong parametersのところに足してみてください。それで解決すると思います。
fields_forでやった場合、見えないですがidが送られるんですよ。

[yamady]

おおおおおお！
できました！！（感動）strong parametersまだ完全に理解しきれてないですね、、よくここミスが多いので、しっかりとやらなければ。。
chelsy7110さん、suzukisさん、goufさん、本当にありがとうございました。！！！