Rubrics::where('id', $request->id)->fill($data)->description()->create($data);

тут тебе логичней firstOrNew использовать.

потом fill и save

$model = Rubrics::where('id', $request->id)->first(); $model->fill($data); $model->save();

BadMethodCallException Call to undefined method Illuminate\Database\Eloquent\Builder::fill()

а точно филл?

а то чет говорит метода нет )

да я уже читаю да

странно

а че оно тогда : - |

$model = Rubrics::firstOrNew('id', $request->id); $model->fill($data); $model->save(); Вот так попробуй

или там массив в параметре? не помню. Возможно вот должно быть $model = Rubrics::firstOrNew(['id' => $request->id]);

или там массив в параметре? не помню. Возможно вот должно быть $model = Rubrics::firstOrNew(['id' => $request->id]);

массив, да

я опять метод перепутал xD

$model = Rubrics::where('id', $request->id)->first(); $model->fill($data); $model->save();

работает

$model = Rubrics::firstOrNew('id', $request->id); $model->fill($data); $model->save(); Вот так попробуй

можно даже findOrNew, раз у него по id поиск

Rubrics::findOrNew($request->id)

$model = Rubrics::where('id', $request->id)->first(); $model->fill($data); $model->save(); $model->description()->create($data);

ну вместо одно строки такое вышло

некрасиво, но сойдет

Rubrics::findOrNew($request->id)

научи вот так форматировать, плз )

научи вот так форматировать, плз )

чтобы код выделять?

ага

три ` в начале и три в конце блока нужного :)

$model->save();

спс

Rubrics::findOrNew($request->id)

а зачем так?

когда именно поис

к

ну или find :)

я просто Rubrics::find($request->id); обычно юзаю

ну вот да, ок

просто мб смысл какой есть

не, я просто за код Алекса зацепился, вот и все если тебе только апдейт - то просто find()

ага, ня

я просто Rubrics::find($request->id); обычно юзаю

я просто всегда думал если find ничего не находит, то null возвращает, а findOrNew если не нашел то вызывает конструктор

разве не так? если он null вернет, то fill ошибку выдаст

Если ты findOrNew делаешь, то если записи не было, то при save он ее создаст Учитывая, что кейс тут именно обновление - такого поведения не нужно

Можно делать findOrFail, тогда сразу ModelNotFoundException бросится, если записи нет

кейс был про создание и апдейт )

у меня на создание метод просто валидацию делает и отдаёт, хотел реюзать как есть для апдейта, данные одинаковые выходит

в таком случае лучше findOrFail

у меня на создание метод просто валидацию делает и отдаёт, хотел реюзать как есть для апдейта, данные одинаковые выходит

я запутался -_-

сейчас попробую

ну да

но у меня валидатор стоит на id

я чет не подумал, что его подделать могут

if($model = Rubrics::find($request->id)) { $model->fill($data); $model->save(); $model->description()->create($data); }

завернул так

если ниче не происходит просто редирект назад делает

ситуация когда id не тот может быть только при подтасовке данных в этом случае

if($model = Rubrics::find($request->id)) { $model->fill($data); $model->save(); $model->description()->create($data); }

а создание у тебя все таки не тут просиходит? )

так что норм

ситуация когда id не тот может быть только при подтасовке данных в этом случае

его может просто не быть ведь

а создание у тебя все таки не тут просиходит? )

нет, а предлагаешь и апдейт и креат делать одним методм через Find or New?

самого значения в случае создания

хм

нет, а предлагаешь и апдейт и креат делать одним методм через Find or New?

я так делаю

я чет никогда даже не задумывался

и как блин этот метод обозвать +_+

save )

сойдёт

❤️

ну такое

если вдруг надо будет проверить - создание идет обновление можно между до save() проверить $model->id если там пусто - значит создаешь

обновление - это обновление создание - это создание зачем перемешивать?

обновление - это обновление создание - это создание зачем перемешивать?

ну я тоже единство отвественности стараюсь поддерживать

обновление - это обновление создание - это создание зачем перемешивать?

дело то хозяйское, если платят за количество строк то можно и на каждое событие методы делать

Платят к сожалению за готовый функционал ))

дело то хозяйское, если платят за количество строк то можно и на каждое событие методы делать

что уж там, можно и в роутинге логику писать, если так мыслить строк меньше, огонь же

обновление - это обновление создание - это создание зачем перемешивать?

тут соглашусь, лучше разделять к тому же, если нет валидации по существованию id(а ее в таком слукчае и не будет, видимо), то в апдейт можно передать что угодно и он это создаст

тут соглашусь, лучше разделять к тому же, если нет валидации по существованию id(а ее в таком слукчае и не будет, видимо), то в апдейт можно передать что угодно и он это создаст

например

когда филлаблы указаны и обращение идёт к модели и связанной модели

умозрительно просто не оч понимаю

если сущности с указанным айди нет, ты должен отдать 404

Смотри, у тебя в базе есть записи с id 1-15 ты заходишь в условную форму редактирования записи с id 5 меняешь в html значение поля id на 400 и делаешь апдейт ты ожидаешь, что при таком коде не обновится ничего, так как записи с id 400 в базе нет, а по факту он ее тебе создаст

потому что в апдейте ты используешь метод findOrNew

если есть желание сократить код - можно часть с заполнением модели и ее сохранением сделать отдельным методом приватным, на вход подавать твою модель

но тут особого смысла нет

если сущности с указанным айди нет, ты должен отдать 404

+

Смотри, у тебя в базе есть записи с id 1-15 ты заходишь в условную форму редактирования записи с id 5 меняешь в html значение поля id на 400 и делаешь апдейт ты ожидаешь, что при таком коде не обновится ничего, так как записи с id 400 в базе нет, а по факту он ее тебе создаст

а тут да, закрыт к созданию и открыт к изменению, логично

значит я норм всё делал xD

$arr, $q - я б порукам дал

Эт временно

Эт временно

Ну да, ну да. Лет на 7, пока проект не умрет

Подскажите, может кто нибудь юзал какой-то пакет для реализации атрибутов у товаров? Цвета, размеры, мощности и вот это вот всё... И объединение атрибутов в группы атрибутов. И назначение групп атрибутов разным категориям товаров)) Или лучше самому запилить?

как ведёт себя EAV на товарах от 500к и 10к атрибутов, интересно

Подскажите, может кто нибудь юзал какой-то пакет для реализации атрибутов у товаров? Цвета, размеры, мощности и вот это вот всё... И объединение атрибутов в группы атрибутов. И назначение групп атрибутов разным категориям товаров)) Или лучше самому запилить?

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

какая примерно схема таблицы была? Как решать когда у атрибутов разные еденицы измерения, которые надо выводить на фронте, и при этом адаптировать под Мега, Кило и тд?

едИницы )

да пофик)

и еще если для разных атрибутов нужны разные типы (селект, чекбокс, радио, текст...)

звучит как задача для json

какая примерно схема таблицы была? Как решать когда у атрибутов разные еденицы измерения, которые надо выводить на фронте, и при этом адаптировать под Мега, Кило и тд?

у меня проще задача была. название параметра - значение

такая простая мне не подойдет :( Ладно, покопаю пока готовые решения, изучу подходы и буду свой придумывать)

такая простая мне не подойдет :( Ладно, покопаю пока готовые решения, изучу подходы и буду свой придумывать)

так а почему не оборачивать это в json который на фронте разбирается?

мне кажется потом фильтровать по json когда много товаров будет не оптимально совсем. Может ошибаюсь. Изучу и такой подход тоже

ну тогда если только релейшеншипами