Lakion\CmsPlugin\Fixture\Factory\RouteExampleFactory

Complexity

Total Complexity

2

Size/Duplication

Total Lines	54
Duplicated Lines	20.37 %

Coupling/Cohesion

Components	1
Dependencies	7

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	17	1
A	create()	11	11	1

<?php

namespace Lakion\CmsPlugin\Fixture\Factory;

use Lakion\CmsPlugin\Document\Route;
use Lakion\CmsPlugin\Document\StaticContent;
use Sylius\Bundle\CoreBundle\Fixture\Factory\ExampleFactoryInterface;
use Sylius\Bundle\CoreBundle\Fixture\OptionsResolver\LazyOption;
use Sylius\Component\Core\Formatter\StringInflector;
use Sylius\Component\Resource\Factory\FactoryInterface;
use Sylius\Component\Resource\Repository\RepositoryInterface;
use Symfony\Component\OptionsResolver\Options;
use Symfony\Component\OptionsResolver\OptionsResolver;

final class RouteExampleFactory implements ExampleFactoryInterface
{
    /**
     * @var FactoryInterface
     */
    private $routeFactory;

    /**
     * @var \Faker\Generator
     */
    private $faker;

    /**
     * @var OptionsResolver
     */
    private $optionsResolver;

    /**
     * @param FactoryInterface $routeFactory
     * @param RepositoryInterface $staticContentRepository
     */
    public function __construct(FactoryInterface $routeFactory, RepositoryInterface $staticContentRepository)
    {
        $this->routeFactory = $routeFactory;

        $this->faker = \Faker\Factory::create();
        $this->optionsResolver =
            (new OptionsResolver())
                ->setDefault('name', function (Options $options) {

The parameter $options is not used and could be removed.

                    return StringInflector::nameToCode($this->faker->words(3, true));

It seems like $this->faker->words(3, true) targeting Faker\Generator::words() can also be of type array; however, Sylius\Component\Core\Formatter\StringInflector::nameToCode() does only seem to accept string, maybe add an additional type check?

                })
                ->setAllowedTypes('name', 'string')

                ->setDefault('content', LazyOption::randomOne($staticContentRepository))
                ->setAllowedTypes('content', ['string', StaticContent::class])
                ->setNormalizer('content', LazyOption::findOneBy($staticContentRepository, 'name'))
        ;
    }

    /**
     * {@inheritdoc}
     */
    public function create(array $options = [])

This method seems to be duplicated in your project.

    {
        $options = $this->optionsResolver->resolve($options);

        /** @var Route $route */
        $route = $this->routeFactory->createNew();
        $route->setName($options['name']);
        $route->setContent($options['content']);

        return $route;
    }
}