Configuration::getConfigTreeBuilder() - Code Metrics - Inspection of "Fixed #1, you can exclude services from injection..." - RunOpenCode/traitor-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — master (2f8908)

by Nikola

created 2016-06-03 10:49 UTC

Configuration::getConfigTreeBuilder() C

↳ Parent: Configuration

Complexity

Conditions	7
Paths	1

Size

Total Lines	80
Code Lines	63

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	54
CRAP Score	7.1867

Importance

Changes	3
Bugs	1	Features	1

Metric	Value
c	3
b	1
f	1
dl	0
loc	80
ccs	54
cts	64
cp	0.8438
rs	6.5597
cc	7
eloc	63
nc	1
nop	0
crap	7.1867

How to fix Long Method

<?php
/*
 * This file is part of the  TraitorBundle, an RunOpenCode project.
 *
 * (c) 2016 RunOpenCode
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
namespace RunOpenCode\Bundle\Traitor\DependencyInjection;

use Symfony\Component\Config\Definition\Builder\TreeBuilder;
use Symfony\Component\Config\Definition\ConfigurationInterface;

class Configuration implements ConfigurationInterface
{
    /**
     * {@inheritdoc}
     */
    public function getConfigTreeBuilder()
    {
        $treeBuilder = new TreeBuilder();

        $rootNode = $treeBuilder->root('run_open_code_traitor');

        $rootNode
            ->addDefaultsIfNotSet()
            ->children()
                ->booleanNode('use_common_traits')
                    ->defaultFalse()
                    ->info('For sake of productivity, some of the common Symfony and vendor traits, as well as traits from this library, can be automatically added to "inject" definition.')
                ->end()
                ->arrayNode('inject')
                    ->useAttributeAsKey('trait')
                    ->prototype('array')
                        ->validate()
                            ->ifTrue(function($value) {

                                if (!is_array($value) || 2 !== count($value)) {
                                    return true;
                                }

                                if (!is_string($value[0])) {
                                    return true;
                                }

                                if (!is_array($value[1])) {
                                    return true;
                                }

                                foreach ($value[1] as $arg) {

                                    if (!is_string($arg)) {
                                        return true;
                                    }
                                }

                                return false;
                            })
                            ->thenInvalid('Expected proper setter injection definition.')
                        ->end()
                        ->prototype('variable')
                        ->end()
                    ->end()
                ->end()
                ->arrayNode('filters')
                    ->info('Analyse services for injection which matches filter criteria.')
                    ->children()
                        ->arrayNode('tags')
                            ->prototype('scalar')
                            ->end()
                        ->end()
                        ->arrayNode('namespaces')
                            ->prototype('scalar')
                            ->end()
                        ->end()
                    ->end()
                ->end()
                ->arrayNode('exclude')
                    ->info('Exclude services from injection which matches filter criteria.')
                    ->children()
                        ->arrayNode('services')
                            ->prototype('scalar')
                            ->end()
                        ->end()
                        ->arrayNode('namespaces')
                            ->prototype('scalar')
                            ->end()
                        ->end()
                        ->arrayNode('classes')
                            ->prototype('scalar')
                            ->end()
                        ->end()
                    ->end()
                ->end()
            ->end();

        return $treeBuilder;
    }
}


1		<?php
2		/*
3		* This file is part of the TraitorBundle, an RunOpenCode project.
4		*
5		* (c) 2016 RunOpenCode
6		*
7		* For the full copyright and license information, please view the LICENSE
8		* file that was distributed with this source code.
9		*/
10		namespace RunOpenCode\Bundle\Traitor\DependencyInjection;
11
12		use Symfony\Component\Config\Definition\Builder\TreeBuilder;
13		use Symfony\Component\Config\Definition\ConfigurationInterface;
14
15		class Configuration implements ConfigurationInterface
16		{
17		/**
18		* {@inheritdoc}
19		*/
20	4	public function getConfigTreeBuilder()
21		{
22	4	$treeBuilder = new TreeBuilder();
23
24	4	$rootNode = $treeBuilder->root('run_open_code_traitor');
25
26		$rootNode
27	4	->addDefaultsIfNotSet()
28	4	->children()
29	4	->booleanNode('use_common_traits')
30	4	->defaultFalse()
31	4	->info('For sake of productivity, some of the common Symfony and vendor traits, as well as traits from this library, can be automatically added to "inject" definition.')
32	4	->end()
33	4	->arrayNode('inject')
34	4	->useAttributeAsKey('trait')
35	4	->prototype('array')
36	4	->validate()
37	4	->ifTrue(function($value) {
38
39		if (!is_array($value) \|\| 2 !== count($value)) {
40	2	return true;
41		}
42
43		if (!is_string($value[0])) {
44		return true;
45		}
46
47		if (!is_array($value[1])) {
48		return true;
49		}
50
51		foreach ($value[1] as $arg) {
52
53		if (!is_string($arg)) {
54		return true;
55		}
56		}
57
58		return false;
59	4	})
60	4	->thenInvalid('Expected proper setter injection definition.')
61	4	->end()
62	4	->prototype('variable')
63	4	->end()
64	4	->end()
65	4	->end()
66	4	->arrayNode('filters')
67	4	->info('Analyse services for injection which matches filter criteria.')
68	4	->children()
69	4	->arrayNode('tags')
70	4	->prototype('scalar')
71	4	->end()
72	4	->end()
73	4	->arrayNode('namespaces')
74	4	->prototype('scalar')
75	4	->end()
76	4	->end()
77	4	->end()
78	4	->end()
79	4	->arrayNode('exclude')
80	4	->info('Exclude services from injection which matches filter criteria.')
81	4	->children()
82	4	->arrayNode('services')
83	4	->prototype('scalar')
84	4	->end()
85	4	->end()
86	4	->arrayNode('namespaces')
87	4	->prototype('scalar')
88	4	->end()
89	4	->end()
90	4	->arrayNode('classes')
91	4	->prototype('scalar')
92	4	->end()
93	4	->end()
94	4	->end()
95	4	->end()
96	4	->end();
97
98	4	return $treeBuilder;
99		}
100		}
101

RunOpenCode / traitor-bundle

Branch — master (2f8908)

Configuration::getConfigTreeBuilder() C

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like