フォロー機能でエラー（申請時）

Question

フォロー機能を追加したのですが、Followの申請でエラーが出てしまいます。
しかしながらUnfollowする際には、エラーなく正常に機能しています。

NoMethodError in FriendshipsController#create
undefined method `user_id' for nil:NilClass
>**active_friendships.create(follower_id: @current.user_id, followed_id: user.id)**

<% if @current_user != @user %>
  <% if @current_user.following?(@user) %>
    <%= button_to "Unfollow", friendship_path(user_id: @user_id), method: :delete, :class => "btn" %>
  <% else %>
    <%= button_to "Follow", friendships_path(user_id: @user_id), method: :post, :class => "btn" %>
  <% end %>
<% end %>

class FriendshipsController < ApplicationController
  before_action :authenticate_user
  before_action :find_user

  def create
    @current_user.follow(@user)
    redirect_back(fallback_location: "something")
  end
  def destroy
    @current_user.unfollow(@user)
    redirect_back(fallback_location: "something")
  end
  def find_user
    @user = User.find_by(id: params[:id])
  end
end

  def follow(user)
    active_friendships.create(follower_id: @current.user_id, followed_id: user.id)
  end

  def unfollow(user)
    active_friendships.find_by(followed_id: user.id).destroy
  end

  def following?(user)
    following.include?(user)
  end

エラーではuser_idの値が正しく取れていないようですが、
unfollow時では正常し、friendshipのdestroyが上手くできてます。

恐れ入りますが、お教え頂ければ幸いです。

Answer 1

def find_user
@user = User.find_by(id: params[:id])
end
を、@user = User.find_by(id: params[:user_id])
に変更したら正常に動きました。
ありがとうございました。

Answer 2

find_userでuser_idではなくidを取っているので、

friendships_path(user_id: @user_id)
を
friendships_path(id: @user_id)
とすると解決するような気がします。
(が、なぜunfollowでは動くのかはわかりません。。)

Comments

[tomtom1]

ご回答ありがとうございます。
残念ながら同様のエラーがでてしまいました。
他に考えられる解決方法があれば、お教え頂ければ幸いです。

[mariguranule]

def follow(user)
active_friendships.create(follower_id: @current.user_id, followed_id: user.id)
end

のところ、
@current.user_id
ではなく
@current_user.id
ですね！

[tomtom1]

ご回答ありがとうございます。
active_friendships.create(follower_id: @current_user.id, followed_id: user_id)
に変更しましたところ、undefined method `id' for nil:NilClassのエラーがでました。

また、確認のために敢えて、follower_id: @current_user.id, を抜いたところ、
undefined local variable or method `user_id' for #<User:000>のエラーがでます。

[tomtom1]

そしてまた敢えて、下記のように値を入れて確認しましたところ、
active_friendships.create(follower_id: 1, followed_id: 2)
正常にuser_id:1がuser_id:2をフォローすることに成功しました。

そこで下記2つで確認しましたところ、
active_friendships.create(follower_id: @current_user.id, followed_id: 2)
active_friendships.create(follower_id: 1, followed_id: user.id)
両方共、undefined method `id' for nil:NilClassのエラーが出現しました。
従って、両方の値の取得がしっかりできていないという現状と考察しています。
Unfollowでは上手くいくところがまた謎が深まります。

[mariguranule]

@current_userではなく@なしのcurrent_userでできる気がします。

[tomtom1]

ありがとうございます。
試しましたが、undefined local variable or method `current_user' のエラーがでます。

また、active_friendships.create(followed_id: 2)で挑戦したところ、
正常にuser_id:1がuser_id:2をフォローできました。

従って、follow_idの値のみ必要としている状況なのですが、
active_friendships.create(followed_id: user_id)ですとやはり、
undefined local variable or method `user_id'のエラーが発生します。

def unfollow(user)
active_friendships.find_by(followed_id: user.id).destroy
end
のアンフォローではしっかしとuser_idの値を取得できているのですが..

[tomtom1]

また、UserControllerに下記の記載がありますが、
エラーの原因となる値の取得を制限している記述なのありましたらご指摘頂ければ幸いです。

before_action :authenticate_user, {only: [:index, :show, :edit, :update]}
before_action :forbid_login_user, {only: [:new, :create, :login_form, :login]}
before_action :ensure_correct_user, {only: [:edit, :update]}