BitBag\SyliusWishlistPlugin\Repository\WishlistRepository

Complexity

Total Complexity

2

Size/Duplication

Total Lines	19
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	findByToken()	0	7	1
A	findOneByShopUser()	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
 * another great project.
 * You can find more information about us on https://bitbag.shop and write us
 * an email on mikolaj.krol@bitbag.pl.
 */

declare(strict_types=1);

namespace BitBag\SyliusWishlistPlugin\Repository;

use BitBag\SyliusWishlistPlugin\Entity\WishlistInterface;
use Sylius\Bundle\ResourceBundle\Doctrine\ORM\EntityRepository;
use Sylius\Component\Core\Model\ShopUserInterface;

class WishlistRepository extends EntityRepository implements WishlistRepositoryInterface
{
    public function findOneByShopUser(ShopUserInterface $shopUser): ?WishlistInterface
    {
        return $this->createQueryBuilder('o')

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

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

    public function findByToken(string $token): ?WishlistInterface
    {
        return $this->createQueryBuilder('o')

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

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