Корректно ли так определять маршруты?

Question

[Александр Мак-Ли]

Добрый день.

Подскажите пожалуйста, по роутингу.

Если мы определим группу маршрутов

Route::group($groupData, function () {
    //BookCategory
    $methods = ['index', 'edit', 'update', 'create', 'store', 'destroy'];
    Route::resource('categories', 'CategoryController')
        ->only($methods)
        ->names('breh.admin.lib.categories');

    Route::resource('books','BookController')
        ->only($methods)
        ->names('breh.admin.lib.book');
});

То на каждый delete action нужно будет определять форму.
Потому что роут будет ждать метод DELETE, а не GET.

Поэтому я создал маршрут delete таким образом:

Route::get('/admin/categories/{id}/delete', 'CategoryController@delete')
    ->middleware('is_admin')
    ->name('breh.admin.lib.categories.delete');

Скажите насколько это корректно?

Comments

[Дмитрий]

Делать delete через GET - это бред. Подумайте про обновление страницы.
Не вижу никакой проблемы послать правильный DELETE через формочку или аякс.

Answer 1

[Александр Аксентьев]

Зачем отдельно его задавать если вы Route::resource используете который всё это внутри себя поддерживает?
Реализуйте в рамках ресурса.

Потому что роут будет ждать метод DELETE, а не GET.

Поэтому я создал маршрут delete таким образом:

Route::get('/admin/categories/{id}/delete'
И где тут у вас метод delete?
Постфикс /delete никакой не нужен, потому что метод другой будет, GET /admin/categories/{id} и DELETE /admin/categories/{id} это разные роуты и без дополнительных разделений.

Route::delete('/admin/categories/{id}'
вот метод delete

Comments

[Александр Мак-Ли]

Я понимаю это.

Я начал не с того.
В админке в списке идут категории, рядом идут кнопки редактировать/удалить/снять с публикации.

Понимаете, если я буду использовать delete, то мне нужно на каждую кнопочку делать формочку, что с ТЗ фронтщиков капец.
Хотелось бы, чтобы на фронте админки было представлено в формате простой ссылки,а не форма submit.

[JhaoDa]

Александр Мак-Ли, ну так фронтовики пусть и придумают, как сделать ссылку вместо формы, но чтобы в итоге получился DELETE. Подсказка: всё уже давно придумано, достаточно загуглить.

Answer 2

[JhaoDa]

Это некорректно настолько, что за такое руки отрубать надо. Максимум POST.

P.S. Вопрос миллион раз задан и отвечен в интернетах, предлагаю сначала приобрести навыки гугления, а уж потом ларавел.

Comments

[Александр Мак-Ли]

Начал я не с того, посмотрите коммент под 1м ответом.
Спасибо за отклик.

Answer 3

[sirenity]

$methods = ['index', 'edit', 'update', 'create', 'store', 'destroy'];

Так как у вас используются методы все кроме show, можно писать except('show'), вместо only($methods)

Поэтому я создал маршрут delete таким образом:

Этот метод отдает форму удаления, которая отличается от обновления? А у удалять по обычному методу будете Route::delete('/admin/categories/{id}? тогда да возможна такая запись. Но можно было тогда оставить show и там сделать удаление.

Comments

[Александр Мак-Ли]

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