Blast\Bundle\CoreBundle\Admin\Traits\Base

Complexity

Total Complexity

12

Size/Duplication

Total Lines	108
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	configureDatagridFilters()	0	4	1
A	configureListFields()	0	4	1
A	configureFormFields()	0	4	1
A	configureShowFields()	0	4	1
A	getBatchActions()	0	14	2
A	getExportFormats()	0	9	2
B	getExportFields()	0	22	4

<?php

/*
 *
 * Copyright (C) 2015-2017 Libre Informatique
 *
 * This file is licenced under the GNU LGPL v3.
 * For the full copyright and license information, please view the LICENSE.md
 * file that was distributed with this source code.
 */

namespace Blast\Bundle\CoreBundle\Admin\Traits;

use Blast\Bundle\CoreBundle\Admin\CoreAdmin;
use Blast\Bundle\CoreBundle\DataSource\Iterator;
use Sonata\AdminBundle\Datagrid\DatagridMapper;
use Sonata\AdminBundle\Datagrid\ListMapper;
use Sonata\AdminBundle\Form\FormMapper;
use Sonata\AdminBundle\Show\ShowMapper;

trait Base
{
    /**
     * @var array
     */
    protected $exportFields = [];

    /**
     * @param DatagridMapper $mapper
     */
    protected function configureDatagridFilters(DatagridMapper $mapper)
    {
        CoreAdmin::configureDatagridFilters($mapper);

The method configureDatagridFilters() cannot be called from this context as it is declared protected in class Blast\Bundle\CoreBundle\Admin\CoreAdmin.

    }

    /**
     * @param ListMapper $mapper
     */
    protected function configureListFields(ListMapper $mapper)
    {
        CoreAdmin::configureListFields($mapper);

The method configureListFields() cannot be called from this context as it is declared protected in class Blast\Bundle\CoreBundle\Admin\CoreAdmin.

    }

    /**
     * @param FormMapper $mapper
     */
    protected function configureFormFields(FormMapper $mapper)
    {
        CoreAdmin::configureFormFields($mapper);

The method configureFormFields() cannot be called from this context as it is declared protected in class Blast\Bundle\CoreBundle\Admin\CoreAdmin.

    }

    /**
     * @param ShowMapper $mapper
     */
    protected function configureShowFields(ShowMapper $mapper)
    {
        CoreAdmin::configureShowFields($mapper);

The method configureShowFields() cannot be called from this context as it is declared protected in class Blast\Bundle\CoreBundle\Admin\CoreAdmin.

    }

    /**
     * {@inheritdoc}
     **/
    public function getBatchActions()
    {
        $actions = [];

        if ($this->isGranted('DELETE')) {

It seems like isGranted() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $actions['delete'] = array(
               'label'              => 'action_delete',
               'translation_domain' => 'SonataAdminBundle',
               'ask_confirmation'   => true,
           );
        }

        return $this->handleBatchActions($actions);

It seems like handleBatchActions() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * {@inheritdoc}
     **/
    public function getExportFormats()
    {
        $formats = [];
        foreach (parent::getExportFormats() as $format) {
            $formats[$format] = [];
        }

        return array_keys($this->addPresetExportFormats($formats));

It seems like addPresetExportFormats() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * {@inheritdoc}
     */
    public function getExportFields()
    {
        // prerequisites
        $fields = parent::getExportFields();

$fields is not used, you could remove the assignment.

        $this->getExportFormats();

        // nothing to add
        if (!$this->exportFields) {
            return parent::getExportFields();
        }

        $request = $this->getConfigurationPool()->getContainer()->get('request_stack')->getMasterRequest();

It seems like getConfigurationPool() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        // nothing specific to add
        if (!($request->get('format')
            && isset($this->exportFields[$request->get('format')]))) {
            return parent::getExportFields();
        }

        // specificities for this format
        return $this->exportFields[$request->get('format')];
    }

    // /**
    //  * {@inheritdoc}
    //  */
    // public function getDataSourceIterator()

50% of this comment could be valid code. Did you maybe forget this after debugging?

    // {
    //     $datagrid = $this->getDatagrid();

55% of this comment could be valid code. Did you maybe forget this after debugging?

    //     $datagrid->buildPager();

72% of this comment could be valid code. Did you maybe forget this after debugging?

    //
    //     return $this->getModelManager()->getDataSourceIterator($datagrid, $this->getExportFields());

72% of this comment could be valid code. Did you maybe forget this after debugging?

    //     // return new Iterator($datagrid->getQuery()->getQuery(), $this->getExportFields());
    // }
}