laravel,フォロー機能でdetatchがうまく動かない

Question

laravelでフォロー機能を実装しています。

https://readouble.com/laravel/6.x/ja/eloquent-relationships.html#updating-many-to-many-relationships
上記を見ながらdetatchを使おうとしているのですが、うまく動きません。(unfollow)

detachの引数部分の指定がおかしいのか、detachしているのが間違っているのかなと予想しています。

public function followers()
    {
        return $this->belongsToMany(self::class, 'follows', 'followed_id', 'following_id');
    }

    public function follows()
    {
        return $this->belongsToMany(self::class, 'follows', 'following_id', 'followed_id');
    }

    // フォローする処理
    public function follow(Int $user_id)
    {
        // フォローするユーザー取得
        $follow_user_id = Auth::user()->id;
        // 配列の形でattachする
        return $this->follows()->attach($user_id, ['following_id' => $follow_user_id]);
    }

    public function unfollow(Int $user_id)
    {
        $follow_user_id = Auth::user()->id;
        return $this->follows()->detach($user_id, ['following_id' => $follow_user_id]);
    }

public function unfollow(User $user)
    {
        $user->unfollow($user->id);

        return back();
    }

Route::post('/mypage/{user}/follows', 'UserController@follow');
Route::post('/mypage/{user}/unfollows', 'UserController@unfollow');

// $user->idは取得できます
<form method="POST" action="{{ route('unfollow', ['user' => $user->id]) }}">
                            @csrf
                            <button type="submit" class="btn btn-outline-info btn-sm" style="width: 100%;">フォロー解除</button>
                        </form>

<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;

class CreateFollowsTable extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        Schema::create('follows', function (Blueprint $table) {
            $table->bigIncrements('id');
            // フォローする側
            $table->bigInteger('following_id')->unsigned();
            // フォローされる側
            $table->bigInteger('followed_id')->unsigned();

            $table->foreign('following_id')->references('id')->on('users')->onDelete('cascade');
            $table->foreign('followed_id')->references('id')->on('users')->onDelete('cascade');
            $table->timestamps();
        });
    }

    /**
     * Reverse the migrations.
     *
     * @return void
     */
    public function down()
    {
        Schema::dropIfExists('follows');
    }
}

試したこと、detatch部分を書き換えましたが無理でした。

return $this->follows()->detach($user_id);
return $this->follows()->detach(['followed_id', $user_id , 'following_id', $follow_user_id]);

どなたか知恵を貸していただきたいです。

よろしくお願いします

php7.4.2
laravel6.18

Answer 1

アンフォローをする側とされる側が同じユーザーになっていないでしょうか？
以下のようにするとどうでしょうか。フォローについても同様です。

UserController

    /**
     * 指定されたユーザーをアンフォローする。
     *
     * @param  User  $user  アンフォローされるユーザー
     */
    public function unfollow(User $user)
    {
        // 認証済ユーザーがユーザーをアンフォロー
        Auth::user()->unfollow($user->id);

        return back();
    }

User

    /**
     * 指定されたIDのユーザーをアンフォローする。
     *
     * @param  int  $user_id  アンフォローされるユーザーのID
     */
    public function unfollow($user_id)
    {
        return $this->follows()->detach($user_id);
    }

Comments

[tenlife]

丁寧にありがとうございます。非常にわかりやすく本当に助かります。
フォローの方も参考にさせていただきました！