ResourceConfiguration::createResourceNode() - Code Metrics - Inspection of "Merge pull request #47 from php-lug/resource" - php-lug/lug - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( f61101...67e7bb )

by Eric

created 2016-06-12 20:50 UTC

ResourceConfiguration::createResourceNode() A

↳ Parent: ResourceConfiguration

Complexity

Conditions	3
Paths	2

Size

Total Lines	51
Code Lines	39

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	43
CRAP Score	3.0007

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	0
loc	51
ccs	43
cts	45
cp	0.9556
rs	9.4109
cc	3
eloc	39
nc	2
nop	2
crap	3.0007

How to fix Long Method

<?php

/*
 * This file is part of the Lug package.
 *
 * (c) Eric GELOEN <[email protected]>
 *
 * For the full copyright and license information, please read the LICENSE
 * file that was distributed with this source code.
 */

namespace Lug\Bundle\ResourceBundle\DependencyInjection\Extension;

use Lug\Bundle\ResourceBundle\ResourceBundleInterface;
use Lug\Component\Resource\Controller\ControllerInterface;
use Lug\Component\Resource\Domain\DomainManagerInterface;
use Lug\Component\Resource\Factory\FactoryInterface;
use Lug\Component\Resource\Model\ResourceInterface;
use Lug\Component\Resource\Repository\RepositoryInterface;
use Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition;
use Symfony\Component\Config\Definition\Builder\NodeBuilder;
use Symfony\Component\Config\Definition\Builder\NodeDefinition;
use Symfony\Component\Config\Definition\Builder\TreeBuilder;
use Symfony\Component\Config\Definition\ConfigurationInterface;
use Symfony\Component\Form\FormTypeInterface;

/**
 * @author GeLo <[email protected]>
 */
class ResourceConfiguration implements ConfigurationInterface
{
    /**
     * @var ResourceBundleInterface
     */
    private $bundle;

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

    /**
     * {@inheritdoc}
     */
    public function getConfigTreeBuilder()
    {
        $treeBuilder = $this->createTreeBuilder();
        $children = $treeBuilder->root($this->bundle->getAlias())->children();

        $this->configureBundle($children);
        $children->append($this->createResourceNodes());

        return $treeBuilder;
    }

    /**
     * @param NodeBuilder $builder
     */
    protected function configureBundle(NodeBuilder $builder)
    {
    }

    /**
     * @return ArrayNodeDefinition
     */
    private function createResourceNodes()
    {
        $resourcesNode = $this->createNode('resources')->addDefaultsIfNotSet();
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        $childrenNode = $resourcesNode->children();

        foreach ($this->bundle->getResources() as $resource) {
            $childrenNode->append($this->createResourceNode($resource));
        }

        return $resourcesNode;
    }

    /**
     * @param ResourceInterface $resource
     * @param string|null       $name
     *
     * @return ArrayNodeDefinition
     */
    private function createResourceNode(ResourceInterface $resource, $name = null)
    {
        $resourceNode = $this->createNode($name ?: $resource->getName())->addDefaultsIfNotSet();
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        $childrenNode = $resourceNode->children()
            ->append($this->createDriverNode($resource))
            ->append($this->createClassNode(
                'model',
                $resource->getModel(),
                $resource->getInterfaces(),
                true
            ))
            ->append($this->createClassNode(
                'repository',
                $resource->getRepository(),
                [RepositoryInterface::class],
                true
            ))
            ->append($this->createClassNode(
                'factory',
                $resource->getFactory(),
                [FactoryInterface::class]
            ))
            ->append($this->createClassNode(
                'form',
                $resource->getForm(),
                [FormTypeInterface::class]
            ))
            ->append($this->createClassNode(
                'choice_form',
                $resource->getChoiceForm(),
                [FormTypeInterface::class]
            ))
            ->append($this->createClassNode(
                'domain_manager',
                $resource->getDomainManager(),
                [DomainManagerInterface::class]
            ))
            ->append($this->createClassNode(
                'controller',
                $resource->getController(),
                [ControllerInterface::class]
            ))
            ->append($this->createNode('id_property_path', 'scalar', $resource->getIdPropertyPath()))
            ->append($this->createNode('label_property_path', 'scalar', $resource->getLabelPropertyPath()));

        if ($resource->getTranslation() !== null) {
            $childrenNode->append($this->createResourceNode($resource->getTranslation(), 'translation'));
        }

        return $resourceNode;
    }

