Laravel can't create model with Model::create() and $model->save()

Question

I want to seed the hero table, but I get the following error.

Argument 1 passed to Illuminate\Database\Query\Builder::insertGetId() must be of the type array, object given, called in

E:\xampp\htdocs\cyberpunk\vendor\laravel\framework\ src\Illuminate\Database\Eloquent\Builder.php on line 1350.

Exception trace: Illuminate\Database\Query\Builder::insertGetId(Object(App\Hero), "id") ...vendor\laravel\framework\src\Illuminate\Database\Eloquent\Builder.php:1350

Seeder

User::find(1)->hero()->create([
    'name' => $faker->userName,
    'level' => $faker->numberBetween(1, 99),
    'strength' => $faker->numberBetween(1, 20),
    'vitality' => $faker->numberBetween(1, 20),
    'stamina' => $faker->numberBetween(1, 20),
    'agility' => $faker->numberBetween(1, 20),
    'perception' => $faker->numberBetween(1, 20),
    'luck' => $faker->numberBetween(1, 20),
]);

I have a one-to-one relation between User and Hero. I want to create a new hero via the user model. This way also causes the same error.

$hero = new Hero;
$hero->user_id = 1;
...
$hero->save();

User

class User extends Authenticatable implements JWTSubject
{
    use Notifiable;

    protected $fillable = [
        'name', 'email', 'password'
    ];

    protected $hidden = [
        'password', 'remember_token',
    ];

    public $appends = ['hashid'];

    public function hero()
    {
        return $this->hasOne(Hero::class);
    }
}

Hero Model

class Hero extends Model
{
    protected $fillable = ['level'];

    public function user()
    {
        return $this->belongsTo(User::class);
    }
}

try adding 'user_id' (and all others fields) to the fillable property in Hero Model : protected $fillable =[ 'user_id', 'level', ...]; — porloscerros Ψ
– porloscerros Ψ, Commented Sep 30, 2019 at 23:55

Kenny Horna · Accepted Answer · 2019-10-01 01:08:40Z

0

The problem is your $fillable config. Try adding your fields to the $fillable:

class Hero extends Model {

    protected $fillable = ['level', 'client_id', ... ]; // <-- your allowed fields

    public function user() {
        return $this->belongsTo(User::class);
    }
}

Now, keep in mind that this will need to be applied for the majority of your fields. An alternative is to make use of the opposite, the $guarded attribute. You list there the fields that you want to protect against mass-assignment:

class Hero extends Model {

    protected $guarded = []; // <-- your protected fields go here

    public function user() {
        return $this->belongsTo(User::class);
    }
}

Check the related section of the documentation.

edited Oct 1, 2019 at 1:08

answered Oct 1, 2019 at 1:02

Kenny Horna

14.3k5 gold badges49 silver badges74 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Laravel can't create model with Model::create() and $model->save()

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related