Как правильно реализовать метод update для объяления?

Question

[ilya17392]

Контроллер

public function edit($id)
    {
        $ad = Ad::find($id);
        
        return view('update', [
            'ad' => $ad
            ]);
    }

    
    public function update(Request $request, $id)
    {
        $ad = Ad::find($id);
        
        $data = $request->all();
        $result = $ad->fill($data)->save();
        return redirect('/');
    }

Роут

Route::get('/edit/{id}', 'AdController@update')->name('adUpdate');

Вьюха update.blade.php

<form method="post" class="col-md-6" action="{{ route(''adUpdate) }}">
    @method('PATCH')
     <div class="form-group">
         <label>Title</label>
         
         <input type="text" name="title" class="form-control">
         @error ('title')
         <small class="form-text text-danger">{{ $message }}</small>
         @enderror
     </div>
     
     
     <div class="form-group">
         <label>Description</label>
         <textarea type="text" name="description" class="form-control"></textarea>
         @error ('description')
         <small class="form-text text-danger">{{ $message }}</small>
         @enderror
     </div>
     
    
     <button type="submit" class="btn btn-primary">Create</button>
     @csrf
 </form>

Comments

[Евгений Вольф]

А в чем у Вас проблема то собственно?

P.S. У моделей есть метод update(), использовать fill() + save() не обязательно.

[ilya17392]

Евгений Вольф, ничего не происходит, должно переходить на /edit/{id} и отображать форму

[Евгений Вольф]

ilya17392, ааа, дык это просто. Примерно так:

$id = $model->update($request->all())->id;
return redirect("/edit/{$id}");

Как-то примерно так (адрес редиректа только поправьте, как Вам нужно).

[ilya17392]

Евгений Вольф,

public function update(Request $request, $id)
    {
        $ad = Ad::find($id);
        $id = $ad->update($request->all());
        return redirect("/update/{$id}");
    }

Route::get('/edit/{id}', 'AdController@update')->name('adUpdate');

адрес пишет норм, но страница пустая 404,как и раньше,ошибок не выдает(

[Евгений Вольф]

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

[Евгений Вольф]

ilya17392, или попробуйте вот так:

$model->update($request->all());
return redirect("/edit/{$model->id}");

P.S. Вечер пятницы, голова уже деревянная... уже элементарные вещи забывать начинаю, извините.

[ilya17392]

Евгений Вольф, не помогает, в общем задача такая, нужно чтоб по нажатию на кнопку

<a href="/update/{{ $oneAd->id }}" class="btn btn-primary">Update</a>

на странице ad-content.blade.php , пользователь переходил по ссылке /edit/id записи , и мне что-то подсказывает, что я накосячил с путями

[Евгений Вольф]

ilya17392, я думаю, что для того, что бы не косячить с путями - Вам нужно использовать именованные роуты. Выглядит это так: <a href="{{route('adUpdate', ['id' => 1])}}

где, adUpdate - это имя роута, которое Вы ему задали, id - параметр в роутах, 1 - значение параметра в роутах.

[ilya17392]

Евгений Вольф, Спасибо)уже хоть ошибки появились

Call to a member function update() on null

$ad = Ad::find($id);
        $id = $ad->update($request->all())->id;
        return redirect("/update/{$id}");

на вторую строку ругается

[Евгений Вольф]

ilya17392,

Call to a member function update() on null

это значит $ad === null. Иными словами, в результате работы первой строчки - ничего не нашлось.

[JhaoDa]

Евгений Вольф, бесполезно, автор не понимает, что пишет.

Answer 1

[Максим Шевченко]

У вас не правельно написан метод routa в форме стоит отправка с помощью метода PATH

Route::patch('/edit/{id}', 'AdController@update')->name('adUpdate');

А у вас обращение к гет

Route::get('/edit/{id}', 'AdController@update')->name('adUpdate');

P.S И еще у вас не правельно написан action формы. Присмотритесь как стоят кавычки в route() и вы не передаете обязательный параметр $id