Как правильно сконструировать policy на laravel 9?

Question

[lexstile]

Есть таблица projects (user_id - идентификатор создателя):

Есть таблица приглашенных в проект пользователей:

Полиси для создателя выглядит так:

public function view(User $user, Project $project) {
        return $user->id === $project->user_id;
    }

Вопрос: как мне проверить еще и модель приглашенных пользователей?
Добавил метод в модель Project, но что дальше делать - не знаю:

public function users()
    {
        return $this->belongsToMany(ProjectUser::class);
    }

Answer 1

[Дмитрий]

public function view(User $user, Project $project) {
        return $user->id === $project->user_id || $project->users()->find($user->id);
    }

Comments

[lexstile]

Спасибо за ответ, попробовал.
Не знаю почему, но вываливается ошибка:

"SQLSTATE[42S02]: Base table or view not found: 1146 Table 'db-admin.project_project_user' doesn't exist (SQL: select `project_users`.*, `project_project_user`.`project_id` as `pivot_project_id`, `project_project_user`.`project_user_id` as `pivot_project_user_id` from `project_users` inner join `project_project_user` on `project_users`.`id` = `project_project_user`.`project_user_id` where `project_project_user`.`project_id` = 1 and `project_users`.`id` = 2 limit 1)"

[Дмитрий]

lexstile, а что тут не знать? Вы либо соблюдаете naming conventions для pivot tables, либо прописываете руками в связи как у вас таблицы называются. Как бы если ошибку прочитать - тут не может быть никаких разночтений.

[lexstile]

Дмитрий, нужно и в модели ProjectUser связь прописывать или достаточно Project?

[Дмитрий]

lexstile,

Вот здесь надо указать имя таблицы связки.

return $this->belongsToMany(ProjectUser::class, "ДОКУМЕНТАЦИЯ ПРИДУМАНА НЕ ПРОСТО ТАК");

Собственно если в ошибке написано что он не может найти таблицу - то какие мысли возникают?

[lexstile]

Дмитрий, там не так много примеров, чтобы разобраться.
B как я понял, первым параметром будет уже не ProjectUser, а User что ли...

[Дмитрий]

lexstile, млин. сразу и не разглядел. Зачем вам доп модель ProjectUser для belongsToMany? Она вообще не обязательно. А если уж нужна - то в доке есть пример как прописывать ее через pivot.

return $this->belongsToMany(User::class, "НАЗВАНИЕ ТАБЛИЦЫ приглашенных в проект пользователей");

А если вам нужен pivot именно как модель то -

return $this->belongsToMany(User::class, "НАЗВАНИЕ ТАБЛИЦЫ приглашенных в проект пользователей")->using(ProjectUser::class);

Но у меня сильное подозрение что на вашем этапе разработки может понадобиться такое.

[lexstile]

Дмитрий, ваш пример сработал, благодарю.
А я пробовал так, мой не сработал:

public function users()
    {
        return $this->belongsToMany(User::class, 'project_users', 'user_id', 'project_id');
    }

Зачем вам доп модель ProjectUser для belongsToMany?

Эм, а как иначе создать связь многие ко многим? (честно, не очень понимаю)

[Дмитрий]

lexstile, связка belongsToMany создается через таблицу связку. То есть у меня есть модель Users с таблицей users есть модель Projects с таблицей projects и есть pivot таблица project_to_user. И связь многие ко многим так и прописывается - Модель Project связана с моделью User через pivot таблицу project_to_user через ключ такой, то ключ такой, то ключ такой то, ключ такой то. Если мы откроем доку ларки то увидем

class Project{
public function users(){
 return $this->belongsToMany(User::class, 'project_to_user', ....ключи);
}

Ровно то что я написал. Ключи и таблицу надо указывать только если вы не соблюдаете naming convetions

[lexstile]

Дмитрий, Ааа... вы про то, что таблица нужна... а без модели можно обойтись?

[lexstile]

Дмитрий, благодарю за совет.

[Дмитрий]

lexstile, То что вы пытаетесь сделать - называется один ко многим. Вы прописываете связку model Project -> model ProjectUser -> model User.

так тоже можно.

class Project{
public function project_users(){
 return $this->hasMany(ProjectUser::class);
}
}

class ProjectUser{
public function user(){
 return $this->belongsTo(User::class);
}
}

И тогда у вас будет в $project->project_users коллекция моделей ProjectUser у которых в переменной user будет лежать модель пользователя. Но нахера?

[Дмитрий]

lexstile, Промежуточная модель к pivot таблице тоже иногда может быть нужна если к связке нужно что то привязать. Например в том проекте что я пишу у меня есть связь пробирка с операцией над ней - и к этой связи надо привязать используемые реагенты и еще кучу всего - и я завел через using модель pivot таблицы. Но в вашем случае - нет

[lexstile]

То что вы пытаетесь сделать - называется один ко многим. Вы прописываете связку model Project -> model ProjectUser -> model User.

То есть, если мы рассматриваем таблицы users и projects через связанную таблицу - это многие ко многим, а можно просто взять таблицы projects и project_users и тогда можно использовать связь один ко многим?

[lexstile]

Дмитрий, понял, благодарю.

Промежуточная модель к pivot таблице тоже иногда может быть нужна если к связке нужно что то привязать. Например в том проекте что я пишу у меня есть связь пробирка с операцией над ней - и к этой связи надо привязать используемые реагенты и еще кучу всего - и я завел через using модель pivot таблицы

[jazzus]

Лучше метод exists

[Дмитрий]

jazzus, чем?

[jazzus]

Дмитрий, он специально для этого создан, возвращает булеан, а не модель,лучше читаемость кода

[Дмитрий]

jazzus, наверное вы правы.