Laravel Policies - 複数の引数を関数に渡す方法

Question

投稿を削除/更新するためにユーザーキャラクタを承認しようとしています。私はこれを行うためのポリシーを使用していましたが、ポリシー関数には1つのパラメータしか渡すことができませんでした。ユーザーと他の変数より多くを渡すと、変数は関数に渡されません。

モデル：ユーザーには複数の文字があり、複数の投稿を投稿することができます。だから、認証目的のために、私は現在の文字のIDで投稿のcharacter_idを比較しなければならない...-

docsごとに、あなたは門ファサードに多くの倍数を渡すことができます。

Gate::define('delete-comment', function ($user, $post, $comment) { 
    // 
}); 

をしかし、私はcouldnとにかくポリシーでそうすることはできません。私がしなければならなかったのは、認証に必要なオブジェクトを取得するためにRequestオブジェクトを注入することでした。基本的に私はUser Objectは必要ありません。

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

Requestオブジェクトを使用すると動作しますが、非常にハッキリです。これを達成するためのより良い方法はありますか？

編集：CharacterLocationControllerで

私は方法showを持っていると私は、リソースを表示する前にアクションを許可します。ポリシーは、このように登録されて

public function show(Request $request, Character $character, Location $location) 
{ 
    $this->authorize([$location, $character]); 
    ... 
} 

：AuthServiceProvider

に'App\Location' => 'App\Policies\LocationPolicy' Iは、ポリシー関数に渡された配列をダンプし、それだけ$locationを出力します。

public function show(User $user, $data) { 
    dd($data); // expecting location and character 
    return !$location->private || $location->authorized->contains($this->character); 
} 

Answer 1

ここでは、何の機能が何をしているのかについて多少の混乱があると思います。

あなたは

Gate::define('delete-comment', function ($user, $post, $comment) { 
    // 
}); 

それともCommentPolicy

public function delete(User $user, Post $post, Comment $comment) 
{ 
    return $user->id === $post->user_id; 
} 

内を使用する場合は、あなたがやっているすべてのルールを定義しています。この時点で、私たちは何かを渡すことを心配することはなく、受け取ったオブジェクトはお互いにやりとりすることができます。この2つの唯一の違いは、ポリシーを使用する場合です。これは、すべてのルールを簡単で読みやすいクラスに抽象化する簡単な方法です。何百ものテーブルやモデルを持つアプリを持っている場合、これらのルールをアプリ全体に散らばってしまうと、混乱が早くなり、ポリシーはそれらをすべて整理しておくのに役立ちます。

あなたが実際にこれらのアイテムを渡すときに何かをする権限を持っているかどうかを確認しているときです。たとえば、あなたはパラメータがポリシーまたはGate::defineメソッドに渡すことしようとしているかを制御するものであるCommentController

$this->authorize('delete', [$post, $comment]); 

で

if (Gate::allows('delete-comment', [$post, $comment])) { 
    // 
} 

場合や、次のようありません。ドキュメントによれば、$userパラメータが既に追加されています。この場合、正しい$postと$commentが変更されることを心配する必要があります。