出品者以外はトップに戻りたい

Question

やりたいこと

出品者以外は商品編集画面には行かずにトップ画面に戻る処理をしたいです

前提

product テーブルにはidカラム、　user_id(出品者のid)カラムがあります。

やったこと

  def move_to_index_if_not_seller
    redirect_to root_path unless user_signed_in? && current_user.id == Product.where(user_id: current_user.id)
  end

上記のメソッドをコントローラーに作成しました。

  before_action :move_to_index_if_not_seller, only: [:edit]

上記をコントローラーに記載しました。

結果

ログアウトしているユーザー、出品者以外のユーザーはちゃんと編集画面に行けなかったです。
しかし出品したユーザーも編集画面に行くことが出来ませんでした。
whereメソッドだはだめかと思い、findメソッドでもだめでした。

お分かりになる方がいらっしゃれば、ご教授願います。

Answer 1

current_user.id == Product.where(user_id: current_user.id)
id(integer)とProductの集合を比べているので、必ず falseです。

params[:id]にProductのidが入っている　として
current_user.id == Product.find(params[:id]).user_id
に。
実在しないidがくると例外で落ちますから、その対策が必要なら入れてください。

Comments

[987654321]

ありがとうございます！
解決出来ました！