Deleted

[Arik]

Не кажется, что моделька (так сказать одноклеточная) начала много на себя брать?)
И теперь трейтом (умудрилась заразиться паразитом) теперь знатно зависима от auth()?
Для контекста web, api можно еще простить, но вот дальше как жить с ней не ясно.
Я пока сам не знаю как лучше реализовать, только начал изучать слои и все такое…

[Ant-kul]

Насчет смешивания ответственности — Вы правы. Буду признателен, если поделитесь идеей, так как сам я не придумал как перегрузить ответственность на политики без оверкода, который, пожалуй, не мой выбор.

[gnus212]

Спасибо! Многое для себя почерпал

[Ant-kul]

Спасибо и Вам за фидбэк. Мне очень приятно)

[Fortop]

if ($user->can('delete-' . $this->getModelClass())) {

Не нужно так. Вы просто на ровном месте усложнили жизнь любому новому/стороннему разработчику на проекте.
Если уж так хочется использовать эту механику, то пишите методы/хелперы.

if ($user->canDelete($this->getModelClass())) {

Тогда заработает автокомплит и разработчик, наткнувшись на это в любом месте кода, при интроиспекции поймет какие вообще есть варианты. А с текстовыми константами нужно ресерчить сколько мест?

[Ant-kul]

Тогда мы перегружаем класс модели. Так как возможно проверять доступность не только стандартного CRUD, но и дополнительных действий. И мы на ровном месте получаем модель User на 300+ строк.

[Fortop]

Правильно.

Следующий шаг улучшения качества кода — не кладите эти проверки в модель.
Положите их, например, в Gate, который будет принимать пользователя и все что нужно для проверки по контексту.