App\DataFixtures\TrickImageFixtures::load()

Complexity

Conditions	6
Paths	5

Size

Total Lines	28
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function load(ObjectManager $manager)
    {
        $faker = Factory::create();

        $tricks = $manager->getRepository(Trick::class)->findAll();


        foreach ($tricks as $trick) {
            $maxTags = rand(0, 8);
            $primaryImageSet = false;
            if ($maxTags > 0) {
                for ($i = 0; $i <= $maxTags; $i++) {
                    $image = new Image();

                    $image->setTitle($faker->words(rand(1, 3), true));

It seems like $faker->words(rand(1, 3), true) can also be of type array; however, parameter $title of App\Entity\Image::setTitle() does only seem to accept null|string, maybe add an additional type check?

                    $image->setImage($faker->image('public/uploads/trick_images', 400, 300, null, false));
                    $image->setTrick($trick);
                    if(rand(0,2)>0 && !$primaryImageSet){
                        $image->setPrimaryImage(true);
                        $primaryImageSet = true;
                    }

                    $manager->persist($image);
                    $manager->persist($trick);
                }
            }
        }
        $manager->flush();
    }
}