ViewRepository::findOneByHomepage() - Code Metrics - Inspection of "Feature/make page settings and seo i18n" - Victoire/victoire - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#374)

by Leny

created 2016-04-01 07:36 UTC

ViewRepository::findOneByHomepage() B

↳ Parent: ViewRepository

Complexity

Conditions	1
Paths	1

Size

Total Lines	27
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
dl	0
loc	27
rs	8.8571
c	1
b	0
f	0
cc	1
eloc	16
nc	1
nop	1

<?php

namespace Victoire\Bundle\CoreBundle\Repository;

use Doctrine\ORM\Query;
use Gedmo\Tree\Entity\Repository\NestedTreeRepository;
use Victoire\Bundle\CoreBundle\Entity\View;
use Victoire\Bundle\PageBundle\Entity\Page;
use Victoire\Bundle\PageBundle\Entity\PageStatus;
use Victoire\Bundle\ViewReferenceBundle\ViewReference\BusinessPageReference;
use Victoire\Bundle\ViewReferenceBundle\ViewReference\ViewReference;

/**
 * The View repository.
 */
class ViewRepository extends NestedTreeRepository
{
    private $queryBuilder;

    /**
     * Get query builder instance.
     */
    public function getInstance()
    {
        return $this->queryBuilder ? $this->queryBuilder : $this->createQueryBuilder('page');
    }

    /**
     * Get the query builder for a view  by url.
     *
     * @param string $url The url
     *
     * @return \Doctrine\ORM\QueryBuilder The query builder
     */
    public function getOneByUrl($url)
    {
        return $this->createQueryBuilder('page')
            ->where('page.url = (:url)')
            ->setMaxResults(1)
            ->setParameter('url', $url);
    }

    /**
     * Filter the query by the sitemap index (=visibility).
     *
     * @param bool $indexed
     *
     * @return ViewRepository
     */
    public function filterBySitemapIndexed($indexed = true)
    {
        $qb = $this->getInstance();
        $qb->innerJoin('page.seo', 'seo')->addSelect('seo')
            ->andWhere('seo.sitemapIndexed = :sitemapIndexed')
            ->setParameter('sitemapIndexed', $indexed);

        return $this;
    }

    /**
     * Get all rentals in the repository.
     *
     * @param bool $excludeUnpublished Should we get only the published Views ?
     *
     * @return ViewRepository
     */
    public function getAll($excludeUnpublished = false)
    {
        $this->queryBuilder = $this->getInstance();

        //If $excludeUnpublished === true, we exclude the non published results
        if ($excludeUnpublished) {

            $this->queryBuilder
                ->andWhere('page.status = :status')
                ->orWhere('page.status = :scheduled_status AND page.publishedAt > :publicationDate')
                ->setParameter('status', PageStatus::PUBLISHED)
                ->setParameter('scheduled_status', PageStatus::SCHEDULED)
                ->setParameter('publicationDate', new \DateTime());
        }

        return $this;
    }

    /**
     * Run instance.
     *
     * @param string $method
     * @param string $hydrationMode
     *
     * @return array
     */
    public function run($method = 'getResult', $hydrationMode = Query::HYDRATE_OBJECT)
    {
        return $this->getInstance()->getQuery()->$method($hydrationMode);
    }

