KleijnWebSwaggerBundle::build() - Code Metrics - Inspection of "Cleanup" - kleijnweb/swagger-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( aa6f89...863f7f )

by John

created 2017-04-05 10:39 UTC

KleijnWebSwaggerBundle::build() A

↳ Parent: KleijnWebSwaggerBundle

Complexity

Conditions	2
Paths	2

Size

Total Lines	10
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php declare(strict_types = 1);
/*
 * This file is part of the KleijnWeb\SwaggerBundle package.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace KleijnWeb\SwaggerBundle;

use KleijnWeb\SwaggerBundle\DependencyInjection\KleijnWebSwaggerExtension;
use KleijnWeb\SwaggerBundle\DependencyInjection\SwaggerRequestAuthorizationFactory;
use Symfony\Bundle\SecurityBundle\DependencyInjection\SecurityExtension;
use Symfony\Component\DependencyInjection\ContainerBuilder;
use Symfony\Component\DependencyInjection\Extension\ExtensionInterface;
use Symfony\Component\HttpKernel\Bundle\Bundle;

/**
 * @author John Kleijn <[email protected]>
 */
class KleijnWebSwaggerBundle extends Bundle
{
    /**
     * @return string The Bundle namespace
     */
    public function getNamespace()
    {
        return __NAMESPACE__;
    }

    public function build(ContainerBuilder $container)
    {
        parent::build($container);

        if ($container->hasExtension('security')) {
            /** @var SecurityExtension $extension */
            $extension = $container->getExtension('security');
            $extension->addSecurityListenerFactory(new SwaggerRequestAuthorizationFactory());
        }
    }

    /**
     * @return ExtensionInterface
     */
    public function getContainerExtension()
    {
        if (null === $this->extension) {
            $this->extension = new KleijnWebSwaggerExtension();
        }

        return $this->extension;

    }
}


1			<?php declare(strict_types = 1);
2			/*
3			* This file is part of the KleijnWeb\SwaggerBundle package.
4			*
5			* For the full copyright and license information, please view the LICENSE
6			* file that was distributed with this source code.
7			*/
8
9			namespace KleijnWeb\SwaggerBundle;
10
11			use KleijnWeb\SwaggerBundle\DependencyInjection\KleijnWebSwaggerExtension;
12			use KleijnWeb\SwaggerBundle\DependencyInjection\SwaggerRequestAuthorizationFactory;
13			use Symfony\Bundle\SecurityBundle\DependencyInjection\SecurityExtension;
14			use Symfony\Component\DependencyInjection\ContainerBuilder;
15			use Symfony\Component\DependencyInjection\Extension\ExtensionInterface;
16			use Symfony\Component\HttpKernel\Bundle\Bundle;
17
18			/**
19			* @author John Kleijn <[email protected]>
20			*/
21			class KleijnWebSwaggerBundle extends Bundle
22			{
23			/**
24			* @return string The Bundle namespace
25			*/
26			public function getNamespace()
27			{
28			return __NAMESPACE__;
29			}
30
31			public function build(ContainerBuilder $container)
32			{
33			parent::build($container);
34
35			if ($container->hasExtension('security')) {
36			/** @var SecurityExtension $extension */
37			$extension = $container->getExtension('security');
38			$extension->addSecurityListenerFactory(new SwaggerRequestAuthorizationFactory());
39			}
40			}
41
42			/**
43			* @return ExtensionInterface
44			*/
45			public function getContainerExtension()
46			{
47			if (null === $this->extension) {
48			$this->extension = new KleijnWebSwaggerExtension();
49			}
50
51			return $this->extension;
			0 ignored issues – show Comprehensibility Best Practice introduced 2016-02-04 19:12 UTC by Report Bug Copy Issue Report The expression `$this->extension;` of type `Symfony\Component\Depend...xtensionInterface\|false` adds `false` to the return on line 51 which is incompatible with the return type declared by the interface `Symfony\Component\HttpKe...::getContainerExtension` of type `Symfony\Component\Depend...ExtensionInterface\|null`. It seems like you forgot to handle an error condition. Loading history...
52			}
53			}
54

kleijnweb / swagger-bundle

Push — master ( aa6f89...863f7f )

KleijnWebSwaggerBundle::build() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like