Как правильно обнавить данные laravel?

Question

[Иван Жук]

Ребят можете подсказать как правильно обнавить данные в ларавел
я думал так но код не работает можете исправить меня)

public function update(Request $request,$id){
        $user = Crud::find($id);

        $update = $user->update([
           'name' => $request->input('name'),
            'email' => $request->input('email')
        ]);
        $user->save();
        return redirect('/')->with('update', 'Вы успешно Изменили Данные');

    }

Comments

[JhaoDa]

Мы должны угадать, как именно он не работает?

Ну тайный смысл $user->save() после $user->update() от меня ускользает.

Answer 1

[pLavrenov]

1) update сам вызывает save (save надо убрать)
2) Можно использовать магию и итоговый вариант будет таким

public function update(Request $request, Crud $crud){
        $crud->update([
            'name' => $request->get('name'),
            'email' => $request->get('email')
        ]);
        return redirect('/')->with('update', 'Вы успешно Изменили Данные');
    }

Важный момент:
- переменная $crud должна быть названа как в роуте
- использовать валидацию

Answer 2

[Aleksey]

Ставлю доллар что в массиве fillable не прописаны name и email. Оттого они и не обновляются. Ну и как вам выше указали - вызов save тут лишний. Update достаточно.

Comments

[jazzus]

или save без update и fillable

Answer 3

[N]

Если банально и без всяких проверок, то...

public function update(Request $request,$id){
        $user = Crud::find($id);
        if($user) {
           $user->name = $request->input('name');
           $user->email = $request->input('email');
           $user->save();
        }

        return redirect('/')->with('update', 'Вы успешно Изменили Данные');
    }

У тебя люди/кони смешались...доку повнимательнее глянь...

Исходя из твоего варианта, то наверное так:

public function update(Request $request,$id){

        $update = Crud::where('id', $id)->update([
           'name' => $request->input('name'),
            'email' => $request->input('email')
        ]);

        return redirect('/')->with('update', 'Вы успешно Изменили Данные');
    }

Comments

[jazzus]

if($user) {
}
return redirect('/')->with('update', 'Вы успешно Изменили Данные');

Вернет сообщение, даже если данные не изменили) Чтобы не писать if есть findOrFail а вообще можно привязать модель к маршруту и Ларавел все сам сделает https://laravel.com/docs/8.x/routing#route-model-b...

[N]

jazzus, Это всё понятно...только в моём первом случае, он не должен вернуть условие, если записи с таким ID не будет...
Я "в стиле" автора написал...и на два примера разложил...а так конечно, как уже тут говорили, магия/маршруты к месту тоже может быть...но Мы весь подтекст задачи автора не знаем...

P.S.: У него же вопрос был не логике приложения, а "почему не обновляет?"...
И первая фраза из моего ответа:

Если банально и без всяких проверок, то...

:)

[jazzus]

N, второй пример ничем от кода автора не отличается в плане, что что-то не работает. Там хоть 10 save можно делать. Но я согласен как вопросы задают так действительно либо экстрасенсов ждать или ставки делать)

[Иван Жук]

N, спасибо помог слушай можешь написать в вк 1 вопрос задам и все) https://vk.com/id349962986

[N]

Иван Жук, Я тебе написал. Только и ты, будь добр...пометь решением, как минимум, не только мой ответ...
Тут человек тоже правильно написал.

[jazzus]

Иван Жук, через save получилось? Значит проблема была с fillable