BitBag\SyliusElasticsearchPlugin\Repository\ProductVariantRepository

Complexity

Total Complexity

2

Size/Duplication

Total Lines	18
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

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

<?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\SyliusElasticsearchPlugin\Repository;

use Doctrine\ORM\EntityRepository;
use Sylius\Component\Core\Model\ProductVariantInterface;
use Sylius\Component\Core\Repository\ProductVariantRepositoryInterface as BaseProductVariantRepositoryInterface;
use Sylius\Component\Product\Model\ProductOptionValueInterface;

class ProductVariantRepository implements ProductVariantRepositoryInterface
{
    /** @var BaseProductVariantRepositoryInterface|EntityRepository */
    private $baseProductVariantRepository;

    public function __construct(BaseProductVariantRepositoryInterface $baseProductVariantRepository)
    {
        $this->baseProductVariantRepository = $baseProductVariantRepository;
    }

    public function findOneByOptionValue(ProductOptionValueInterface $productOptionValue): ?ProductVariantInterface
    {
        return $this->baseProductVariantRepository->createQueryBuilder('o')

The method createQueryBuilder() does not exist on Sylius\Component\Core\Repository\ProductVariantRepositoryInterface. Did you maybe mean createQueryBuilderByProductId()?

The expression return $this->baseProductVariantRepository->createQueryBuilder('o')->where(':optionValue MEMBER OF o.optionValues')->setParameter('optionValue', $productOptionValue)->getQuery()->setMaxResults(1)->getOneOrNullResult() could return the type integer which is incompatible with the type-hinted return Sylius\Component\Core\Model\ProductVariantInterface|null. Consider adding an additional type-check to rule them out.

            ->where(':optionValue MEMBER OF o.optionValues')
            ->setParameter('optionValue', $productOptionValue)
            ->getQuery()
            ->setMaxResults(1)
            ->getOneOrNullResult()
        ;
    }
}