    /**
     * Find a large amount of views by ViewReferences and optimizing queries with translation walker.
     *
     * @param ViewReference[] $viewReferences
     *
     * @return View[]|null The entity instance or NULL if the entities cannot be found.
     */
    public function findByViewReferences(array $viewReferences)
    {
        $views = [];
        $localizedViewReferences = [];
        foreach ($viewReferences as $viewReference) {
            if (array_key_exists($viewReference->getLocale(), $localizedViewReferences)) {
                $localizedViewReferences[$viewReference->getLocale()] = [];
            }
            $localizedViewReferences[$viewReference->getLocale()][] = $viewReference;
        }

        foreach ($localizedViewReferences as $locale => $_viewReferences) {
            //the query builder
            $queryBuilder = $this->createQueryBuilder('page');;

            $pageIds = [];
            foreach ($_viewReferences as $viewReference) {
                if ($viewReference instanceof BusinessPageReference) {
                    $pageIds[] = $viewReference->getTemplateId();
                } else {
                    $pageIds[] = $viewReference->getViewId();
                }
            }

            $queryBuilder->andWhere('page.id IN (:pageIds)')
                ->setParameter('pageIds', $pageIds);


            // Use Translation Walker
            $query = $queryBuilder->getQuery();
            $query->setHint(
                \Doctrine\ORM\Query::HINT_CUSTOM_OUTPUT_WALKER,
                'Gedmo\\Translatable\\Query\\TreeWalker\\TranslationWalker'
            );

            // Force the locale
            $query->setHint(
                \Gedmo\Translatable\TranslatableListener::HINT_TRANSLATABLE_LOCALE,
                $locale
            );

            /** @var View[] $query */
            $_views = $query->getResult();

            //Parse views to set locale and reference
            $views = array_merge($views, $_views);
        }

        return $views;
    }

    /**
     * Finds a single entity by a set of criteria.
     *
     * @param array $criteria
     * @param array|null $orderBy
     *
     * @return object|null The entity instance or NULL if the entity can not be found.
     */
    public function findOneBy(array $criteria, array $orderBy = null)
    {
        $persister = $this->_em->getUnitOfWork()->getEntityPersister($this->_entityName);

        $hints = [];
        if (isset($criteria['locale'])) {
            $hints = [
                \Doctrine\ORM\Query::HINT_CUSTOM_OUTPUT_WALKER => 'Gedmo\\Translatable\\Query\\TreeWalker\\TranslationWalker',
                \Gedmo\Translatable\TranslatableListener::HINT_TRANSLATABLE_LOCALE => $criteria['locale']
            ];
            unset($criteria['locale']);
        }

        return $persister->load($criteria, null, null, $hints, null, 1, $orderBy);
    }

    /**
     * Get the the view that is a homepage and a published one.
     *
     * @param string $locale
     *
     * @return Page
     */
    public function findOneByHomepage($locale = 'fr')
    {
        //the query builder
        $queryBuilder = $this->createQueryBuilder('page');

        $queryBuilder
            ->where('page.homepage = true')
            ->andWhere('page.status = :status')
            ->setMaxResults(1)
            ->setParameter('status', PageStatus::PUBLISHED);

        // Use Translation Walker
        $query = $queryBuilder->getQuery();
        $query->setHint(
            \Doctrine\ORM\Query::HINT_CUSTOM_OUTPUT_WALKER,
            'Gedmo\\Translatable\\Query\\TreeWalker\\TranslationWalker'
        );
        // Force the locale
        $query->setHint(
            \Gedmo\Translatable\TranslatableListener::HINT_TRANSLATABLE_LOCALE,
            $locale
        );

        $view = $query->getOneOrNullResult();

        return $view;
    }

    /**
     * Get PageSeo.
     *
     * @param string $method leftJoin|innerJoin
     *
     * @return ViewRepository
     */
    public function joinSeo($method = 'leftJoin')
    {
        $this->getInstance()->$method('page.seo', 'seo')->addSelect('seo');

        return $this;
    }

    /**
     * Filter the query by the sitemap index (=visibility).
     *
     * @param array $ids
     *
     * @return ViewRepository
     */
    public function filterByIds($ids)
    {
        $this->getInstance()->andWhere('page.id IN (:ids)')->setParameter('ids', $ids);

        return $this;
    }
}


