PageRepository::findFrontPages() - Code Metrics - Orbitale/CmsBundle - Measure and Improve Code Quality continuously with Scrutinizer

PageRepository::findFrontPages() C
last analyzed 2017-06-15 07:33 UTC

↳ Parent: PageRepository

Complexity

Conditions	7
Paths	32

Size

Total Lines	68
Code Lines	41

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	68
rs	6.9654
c	0
b	0
f	0
cc	7
eloc	41
nc	32
nop	3

How to fix Long Method

<?php

/*
* This file is part of the OrbitaleCmsBundle package.
*
* (c) Alexandre Rock Ancelet <[email protected]>
*
* For the full copyright and license information, please view the LICENSE
* file that was distributed with this source code.
*/

namespace Orbitale\Bundle\CmsBundle\Repository;

use Doctrine\ORM\Tools\Pagination\Paginator;
use Orbitale\Bundle\CmsBundle\Entity\Category;
use Orbitale\Bundle\CmsBundle\Entity\Page;

class PageRepository extends AbstractCmsRepository
{
    /**
     * @param Category $category
     * @param string   $order
     * @param string   $orderBy
     * @param int      $page
     * @param int      $limit
     *
     * @return Paginator
     */
    public function findByCategory(Category $category, $order, $orderBy, $page, $limit): Paginator
    {
        $qb = $this->createQueryBuilder('page')
            ->where('page.category = :category')
            ->orderBy('page.'.$orderBy, $order)
            ->setMaxResults($limit)
            ->setFirstResult($limit * ($page-1))
            ->setParameter('category', $category)
        ;

        return new Paginator($qb->getQuery()->useResultCache($this->cacheEnabled, $this->cacheTtl));
    }

    /**
     * Will search for pages to show in front depending on the arguments.
     * If slugs are defined, there's no problem in looking for nulled host or locale,
     * because slugs are unique, so it does not.
     *
     * @param array       $slugs
     * @param string|null $host
     * @param string|null $locale
     *
     * @return Page[]
     */
    public function findFrontPages(array $slugs = [], $host = null, $locale = null)
    {
        $qb = $this->createQueryBuilder('page')
            ->where('page.enabled = :enabled')
            ->leftJoin('page.category', 'category')
            ->andWhere('page.category is null OR category.enabled = :enabled')
            ->setParameter('enabled', true)
        ;

        // Will search differently if we're looking for homepage.
        $searchForHomepage = 0 === count($slugs);

        if (true === $searchForHomepage) {
            // If we are looking for homepage, let's get only the first one.
            $qb
                ->andWhere('page.homepage = :homepage')
                ->setParameter('homepage', true)
                ->setMaxResults(1)
            ;
        } else {
            $qb
                ->andWhere('page.slug IN ( :slugs )')
                ->setParameter('slugs', $slugs)
            ;
        }

        $hostWhere = 'page.host IS NULL';
        if (null !== $host) {
            $hostWhere .= ' OR page.host = :host';
            $qb->setParameter('host', $host);
            $qb->addOrderBy('page.host', 'asc');
        }
        $qb->andWhere($hostWhere);

        $localeWhere = 'page.locale IS NULL';
        if (null !== $locale) {
            $localeWhere .= ' OR page.locale = :locale';
            $qb->setParameter('locale', $locale);
            $qb->addOrderBy('page.locale', 'asc');
        }
        $qb->andWhere($localeWhere);

        // Then the last page will automatically be one that has both properties.
        $qb
            ->orderBy('page.host', 'asc')
            ->addOrderBy('page.locale', 'asc')
        ;

        /** @var Page[] $results */
        $results = $qb->getQuery()
            ->useResultCache($this->cacheEnabled, $this->cacheTtl)
            ->getResult()
        ;

        // If we're looking for a homepage, only get the last result (matching more properties).
        if (true === $searchForHomepage && count($results) > 0) {
            reset($results);
            $results = [$results[0]];
        }

        $resultsSortedBySlug = [];

        foreach ($results as $page) {
            $resultsSortedBySlug[$page->getSlug()] = $page;
        }

        return $resultsSortedBySlug;
    }
}