    /**
     * @param ResourceInterface $resource
     *
     * @return ArrayNodeDefinition
     */
    private function createDriverNode(ResourceInterface $resource)
    {
        $driverNode = $this->createNode('driver')->addDefaultsIfNotSet();
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        $driverNode->children()
            ->append($this->createNode('name', 'scalar', $resource->getDriver()))
            ->append($this->createNode('manager', 'scalar', $resource->getDriverManager()))
            ->append($this->createDriverMappingNode($resource));

        return $driverNode;
    }

    /**
     * @param ResourceInterface $resource
     *
     * @return ArrayNodeDefinition
     */
    private function createDriverMappingNode(ResourceInterface $resource)
    {
        $mappingNode = $this->createNode('mapping')->addDefaultsIfNotSet();
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        $mappingNode->children()
            ->append($this->createNode('path', 'scalar', $resource->getDriverMappingPath()))
            ->append($this->createNode('format', 'scalar', $resource->getDriverMappingFormat()));

        return $mappingNode;
    }

    /**
     * @param string   $name
     * @param string   $class
     * @param string[] $interfaces
     * @param bool     $required
     *
     * @return NodeDefinition
     */
    private function createClassNode($name, $class, array $interfaces = [], $required = false)
    {
        return $this->createNode($name, 'scalar', $class, function ($class) use ($interfaces, $required) {
            if ($class === null) {
                return $required;
            }

            if (!class_exists($class)) {
                return true;
            }

            $classInterfaces = class_implements($class);

            foreach ($interfaces as $interface) {
                if (!in_array($interface, $classInterfaces, true)) {
                    return true;
                }
            }

            return false;
        }, 'The %s %%s does not exist.');
    }

    /**
     * @param string        $name
     * @param string        $type
     * @param mixed         $default
     * @param callable|null $if
     * @param string|null   $then
     *
     * @return ArrayNodeDefinition|NodeDefinition
     */
    private function createNode($name, $type = 'array', $default = null, $if = null, $then = null)
    {
        $node = $this->createTreeBuilder()->root($name, $type);

        if ($default !== null) {
            $node->defaultValue($default);
        }

        if ($if !== null && $then !== null) {
            $node->validate()->ifTrue($if)->thenInvalid(sprintf($then, $name));
        }

        return $node;
    }

    /**
     * @return TreeBuilder
     */
    private function createTreeBuilder()
    {
        return new TreeBuilder();
    }
}


Push — master ( f61101...67e7bb )

ResourceConfiguration::createResourceNode() A

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method

Long Method

Available Fixes

Available Fixes

Available Fixes

Available Fixes

