App\DataFixtures\TagFixtures::load()

Complexity

Conditions	6
Paths	3

Size

Total Lines	37
Code Lines	22

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 = strtolower($faker->words(rand(1,3), true));

It seems like $faker->words(rand(1, 3), true) can also be of type array; however, parameter $str of strtolower() does only seem to accept string, maybe add an additional type check?

                    //Avoid duplicate tags
                    $tag = $tagRepository->findOneBy(['name'=>$tagName]);
                    if($tag === null){
                        $tag=new Tag();
                        $tag->setName($tagName);
                        $manager->persist($tag);
                        $manager->flush(); //create the tag in DB
                    }

                    $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();

}