1		<?php
2
3		namespace Victoire\Bundle\CoreBundle\Repository;
4
5		use Doctrine\ORM\Query;
6		use Gedmo\Tree\Entity\Repository\NestedTreeRepository;
7		use Victoire\Bundle\CoreBundle\Entity\View;
8		use Victoire\Bundle\PageBundle\Entity\Page;
9		use Victoire\Bundle\PageBundle\Entity\PageStatus;
10		use Victoire\Bundle\ViewReferenceBundle\ViewReference\BusinessPageReference;
11		use Victoire\Bundle\ViewReferenceBundle\ViewReference\ViewReference;
12
13		/**
14		* The View repository.
15		*/
16		class ViewRepository extends NestedTreeRepository
17		{
18		private $queryBuilder;
19
20		/**
21		* Get query builder instance.
22		*/
23		public function getInstance()
24		{
25		return $this->queryBuilder ? $this->queryBuilder : $this->createQueryBuilder('page');
26		}
27
28		/**
29		* Get the query builder for a view by url.
30		*
31		* @param string $url The url
32		*
33		* @return \Doctrine\ORM\QueryBuilder The query builder
34		*/
35		public function getOneByUrl($url)
36		{
37		return $this->createQueryBuilder('page')
38		->where('page.url = (:url)')
39		->setMaxResults(1)
40		->setParameter('url', $url);
41		}
42
43		/**
44		* Filter the query by the sitemap index (=visibility).
45		*
46		* @param bool $indexed
47		*
48		* @return ViewRepository
49		*/
50		public function filterBySitemapIndexed($indexed = true)
51		{
52		$qb = $this->getInstance();
53		$qb->innerJoin('page.seo', 'seo')->addSelect('seo')
54		->andWhere('seo.sitemapIndexed = :sitemapIndexed')
55		->setParameter('sitemapIndexed', $indexed);
56
57		return $this;
58		}
59
60		/**
61		* Get all rentals in the repository.
62		*
63		* @param bool $excludeUnpublished Should we get only the published Views ?
64		*
65		* @return ViewRepository
66		*/
67		public function getAll($excludeUnpublished = false)
68		{
69		$this->queryBuilder = $this->getInstance();
70
71		//If $excludeUnpublished === true, we exclude the non published results
72	View Code Duplication	if ($excludeUnpublished) {
		0 ignored issues – show Duplication introduced 2016-02-24 23:56 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
73		$this->queryBuilder
74		->andWhere('page.status = :status')
75		->orWhere('page.status = :scheduled_status AND page.publishedAt > :publicationDate')
76		->setParameter('status', PageStatus::PUBLISHED)
77		->setParameter('scheduled_status', PageStatus::SCHEDULED)
78		->setParameter('publicationDate', new \DateTime());
79		}
80
81		return $this;
82		}
83
84		/**
85		* Run instance.
86		*
87		* @param string $method
88		* @param string $hydrationMode
89		*
90		* @return array
91		*/
92		public function run($method = 'getResult', $hydrationMode = Query::HYDRATE_OBJECT)
93		{
94		return $this->getInstance()->getQuery()->$method($hydrationMode);
95		}
96
97		/**
98		* Find a large amount of views by ViewReferences and optimizing queries with translation walker.
99		*
100		* @param ViewReference[] $viewReferences
101		*
102		* @return View[]\|null The entity instance or NULL if the entities cannot be found.
103		*/
104		public function findByViewReferences(array $viewReferences)
105		{
106		$views = [];
107		$localizedViewReferences = [];
108		foreach ($viewReferences as $viewReference) {
109		if (array_key_exists($viewReference->getLocale(), $localizedViewReferences)) {
110		$localizedViewReferences[$viewReference->getLocale()] = [];
111		}
112		$localizedViewReferences[$viewReference->getLocale()][] = $viewReference;
113		}
114
115		foreach ($localizedViewReferences as $locale => $_viewReferences) {
116		//the query builder
117		$queryBuilder = $this->createQueryBuilder('page');;
118
119		$pageIds = [];
120		foreach ($_viewReferences as $viewReference) {
121		if ($viewReference instanceof BusinessPageReference) {
122		$pageIds[] = $viewReference->getTemplateId();
123		} else {
124		$pageIds[] = $viewReference->getViewId();
125		}
126		}
127
128		$queryBuilder->andWhere('page.id IN (:pageIds)')
129		->setParameter('pageIds', $pageIds);
130
131
132		// Use Translation Walker
133		$query = $queryBuilder->getQuery();
134		$query->setHint(
135		\Doctrine\ORM\Query::HINT_CUSTOM_OUTPUT_WALKER,
136		'Gedmo\\Translatable\\Query\\TreeWalker\\TranslationWalker'
137		);
138
139		// Force the locale
140		$query->setHint(
141		\Gedmo\Translatable\TranslatableListener::HINT_TRANSLATABLE_LOCALE,
142		$locale
143		);
144
145		/** @var View[] $query */
146		$_views = $query->getResult();
		0 ignored issues – show Bug introduced 2016-04-01 08:04 UTC by Report Bug Copy Issue Report The method `getResult` cannot be called on `$query` (of type `array<integer,object<Vic...oreBundle\Entity\View>>`). Methods can only be called on objects. This check looks for methods being called on variables that have been inferred to never be objects. Loading history...
147		//Parse views to set locale and reference
148		$views = array_merge($views, $_views);
149		}
150
151		return $views;
152		}
153
154		/**
155		* Finds a single entity by a set of criteria.
156		*
157		* @param array $criteria
158		* @param array\|null $orderBy
159		*
160		* @return object\|null The entity instance or NULL if the entity can not be found.
161		*/
162		public function findOneBy(array $criteria, array $orderBy = null)
163		{
164		$persister = $this->_em->getUnitOfWork()->getEntityPersister($this->_entityName);
165
166		$hints = [];
167		if (isset($criteria['locale'])) {
168		$hints = [
169		\Doctrine\ORM\Query::HINT_CUSTOM_OUTPUT_WALKER => 'Gedmo\\Translatable\\Query\\TreeWalker\\TranslationWalker',
170		\Gedmo\Translatable\TranslatableListener::HINT_TRANSLATABLE_LOCALE => $criteria['locale']
171		];
172		unset($criteria['locale']);
173		}
174
175		return $persister->load($criteria, null, null, $hints, null, 1, $orderBy);
176		}
177
178		/**
179		* Get the the view that is a homepage and a published one.
180		*
181		* @param string $locale
182		*
183		* @return Page
184		*/
185		public function findOneByHomepage($locale = 'fr')
186		{
187		//the query builder
188		$queryBuilder = $this->createQueryBuilder('page');
189
190		$queryBuilder
191		->where('page.homepage = true')
192		->andWhere('page.status = :status')
193		->setMaxResults(1)
194		->setParameter('status', PageStatus::PUBLISHED);
195
196		// Use Translation Walker
197		$query = $queryBuilder->getQuery();
198		$query->setHint(
199		\Doctrine\ORM\Query::HINT_CUSTOM_OUTPUT_WALKER,
200		'Gedmo\\Translatable\\Query\\TreeWalker\\TranslationWalker'
201		);
202		// Force the locale
203		$query->setHint(
204		\Gedmo\Translatable\TranslatableListener::HINT_TRANSLATABLE_LOCALE,
205		$locale
206		);
207
208		$view = $query->getOneOrNullResult();
209
210		return $view;
211		}
212
213		/**
214		* Get PageSeo.
215		*
216		* @param string $method leftJoin\|innerJoin
217		*
218		* @return ViewRepository
219		*/
220		public function joinSeo($method = 'leftJoin')
221		{
222		$this->getInstance()->$method('page.seo', 'seo')->addSelect('seo');
223
224		return $this;
225		}
226
227		/**
228		* Filter the query by the sitemap index (=visibility).
229		*
230		* @param array $ids
231		*
232		* @return ViewRepository
233		*/
234		public function filterByIds($ids)
235		{
236		$this->getInstance()->andWhere('page.id IN (:ids)')->setParameter('ids', $ids);
237
238		return $this;
239		}
240		}
241

Victoire / victoire

Pull Request — master (#374)

ViewRepository::findOneByHomepage() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like