Почему при массовой сохранении вставляет пустые значения?

Question

[Анатолий]

Добрый день!

Подскажите, почему может при массовом заполнении, запись создает, а все поля пустые?
Objects::create($request->all());

Хотя в модели Objects, массовая вставка разрешена

protected $guarded = [];

Comments

[Кирилл Санников]

$request->all() дампили ? Может там и значения пустые

[Анатолий]

Кирилл Санников, Да, конечно. он не пустой 100%

[Анатолий]

Кирилл Санников, Даже просто подставляю массив с полями и значениями необходимыми для вставки, и все равно, запись создает, а все значения пустые

[Кирилл Санников]

Анатолий, конечно очень тупой совет, но может попробовать прописать fillable в модели место guarded

[Konata Izumi]

Анатолий, А ключи там верные?
Отвардампите пример реквеста и выложите сюда.
Да и полный код класса модели не помешает.

[Анатолий]

Кирилл Санников, не удобно, просто кол-во полей может меняться

[Konata Izumi]

Кирилл Санников, согласно документации в модели нужно указать либо массив филабл либо гвардед, но не оба.
Если модель наследуется от абстрактной елоквент модели, то проблем быть не должно.
Разве что в модель суют некорректные данные.

[Кирилл Санников]

Анатолий, согласен, но это лишь для того, чтобы понять тут ли кроется проблема

[Анатолий]

Konata Izumi,
Код модели

<?php namespace App\Models;

use Illuminate\Database\Eloquent\Model;

class Objects extends Model
{
		protected $table = 'objects';

		protected $guarded = [];

}

[Кирилл Санников]

Konata Izumi, не не не, вместе точно их указывать не нужно)

[Konata Izumi]

Анатолий, тут всё ок.
Остается только данные проверить, которые приходят в Model::create.

[Анатолий]

Konata Izumi, А не подскажете, при guarded = [];
Ему нужно передавать все поля в обязательном порядке?
У меня просто на вставку кидаются не все поля которые есть в таблице

[Андрей]

Может надо прописать все атрибуты в fillable?

protected $fillable = [

    ];

[Анатолий]

Андрей, Не подходит, потому что кол-во полей может меняться

[Konata Izumi]

Анатолий, Андрей,
Да что вы с гвардом и филабл носитесь?
Покажите уже список данных с ключами по которым вставляете запись.

Что сюда приходит?
var_dump($request->all())

[Konata Izumi]

Есть предположение, что в форме забыли name указать у полей, или указали неверный.

[Анатолий]

Konata Izumi,

array:19 [▼
  "program" => "1"
  "type" => "2"
  "rubric_id" => null
  "title_ru" => "фывфывфыв"
  "about_ru" => null
  "infrastructure_ru" => "<p>фывфывфыфвы</p>"
  "plans_ru" => null
  "circs_ru" => null
  "developer_ru" => null
  "location_ru" => null
  "title_kz" => null
  "about_kz" => null
  "infrastructure_kz" => null
  "plans_kz" => null
  "circs_kz" => null
  "developer_kz" => null
  "location_kz" => null
  "section_id" => 3
  "published_at" => "2019-08-27 16:55"
]

Все эти поля имеются в таблице

[Андрей]

Анатолий, И.. что религия не позволяет добавить.. хм.. странно...

[Андрей]

Анатолий, https://medium.com/@kshitij206/laravel-mass-assign...
ну попробуйте
protected $fillable = ['*'];

[Konata Izumi]

Андрей, в документации ровно то же самое написано.

[Анатолий]

Konata Izumi,

dd(Objects::create($data));
Выдает следующее:

[Konata Izumi]

Анатолий, Если в таблице objects есть все эти поля, я уже не знаю.
Берите xdebug и смотрите, где контроллер (или где там у вас сохранение) не так отрабатывает.

[JhaoDa]

Анатолий, запрос и его параметры дампил? Если там всё ок, то вопросы к БД.

[Konata Izumi]

Анатолий, Эксепшен MassAssignmentException не вылез, следовательно, проблема не в гвардед/филлабл.

Answer 1

[Konata Izumi]

Не выдержал и решил проверить это дело кодом.
PHP 7.3.3, Laravel 5.8

Миграция:

class CreateTestTable extends Migration
{
    public function up()
    {
        Schema::create('test', function (Blueprint $table) {
            $table->bigIncrements('id');
            $table->string('name')->nullable();
            $table->integer('age')->nullable();
            $table->timestamps();
        });
    }
}

Модель:

class Test extends Model
{
    protected $table = 'test';
    protected $guarded = [];
}

Вывод php artisan tinker

Psy Shell v0.9.9 (PHP 7.3.3 — cli) by Justin Hileman
>>> use App\Models\Test;
>>> Test::create(['age'=>'21','name'=>'Ivan']);
=> App\Models\Test {#2985
     age: "21",
     name: "Ivan",
     updated_at: "2019-08-27 12:44:11",
     created_at: "2019-08-27 12:44:11",
     id: 1,
   }

Убираем guard у модели:

class Test extends Model
{
    protected $table = 'test';
}

tinker:

Psy Shell v0.9.9 (PHP 7.3.3 — cli) by Justin Hileman
>>> use App\Models\Test;
>>> Test::create(['age'=>'24','name'=>'Ivan3']);
Illuminate/Database/Eloquent/MassAssignmentException with message 'Add [age] to fillable property to allow mass assignment on [App/Models/Test].'

Comments

[Анатолий]

Спасибо, я разобрался в чем проблема.

У меня в этой модели есть __construct(), а так как при наследовании в абстрактном классе Model тоже есть __construct(), и он по умолчанию принимает

public function __construct(array $attributes = [])

А я при наследовании не указал этого.
Прошу прощения за невнимательность.

Спасибо за помощь

[Konata Izumi]

Анатолий, в parent::__construct забыли аргументы прокинуть?)

[Анатолий]

Konata Izumi, Да