Alpixel\Bundle\CMSBundle\Entity\Repository\NodeRepository

Complexity

Total Complexity

4

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	8
Bugs	3	Features	0

2 Methods

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

<?php

namespace Alpixel\Bundle\CMSBundle\Entity\Repository;

use Alpixel\Bundle\CMSBundle\Entity\Node;
use Doctrine\ORM\EntityRepository;

class NodeRepository extends EntityRepository
{
    public function findAllWithLocale($locale)
    {
        return $this
            ->getEntityManager()
            ->createQueryBuilder('n')
            ->andWhere('n.published = true')
            ->andWhere('n.locale = :locale')
            ->setParameter('locale', $locale)
            ->orderBy('n.position', 'ASC')
            ->getQuery()
            ->getResult();
    }

    public function findTranslation(Node $node, $locale)
    {
        $nodeSource = null;

$nodeSource 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 ($node->getTranslationSource() !== null) {
            $nodeSource = $node->getTranslationSource();
            if ($nodeSource->getLocale() == $locale) {
                return $nodeSource;
            }
        } else {
            $nodeSource = $node;
        }

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