BitBag\SyliusShippingExportPlugin\Repository\ShippingExportRepository

Complexity

Total Complexity

3

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	createListQueryBuilder()	0	4	1
A	findAllWithNewOrPendingState()	0	8	1
A	findAllWithNewState()	0	9	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\ShippingExportInterface;
use Doctrine\ORM\QueryBuilder;
use Sylius\Bundle\ResourceBundle\Doctrine\ORM\EntityRepository;

class ShippingExportRepository extends EntityRepository implements ShippingExportRepositoryInterface
{
    public const NEW_STATE_PARAMETER = 'newState';

    public const PENDING_STATE_PARAMETER = 'pendingState';

    public function createListQueryBuilder(): QueryBuilder
    {
        return $this->createQueryBuilder('o')
            ->leftJoin('o.shipment', 'shipment')
        ;
    }

    /**
     * @inheritdoc
     */
    public function findAllWithNewState(): array
    {
        trigger_deprecation('bitbag/shipping-export-plugin', '1.6', 'The "%s()" method is deprecated, use "ShippingExportRepository::findAllWithNewOrPendingState" instead.', __METHOD__);

        return $this->createQueryBuilder('o')

The expression return $this->createQueryBuilder('o')->where('o.state = :newState')->setParameter(self::NEW_STATE_PARAMETER, BitBag\SyliusShippingExportPlugin\Entity\ShippingExportInterface::STATE_NEW)->getQuery()->getResult() could return the type integer which is incompatible with the type-hinted return array. Consider adding an additional type-check to rule them out.

            ->where('o.state = :newState')
            ->setParameter(self::NEW_STATE_PARAMETER, ShippingExportInterface::STATE_NEW)
            ->getQuery()
            ->getResult()
        ;
    }

    public function findAllWithNewOrPendingState(): array
    {
        return $this->createQueryBuilder('o')

The expression return $this->createQueryBuilder('o')->where('o.state = :newState OR o.state = :pendingState')->setParameter(self::NEW_STATE_PARAMETER, BitBag\SyliusShippingExportPlugin\Entity\ShippingExportInterface::STATE_NEW)->setParameter(self::PENDING_STATE_PARAMETER, BitBag\SyliusShippingExportPlugin\Entity\ShippingExportInterface::STATE_PENDING)->getQuery()->getResult() could return the type integer which is incompatible with the type-hinted return array. Consider adding an additional type-check to rule them out.

            ->where('o.state = :newState OR o.state = :pendingState')
            ->setParameter(self::NEW_STATE_PARAMETER, ShippingExportInterface::STATE_NEW)
            ->setParameter(self::PENDING_STATE_PARAMETER, ShippingExportInterface::STATE_PENDING)
            ->getQuery()
            ->getResult()
        ;
    }
}