Ruby on Railsの友情を破壊する

Question

私は現在、ruby on railで友情を破壊することに問題があります。友情は正常に作成されています（私はこれをレールコンソールでテストしました）が、友情を削除することはできません。私はちょうどエラーを取得して保持するので、既存の友情のためのチェックを追加しました

<%= form_for(current_user.friendships.find_by(friend_id: @user.id), 
     html: { method: :delete }) do |f| %> 
    <%= f.submit "Unfriend", class: "btn" %> 
<% end %> 

注：以下は

class FriendshipsController < ApplicationController 

    def create 
    @current_friend = User.find(params[:friend_id]) 
    @friendship = current_user.friendships.build(:friend_id => @current_friend.id) 
    @friendship_2 = @current_friend.friendships.build(:friend_id => current_user.id) 
    if @friendship.save && @friendship_2.save 
     flash[:notice] = "Friend added" 
     redirect_to current_user 
    else 
     flash[:notice] = "Can not add friend" 
     redirect_to current_user 
    end 
    end 

    def destroy 
    @friendship = current_user.friendships.find_by(friend_id: params[:id]).first 
    if @friendship.exists? 
     @friendship.destroy 
    end 
    @friendship_2 = Friendship.where(user_id: params[:id], friend_id: current_user.id).first 
    if @friendship_2.exists? 
     @friendship_2.destroy 
    end 
    flash[:notice] = "Friendship destroyed" 
    redirect_to current_user 
    end 
end 

は友情を削除するための私の形式である：以下

は私のコントローラのコードです友情は無かったと言いましたが、これは私の現在の問題です。

ありがとうございました！あなたのビューで

Answer 1

あなたは（あなたが@user.idへの供給をしていた）friend_idとの友情を探している、とFriendshipことは、あなたがform_forに送信しているものであるので、あなたのフォームはの:idを持っていますFriendship。 は以前User IDませんFriendship IDでなければなりません実証してきましたを（friend_idとして

はしかし、その後、あなたのdestroyアクションで、あなたのクエリでparams[:id]（友情ID）を使用しています。

あなたのコードがすべき実際にあること：

@friendship = current_user.friendships.find_by(id: params[:id]) 

（とfind_byはすでに単一のレコードを返しますので、あなたは、すべてのことに.firstを呼び出すべきではありません）