Почему внутри foreach работает только первая форма, остальные отдают 403?

Question

[vladislav997]

Подскажите, почему внутри foreach работает только первая форма, остальные отдают 403?

@foreach($posts as $post)
    Name: {{ $post->name }}

    <form method="POST" action="{{ route('admin.post.delete', ['post' => $post->_id]) }}">
        @csrf
        <button type="submit">Удалить</button>
    </form>
@endforeach

Полагаю что проблема как раз таки в csrf, но как фиксануть?

Comments

[Алексей Уколов]

Нет, проблема не в CSRF - их может быть сколько угодно.
В нормальном виде у вас ещё не хватает @method('delete'), но это зависит от того, какой метод ожидает роутер.
Проблему и решение надо искать скорее в методе удаления в контроллере.

[Adamos]

Не выводить на странице херову тучу форм, а создать одну и динамически, по кнопке, подставлять в нее id поста, который хочется удалить.

[Алексей Уколов]

Ну хрень же, а не совет. Уж такую простую-то форму можно сделать независимой от js.

[vladislav997]

Алексей Уколов,
web.php

Route::group(['prefix' => 'post', 'as' => 'post.'], function() {
    Route::post('/delete/{post}', [PostController::class, 'delete'])->name('delete');
});

PostController

public function delete(Request $request, Post $post)
{
    dd(1);
}

Answer 1

[Сергей delphinpro]

В целом ваш вариант выглядит правильным, и проблема не в форме, скорее всего.

Кстати, так немного компактнее:

<form method="POST" if="myform">@csrf</form>
@foreach($posts as $post)
    <button type="submit"
        form="myform"
        fromaction="{{ route('admin.post.delete', ['post' => $post->_id]) }}"
    >Удалить</button>
@endforeach

Comments

[Adamos]

Кнопка "удалить", надо полагать, где-то рядом с постом. Форма захватит всю страницу, а внутри поста может оказаться голосовалка, например, со своей формой.
Неаккуратненько.

[vladislav997]

попробовал так:
blade:

@foreach($posts as $post)
    Name: {{ $post->name }}

    <input type="hidden" form="myform" name="post" value="{{ $post->_id }}">
    <button type="submit" form="myform">Удалить</button>
@endforeach

<form method="POST" id="myform" action="{{ route('admin.post.delete') }}">@csrf</form>

web.php

Route::group(['prefix' => 'post', 'as' => 'post.'], function() {
    Route::post('/delete', [PostController::class, 'delete'])->name('delete');
});

PostController:

public function delete(Request $request)
    {
        dd($request->post);
    }

и теперь дампится айдишник последнего элемента в форыче, даже если кликаю на первую кнопку

а вот что вижу в исходном коде браузера

[Сергей delphinpro]

Adamos, нет. я имел ввиду форму именно так, как написал. пустую, там только токен.
а экшен уже передается в нее через привязанную кнопку.
в laravel роуты удаления выглядят так [DELETE]/users/1, достаточно передаать только урл.
ну и директиву@method('DELETE') в саму форму нужно добавить

[Сергей delphinpro]

vladislav997, в вашем случае в форму будет попадать весь список инпутов. Так как я написал только кнопку можно передать индивидуально. И вам не нужен input. Нужно только action передать. В нем уже id присутствует

[Сергей delphinpro]

* не весь список. так имя не массив, то попадет только последний в разметке

[Adamos]

Сергей delphinpro, да, это я был невнимателен.
Но там же, скорее всего, не только delete, но и edit и прочие like
И мне лично было бы проще вывести пустую форму, после нее - скрипт с функцией ее заполнения (выверенный один раз), а на кнопки прописывать запуск этого скрипта с параметрами. Хоть Алексей и против ;)

[vladislav997]

отбой, там проверка стояла в middleware по правам у меня кастомная