ArrayDriver::loadMetadataForClass() - Code Metrics - Inspection of "Merge pull request #42 from psiphp/views" - psiphp/grid - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( fa25fc...8f184f )

by Daniel

created 2016-12-11 17:07 UTC

ArrayDriver::loadMetadataForClass() C

↳ Parent: ArrayDriver

Complexity

Conditions	7
Paths	19

Size

Total Lines	93
Code Lines	68

Duplication

Lines	24
Ratio	25.81 %

Importance

Changes

Metric	Value
dl	24
loc	93
rs	6.4932
c	0
b	0
f	0
cc	7
eloc	68
nc	19
nop	1

How to fix Long Method

<?php

declare(strict_types=1);

namespace Psi\Component\Grid\Metadata\Driver;

use Metadata\Driver\AdvancedDriverInterface;
use Psi\Component\Grid\Metadata\ActionMetadata;
use Psi\Component\Grid\Metadata\ClassMetadata;
use Psi\Component\Grid\Metadata\ColumnMetadata;
use Psi\Component\Grid\Metadata\FilterMetadata;
use Psi\Component\Grid\Metadata\GridMetadata;
use Psi\Component\Grid\Metadata\QueryMetadata;

class ArrayDriver implements AdvancedDriverInterface
{
    /**
     * @var array
     */
    private $config;

    /**
     * @param array $config
     */
    public function __construct(array $config)
    {
        $this->config = $config;
    }

    /**
     * {@inheritdoc}
     */
    public function loadMetadataForClass(\ReflectionClass $class)
    {
        if (!isset($this->config[$class->getName()])) {

            return;
        }

        $config = $this->resolveConfig([
            'grids' => [],
            'queries' => [],
        ], $this->config[$class->getName()]);


        $queries = [];
        foreach ($config['queries'] as $queryName => $queryConfig) {
            $queryConfig = $this->resolveConfig([
                'name' => null,
                'selects' => [],
                'joins' => [],
                'criteria' => [],
            ], $queryConfig);
            $queries[$queryName] = new QueryMetadata(
                $queryName, $queryConfig['selects'], $queryConfig['joins'], $queryConfig['criteria']
            );
        }

        $grids = [];
        foreach ($config['grids'] as $gridName => $gridConfig) {
            $gridConfig = $this->resolveConfig([
                'name' => null,
                'query' => null,
                'columns' => [],
                'filters' => [],
                'actions' => [],
                'page_size' => 50,
            ], $gridConfig);

            $columns = [];
            foreach ($gridConfig['columns'] as $columnName => $columnConfig) {

                $columnConfig = $this->resolveConfig([
                    'type' => null,
                    'options' => [],
                ], $columnConfig);

                $columns[$columnName] = new ColumnMetadata(
                    $columnName,
                    $columnConfig['type'],
                    $columnConfig['options']
                );
            }

            $filters = [];
            foreach ($gridConfig['filters'] as $filterName => $filterConfig) {
                $filterConfig = $this->resolveConfig([
                    'type' => null,
                    'field' => null,
                    'options' => [],
                ], $filterConfig);

                $filters[$filterName] = new FilterMetadata(
                    $filterName,
                    $filterConfig['type'],
                    $filterConfig['field'],
                    $filterConfig['options']
                );
            }

            $actions = [];
            foreach ($gridConfig['actions'] as $actionName => $actionConfig) {

                $actionConfig = $this->resolveConfig([
                    'type' => null,
                    'options' => [],
                ], $actionConfig);

                $actions[$actionName] = new ActionMetadata(
                    $actionName,
                    $actionConfig['type'],
                    $actionConfig['options']
                );
            }

            $grids[$gridName] = new GridMetadata(
                $gridName,
                $columns,
                $filters,
                $actions,
                $gridConfig['page_size'],
                $gridConfig['query']
            );
        }

        $classMetadata = new ClassMetadata($class->getName(), $grids, $queries);


        return $classMetadata;
    }

    /**
     * {@inheritdoc}
     */
    public function getAllClassNames()
    {
        return array_keys($this->config);
    }

