App\DataFixtures\TagFixtures::load()

Complexity

Conditions	6
Paths	3

Size

Total Lines	35
Code Lines	20

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

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

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

        $fixedTag = new Tag();
        $fixedTag->setName("Starbug tag");

        foreach ($tricks as $trick){
            $maxTags = rand(0,5);
            if($maxTags >0){
                for($i=0; $i<=$maxTags; $i++){

                    $tagName = $faker->words(rand(1,3), true);
                    //Avoid duplicate tags
                    $tag = $tagRepository->findOneBy(['name'=>$tagName]);
                    if($tag === null){
                        $tag=new Tag();
                        $tag->setName($tagName);

It seems like $tagName can also be of type array; however, parameter $name of App\Entity\Tag::setName() does only seem to accept string, maybe add an additional type check?

                    }

                    $trick->addTag($tag);

                    if(rand(0,6)===1){
                        $trick->addTag($fixedTag);
                    }
                }
            }
            $manager->persist($trick);
            $manager->flush(); //need to flush or next DB query for unicity will fail
        }

        $manager->flush();

}