Kunstmaan\AdminListBundle\Tests\unit\Model\ConcreteConfigurator

Complexity

Total Complexity

9

Size/Duplication

Total Lines	88
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	getBundleName()	0	4	1
A	getEntityName()	0	4	1
A	buildFields()	0	4	1
A	getEditUrlFor()	0	9	1
A	getDeleteUrlFor()	0	9	1
A	getCount()	0	4	1
A	getItems()	0	6	1
A	getPagerfanta()	0	4	1
A	getIterator()	0	4	1

<?php
namespace Kunstmaan\AdminListBundle\Tests\unit\Model;

use ArrayIterator;
use Kunstmaan\AdminListBundle\AdminList\Configurator\AbstractAdminListConfigurator;
use Pagerfanta\Pagerfanta;

class ConcreteConfigurator extends AbstractAdminListConfigurator
{
    /**
     * @return string
     */
    public function getBundleName()
    {
        return 'xyz';
    }

    /**
     * @return string
     */
    public function getEntityName()
    {
        return 'Xyz';
    }

    /**
     * @return mixed

Consider making the return type a bit more specific; maybe use boolean.

     */
    public function buildFields()
    {
        return true;
    }

    /**
     * @param array|object $item
     * @return array

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

     */
    public function getEditUrlFor($item)
    {
        return [
            'Xyz' =>  [
                'path' => 'xyz_admin_xyz_edit',
                'params' => [],
            ]
        ];
    }

    /**
     * @param array|object $item
     * @return array

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

     */
    public function getDeleteUrlFor($item)
    {
        return [
            'Xyz' =>  [
                'path' => 'xyz_admin_xyz_delete',
                'params' => [],
            ]
        ];
    }

    /**
     * @return int

Should the return type not be integer|null?

     */
    public function getCount()
    {
        // TODO: Implement getCount() method.
    }

    /**
     * @return mixed

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

     */
    public function getItems()
    {
        return [
            'some' => 'item',
        ];
    }

    /**
     * @return Pagerfanta

Should the return type not be string?

     */
    public function getPagerfanta()
    {
        return 'fake pagerfanta';
    }

    /**
     * @return mixed

Consider making the return type a bit more specific; maybe use ArrayIterator.

     */
    public function getIterator()
    {
        return new ArrayIterator();
    }
}