    private function resolveConfig(array $defaultConfig, array $config)
    {
        if ($diff = array_diff(array_keys($config), array_keys($defaultConfig))) {

            throw new \InvalidArgumentException(sprintf(
                'Invalid configuration keys "%s" for grid, valid keys: "%s"',
                implode('", "', $diff), implode('", "', array_keys($defaultConfig))
            ));
        }

        return array_merge($defaultConfig, $config);
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Psi\Component\Grid\Metadata\Driver;
6
7		use Metadata\Driver\AdvancedDriverInterface;
8		use Psi\Component\Grid\Metadata\ActionMetadata;
9		use Psi\Component\Grid\Metadata\ClassMetadata;
10		use Psi\Component\Grid\Metadata\ColumnMetadata;
11		use Psi\Component\Grid\Metadata\FilterMetadata;
12		use Psi\Component\Grid\Metadata\GridMetadata;
13		use Psi\Component\Grid\Metadata\QueryMetadata;
14
15		class ArrayDriver implements AdvancedDriverInterface
16		{
17		/**
18		* @var array
19		*/
20		private $config;
21
22		/**
23		* @param array $config
24		*/
25		public function __construct(array $config)
26		{
27		$this->config = $config;
28		}
29
30		/**
31		* {@inheritdoc}
32		*/
33		public function loadMetadataForClass(\ReflectionClass $class)
34		{
35		if (!isset($this->config[$class->getName()])) {
		0 ignored issues – show Bug introduced 2016-11-06 14:07 UTC by Report Bug Copy Issue Report Consider using `$class->name`. There is an issue with `getName()` and APC-enabled PHP versions. Loading history...
36		return;
37		}
38
39		$config = $this->resolveConfig([
40		'grids' => [],
41		'queries' => [],
42		], $this->config[$class->getName()]);
		0 ignored issues – show Bug introduced 2016-11-06 14:07 UTC by Report Bug Copy Issue Report Consider using `$class->name`. There is an issue with `getName()` and APC-enabled PHP versions. Loading history...
43
44		$queries = [];
45		foreach ($config['queries'] as $queryName => $queryConfig) {
46		$queryConfig = $this->resolveConfig([
47		'name' => null,
48		'selects' => [],
49		'joins' => [],
50		'criteria' => [],
51		], $queryConfig);
52		$queries[$queryName] = new QueryMetadata(
53		$queryName, $queryConfig['selects'], $queryConfig['joins'], $queryConfig['criteria']
54		);
55		}
56
57		$grids = [];
58		foreach ($config['grids'] as $gridName => $gridConfig) {
59		$gridConfig = $this->resolveConfig([
60		'name' => null,
61		'query' => null,
62		'columns' => [],
63		'filters' => [],
64		'actions' => [],
65		'page_size' => 50,
66		], $gridConfig);
67
68		$columns = [];
69	View Code Duplication	foreach ($gridConfig['columns'] as $columnName => $columnConfig) {
		0 ignored issues – show Duplication introduced 2016-11-15 13:38 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...
70		$columnConfig = $this->resolveConfig([
71		'type' => null,
72		'options' => [],
73		], $columnConfig);
74
75		$columns[$columnName] = new ColumnMetadata(
76		$columnName,
77		$columnConfig['type'],
78		$columnConfig['options']
79		);
80		}
81
82		$filters = [];
83		foreach ($gridConfig['filters'] as $filterName => $filterConfig) {
84		$filterConfig = $this->resolveConfig([
85		'type' => null,
86		'field' => null,
87		'options' => [],
88		], $filterConfig);
89
90		$filters[$filterName] = new FilterMetadata(
91		$filterName,
92		$filterConfig['type'],
93		$filterConfig['field'],
94		$filterConfig['options']
95		);
96		}
97
98		$actions = [];
99	View Code Duplication	foreach ($gridConfig['actions'] as $actionName => $actionConfig) {
		0 ignored issues – show Duplication introduced 2016-11-15 13:38 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...
100		$actionConfig = $this->resolveConfig([
101		'type' => null,
102		'options' => [],
103		], $actionConfig);
104
105		$actions[$actionName] = new ActionMetadata(
106		$actionName,
107		$actionConfig['type'],
108		$actionConfig['options']
109		);
110		}
111
112		$grids[$gridName] = new GridMetadata(
113		$gridName,
114		$columns,
115		$filters,
116		$actions,
117		$gridConfig['page_size'],
118		$gridConfig['query']
119		);
120		}
121
122		$classMetadata = new ClassMetadata($class->getName(), $grids, $queries);
		0 ignored issues – show Bug introduced 2016-12-10 18:16 UTC by Report Bug Copy Issue Report Consider using `$class->name`. There is an issue with `getName()` and APC-enabled PHP versions. Loading history...
123
124		return $classMetadata;
125		}
126
127		/**
128		* {@inheritdoc}
129		*/
130		public function getAllClassNames()
131		{
132		return array_keys($this->config);
133		}
134
135		private function resolveConfig(array $defaultConfig, array $config)
136		{
137	View Code Duplication	if ($diff = array_diff(array_keys($config), array_keys($defaultConfig))) {
		0 ignored issues – show Duplication introduced 2016-11-06 16:25 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...
138		throw new \InvalidArgumentException(sprintf(
139		'Invalid configuration keys "%s" for grid, valid keys: "%s"',
140		implode('", "', $diff), implode('", "', array_keys($defaultConfig))
141		));
142		}
143
144		return array_merge($defaultConfig, $config);
145		}
146		}
147

psiphp / grid

Push — master ( fa25fc...8f184f )

ArrayDriver::loadMetadataForClass() C

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like