CustomerGroupFixture::configureResourceNode() - Code Metrics - Inspection of "Merge pull request #6985 from Zales0123/reviews-fi..." - Sylius/Sylius - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 4e7b1e...2ea0a3 )

by Paweł

created 2016-12-05 11:46 UTC

CustomerGroupFixture::configureResourceNode() A

↳ Parent: CustomerGroupFixture

Complexity

Conditions	1
Paths	1

Size

Total Lines	8
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	8
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	5
nc	1
nop	1

<?php

/*
 * This file is part of the Sylius package.
 *
 * (c) Paweł Jędrzejewski
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Sylius\Bundle\CoreBundle\Fixture;

use Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition;

/**
 * @author Grzegorz Sadowski <[email protected]>
 */
class CustomerGroupFixture extends AbstractResourceFixture
{
    /**
     * {@inheritdoc}
     */
    public function getName()
    {
        return 'customer_group';
    }

    /**
     * {@inheritdoc}
     */
    protected function configureResourceNode(ArrayNodeDefinition $resourceNode)
    {
        $resourceNode
class User
{
    private $email;

    public function getEmail()
    {
        return $this->email;
    }

    public function setEmail($email)
    {
        $this->email = $email;
    }
}
            ->children()
                ->scalarNode('name')->cannotBeEmpty()->end()
                ->scalarNode('code')->cannotBeEmpty()->end()
        ;
    }
}


1			<?php
2
3			/*
4			* This file is part of the Sylius package.
5			*
6			* (c) Paweł Jędrzejewski
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			namespace Sylius\Bundle\CoreBundle\Fixture;
13
14			use Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition;
15
16			/**
17			* @author Grzegorz Sadowski <[email protected]>
18			*/
19			class CustomerGroupFixture extends AbstractResourceFixture
20			{
21			/**
22			* {@inheritdoc}
23			*/
24			public function getName()
25			{
26			return 'customer_group';
27			}
28
29			/**
30			* {@inheritdoc}
31			*/
32			protected function configureResourceNode(ArrayNodeDefinition $resourceNode)
33			{
34			$resourceNode
			0 ignored issues – show Unused Code introduced 2016-12-05 08:10 UTC by Report Bug Copy Issue Report The call to the method `Symfony\Component\Config...r\NodeDefinition::end()` seems un-needed as the method has no side-effects. PHP Analyzer performs a side-effects analysis of your code. A side-effect is basically anything that might be visible after the scope of the method is left. Let’s take a look at an example: class User { private $email; public function getEmail() { return $this->email; } public function setEmail($email) { $this->email = $email; } } If we look at the `getEmail()` method, we can see that it has no side-effect. Whether you call this method or not, no future calls to other methods are affected by this. As such code as the following is useless: $user = new User(); $user->getEmail(); // This line could safely be removed as it has no effect. On the hand, if we look at the `setEmail()`, this method _has_ side-effects. In the following case, we could not remove the method call: $user = new User(); $user->setEmail('email@domain'); // This line has a side-effect (it changes an // instance variable). Loading history...
35			->children()
36			->scalarNode('name')->cannotBeEmpty()->end()
37			->scalarNode('code')->cannotBeEmpty()->end()
38			;
39			}
40			}
41

Sylius / Sylius

Push — master ( 4e7b1e...2ea0a3 )

CustomerGroupFixture::configureResourceNode() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like