Couldn't find ＊＊＊ without an IDというエラーに困っております

Question

Couldn't find ＊＊＊ without an IDというエラーについて

お世話になります。
現在、railsでtwitterクローンを作成中です。
フォロー機能に加えて、お気に入り機能を追加しようとしております。

上記お気に入り機能を実装中に以下のエラーメッセージが発生しました。

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

ActiveRecord::RecordNotFound in FavoritesController#create
Couldn't find Minipost without an ID
エラーメッセージ
Couldn't find Minipost without an ID
Extracted source (around line #5):
４def create
５ @minipost = Minipost.find(params[:minipost_id])
６ current_user.like(minipost)
７ flash[:success] = 'このメッセージをお気に入りにしました'
８ redirect_back(fallback_location: root_path)
app/controllers/favorites_controller.rb:5:in `create'

Request
Parameters:

{"utf8"=>"✓", "authenticity_token"=>"s8/u2oD4WEXwIo/Qu7gUiKBNPOdr44g1Fp2SaaoPX6dEhh6yrSrFWNesapJiBDPXEUEfGMvY9OXswX8qNtp22A==", "user_id"=>"", "commit"=>"Like"}

該当のソースコード

class User < ApplicationRecord
  before_save { self.email.downcase! }
  validates :name, presence: true, length: {maximum:50}
  validates :email, presence: true, length: {maximum:255},
                    format: {with:/\A[\w+\-.]+@[a-z\d\-.]+.[a-z]+\z/i },
                    uniqueness: {case_sesitive: false}
  has_secure_password
  has_many :miniposts
  has_many :relationships
  has_many :followings, through: :relationships, source: :follow
  has_many :reverses_of_relationship, class_name: 'Relationship', foreign_key: 'follow_id'
  has_many :followers, through: :reverses_of_relationship, source: :user
  has_many :favorites, dependent: :destroy
  has_many :likes, through: :favorites, source: :minipost
  
  def follow(other_user)
    unless self == other_user
      self.relationships.find_or_create_by(follow_id: other_user.id)
    end
  end
  
  def unfollow(other_user)
    relationship = self.relationships.find_by(follow_id: other_user.id)
    relationship.destroy if relationship
  end
  
  def following?(other_user)
    self.followings.include?(other_user)
  end
  
  def feed_miniposts
    Minipost.where(user_id: self.following_ids + [self.id])
  end
  
  
  def like(other_minipost)
    favorites.find_or_create_by(other_minipost, minipost.id)
  end
  
  def unlike(other_minipost)
    favorite = favorites.find_by(other_minipost, minipost.id)
    favorite.destroy if favorite
  end  
    
  def like?(other_minipost)
    likes.include?(other_minipost)
  end
end

class UsersController < ApplicationController
  before_action :require_user_logged_in, only: %i(index show followings followers likes)
  
  def index
    @users = User.order(id: :desc).page(params[:page]).per(15)
  end

  def show
    @user = User.find(params[:id])
    @miniposts = @user.miniposts.order(id: :desc).page(params[:page])
    counts(@user)
  end

  def new
    @user = User.new
  end

  def create
    @user = User.new(user_params)
  
    if @user.save
      flash[:success] = 'ユーザーを登録しました'
      redirect_to @user
    else
      flash.now[:danger] = 'ユーザーの登録に失敗しました'
      render :new
    end
  end
  
  def followings
    @user = User.find(params[:id])
    @followings = @user.followings.page(params[:page])
    counts(@user)
  end
  
  def followers
    @user = User.find(params[:id])
    @followers = @user.followers.page(params[:page])
    counts(@user)
  end
  
  def likes
    @minipost = Minipost.find(params[:id])
    @likes = @user.likes.page(params[:page])
    counts(@user)
  end
  
  private
  
    def user_params
      params.require(:user).permit(:name, :email, :password, :password_confirmation)
    end
    
end

class Favorite < ApplicationRecord
  belongs_to :user
  belongs_to :minipost
  
  validates :user, presence: true
  validates :user_id, uniqueness: { scope: :minipost_id }
  validates :minipost, presence: true
end

class FavoritesController < ApplicationController
  before_action :require_user_logged_in
  
  def create
    @minipost = Minipost.find(params[:minipost_id])
    current_user.like(minipost)
    flash[:success] = 'このメッセージをお気に入りにしました'
    redirect_back(fallback_location: root_path)
  end

  def destroy
    @minipost = Favorite.find(params[:id]).minipost
    current_user.unlike(minipost)
    flash[:danger] = 'このメッセージをお気に入りから外しました'
    redirect_back(fallback_location: root_path)
  end
end

class Minipost < ApplicationRecord
  belongs_to :user
  
  validates :content, presence: true, length: {maximum:255}
  
  has_many :favorites, dependent: :destroy
  has_many :likes, through: :favorites, source: :user
end

class MinipostsController < ApplicationController
  before_action :require_user_logged_in
  before_action :correct_user, only: %i(destroy)
  
  def create
    @minipost = current_user.miniposts.build(minipost_params)
    if @minipost.save
      flash[:success] = 'メッセージを投稿しました'
      redirect_to root_url
    else
      @miniposts = current_user.feed_miniposts.order(id: :desc).page(params[:page])
      flash.now[:alert] = 'メッセージの投稿に失敗しました'
      render 'toppages/index'
    end
  end

  def destroy
    @minipost.destroy
    flash[:success] = 'メッセージを削除しました'
    redirect_back(fallback_location: root_path)
  end
  
  private
  
    def minipost_params
      params.require(:minipost).permit(:content)
    end
    
    def correct_user
      @minipost = current_user.miniposts.find_by(id: params[:id])
      unless @minipost
        redirect_to root_url
      end
    end
end

<% if current_user.like?(minipost) %>
  <%= form_with(model: current_user.favorites.find_by(user_id: minipost.id), local: true, method: :delete) do |f| %>
    <%= hidden_field_tag :user_id, minipost.id %>
    <%= f.submit 'Unlike', class: 'btn btn-danger btn-sm' %>
  <% end %>
<% else %>
  <%= form_with(model: current_user.favorites.create, local: true) do |f| %>
    <%= hidden_field_tag :user_id, minipost.id %>
    <%= f.submit 'Like', class: 'btn btn-info btn-sm' %>
  <% end %>
<% end %>

Rails.application.routes.draw do
  root to: 'toppages#index'
  
  get 'login', to: 'sessions#new'
  post 'login', to: 'sessions#create'
  delete 'logout', to: 'sessions#destroy'
  
  get 'signup', to: 'users#new'
  resources :users, only: %i(index show new create) do
    member do
      get :followings
      get :followers
      get :likes
    end
  end
  
  resources :miniposts, only: %i(create destroy) do
  end
  
  resources :relationships, only: %i(create destroy)
  resources :favorites, only: %i(create destroy)

試したこと

ルーティング⇨いいね！をするボタン（like_button)はusers/id:/likesとしたいため、上記のようにしています。user.rbのlikeメソッドの引数とfavoritesControllerの引数を(user_id, minipost.id)としてみたのですが、エラーは”Couldn't find Minipost with 'id'=”となるだけで解決できずにおります。

補足情報（FW/ツールのバージョンなど）

AWS cloud9
ruby 2.5.3
rails 5.2.4.3

初投稿で拙い記載、不足があるかと思いますが、ご教授いただけますと幸いです。
よろしくお願いします。

Answer 1

<%= hidden_field_tag :user_id, minipost.id %>

なぜminipost.idを:user_idとしているのでしょうか？
2箇所とも<%= hidden_field_tag :minipost_id, minipost.id %>にすると改善するかと思います。

---

<%= form_with(model: current_user.favorites.create, local: true) do |f| %>

createになっているのが気になります。newでいいのではないでしょうか？

Comments

[hirome5]

ご回答いただきありがとうございます。ご指摘の箇所につきましては<%= form_with(model: current_user.favorites.find_by(minipost_id: minipost.id)と２箇所の<%= hidden_field_tag :minipost_id, minipost.id %>のように訂正いたしました。もう１箇所も確かにnewで良いですね。
訂正後、再度出力確認しましたところ、”@minipost = Minipost.find(params[:minipost_id])”の箇所について”Couldn't find Minipost with 'id'=”のエラーとなります。パラメータには"minipost_id"=>"", "commit"=>"Like"}のようにidが空となっているためにエラーだということなのでしょうが、users_controllerのdef likesが誤っているのでしょうか？お分かりになればご指摘いただければと存じます。

[asm]

likesも大分不思議ですが(params[:id]がuserなのかminipostなのかハッキリしない)
どうも、このViewの時点でminipostが空なようなので
ボタンを表示する側のViewおよびControllerについて提示をお願いします。

[hirome5]

ご回答ありがとうございます。そして、返信が遅くなってしまい申し訳ございません。

表示するViewにつきましては
<ul class="list-unstyled">
<% miniposts.each do |minipost| %>
<li class="media mb-3">
<img class="mr-2 rounded" src="<%= gravatar_url(minipost.user, { size: 50 }) %>" alt="">
<div class="media-body">
<div>
<%= link_to minipost.user.name, user_path(minipost.user) %> <span class="text-muted">posted at <%= minipost.created_at %></span>
</div>
<div>
<p class="mb-0"><%= minipost.content %></p>
</div>
<div style="display:inline-flex">
<% if current_user == minipost.user %>
<%= link_to "Delete", minipost, method: :delete, data: { confirm: "You sure?" }, class: 'btn btn-danger btn-sm' %><%= render 'favorites/like_button' , minipost: @minipost %>
<% end %>
</div>
</div>
</li>
<% end %>
<%= paginate miniposts %>
</ul>

となりまして、

コントローラは
class FavoritesController < ApplicationController
before_action :require_user_logged_in

def create
@minipost = Minipost.find(params[:minipost_id])
current_user.like(minipost)
flash[:success] = 'このメッセージをお気に入りにしました'
redirect_back(fallback_location: root_path)
end

def destroy
@minipost = Favorite.find(params[:id]).minipost
current_user.unlike(minipost)
flash[:danger] = 'このメッセージをお気に入りから外しました'
redirect_back(fallback_location: root_path)
end
end

になります。
コメントでマークアップ記法の仕方がわかりかねるため、直接の表記となり、読みづらいことかと存じますがご確認をお願いいたします。

[asm]

> <%= render 'favorites/like_button' , minipost: @minipost %>

paramsに渡らない問題はここですね。
<%= render 'favorites/like_button' , minipost: minipost %>
が正しいかと思います。

[hirome5]

ご回答ありがとうございます。
おかげさまで、表題のエラーが解決いたしました。
別なエラーは出ておりますが、ボタンへの値の受け渡しに詰まっておりましたので大変助かりました。
また何かお訊ねするかもしれませんが、どうぞよろしくお願いします。