Ребят, так правильно ли прописывать права на удаление только своего

Question

Ребят, так правильно ли прописывать права на удаление только своего

блога через политики?
== BlogPolicy
public function delete(User $user, Blog $blog)
{
return $user->id === $blog->user_id;
}
== BlogController
public function destroy(Blog $blog)
{
if (auth()->user()->can('delete',$blog)) {
// можно удалить
}
Все работает. Просто стоит ли для этого прописывать политику или есть другие способы?

#laravel #programming #russian

0

06.01.2022

4 ответов

16 просмотров

Алексей Лукин

Алексей Лукин
Политики - правильный путь. И пара утренних замеча...

Ну и да, для API-контроллеров хорошо помогает `$this->authorize(BlogPolicy::ABILITY_DELETE, $blog); вместо всего `if

0

07.01.2022

Dmytro 🇺🇦

можно еще через Gate делать, а также вынести проверку из контролера в роуты через can Route::put('/post/{post}', function (Post $post) { // The current user may update the post... })->can('update', 'post');

0

07.01.2022

Roman Gill Автор вопроса

Алексей Лукин
Политики - правильный путь. И пара утренних замеча...

Спасибо за полезный подробный ответ. Учту.

0

07.01.2022

Алексей Лукин · Accepted Answer

Политики - правильный путь. И пара утренних замечаний за чашкой кофе :) Во-первых, как одно из ТОП-10 правил - загоняйте все константные значения в константы :) Т.е. к этому примеру определите в классе BlogPolicy константу, скажем, public const ABILITY_DELETE = 'delete'; и в котроллере вместо 'delete' используйте BlogPolicy::ABILITY_DELETE - поверьте, когда у вас появится какой-нибудь API где вы переиспользуете эту политику, а потом может в каком-нибудь ресурсе вы будете отдавать список прав к блогу... В общем, человек, которому потребуется найти все места, где вы вы используете эту абилити скажет вам огромное спасибо, сделав это одним кликом в IDE, а не поиском всех возможных 'delete' от разных политик. Во-вторых, я бы всё же посоветовал в контроллере использовать cannot и обрабатывать сначала не нормальную, прерывающую логику - как правило такие вещи читаются легче.

169 похожих чатов

Ребят, так правильно ли прописывать права на удаление только своего

4 ответов

Похожие вопросы