laravel,認可処理で上手く行かない

Question

初心者です。

エラーハンドリングを実装しています。

"ユーザーとskidayが紐づいているときのみ許可する"という事をやりたいです。

現状は自分も編集画面にアクセスできなくなってしまいます。

Route::get('/calendar/{skiday}', 'CalendarController@show')->name('calendar.show');
Route::group(['middleware' => 'can:update,skiday'], function(){
    Route::get('/calendar/{skiday}/edit', 'CalendarController@edit')->name('calendar.edit');
    Route::post('/calendar/{skiday}/edit', 'CalendarController@update')->name('calendar.update');
});

<?php

namespace App\Policies;

use App\User;
use App\Skiday;
use Illuminate\Auth\Access\HandlesAuthorization;

class SkidayPolicy
{
    use HandlesAuthorization;

    /**
     * Create a new policy instance.
     *
     * @return void
     */
    public function __construct()
    {
        //
    }

    public function update(User $user, Skiday $skiday)
    {
        return $user->id === $skiday->user_id;
    }
}

use App\Skiday;
use App\User;
use App\Policies\SkidayPolicy;

class AuthServiceProvider extends ServiceProvider
{
    protected $policies = [
        Skiday::class => SkidayPolicy::class,
    ];
}

public function edit($skiday)
    {
        $skiday = Skiday::find($skiday);
        return view('calendar.edit', ['skiday' => $skiday]);
    }

どこに原因があるのかわからず困っています。

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

よろしくお願いします。

php7.4.3
laravel6.18

Comments

[Lulucom]

「エラーハンドリング」という表現よりも
「認可」と表現された方が正確かなと思いました。
https://readouble.com/laravel/6.x/ja/authorization.html

[tenlife]

ありがとうございます。修正しました。

Answer 1

$this->check($skiday);

これが邪魔をしていました。これをコメントアウトして下記のように書きました。

$this->authorize('update', $skiday);

ただ、できればwebで下のような感じで処理したかった。なぜかハマらない。こっちの方ができたら見やすいかなと。書き方に問題がありそうだなと思っています。

Route::group(['middleware' => 'can:update,skiday'], function(){
    Route::get('/calendar/{skiday}/edit', 'CalendarController@edit')->name('calendar.edit');
    Route::post('/calendar/{skiday}/edit', 'CalendarController@update')->name('calendar.update');
});

Answer 2

SkidayPolicyクラスのupdateメソッドで、===を==にしてみるとうまくいかないでしょうか。
もしそうなら、どちらかのidの値が文字列になってしまっているのではないかと思います。

    public function update(User $user, Skiday $skiday)
    {
        return $user->id == $skiday->user_id;
    }

Comments

[tenlife]

回答ありがとうございます。他の部分に問題がありました。
ただ、webに処理をかければ尚良かったです。

Answer 3

エラーは表示されていないのですか？
debugbarの導入と使い方を学ばれることをオススメします。

Comments

[tenlife]

debugbarを使用すると下記のエラーが出ました。
error
array:4 [▼
"ability" => "update"
"result" => null
"user" => 1
"arguments" => "[0 => 2]"
]