Sylius\Bundle\CoreBundle\Fixture\SimilarProductAssociationFixture

Complexity

Total Complexity

7

Size/Duplication

Total Lines	115
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	10

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	16	1
A	getName()	0	4	1
B	load()	0	24	2
A	configureOptionsNode()	0	7	1
A	getAssociatedProductsAsArray()	0	13	2

<?php

/*
 * This file is part of the Sylius package.
 *
 * (c) Paweł Jędrzejewski
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Sylius\Bundle\CoreBundle\Fixture;

use Sylius\Bundle\FixturesBundle\Fixture\AbstractFixture;
use Sylius\Component\Core\Model\ProductInterface;
use Sylius\Component\Core\Repository\ProductRepositoryInterface;
use Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition;
use Symfony\Component\OptionsResolver\OptionsResolver;

/**
 * @author Grzegorz Sadowski <grzegorz.sadowski@lakion.com>
 */
class SimilarProductAssociationFixture extends AbstractFixture
{
    /**
     * @var AbstractResourceFixture
     */
    private $productAssociationTypeFixture;

    /**
     * @var AbstractResourceFixture
     */
    private $productAssociationFixture;

    /**
     * @var ProductRepositoryInterface
     */
    private $productRepository;

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

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

    /**
     * @param AbstractResourceFixture $productAssociationTypeFixture
     * @param AbstractResourceFixture $productAssociationFixture
     * @param ProductRepositoryInterface $productRepository
     */
    public function __construct(
        AbstractResourceFixture $productAssociationTypeFixture,

The variable name $productAssociationTypeFixture exceeds the maximum configured length of 20.

        AbstractResourceFixture $productAssociationFixture,

The variable name $productAssociationFixture exceeds the maximum configured length of 20.

        ProductRepositoryInterface $productRepository
    ) {
        $this->productAssociationTypeFixture = $productAssociationTypeFixture;
        $this->productAssociationFixture = $productAssociationFixture;
        $this->productRepository = $productRepository;

        $this->faker = \Faker\Factory::create();
        $this->optionsResolver =
            (new OptionsResolver())
                ->setRequired('amount')
                ->setAllowedTypes('amount', 'int')
        ;
    }

    /**
     * {@inheritdoc}
     */
    public function getName()
    {
        return 'similar_product_association';
    }

    /**
     * {@inheritdoc}
     */
    public function load(array $options)
    {
        $options = $this->optionsResolver->resolve($options);

        $this->productAssociationTypeFixture->load(['custom' => [[
            'code' => 'similar_products',
            'name' => 'Similar products',
        ]]]);

        $products = $this->productRepository->findAll();
        $products = $this->faker->randomElements($products, $options['amount']);

        $productAssociations = [];
        /** @var ProductInterface $product */
        foreach ($products as $product) {
            $productAssociations[] = [
                'type' => 'similar_products',
                'owner' => $product->getCode(),
                'associated_products' => $this->getAssociatedProductsAsArray($product),
            ];
        }

        $this->productAssociationFixture->load(['custom' => $productAssociations]);
    }

    /**
     * {@inheritdoc}
     */
    protected function configureOptionsNode(ArrayNodeDefinition $optionsNode)
    {
        $optionsNode

It seems like you code against a specific sub-type and not the parent class Symfony\Component\Config\Definition\Builder\NodeDefinition as the method min() does only exist in the following sub-classes of Symfony\Component\Config\Definition\Builder\NodeDefinition: Symfony\Component\Config\Definition\Builder\FloatNodeDefinition, Symfony\Component\Config\Definition\Builder\IntegerNodeDefinition, Symfony\Component\Config\Definition\Builder\NumericNodeDefinition. Maybe you want to instanceof check for one of these explicitly?

            ->children()
                ->integerNode('amount')->isRequired()->min(0)->end()
        ;
    }

    /**
     * @param ProductInterface $owner
     *
     * @return string[]
     */
    private function getAssociatedProductsAsArray(ProductInterface $owner)
    {
        $products = $this->productRepository->findBy(['mainTaxon' => $owner->getMainTaxon()]);
        $products = $this->faker->randomElements($products, 3);

        $associatedProducts = [];
        /** @var ProductInterface $product */
        foreach ($products as $product) {
            $associatedProducts[] = $product->getCode();
        }

        return $associatedProducts;
    }
}