BitBag\SyliusCmsPlugin\Assigner\TaxonsAssigner

Complexity

Total Complexity

3

Size/Duplication

Total Lines	18
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	assign()	0	8	2

<?php

/*
 * This file was created by developers working at BitBag
 * Do you need more information about us and what we do? Visit our https://bitbag.io website!
 * We are hiring developers from all over the world. Join us and start your new, exciting adventure and become part of us: https://bitbag.io/career
*/

declare(strict_types=1);

namespace BitBag\SyliusCmsPlugin\Assigner;

use BitBag\SyliusCmsPlugin\Entity\TaxonAwareInterface;
use Sylius\Component\Core\Model\TaxonInterface;
use Sylius\Component\Taxonomy\Repository\TaxonRepositoryInterface;
use Webmozart\Assert\Assert;

final class TaxonsAssigner implements TaxonsAssignerInterface
{
    /** @var TaxonRepositoryInterface */
    private $taxonRepository;

    public function __construct(TaxonRepositoryInterface $taxonRepository)
    {
        $this->taxonRepository = $taxonRepository;
    }

    public function assign(TaxonAwareInterface $taxonAware, array $taxonCodes): void
    {
        foreach ($taxonCodes as $taxonCode) {
            /** @var TaxonInterface|null $taxon */
            $taxon = $this->taxonRepository->findOneBy(['code' => $taxonCode]);

            Assert::notNull($taxon, sprintf('Taxon with %s code not found.', $taxonCode));
            $taxonAware->addTaxon($taxon);

It seems like $taxon can also be of type null; however, parameter $taxon of BitBag\SyliusCmsPlugin\Entity\TaxonAwareInterface::addTaxon() does only seem to accept Sylius\Component\Core\Model\TaxonInterface, maybe add an additional type check?

        }
    }
}