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

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

блога через политики?
== 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)) {
// можно удалить
}
Все работает. Просто стоит ли для этого прописывать политику или есть другие способы?

4 ответов

17 просмотров

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

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

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

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

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

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

Обсуждают сегодня

30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
14
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Ребят в СИ можно реализовать ООП?
Николай
33
Как передать управляющий символ в открытую через CreateProcess консоль? Собсна, есть процедура: procedure TRedirectThread.WriteData(Data: OEMString); var Written: Cardinal;...
Serjone
6
в JclConsole объявлено так: function CtrlHandler(CtrlType: DWORD): BOOL; stdcall; - где ваше объявление с stdcall? у вас на картинке нет stdcall
Karagy
8
https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_h_common.erl#L174 https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_olp.erl#L76 15 лет назад...
Maksim Lapshin
20
~ 2m21s  nix shell github:nixos/nixpkgs#stack ~  stack ghc -- --version error: … while calling the 'derivationStrict' builtin at /builtin/derivation.nix:...
Rebuild your mind.
6
Карта сайта