ParameterMissing を解決できない。

Question

あるユーザーが特定の記事をお気に入り登録して、お気に入り記事のみを一覧表示できる機能を実装しようとしているのですが、お気に入り登録を解除する機能でParameterMissingエラーが起きました。

データベース上では、id=11のfavoriteのなかに今のユーザーID(5)とお気に入りを解除したい記事のID(5)を格納していて、このfavoriteを消去することでお気に入りを解除します。

以下が、favoriteクラスのコントローラのコードです。

class FavoritesController < ApplicationController
    def destroy
        @favorite = Favorite.find(favorite_params)
        @favorite.destroy
        redirect_to favorite_path(current_user)
    end

    private

    def favorite_params
      params.require(:favorite).permit(:user_id, :article_id)
    end

end

user, favorite, articleの関連付けは済ませてあります。

この時、以下のエラーが発生しました。

不足してるコードがあったらすみません。パラメータについての知識が不足しているのかもしれませんが、解決の仕方がわかりません。よろしくお願いします！

Answer 1

テーブル間の関係は推測で書いてみましたが、こんな感じになるのではないでしょうか？

def destroy
  article = Article.find(params[:article_id])
  favorite = article.favorites.find_by(article_id: article.id)
  favorite.destroy if favorite
end

Comments

[yutann]

情報不足で失礼しました。各テーブルにはカラムが以下のようにあります。
favorite→id, user_id, article_id
article→id
user→id
教えていただいたコードでは、ActiveRecord::RecordNotFound in FavoritesController#destroyというタイトルでエラーが発生しました。

article = Article.find(params[:article_id])についてCouldn't find Article without an IDと指摘されました。どうしたらよいでしょうか？

[yutann]

あ、articleにはarticle_idカラムがないから、params[:id]にすればよいということでしょうか。実際に実行してみるとエラーが出なくなり、無事実行できました!

[bamboo-nova]

あ、userからarticleへは別途で1対多の関係はなかったんですね汗
良かったです。

[yutann]

遅れての再コメントで申し訳ありません。
回答にあるdestroyアクションの中の2行目で何をしているのか教えていただきたいです。
必要であれば再度質問を作成します。

[bamboo-nova]

これは、もしもフォーム画面で例えばですが
<%= form_for(@article.favorites.find_by(article_id: @article.id), html: { method: :delete }) do |f| %>
<%= hidden_field_tag :article_id, article.id %>
などとして、article_idという属性にarticleのidを入れていた場合にparams[:article_id]として取得されるので、
favorite = article.favorites.find_by(article_id: article.id)
として、params[article_id]がarticleのidであることを示す必要があったためです。

今回は、そのままparams[:id]で参照させているみたいなので問題ないかなと思います。

[rhiroe]

解決済みですが、コメント失礼しますね。
favorite = article.favorites.find_by(article_id: article.id)
この部分ですが僕もちょっと疑問で、
あんまり意味のないことをしてるんじゃないかなぁ...と
もしかして
favorite = article.favorites.find_by(user_id: current_user.id)
がしたかった...とかではないですか？というのも
article.favorites
の時点で
article_id == article.id
のものしかないはずなんですよね。
なのでここで行うべきことは
user_idでの絞り込みじゃないかなぁと思います。

[bamboo-nova]

あっ…本当ですね汗

すみません、補足ありがとうございます…！