Alpixel\Bundle\CMSBundle\Entity\Repository\NodeRepository

Complexity

Total Complexity

3

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes	4
Bugs	1	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
B	findTranslation()	0	28	3

<?php

namespace Alpixel\Bundle\CMSBundle\Entity\Repository;

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

class NodeRepository extends EntityRepository
{
    public function findTranslation(NodeInterface $nodeItem, $locale)
    {
        $node = $nodeItem->getNode();

The method getNode() does not seem to exist on object<Alpixel\Bundle\CMSBundle\Entity\NodeInterface>.

        $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();
        }
}