Как правильно получать элемент, который связан с помощью hasMany?

Question

[lexstile]

Нужно показать конкретный элемент, в моем случае - меню.
Попробовал 2 варианта - оба работают - метод show.
Вопрос: как лучше получать в данном кейсе? Возможно, есть еще вариант, более правильный?

# MenuController
    public function show(Project $project, Menu $menu)
    {
        return $this->baseJson(data: new MenuResource($menu));
    }

    public function show(Project $project, Menu $menu)
    {
        $menu = $project->menu()->find($menu->id);

        return $this->baseJson(data: new MenuResource($menu));
    }

    # ProjectModel
    public function menu()
    {
        return $this->hasMany(Menu::class);
    }

Comments

[Дмитрий]

разницу в методах я вижу. а смысл этой разницы? вы хотите что бы была проверка что модель меню принадлежит этой конкретной модели проекта?

[lexstile]

Дмитрий, я так понимаю, что во 2-ом случае модель меню будет принадлежать модели проекта?
А в первом я смогу показать любое меню при любом вызове существующего для данного пользователя проекта.

Вопрос наверно такой: использовать второй вариант совместно с методом findOrFail или на уровне policy решить данную проблему?

Если брать policy, придется закидывать policy проекта в policy menu? (т. к. в ProjectPolicy проверяется соотвествие модели пользователя и проекта; а здесь требуется дополнительно проверить соответствие проекта и меню)

[Дмитрий]

lexstile, если внимательно глянуть доку то при соблюдении naming conventions оно автоматом будет проверять scope модели.

/**
     * Retrieve the child model query for a bound value.
     *
     * @param  string  $childType
     * @param  mixed  $value
     * @param  string|null  $field
     * @return \Illuminate\Database\Eloquent\Relations\Relation
     */
    protected function resolveChildRouteBindingQuery($childType, $value, $field)
    {
        $relationship = $this->{Str::plural(Str::camel($childType))}();

        $field = $field ?: $relationship->getRelated()->getRouteKeyName();

        if ($relationship instanceof HasManyThrough ||
            $relationship instanceof BelongsToMany) {
            $field = $relationship->getRelated()->getTable().'.'.$field;
        }

        return $relationship instanceof Model
                ? $relationship->resolveRouteBindingQuery($relationship, $value, $field)
                : $relationship->getRelated()->resolveRouteBindingQuery($relationship, $value, $field);
    }

Ежели вы не хотите соблюдать эти конвенции, к чему это - то вы можете в модели ProjectModel переопределить метод resolveChildRouteBinding и сами руками выставить проверку вложенности какую вам надо.

/**
 * Retrieve the child model for a bound value.
 *
 * @param  string  $childType
 * @param  mixed  $value
 * @param  string|null  $field
 * @return \Illuminate\Database\Eloquent\Model|null
 */
public function resolveChildRouteBinding($childType, $value, $field)
{
    //Смотрим что нам прилетело - и возвращаем запрос
}

[lexstile]

при соблюдении naming conventions оно автоматом будет проверять scope модели

То есть, если в методе show определяются необходимые Project $project и Menu $menu, это еще не значит, что scope проверяется?

[Дмитрий]

lexstile, при соблюдении naming conventions проверяется. в вашем случае мне роуты не видны да и названия ваших моделей тоже. Но к чему теоретические изыскания - вы можете все это проверить поставьте в AppServiceProvider в boot

DB::listen(function($query) {
            Log::info(
                $query->sql,
                $query->bindings,
                $query->time
            );
        });

И посмотрите накладывает ли он ограничения или нет. Если нет - переопределите метод

[lexstile]

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

Ни один из вариантов из тела вопроса корректно не работает:
1. Первый удаляет/показывает любое меню, даже от другого проекта.
2. Второй вроде отрабатывает, но find пришлось заменить на findOrFail, иначе кидал 500 ошибку, когда не совпадали проект и меню.

Благодарю.

[Дмитрий]

lexstile, он в логи выводит. cat storage/logs/laravel.log сделайте.
2. Вариант не правильный. Вы дважды делаете запрос что бы получить одну и ту же сущность- это хреново

[lexstile]

Дмитрий, спасибо, нашел логи.
2. Согласен, два раза - это не то, что нужно.

Answer 1

[Dastan_UI]

Первый вариант самый лучший. Вы используете Implicit Binding. Вы переходите по роуту /my-route/menu/1. Таким образом laravel автоматически ищет Menu::find(1).

Во втором случае вы выполняете два действия.
1. То что выше.
2. Дополнительно ищете Menu::where(relationship)->find(1)

Минусы 2 варианта - вы два раза ищете одно и то же меню, дополнительно во втором разе добавляете условия Relationship, что более медленнее если не указаны индексы

Comments

[Dastan_UI]

При поиске по Идентификатору вам не нужно ставить relationship, если только вы не хотите не показывать, если у него нет доступа к этому меню.

В таком случае используйте 2 вариант сразу с 2 шага.

//Route
Route::get('menu/{project}/{id}', [MenuController::class, 'show']);

//Controller method
public function show(Project $project, int $id){
$menu = $project->menu()->findOrFail($id);
}