Alpixel\Bundle\CMSBundle\Entity\Repository\BlockRepository

Complexity

Total Complexity

4

Size/Duplication

Total Lines	50
Duplicated Lines	52 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	findBlock()	0	12	1
A	findTranslation()	26	26	3

<?php

namespace Alpixel\Bundle\CMSBundle\Entity\Repository;

use Doctrine\ORM\EntityRepository;
use Sonata\BlockBundle\Model\Block;

class BlockRepository extends EntityRepository
{
    /**
     * @param $type
     * @param $locale
     *
     * @throws \Doctrine\ORM\NonUniqueResultException
     *
     * @return \Alpixel\Bundle\CMSBundle\Entity\Block
     */
    public function findBlock($type, $locale)
    {
        return $this
            ->createQueryBuilder('b')
            ->andWhere('b.locale = :locale')
            ->andWhere('b.slug = :type')
            ->setParameter('locale', $locale)
            ->setParameter('type', $type)
            ->setMaxResults(1)
            ->getQuery()
            ->getOneOrNullResult();
    }

    public function findTranslation(Block $block, $locale)

This method seems to be duplicated in your project.

    {
        $source = null;

$source is not used, you could remove the assignment.

        // We are checking if the node is the translation provider or translated
        // from an other node
        if ($block->getTranslationSource() !== null) {

The method getTranslationSource() does not seem to exist on object<Sonata\BlockBundle\Model\Block>.

            $source = $block->getTranslationSource();

The method getTranslationSource() does not seem to exist on object<Sonata\BlockBundle\Model\Block>.

            if ($source->getLocale() == $locale) {
                return $source;
            }
        } else {
            $source = $block;
        }

        return $this->createQueryBuilder('n')
            ->addSelect('n')
            ->andWhere('n.translationSource = :source')
            ->andWhere('n.locale = :locale')
            ->setParameters([
                'source' => $source,
                'locale' => $locale,
            ])
            ->getQuery()
            ->getOneOrNullResult();
    }
}