Как поставить значение по умолчанию атрибута модели в процессе update с fillable полями?

Question

[Илья]

Приветствую всех

Не могу найти решение проблемы.

В общем есть поле в форме checkbox с именем "published" и значением 1, если он в состоянии "checked", логично, в update контроллера отправляется циферка 1 и этот код отрабатывает прекрасно

public function update(Request $request, $id)
    {    
        $this->model->find($id)->update($request->all());

        return back();
    }

Как только checkbox В состоянии not checked (просто галочку не поставили) update не происходит. По идее это нормально, нет галочки - нет поля "published" в Request объекте. Галочка стоит - получаем поле в реквесте published = 1 и апдейт происходит.

Вопрос вот в чем, можно ли как-то сделать обновление этого поля через модель !важно!
То есть имеем значение чекбокса = 1 - обновляем с единичкой, не имеем ничего в процессе update, - устанавливаем значение по умолчанию атрибута модели published = 0. Я уже как только не пробовал и через мутатор "setPublishedAttribute" и через protected $attributes - все тщетно.

Конечно есть варианты сделать через Select с 1 или 0. radio, или проверить реквест в контроллере, но не хотелось бы логику save/create/update выносить в контроллер. Да и во фронтенде есть отличный встроенный switch input в виде прикольного переключателя, но он завязан на input checkbox к сожалению =(

Answer 1

[Yan-s]

но не хотелось бы логику save/create/update выносить в контроллер

переопределите update() модели или добавьте в ней отдельный метод

Comments

[Илья]

А это вариант =)

[Yan-s]

Илья, там обратите внимание, что значение по умолчанию при обновлении подразумевает, что если вы вдруг решите обновить не все поля, то это самое значение тоже может обновиться нежданно.

Еще есть вариант решения на клиенте, то есть в случае, если отключен чекбокс всё равно передавать значение 0.

[Илья]

Да, в процессе посмотрим, может и на клиент переложить эту функцию. Как не догадался с переопределением метода, главное до laravel работал с yii там постоянно это делать надо было =)

public function update(array $attributes = [], array $options = [])
    {
        $attributes['published'] = isset($attributes['published']) ? $attributes['published'] : 0;

        parent::update($attributes,$options);
    }

Надеюсь все правильно

Answer 2

[pLavrenov]

1) Вместо $request->all() лучше использвать $request->only(['необходимые', 'поля', 'published'])
2) Мутаторы работают только с получаемыми данными из базы.
3) Самый просто и правильный вариант указать в миграции дефолтное значение == 0. тк значение может быть или 0 или 1 и никаким другим. И даже если в данных от фронтенда published не придет вообще save отработает правильно без ошибки от PDO
$table->integer('published')->default(0);

Comments

[Илья]

1. Попробую ради эксперимента $request->only(['необходимые', 'поля', 'published']) - хоть это и не совсем удобно, что надо в упор, каждый раз, показывать реквесту поля =)
3.
Не отрабатывает, точнее отрабатывает только при create. При update если в request поля published нет, ларавел его пропускает вообще, при этом мутатор запускается, но не видит совсем получаемое $value (ни isset($value) ни is_null($value) - не помогают). Соотвественно остается значение, которое было записано в базу ранее. Что, собственно, вполне логично, если пользователь не использует в форме поле - значит не надо его трогать

[pLavrenov]

1) Это необходимо. В конечном итоге ты все равно знаешь какие поля должны быть для конкретного действия. Например при использовании методов create и update если в all придут какие либо другие данные то выдаст ошибку что не знает куда запихнуть эти новые поля.
Или например такая ситуация:
Из пользовательского фронтенда можно создать пользователя но тк это возможно и из админки с применением "isAdmin" то при $request->all() пользователи смогут отправить поле isAdmin и сделать себя админом. Такие случаи надо сразу отсекать т.к. в дальнейшем можно запутаться или забыть про это. Оставляешь только нужные для действия поля и нет проблем.
3) АА я понял то есть если в базе раньше было 1 а при update не пришло поле то надо его сделать 0.

$content = Version::update([
    'published' => $request->has('published') ? 1 : 0
]);

[Илья]

lavren, В плане пользователей все ситуативно, я делю на 2 группы таблиц для простых пользователей и администраторов =). Админов никто не может никак зарегать, кроме root админа. Для проверки полей вроде как есть валидаторы и мидлвейры, мидлвейр может не дать пользователю сделать себя админом без каких-либо прав заранее обозначенных

$request->has('published') ? 1 : 0;

Это поприятнее чем переопределение метода update. Надо будет потестить потом

В ларавел тысячи методов, но блин, хреново когда их не знаешь или не догадаешься что они есть =)