1			<?php
2
3			/*
4			* This file is part of the OrbitaleCmsBundle package.
5			*
6			* (c) Alexandre Rock Ancelet <[email protected]>
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			namespace Orbitale\Bundle\CmsBundle\Repository;
13
14			use Doctrine\ORM\Tools\Pagination\Paginator;
15			use Orbitale\Bundle\CmsBundle\Entity\Category;
16			use Orbitale\Bundle\CmsBundle\Entity\Page;
17
18			class PageRepository extends AbstractCmsRepository
19			{
20			/**
21			* @param Category $category
22			* @param string $order
23			* @param string $orderBy
24			* @param int $page
25			* @param int $limit
26			*
27			* @return Paginator
28			*/
29			public function findByCategory(Category $category, $order, $orderBy, $page, $limit): Paginator
30			{
31			$qb = $this->createQueryBuilder('page')
32			->where('page.category = :category')
33			->orderBy('page.'.$orderBy, $order)
34			->setMaxResults($limit)
35			->setFirstResult($limit * ($page-1))
36			->setParameter('category', $category)
37			;
38
39			return new Paginator($qb->getQuery()->useResultCache($this->cacheEnabled, $this->cacheTtl));
40			}
41
42			/**
43			* Will search for pages to show in front depending on the arguments.
44			* If slugs are defined, there's no problem in looking for nulled host or locale,
45			* because slugs are unique, so it does not.
46			*
47			* @param array $slugs
48			* @param string\|null $host
49			* @param string\|null $locale
50			*
51			* @return Page[]
52			*/
53			public function findFrontPages(array $slugs = [], $host = null, $locale = null)
54			{
55			$qb = $this->createQueryBuilder('page')
56			->where('page.enabled = :enabled')
57			->leftJoin('page.category', 'category')
58			->andWhere('page.category is null OR category.enabled = :enabled')
59			->setParameter('enabled', true)
60			;
61
62			// Will search differently if we're looking for homepage.
63			$searchForHomepage = 0 === count($slugs);
64
65			if (true === $searchForHomepage) {
66			// If we are looking for homepage, let's get only the first one.
67			$qb
68			->andWhere('page.homepage = :homepage')
69			->setParameter('homepage', true)
70			->setMaxResults(1)
71			;
72			} else {
73			$qb
74			->andWhere('page.slug IN ( :slugs )')
75			->setParameter('slugs', $slugs)
76			;
77			}
78
79			$hostWhere = 'page.host IS NULL';
80			if (null !== $host) {
81			$hostWhere .= ' OR page.host = :host';
82			$qb->setParameter('host', $host);
83			$qb->addOrderBy('page.host', 'asc');
84			}
85			$qb->andWhere($hostWhere);
86
87			$localeWhere = 'page.locale IS NULL';
88			if (null !== $locale) {
89			$localeWhere .= ' OR page.locale = :locale';
90			$qb->setParameter('locale', $locale);
91			$qb->addOrderBy('page.locale', 'asc');
92			}
93			$qb->andWhere($localeWhere);
94
95			// Then the last page will automatically be one that has both properties.
96			$qb
97			->orderBy('page.host', 'asc')
98			->addOrderBy('page.locale', 'asc')
99			;
100
101			/** @var Page[] $results */
102			$results = $qb->getQuery()
103			->useResultCache($this->cacheEnabled, $this->cacheTtl)
104			->getResult()
105			;
106
107			// If we're looking for a homepage, only get the last result (matching more properties).
108			if (true === $searchForHomepage && count($results) > 0) {
109			reset($results);
110			$results = [$results[0]];
111			}
112
113			$resultsSortedBySlug = [];
114
115			foreach ($results as $page) {
116			$resultsSortedBySlug[$page->getSlug()] = $page;
117			}
118
119			return $resultsSortedBySlug;
120			}
121			}
122

Orbitale / CmsBundle

PageRepository::findFrontPages() C last analyzed 2017-06-15 07:33 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like

PageRepository::findFrontPages() C
last analyzed 2017-06-15 07:33 UTC