laraveleloquentlaravel-seedingfactories

Laravel Seeding produces 3 times more results than expected


I'm working on a Laravel project and trying to seed the database by using model factories.

I have a Banners table and related BannerTranslations model to hold translated values. The project will have 3 languages. I'm trying to seed 5 dummy values into Banners table relating to BannerTranslations. In theory i expect 3 translation rows created for each Banner row, which should result 15 entries (3x5) to the BannerTranslations table in the end. But i get 45 entries instead. Each translation row is seeded 3 times instead of 1. I'm sure i'm missing something very obvious but i can't figure out what.

translation migration:

public function up()
{
    Schema::create('banner_translations', function (Blueprint $table) {
        $table->bigIncrements('id');
        $table->unsignedInteger('banner_id');
        $table->string('locale')->index();

        $table->string('title');
        $table->string('spot')->nullable();
        $table->text('body')->nullable();
        $table->string('alt')->nullable();

        //$table->unique(['banner_id', 'locale']);
        $table->foreign('banner_id')->references('id')->on('banners')->onDelete('cascade');

        $table->timestamps();
    });
}

BannerTranslationsFactory:

use App\BannerTranslation;
use Faker\Generator as Faker;

$factory->define(BannerTranslation::class, function (Faker $faker) {
    return [
        'banner_id' => function (array $banner) {
            return App\Banner::find($banner['id'])->id;
        },
        'locale' => function (array $banner) {
            return App\BannerTranslation::find($banner['banner_id'])->locale;
        },
        'title' => $faker->sentence(2),
        'spot' => $faker->sentence(2),
        'body' => $faker->realText(rand(80, 200)),
        'alt' => null,
    ];
});

and BannersTableSeeder:

$banners = factory(App\Banner::class, 5)->create()->each(function ($banner) {

    $lang_count = count(config('laravellocalization.supportedLocales'));

    foreach(LaravelLocalization::getSupportedLanguagesKeys() as $i => $key) {
        factory(App\BannerTranslation::class, $lang_count)->create([
            'banner_id' => $banner->id,
            'locale' => $key,
        ]);
    }
});

This results 3 same key rows per translation ('en','en','en','de','de','de','fr','fr','fr') instead of ('en','de','fr'). And because of this it returns a unique contstraint error.


Solution

  • I'm not well acquainted with using factories but looking at you're and checking the docs, your problem is with the second parameter in factory(App\BannerTranslation::class, $lang_count). You're already looping through LaravelLocalization::getSupportedLanguagesKeys() and in for each one you're creating 3 records.

    Give this a try:

    $banners = factory(App\Banner::class, 5)->create()->each(function ($banner) {
        foreach(LaravelLocalization::getSupportedLanguagesKeys() as $i => $key) {
            factory(App\BannerTranslation::class)->create([
                'banner_id' => $banner->id,
                'locale' => $key,
            ]);
        }
    });