BitBag\SyliusCmsPlugin\Repository\SectionRepository

Complexity

Total Complexity

6

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	createListQueryBuilder()	0	6	1
A	findByNamePart()	0	7	1
A	findByCodesAndLocale()	0	10	1
A	createTranslationBasedQueryBuilder()	0	15	2
A	findOneByCode()	0	10	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\SyliusCmsPlugin\Repository;

use BitBag\SyliusCmsPlugin\Entity\SectionInterface;
use Doctrine\ORM\QueryBuilder;
use Sylius\Bundle\ResourceBundle\Doctrine\ORM\EntityRepository;

class SectionRepository extends EntityRepository implements SectionRepositoryInterface
{
    public function createListQueryBuilder(string $localeCode): QueryBuilder
    {
        return $this->createQueryBuilder('o')
            ->addSelect('translation')
            ->leftJoin('o.translations', 'translation', 'WITH', 'translation.locale = :localeCode')
            ->setParameter('localeCode', $localeCode)
        ;
    }

    public function findByNamePart(string $phrase, ?string $locale = null): array
    {
        return $this->createTranslationBasedQueryBuilder($locale)

The expression return $this->createTranslationBasedQueryBuilder($locale)->andWhere('translation.name LIKE :name')->setParameter('name', '%' . $phrase . '%')->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.

            ->andWhere('translation.name LIKE :name')
            ->setParameter('name', '%' . $phrase . '%')
            ->getQuery()
            ->getResult()
        ;
    }

    private function createTranslationBasedQueryBuilder(?string $locale = null): QueryBuilder
    {
        $queryBuilder = $this->createQueryBuilder('o')
            ->addSelect('translation')
            ->leftJoin('o.translations', 'translation')
        ;

        if (null !== $locale) {
            $queryBuilder
                ->andWhere('translation.locale = :locale')
                ->setParameter('locale', $locale)
            ;
        }

        return $queryBuilder;
    }

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

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

            ->leftJoin('o.translations', 'translation')
            ->where('translation.locale = :localeCode')
            ->andWhere('o.code = :code')
            ->setParameter('code', $code)
            ->setParameter('localeCode', $localeCode)
            ->getQuery()
            ->getOneOrNullResult()
        ;
    }

    public function findByCodesAndLocale(string $codes, string $localeCode): array
    {
        return $this->createQueryBuilder('o')

The expression return $this->createQueryBuilder('o')->leftJoin('o.translations', 'translation')->where('translation.locale = :localeCode')->andWhere('o.code IN(:codes)')->setParameter('codes', explode(',', $codes))->setParameter('localeCode', $localeCode)->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.

            ->leftJoin('o.translations', 'translation')
            ->where('translation.locale = :localeCode')
            ->andWhere('o.code IN(:codes)')
            ->setParameter('codes', explode(',', $codes))
            ->setParameter('localeCode', $localeCode)
            ->getQuery()
            ->getResult()
            ;
    }
}