AffiliateRepository::findOneByCustomerAndProductNotExpired()

Complexity

Conditions	1
Paths	1

Size

Total Lines	14
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function findOneByCustomerAndProductNotExpired(
        CustomerInterface $customer,
        ProductInterface $product
    ): ?AffiliateInterface {
        return $this->createQueryBuilder('o')

The expression return $this->createQueryBuilder('o')->andWhere('o.customer = :customer')->andWhere('o.product = :product')->andWhere('o.expiresAt > :now')->setParameter('product', $product)->setParameter('customer', $customer)->setParameter('now', new DateTime())->setMaxResults(1)->getQuery()->getOneOrNullResult() could return the type integer which is incompatible with the type-hinted return Odiseo\SyliusReferralsPlugin\Entity\AffiliateInterface|null. Consider adding an additional type-check to rule them out.

            ->andWhere('o.customer = :customer')
            ->andWhere('o.product = :product')
            ->andWhere('o.expiresAt > :now')
            ->setParameter('product', $product)
            ->setParameter('customer', $customer)
            ->setParameter('now', new \DateTime())
            ->setMaxResults(1)
            ->getQuery()
            ->getOneOrNullResult()
        ;