Kunstmaan\ArticleBundle\AdminList\AbstractArticlePageAdminListConfigurator

Complexity

Total Complexity

15

Size/Duplication

Total Lines	181
Duplicated Lines	17.68 %

Coupling/Cohesion

Components	2
Dependencies	10

Importance

Changes

0

14 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	8	8	1
A	getBundleName()	0	4	1
A	getEntityName()	0	4	1
A	buildFilters()	7	7	1
A	buildFields()	7	7	1
A	getQueryBuilder()	0	8	1
A	adaptQueryBuilder()	0	12	1
A	getEditUrlFor()	10	10	1
A	getDeleteUrlFor()	0	10	1
A	getOverviewPage()	0	11	3
A	getOverviewPages()	0	4	1
A	getListTemplate()	0	4	1
A	getEntityClassName()	0	4	1
	getOverviewPageRepository()	0	1	?

<?php

namespace Kunstmaan\ArticleBundle\AdminList;

use Doctrine\ORM\EntityManager;
use Doctrine\ORM\QueryBuilder;
use Kunstmaan\AdminBundle\Helper\Security\Acl\AclHelper;
use Kunstmaan\AdminBundle\Helper\Security\Acl\Permission\PermissionDefinition;
use Kunstmaan\AdminListBundle\AdminList\Configurator\AbstractDoctrineORMAdminListConfigurator;
use Kunstmaan\AdminListBundle\AdminList\FilterType\ORM\BooleanFilterType;
use Kunstmaan\AdminListBundle\AdminList\FilterType\ORM\DateFilterType;
use Kunstmaan\AdminListBundle\AdminList\FilterType\ORM\StringFilterType;
use Kunstmaan\ArticleBundle\Entity\AbstractArticleOverviewPage;
use Kunstmaan\NodeBundle\Entity\Node;

/**
 * The AdminList configurator for the AbstractArticlePage
 */
abstract class AbstractArticlePageAdminListConfigurator extends AbstractDoctrineORMAdminListConfigurator
{
    /**
     * @var string
     */
    protected $locale;

    /**
     * @var string
     */
    protected $permission;

    /**
     * @param EntityManager $em         The entity manager
     * @param AclHelper     $aclHelper  The ACL helper
     * @param string        $locale     The current locale
     * @param string        $permission The permission
     */
    public function __construct(EntityManager $em, AclHelper $aclHelper, $locale, $permission)

This method seems to be duplicated in your project.

    {
        parent::__construct($em, $aclHelper);
        $this->locale = $locale;
        $this->setPermissionDefinition(
            new PermissionDefinition(array($permission), 'Kunstmaan\NodeBundle\Entity\Node', 'n')
        );
    }

    /**
     * Return current bundle name.
     *
     * @return string
     */
    public function getBundleName()
    {
        return 'KunstmaanArticleBundle';
    }

    /**
     * Return current entity name.
     *
     * @return string
     */
    public function getEntityName()
    {
        return 'AbstractArticlePage';
    }

    /**
     * Configure filters
     */
    public function buildFilters()

This method seems to be duplicated in your project.

    {
        $this->addFilter('title', new StringFilterType('title'), 'article.page.list.filter.title')
            ->addFilter('online', new BooleanFilterType('online'), 'article.page.list.filter.online')
            ->addFilter('created', new DateFilterType('created', 'nv'), 'article.page.list.filter.created_at')
            ->addFilter('updated', new DateFilterType('updated', 'nv'), 'article.page.list.filter.updated_at');
    }

    /**
     * Configure the visible columns
     */
    public function buildFields()

This method seems to be duplicated in your project.

    {
        $this->addField('title', 'article.page.list.header.title', true, 'KunstmaanNodeBundle:Admin:title.html.twig')

true is of type boolean, but the function expects a string.

            ->addField('created', 'article.page.list.header.created_at', true)

true is of type boolean, but the function expects a string.

            ->addField('updated', 'article.page.list.header.updated_at', true)

true is of type boolean, but the function expects a string.

            ->addField('online', 'article.page.list.header.online', true, 'KunstmaanNodeBundle:Admin:online.html.twig');

true is of type boolean, but the function expects a string.

    }

    /**
     * @return QueryBuilder
     */
    protected function getQueryBuilder()
    {
        $queryBuilder = $this->em
            ->getRepository('KunstmaanNodeBundle:NodeTranslation')
            ->createQueryBuilder('b');

        return $queryBuilder;
    }

    /**
     * @param QueryBuilder $queryBuilder The query builder
     */
    public function adaptQueryBuilder(QueryBuilder $queryBuilder)
    {
        parent::adaptQueryBuilder($queryBuilder);

        $queryBuilder->innerJoin('b.node', 'n', 'WITH', 'b.node = n.id');
        $queryBuilder->innerJoin('b.nodeVersions', 'nv', 'WITH', 'b.publicNodeVersion = nv.id');
        $queryBuilder->andWhere('b.lang = :lang');
        $queryBuilder->andWhere('n.deleted = 0');
        $queryBuilder->andWhere('n.refEntityName = :class');
        $queryBuilder->addOrderBy('b.updated', 'DESC');
        $queryBuilder->setParameter('lang', $this->locale);
    }

    /**
     * @param mixed $item
     *
     * @return array

Consider making the return type a bit more specific; maybe use array<string,string|array<string,integer>>.

     */
    public function getEditUrlFor($item)

This method seems to be duplicated in your project.

    {
        /* @var Node $node */
        $node = $item->getNode();

        return array(
            'path' => 'KunstmaanNodeBundle_nodes_edit',
            'params' => array('id' => $node->getId()),
        );
    }

    /**
     * Get the delete url for the given $item
     *
     * @param object $item
     *
     * @return array

Consider making the return type a bit more specific; maybe use array<string,string|array<string,integer>>.

     */
    public function getDeleteUrlFor($item)
    {
        /* @var Node $node */
        $node = $item->getNode();

        return array(
            'path' => 'KunstmaanNodeBundle_nodes_delete',
            'params' => array('id' => $node->getId()),
        );
    }

    /**
     * Returns the OverviewPage of these articles
     *
     * @return AbstractArticleOverviewPage
     */
    public function getOverviewPage()
    {
        $repository = $this->getOverviewPageRepository();
        $pages = $repository->findActiveOverviewPages();

        if (isset($pages) && count($pages) > 0) {
            return $pages[0];
        }

        return null;
    }

    /**
     * Returns all overview pages
     *
     * @return mixed
     */
    public function getOverviewPages()
    {
        return $this->getOverviewPageRepository()->findActiveOverviewPages();
    }

    /**
     * @return string
     */
    public function getListTemplate()
    {
        return 'KunstmaanArticleBundle:AbstractArticlePageAdminList:list.html.twig';
    }

    /**
     * Returns the full entity class name
     *
     * @return string
     */
    public function getEntityClassName()
    {
        return $this->em->getRepository($this->getRepositoryName())->getClassName();
    }

    /**
     * @return \Doctrine\ORM\EntityRepository
     */
    abstract public function getOverviewPageRepository();
}