App\DataFixtures\TagFixtures

Complexity

Total Complexity

7

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getDependencies()	0	4	1
B	load()	0	35	6

<?php

namespace App\DataFixtures;

use App\Entity\Tag;
use App\Entity\Trick;
use Doctrine\Bundle\FixturesBundle\Fixture;
use Doctrine\Common\DataFixtures\DependentFixtureInterface;
use Doctrine\Common\Persistence\ObjectManager;
use Faker\Factory;

class TagFixtures extends Fixture implements DependentFixtureInterface
{

    /**
     * This method must return an array of fixtures classes
     * on which the implementing class depends on
     *
     * @return array
     */
    public function getDependencies()
    {
        return array(
            TrickFixtures::class,
        );
    }

    /**
     * Load data fixtures with the passed EntityManager
     *
     * @param ObjectManager $manager
     */
    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();

    }
}