LakionCmsExtension::load() - Code Metrics - Inspection of "Rename to CmsPlugin + maintenance" - Lakion/SyliusCmsBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#45)

by Jan

created 2017-04-11 13:44 UTC

LakionCmsExtension::load() A

↳ Parent: LakionCmsExtension

Complexity

Conditions	1
Paths	1

Size

Total Lines	12
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

namespace Lakion\CmsPlugin\DependencyInjection;

use Sylius\Bundle\ResourceBundle\DependencyInjection\Extension\AbstractResourceExtension;
use Sylius\Bundle\ResourceBundle\SyliusResourceBundle;
use Symfony\Component\Config\FileLocator;
use Symfony\Component\DependencyInjection\ContainerBuilder;
use Symfony\Component\DependencyInjection\Extension\PrependExtensionInterface;
use Symfony\Component\DependencyInjection\Loader\XmlFileLoader;
use Symfony\Component\DependencyInjection\Parameter;

final class LakionCmsExtension extends AbstractResourceExtension implements PrependExtensionInterface
{
    /**
     * {@inheritdoc}
     */
    public function load(array $config, ContainerBuilder $container)
    {
        $config = $this->processConfiguration($this->getConfiguration([], $container), $config);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $loader = new XmlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));

        $this->registerResources('lakion_cms', SyliusResourceBundle::DRIVER_DOCTRINE_PHPCR_ODM, $config['resources'], $container);

        $loader->load('services.xml');

        $staticContentRepository = $container->getDefinition('lakion_cms.repository.static_content');
        $staticContentRepository->addArgument(new Parameter('cmf_content.persistence.phpcr.content_basepath'));
    }

    /**
     * {@inheritdoc}
     */
    public function prepend(ContainerBuilder $container)
    {
        $config = $this->processConfiguration($this->getConfiguration([], $container), $container->getExtensionConfig($this->getAlias()));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        $this->registerResources('lakion_cms', SyliusResourceBundle::DRIVER_DOCTRINE_PHPCR_ODM, $config['resources'], $container);
    }
}


1			<?php
2
3			namespace Lakion\CmsPlugin\DependencyInjection;
4
5			use Sylius\Bundle\ResourceBundle\DependencyInjection\Extension\AbstractResourceExtension;
6			use Sylius\Bundle\ResourceBundle\SyliusResourceBundle;
7			use Symfony\Component\Config\FileLocator;
8			use Symfony\Component\DependencyInjection\ContainerBuilder;
9			use Symfony\Component\DependencyInjection\Extension\PrependExtensionInterface;
10			use Symfony\Component\DependencyInjection\Loader\XmlFileLoader;
11			use Symfony\Component\DependencyInjection\Parameter;
12
13			final class LakionCmsExtension extends AbstractResourceExtension implements PrependExtensionInterface
14			{
15			/**
16			* {@inheritdoc}
17			*/
18			public function load(array $config, ContainerBuilder $container)
19			{
20			$config = $this->processConfiguration($this->getConfiguration([], $container), $config);
			0 ignored issues – show Documentation introduced 2017-01-05 15:18 UTC by Report Bug Copy Issue Report `$this->getConfiguration(array(), $container)` is of type `object\|null`, but the function expects a `object<Symfony\Component...ConfigurationInterface>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
21			$loader = new XmlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
22
23			$this->registerResources('lakion_cms', SyliusResourceBundle::DRIVER_DOCTRINE_PHPCR_ODM, $config['resources'], $container);
24
25			$loader->load('services.xml');
26
27			$staticContentRepository = $container->getDefinition('lakion_cms.repository.static_content');
28			$staticContentRepository->addArgument(new Parameter('cmf_content.persistence.phpcr.content_basepath'));
29			}
30
31			/**
32			* {@inheritdoc}
33			*/
34			public function prepend(ContainerBuilder $container)
35			{
36			$config = $this->processConfiguration($this->getConfiguration([], $container), $container->getExtensionConfig($this->getAlias()));
			0 ignored issues – show Documentation introduced 2017-01-05 15:18 UTC by Report Bug Copy Issue Report `$this->getConfiguration(array(), $container)` is of type `object\|null`, but the function expects a `object<Symfony\Component...ConfigurationInterface>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
37
38			$this->registerResources('lakion_cms', SyliusResourceBundle::DRIVER_DOCTRINE_PHPCR_ODM, $config['resources'], $container);
39			}
40			}
41

Lakion / SyliusCmsBundle

Pull Request — master (#45)

LakionCmsExtension::load() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like