BitBag\SyliusMolliePlugin\Repository\SubscriptionRepository

Complexity

Total Complexity

1

Size/Duplication

Total Lines	9
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	findOneByOrderId()	0	7	1

<?php

/*
 * This file has been created by developers from BitBag.
 * Feel free to contact us once you face any issues or want to start
 * You can find more information about us on https://bitbag.io and write us
 * an email on hello@bitbag.io.
 */

declare(strict_types=1);

namespace BitBag\SyliusMolliePlugin\Repository;

use BitBag\SyliusMolliePlugin\Entity\SubscriptionInterface;
use Sylius\Bundle\ResourceBundle\Doctrine\ORM\EntityRepository;

class SubscriptionRepository extends EntityRepository implements SubscriptionRepositoryInterface
{
    public function findOneByOrderId($orderId): ?SubscriptionInterface
    {
        return $this->createQueryBuilder('o')

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

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