BitBag\SyliusShippingExportPlugin\Repository\ShippingGatewayRepository

Complexity

Total Complexity

3

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	findOneByShippingMethod()	0	7	1
A	findOneByCode()	0	7	1
A	createListQueryBuilder()	0	3	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\SyliusShippingExportPlugin\Repository;

use BitBag\SyliusShippingExportPlugin\Entity\ShippingGatewayInterface;
use Doctrine\ORM\QueryBuilder;
use Sylius\Bundle\ResourceBundle\Doctrine\ORM\EntityRepository;
use Sylius\Component\Shipping\Model\ShippingMethodInterface;

class ShippingGatewayRepository extends EntityRepository implements ShippingGatewayRepositoryInterface
{
    public function createListQueryBuilder(): QueryBuilder
    {
        return $this->createQueryBuilder('o');
    }

    public function findOneByCode(string $code): ?ShippingGatewayInterface
    {
        return $this->createQueryBuilder('o')

The expression return $this->createQueryBuilder('o')->where('o.code = :code')->setParameter('code', $code)->getQuery()->getOneOrNullResult() could return the type integer which is incompatible with the type-hinted return BitBag\SyliusShippingExportPlugin\Entity\ShippingGatewayInterface|null. Consider adding an additional type-check to rule them out.

            ->where('o.code = :code')
            ->setParameter('code', $code)
            ->getQuery()
            ->getOneOrNullResult()
        ;
    }

    public function findOneByShippingMethod(ShippingMethodInterface $shippingMethod): ?ShippingGatewayInterface
    {
        return $this->createQueryBuilder('o')

The expression return $this->createQueryBuilder('o')->where(':shippingMethod MEMBER OF o.shippingMethods')->setParameter('shippingMethod', $shippingMethod)->getQuery()->getOneOrNullResult() could return the type integer which is incompatible with the type-hinted return BitBag\SyliusShippingExportPlugin\Entity\ShippingGatewayInterface|null. Consider adding an additional type-check to rule them out.

            ->where(':shippingMethod MEMBER OF o.shippingMethods')
            ->setParameter('shippingMethod', $shippingMethod)
            ->getQuery()
            ->getOneOrNullResult()
        ;
    }
}