BitBag\SyliusElasticsearchPlugin\PropertyBuilder\ProductMainTaxonPositionPropertyBuilder

Complexity

Total Complexity

3

Size/Duplication

Total Lines	23
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	consumeEvent()	0	13	2
A	__construct()	0	3	1

<?php

namespace BitBag\SyliusElasticsearchPlugin\PropertyBuilder;

use BitBag\SyliusElasticsearchPlugin\PropertyNameResolver\ConcatedNameResolverInterface;
use Elastica\Document;
use FOS\ElasticaBundle\Event\TransformEvent;
use Sylius\Component\Core\Model\ProductInterface;

final class ProductMainTaxonPositionPropertyBuilder extends AbstractBuilder
{
    /** @var ConcatedNameResolverInterface */
    private $taxonPositionNameResolver;

    public function __construct(ConcatedNameResolverInterface $taxonPositionNameResolver)
    {
        $this->taxonPositionNameResolver = $taxonPositionNameResolver;
    }

    public function consumeEvent(TransformEvent $event): void
    {
        $this->buildProperty($event, ProductInterface::class,
            function (ProductInterface $product, Document $document): void {
                $mainTaxon = $product->getMainTaxon();

                if (null === $mainTaxon) {
                    return;
                }

                $document->set(
                    $this->taxonPositionNameResolver->resolvePropertyName($mainTaxon->getCode()),

It seems like $mainTaxon->getCode() can also be of type null; however, parameter $suffix of BitBag\SyliusElasticsearchPlugin\PropertyNameResolver\ConcatedNameResolverInterface::resolvePropertyName() does only seem to accept string, maybe add an additional type check?

                    $mainTaxon->getPosition()
                );
            }
        );
    }
}