Laravel 5.8. Почему не работает мутатор?

Question

[truevolkolak]

Здравствуйте!
Есть таблица БД с полем "slug". В миграции:
$table->string('slug');

Так как поле должно быть уникальным и генерироваться автоматически при добавлении новой записи, в модели использую мутатор:

public function setSlugAttribute($value) {
        $this->attributes['slug'] = Str::slug(mb_substr($this->manufacture, 0, 60) . "-", "-");
    }

Метод store в соответствующем контроллере:

public function store(Request $request)
    {
        $manufacture = Manufacture::create($request->all());
        
        return redirect()->route('admin.manufactures.index');
    }

При отправке формы поле 'slug' отправляется пустым и по идее должно до сохранения новой записи быть изменено этим мутатором. Но этого не происходит. Появляется в браузере такая ошибка:

SQLSTATE[HY000]: General error: 1364 Field 'slug' doesn't have a default value 
(SQL: insert into `manufactures` 
(`manufacture`, `description`, `meta_description`, `meta_keywords`, `country`) 
values 
(Производитель 2, описание, meta description, meta keywords, Россия))

Причем, как видно из ошибки, при массовом заполнении даже нет попытки сохранить значение поля 'slug', хотя в модели оно объявлено в массиве $fillable:

protected $fillable = [
        'manufacture',
        'slug',
        'country',
        'image',
        'description',
        'meta_description',
        'meta_keywords',
    ];

P.S. В другом проекте на Laravel 5.7 ровно такой же мутатор отлично работает.

Answer 1

[JhaoDa]

А почему он должен работать? Мутатор мутирует значение, когда оно устанавливается, явно или неявно.

даже нет попытки сохранить значение поля 'slug', хотя в модели оно объявлено в массиве $fillable

Рекомендую почитать в документации для чего придумано это поле. Уж точно не для извлечения данных из реквеста...

Comments

[truevolkolak]

Ну верно. Я отправляю форму с заполненными данными. Среди полей формы есть и 'slug', который пока что является пустым. Мутатор должен его мутировать перед сохранением, верно?

[JhaoDa]

truevolkolak, ты проверял, что в массиве, который возвращается $request->all() есть это поле? Или думаешь, что оно там есть?

[truevolkolak]

JhaoDa, хм, его там нет.

[truevolkolak]

JhaoDa, сейчас в форме (используется и для добавления и для редактирования записи) указал такой value для 'slug':
value="{{ $manufacture->slug ?? '0' }}"
Теперь это поле отправляется (проверил dd($request->all())), но ошибка сохранилась.

[NubasLol]

truevolkolak, Либо его нет в реквесте, либо в fillable если ошибка такая же. Ксли другая, наверняка ты null передаешь

[truevolkolak]

NubasLol, в реквесте есть:

array:7 [▼
  "_token" => "aHoKleiOnAdLyulvpDZcR8IeyNkypyNzrPy0nbHw"
  "manufacture" => "Производитель 2"
  "description" => null
  "meta_description" => null
  "meta_keywords" => null
  "country" => null
  "slug" => "0"
]

Передается не null.
Ошибка такая же. В fillable сто раз перепроверил. Ну есть там это поле. А в ошибке все равно видно, что нет попытки заполнить это поле:

SQL: insert into `manufactures` (`manufacture`, `description`, `meta_description`, `meta_keywords`, `country`)

[truevolkolak]

NubasLol, какая-то чертовщина. Снес модель полностью, заново ее создал и заполнил. Теперь все работает. Спасибо за участие в решении вопроса!

Answer 2

[Евгений]

https://laravel.com/docs/5.8/eloquent#events

Comments

[truevolkolak]

хотелось бы использовать именно мутаторы, которые, вроде как, именно для подобных случаев подходят отлично. Но как запасной вариант буду иметь в виду. Спасибо!

[Евгений]

truevolkolak, нет, мутатор это геттер/сеттер.
Либо хук на beforeSave, либо так

public function setManufactureAttribute($value) {
  $this->attributes['manufacture'] = $value;
  $this->attributes['slug'] = Str::slug(mb_substr($value, 0, 60) . "-", "-");
}

[truevolkolak]

Евгений, я к предыдущему ответу написал коммент, что все заработало. Причем, непонятно, в чем была ошибка. Я просто заново полностью переписал модель. И да, мутатор в данном случае я использую как сеттер.
Так что этот метод работает:

public function setSlugAttribute($value) 
    {
        $this->attributes['slug'] = Str::slug(mb_substr($this->manufacture, 0, 60), "-");
    }

[Евгений]

truevolkolak, это срабает только если где-то вызывается $model->slug = 'whatever';

[truevolkolak]

Евгений, так в том-то и дело, что нигде я такого присваивания не делаю. Но все равно slug ставится как надо в момент сохранения записи.

[NubasLol]

Евгений, нет

[Евгений]

truevolkolak, ты его передаешь в POST запросе как 0, вот тут и присваивание.
Передавать slug когда он не нужен тот еще костыль...