Ошибка с Route::resource() в Laravel?

Question

[iiideb]

Происходит ошибка

Missing required parameters for [Route: post.show] [URI: post/{post}]. (View: E:\ospanel\OSPanel\domains\akademka\resources\views\posts\index.blade.php)

когда поменял маршруты с изначального значения

Route::get('post/', 'PostController@index')->name('post.index');
Route::get('post/show/{id}', 'PostController@show')->name('post.show');
Route::get('post/create', 'PostController@create')->name('post.create');
Route::get('post/edit/{id}', 'PostController@edit')->name('post.edit');
Route::post('post/', 'PostController@store')->name('post.store');
Route::patch('post/show/{id}', 'PostController@update')->name('post.update');
Route::delete('post/{id}', 'PostController@destroy')->name('post.destroy');

на маршрут
Route::resource('/post', 'PostController');
В контроллере все методы родные(если понадобятся, скину в комментарии)

Вот index.blade.php

@extends('layouts.layout')

@section('content')
  @if(isset($_GET['search']))
    @if( count($posts)>0)
      <h2>Результат поиска:</h2>
      <p class="lead">Всего найдено {{ count($posts) }} постов</p>
    @else
      <h2>По запросу <?php echo $_GET['search']?> ничего не найдено</h2>
      <a class="btn btn-outline-primary" href="{{ route('post.index') }}"> Все посты </a>
    @endif
  @endif

  <div class="row">
    @foreach($posts as $post)
    <div class="col-6">
      <div class="card">
        <div class="card-header"><h2>{{ $post->short_title }}</h2></div>
        <div class="card-body">
          <div class="card-img " style="background-image: url({{$post->img ?? asset('img/who.jpg')}})"></div>
          <div class="card-author">Автор: {{$post->name}}</div>
          <a href="{{route('post.show', ['id'=> $post->post_id])}}" class="btn btn-outline-primary">Посмотреть пост</a>
        </div>
      </div>
    </div>
    @endforeach
  </div>

  @if(!isset($_GET['search']))
  {{ $posts->links() }}
  @endif

@endsection

Comments

[Максим Федоров]

{{route('post.show', ['id'=> $post->post_id])}}"

Missing required parameters for [Route: post.show] [URI: post/{post}]

Сравните название параметра в первом случае (в шаблоне когда генерируете) и втором (ожидаемом)

Answer 1

[jazzus]

В ресурсные маршруты не нужно передавать названия параметров.
route('posts.show', $post)
Т.е. либо ресурс либо его идентификатор если ресурса нет. Ларавел автоматом свяжет и в контроллере уже будет нужный ресурс

public function show(Post $post)
{
  dd($post);
}

Если key не id, а например slug то определить в модели так
protected $primaryKey = 'slug';
В бд не нужно давать префиксы для полей типа $post->post_id. Правильно делать $post->id, name, active и т.д. т.к. post всегда и так будет перед ними в названии переменной.
И ресурсы нужно называть не post, а posts в роутах
Route::resource('posts', 'PostController');

Comments

[Антон Антон]

В бд не нужно давать префиксы для полей типа $post->post_id

Об это множество копий сломано. Но вообще даже если указывать не id, то в модели имя поля первичного ключа можно переопределить.
https://laravel.com/docs/7.x/eloquent#eloquent-mod... см. раздел primary keys. Тогда он сам будет подставляться в поиск по маршруту, без дополнительного переопределения в getRouteKeyName().

[jazzus]

Антон Антон, подправил ответ, спасибо. Насчет ключа не знаю кто там копья ломает post_id видел только у начинающих, в более менее примерах -модулях id и Ларавел по дефолту ждет ид. Ну и сам код гораздо логичней выглядит с $post->id чем с $post->post_id когда вижу enity_id я сразу про связь думаю

[Антон Антон]

jazzus,

post_id видел только у начинающих

дело не в начинающих/продолжающих, эти копья безотносительно к ларавелю идут. В ларавеле по дефолту поле называется id, да и гемора по описанию в модели меньше. В последних еще и заляпухифункции для быстрого создания такого поля (с именем id, типом int, первичным ключем и автоинкрементом) в миграциях добавили.

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

Я, например, предпочитаю UUID в качестве ключа. И имя - id.

Answer 2

[Антон Антон]

Проще всего вместо route использовать action и передавать в него экземпляр модели:
action('PostController@show', $post);

Comments

[jazzus]

Чем это проще route('posts.show', $post)?

[Антон Антон]

jazzus, В момент написания этого ответа вроде как в ответе, выбранном решением про route('posts.show', $post) не было. Ну и эта функция на действия контроллеров навешивается, а не на именованный маршрут. Если вдруг соглашение по именованию маршрутов у Route::resource поменяют - оно не сломается.

[jazzus]

Антон Антон, все там было там где есть. Если так думать, они что угодно могут поменять. Например, отменить хелпер action