Configuration::getConfigTreeBuilder() - Code Metrics - Inspection of "GH-30 basic implementation of a annotation-based c..." - Ma27/Ma27ApiKeyAuthenticationBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#31)

by Maximilian

created 2016-01-19 18:26 UTC

Configuration::getConfigTreeBuilder() A

↳ Parent: Configuration

Complexity

Conditions	1
Paths	1

Size

Total Lines	52
Code Lines	47

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	39
CRAP Score	1

Importance

Changes	6
Bugs	1	Features	1

Metric	Value
c	6
b	1
f	1
dl	0
loc	52
ccs	39
cts	39
cp	1
rs	9.4929
cc	1
eloc	47
nc	1
nop	0
crap	1

How to fix Long Method

<?php

namespace Ma27\ApiKeyAuthenticationBundle\DependencyInjection;

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

/**
 * This is the class that validates and merges configuration from your app/config files.
 *
 * To learn more see {@link http://symfony.com/doc/current/cookbook/bundles/extension.html#cookbook-bundles-extension-config-class}
 */
class Configuration implements ConfigurationInterface
{
    /**
     * {@inheritdoc}
     */
    public function getConfigTreeBuilder()
    {
        $treeBuilder = new TreeBuilder();
        $rootNode = $treeBuilder->root('ma27_api_key_authentication');

        $rootNode
            ->children()
                ->arrayNode('user')
                    ->children()
                        ->integerNode('api_key_length')
                            ->min(50)
                            ->defaultValue(200)
                        ->end()
                        ->scalarNode('object_manager')->isRequired()->end()
                        ->scalarNode('model_name')->defaultValue('AppBundle\\Entity\\User')->end()
                        ->arrayNode('password')
                            ->children()
                                ->scalarNode('strategy')
                                    ->validate()
                                        ->ifNotInArray(array('php55', 'crypt', 'sha512', 'phpass'))
                                        ->thenInvalid(
                                            'Invalid password strategy "%s"! '
                                            .'Allowed strategies are "password", "crypt", "sha512", "phpass"!'
                                        )
                                    ->end()
                                ->end()
                                ->integerNode('phpass_iteration_length')
                                    ->defaultValue(8)
                                ->end()
                            ->end()
                        ->end()
                    ->end()
                ->end()
                ->arrayNode('api_key_purge')
                    ->canBeEnabled()
                    ->children()
                        ->booleanNode('log_state')->defaultFalse()->end()
                        ->scalarNode('logger_service')->defaultValue('logger')->end()
                    ->end()
                ->end()
                ->arrayNode('services')
                    ->addDefaultsIfNotSet()
                    ->children()
                        ->scalarNode('auth_handler')->defaultNull()->end()
                        ->scalarNode('key_factory')->defaultNull()->end()
                        ->scalarNode('password_hasher')->defaultNull()->end()
                    ->end()
                ->end()
            ->end();

        return $treeBuilder;
    }
}


1		<?php
2
3		namespace Ma27\ApiKeyAuthenticationBundle\DependencyInjection;
4
5		use Symfony\Component\Config\Definition\Builder\TreeBuilder;
6		use Symfony\Component\Config\Definition\ConfigurationInterface;
7
8		/**
9		* This is the class that validates and merges configuration from your app/config files.
10		*
11		* To learn more see {@link http://symfony.com/doc/current/cookbook/bundles/extension.html#cookbook-bundles-extension-config-class}
12		*/
13		class Configuration implements ConfigurationInterface
14		{
15		/**
16		* {@inheritdoc}
17		*/
18		public function getConfigTreeBuilder()
19	10	{
20		$treeBuilder = new TreeBuilder();
21	10	$rootNode = $treeBuilder->root('ma27_api_key_authentication');
22	10
23		$rootNode
24		->children()
25	10	->arrayNode('user')
26	10	->children()
27	10	->integerNode('api_key_length')
28	10	->min(50)
29	10	->defaultValue(200)
30	10	->end()
31	10	->scalarNode('object_manager')->isRequired()->end()
32	10	->scalarNode('model_name')->defaultValue('AppBundle\\Entity\\User')->end()
33	10	->arrayNode('password')
34	10	->children()
35	10	->scalarNode('strategy')
36		->validate()
37		->ifNotInArray(array('php55', 'crypt', 'sha512', 'phpass'))
38	9	->thenInvalid(
39	9	'Invalid password strategy "%s"! '
40	9	.'Allowed strategies are "password", "crypt", "sha512", "phpass"!'
41		)
42	9	->end()
43	1	->end()
44		->integerNode('phpass_iteration_length')
45		->defaultValue(8)
46	8	->end()
47	1	->end()
48	1	->end()
49	1	->end()
50	1	->end()
51		->arrayNode('api_key_purge')
52	1	->canBeEnabled()
53		->children()
54	1	->booleanNode('log_state')->defaultFalse()->end()
55	1	->scalarNode('logger_service')->defaultValue('logger')->end()
56	1	->end()
57	1	->end()
58	1	->arrayNode('services')
59	1	->addDefaultsIfNotSet()
60		->children()
61		->scalarNode('auth_handler')->defaultNull()->end()
62	7	->scalarNode('key_factory')->defaultNull()->end()
63	10	->scalarNode('password_hasher')->defaultNull()->end()
64	10	->end()
65	10	->end()
66	10	->end();
67	10
68	10	return $treeBuilder;
69	10	}
70		}
71

Ma27 / Ma27ApiKeyAuthenticationBundle

GitHub Access Token became invalid

Pull Request — master (#31)

Configuration::getConfigTreeBuilder() A

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like