Field doesn't have default value upon factory seeding against a one to many relationship

Question

I have one speciality that can have many procedures.

I am wanting to seed both the speciality with many procedures.

Models:

Speciality

class Speciality extends Model
{

    protected $fillable = ['name'];

    public function procedures() {
        return $this->hasMany('App\Procedure');
    }
}

Procedure

class Procedure extends Model
{
    protected $fillable = ['speciality_id', 'name'];

    public function speciality() {
        return $this->belongsTo('App\Speciality');
    }
}

Factories:

Speciality

$factory->define(App\Speciality::class, function (Faker $faker) {
    return [
        'name' => 'Example'
    ];
});

Procedure

$factory->define(App\Procedure::class, function (Faker $faker) {
    return [
        'name' => $faker->realText(20)
    ];
});

Seeding:

public function run()
    {
        factory(App\Speciality::class)->create()
            ->each(function($speciality) {
                $speciality->procedures()
                    ->saveMany(factory(App\Procedure::class, rand(5,20))
                    ->make()
                );
        });
    }

The error shown is:

SQLSTATE[HY000]: General error: 1364 Field 'speciality_id' doesn't have a default value (SQL: insert into procedures (name, updated_at, created_at) values (It means much the., 2018-04-17 19:09:31, 2018-04-17 19:09:31))

speciality_id isn't set as a value within the procedure factory, because it's the foreign key to the speciality. But my understanding of using $speciality->procedures() is it would automatically assign the speciality_id upon seeding.

Why is this error happening please. Many thanks.

check without using procedure factory like ->saveMany([ new Procedure(['name' => 'A new name']) ]); — afsal c, Apr 17 '18 at 20:13
Your code works for me. Try debugging `HasOneOrMany::setForeignAttributesForCreate()`. — Jonas Staudenmeir, Apr 17 '18 at 23:35

Josh Rumbut · Answer 1 · 2018-04-17T20:23:57.177

0

I may be wrong here but I think there is just a small issue of when you call make:

public function run()
    {
        factory(App\Speciality::class)->create()
            ->each(function($speciality) {
                $speciality->procedures()
                    ->saveMany(factory(App\Procedure::class, rand(5,20))->make());     

        });
    }

Never used this with saveMany() before though, only save(), but try it out?

edited Apr 17 '18 at 20:23

answered Apr 17 '18 at 19:58

Josh Rumbut

2,640
2
32
43

Thank you for your reply. It's throwing an error on the `->saveMany` line - `Call to a member function make() on integer`. It looks like the `->make()` should be outside the brackets? Which if it is, the error reappears. – Apr 17 '18 at 20:02
@Ben what version are you using? – Josh Rumbut Apr 17 '18 at 20:08
Latest 5.6. I have to keep typing to allow submission of comment! – Apr 17 '18 at 20:10
@JoshRumbut I thought that there is a typo in OP, but its actually correct. – Kyslik Apr 17 '18 at 20:11
@Ben I messed up the parens, what I have now is working for me in my app, try it out – Josh Rumbut Apr 17 '18 at 20:24
Thanks @JoshRumbut, unfortunately the same error still applies. What's different on yours than on mine ? :) – Apr 17 '18 at 20:30
@Ben Did you get `member function make() on integer` error or the original error? – Josh Rumbut Apr 17 '18 at 20:33
@Ben could you check that value of `$specialty` inside the `each` function? – Josh Rumbut Apr 17 '18 at 21:57

Field doesn't have default value upon factory seeding against a one to many relationship

1 Answers1