1		<?php
2
3		/*
4		* This file is part of the Lug package.
5		*
6		* (c) Eric GELOEN <[email protected]>
7		*
8		* For the full copyright and license information, please read the LICENSE
9		* file that was distributed with this source code.
10		*/
11
12		namespace Lug\Bundle\ResourceBundle\DependencyInjection\Extension;
13
14		use Lug\Bundle\ResourceBundle\ResourceBundleInterface;
15		use Lug\Component\Resource\Controller\ControllerInterface;
16		use Lug\Component\Resource\Domain\DomainManagerInterface;
17		use Lug\Component\Resource\Factory\FactoryInterface;
18		use Lug\Component\Resource\Model\ResourceInterface;
19		use Lug\Component\Resource\Repository\RepositoryInterface;
20		use Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition;
21		use Symfony\Component\Config\Definition\Builder\NodeBuilder;
22		use Symfony\Component\Config\Definition\Builder\NodeDefinition;
23		use Symfony\Component\Config\Definition\Builder\TreeBuilder;
24		use Symfony\Component\Config\Definition\ConfigurationInterface;
25		use Symfony\Component\Form\FormTypeInterface;
26
27		/**
28		* @author GeLo <[email protected]>
29		*/
30		class ResourceConfiguration implements ConfigurationInterface
31		{
32		/**
33		* @var ResourceBundleInterface
34		*/
35		private $bundle;
36
37		/**
38		* @param ResourceBundleInterface $bundle
39		*/
40	7	public function __construct(ResourceBundleInterface $bundle)
41		{
42	7	$this->bundle = $bundle;
43	7	}
44
45		/**
46		* {@inheritdoc}
47		*/
48	7	public function getConfigTreeBuilder()
49		{
50	7	$treeBuilder = $this->createTreeBuilder();
51	7	$children = $treeBuilder->root($this->bundle->getAlias())->children();
52
53	7	$this->configureBundle($children);
54	7	$children->append($this->createResourceNodes());
55
56	7	return $treeBuilder;
57		}
58
59		/**
60		* @param NodeBuilder $builder
61		*/
62		protected function configureBundle(NodeBuilder $builder)
63		{
64		}
65
66		/**
67		* @return ArrayNodeDefinition
68		*/
69	7	private function createResourceNodes()
70		{
71	7	$resourcesNode = $this->createNode('resources')->addDefaultsIfNotSet();
		0 ignored issues – show Bug introduced 2016-01-19 20:19 UTC by Report Bug Copy Issue Report It seems like you code against a specific sub-type and not the parent class `Symfony\Component\Config...\Builder\NodeDefinition` as the method `addDefaultsIfNotSet()` does only exist in the following sub-classes of `Symfony\Component\Config...\Builder\NodeDefinition`: `Symfony\Component\Config...der\ArrayNodeDefinition`. Maybe you want to `instanceof` check for one of these explicitly? Let’s take a look at an example: abstract class User { /** @return string / abstract public function getPassword(); } class MyUser extends User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different sub-classes of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the parent class: abstract class User { /** @return string / abstract public function getPassword(); /* @return string */ abstract public function getDisplayName(); } Loading history...
72	7	$childrenNode = $resourcesNode->children();
73
74	7	foreach ($this->bundle->getResources() as $resource) {
75	7	$childrenNode->append($this->createResourceNode($resource));
76	7	}
77
78	7	return $resourcesNode;
79		}
80
81		/**
82		* @param ResourceInterface $resource
83		* @param string\|null $name
84		*
85		* @return ArrayNodeDefinition
86		*/
87	7	private function createResourceNode(ResourceInterface $resource, $name = null)
88		{
89	7	$resourceNode = $this->createNode($name ?: $resource->getName())->addDefaultsIfNotSet();
		0 ignored issues – show Bug introduced 2016-01-19 20:19 UTC by Report Bug Copy Issue Report It seems like you code against a specific sub-type and not the parent class `Symfony\Component\Config...\Builder\NodeDefinition` as the method `addDefaultsIfNotSet()` does only exist in the following sub-classes of `Symfony\Component\Config...\Builder\NodeDefinition`: `Symfony\Component\Config...der\ArrayNodeDefinition`. Maybe you want to `instanceof` check for one of these explicitly? Let’s take a look at an example: abstract class User { /** @return string / abstract public function getPassword(); } class MyUser extends User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different sub-classes of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the parent class: abstract class User { /** @return string / abstract public function getPassword(); /* @return string */ abstract public function getDisplayName(); } Loading history...
90	7	$childrenNode = $resourceNode->children()
91	7	->append($this->createDriverNode($resource))
92	7	->append($this->createClassNode(
93	7	'model',
94	7	$resource->getModel(),
95	7	$resource->getInterfaces(),
96		true
97	7	))
98	7	->append($this->createClassNode(
99	7	'repository',
100	7	$resource->getRepository(),
101	7	[RepositoryInterface::class],
102		true
103	7	))
104	7	->append($this->createClassNode(
105	7	'factory',
106	7	$resource->getFactory(),
107	7	[FactoryInterface::class]
108	7	))
109	7	->append($this->createClassNode(
110	7	'form',
111	7	$resource->getForm(),
112	7	[FormTypeInterface::class]
113	7	))
114	7	->append($this->createClassNode(
115	7	'choice_form',
116	7	$resource->getChoiceForm(),
117	7	[FormTypeInterface::class]
118	7	))
119	7	->append($this->createClassNode(
120	7	'domain_manager',
121	7	$resource->getDomainManager(),
122	7	[DomainManagerInterface::class]
123	7	))
124	7	->append($this->createClassNode(
125	7	'controller',
126	7	$resource->getController(),
127	7	[ControllerInterface::class]
128	7	))
129	7	->append($this->createNode('id_property_path', 'scalar', $resource->getIdPropertyPath()))
130	7	->append($this->createNode('label_property_path', 'scalar', $resource->getLabelPropertyPath()));
131
132	7	if ($resource->getTranslation() !== null) {
133		$childrenNode->append($this->createResourceNode($resource->getTranslation(), 'translation'));
134		}
135
136	7	return $resourceNode;
137		}
138
139		/**
140		* @param ResourceInterface $resource
141		*
142		* @return ArrayNodeDefinition
143		*/
144	7	private function createDriverNode(ResourceInterface $resource)
145		{
146	7	$driverNode = $this->createNode('driver')->addDefaultsIfNotSet();
		0 ignored issues – show Bug introduced 2016-01-19 20:19 UTC by Report Bug Copy Issue Report It seems like you code against a specific sub-type and not the parent class `Symfony\Component\Config...\Builder\NodeDefinition` as the method `addDefaultsIfNotSet()` does only exist in the following sub-classes of `Symfony\Component\Config...\Builder\NodeDefinition`: `Symfony\Component\Config...der\ArrayNodeDefinition`. Maybe you want to `instanceof` check for one of these explicitly? Let’s take a look at an example: abstract class User { /** @return string / abstract public function getPassword(); } class MyUser extends User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different sub-classes of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the parent class: abstract class User { /** @return string / abstract public function getPassword(); /* @return string */ abstract public function getDisplayName(); } Loading history...
147	7	$driverNode->children()
148	7	->append($this->createNode('name', 'scalar', $resource->getDriver()))
149	7	->append($this->createNode('manager', 'scalar', $resource->getDriverManager()))
150	7	->append($this->createDriverMappingNode($resource));
151
152	7	return $driverNode;
153		}
154
155		/**
156		* @param ResourceInterface $resource
157		*
158		* @return ArrayNodeDefinition
159		*/
160	7	private function createDriverMappingNode(ResourceInterface $resource)
161		{
162	7	$mappingNode = $this->createNode('mapping')->addDefaultsIfNotSet();
		0 ignored issues – show Bug introduced 2016-01-19 20:19 UTC by Report Bug Copy Issue Report It seems like you code against a specific sub-type and not the parent class `Symfony\Component\Config...\Builder\NodeDefinition` as the method `addDefaultsIfNotSet()` does only exist in the following sub-classes of `Symfony\Component\Config...\Builder\NodeDefinition`: `Symfony\Component\Config...der\ArrayNodeDefinition`. Maybe you want to `instanceof` check for one of these explicitly? Let’s take a look at an example: abstract class User { /** @return string / abstract public function getPassword(); } class MyUser extends User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different sub-classes of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the parent class: abstract class User { /** @return string / abstract public function getPassword(); /* @return string */ abstract public function getDisplayName(); } Loading history...
163	7	$mappingNode->children()
164	7	->append($this->createNode('path', 'scalar', $resource->getDriverMappingPath()))
165	7	->append($this->createNode('format', 'scalar', $resource->getDriverMappingFormat()));
166
167	7	return $mappingNode;
168		}
169
170		/**
171		* @param string $name
172		* @param string $class
173		* @param string[] $interfaces
174		* @param bool $required
175		*
176		* @return NodeDefinition
177		*/
178		private function createClassNode($name, $class, array $interfaces = [], $required = false)
179		{
180	7	return $this->createNode($name, 'scalar', $class, function ($class) use ($interfaces, $required) {
181		if ($class === null) {
182		return $required;
183		}
184
185		if (!class_exists($class)) {
186		return true;
187		}
188
189		$classInterfaces = class_implements($class);
190
191		foreach ($interfaces as $interface) {
192		if (!in_array($interface, $classInterfaces, true)) {
193		return true;
194		}
195		}
196
197		return false;
198	7	}, 'The %s %%s does not exist.');
199		}
200
201		/**
202		* @param string $name
203		* @param string $type
204		* @param mixed $default
205		* @param callable\|null $if
206		* @param string\|null $then
207		*
208		* @return ArrayNodeDefinition\|NodeDefinition
209		*/
210	7	private function createNode($name, $type = 'array', $default = null, $if = null, $then = null)
211		{
212	7	$node = $this->createTreeBuilder()->root($name, $type);
213
214	7	if ($default !== null) {
215	7	$node->defaultValue($default);
216	7	}
217
218	7	if ($if !== null && $then !== null) {
219	7	$node->validate()->ifTrue($if)->thenInvalid(sprintf($then, $name));
220	7	}
221
222	7	return $node;
223		}
224
225		/**
226		* @return TreeBuilder
227		*/
228	7	private function createTreeBuilder()
229		{
230	7	return new TreeBuilder();
231		}
232		}
233

php-lug / lug

Push — master ( f61101...67e7bb )

ResourceConfiguration::createResourceNode() A

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method

Long Method

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like