Почему не работает массовое изменение атрибута в Laravel?

Question

[truevolkolak]

Доброго дня!
В таблице товаров 'products' есть поле 'autoscu' (имеет вид "xxx-xxx", где "х" - цифра 0-9). Мне нужно для всех товаров (до 1000 штук) поменять его, убрав символ дефиса.
В контроллере у меня так:

public function modifyAutoScu(Request $request) {
        $products = Product::all();
        foreach ($products as $product) {
              $product->update(['autoscu' => preg_replace("/[^0-9]/", "", $product->autoscu )]);
        }
        return redirect()->route('admin.products.index');
    }

Изменения не сохраняются. Более того, при такой проверке:

$products = Product::all();
foreach ($products as $product) {
   $product->autoscu = '123456';
   dd($product->autoscu);
}

Выводит старое неизмененное значение.

Поле 'autoscu' присутствует в модели Product в массиве $fillable.
Если менять по одному продукту, всё получается:

$product = Product::first();
$product->update(['autoscu' => preg_replace("/[^0-9]/", "", $product->autoscu )]); //это работает

Вариант через save(), соответственно, тоже не работает:

$products = Product::all();
foreach ($products as $product) {
   $product->autoscu = preg_replace("/[^0-9]/", "", $product->autoscu );
   $product->save();
}

Как решить эту простую проблему?

Comments

[JhaoDa]

Под scu ты, случаем, имел ввиду не Stock Keeping Unit?

[truevolkolak]

JhaoDa, типа того, и теперь мне нужно сделать его чисто цифровым.

[JhaoDa]

truevolkolak, ну так оно SKU, а не SCU.

[truevolkolak]

Отлично! Как это решает проблему?

[JhaoDa]

truevolkolak, ну как минимум одну проблему это решает — некорректный нейминг. А по основному вопросу — на первый взгляд всё ок, так что дебагай.

А вообще, такую задачу надо решать на уровне БД одним запросом, а не тянуть все модели ради обновления.

[truevolkolak]

JhaoDa, это частный случай бОльшей задачи по смене этого поля по тому или иному шаблону для всех или некоторых товаров, так что хочется сделать именно таким образом.

Answer 1

[Дмитрий]

А можно не разбираться чо там и как

Product::update(['autoscu' => DB::raw('replace(autoscu, "-", "")')]);