ResourceExtension::getAlias() - Code Metrics - Inspection of "[ResourceBundle] Track resource change" - php-lug/lug - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#36)

by Eric

created 2016-05-18 20:09 UTC

ResourceExtension::getAlias() A

↳ Parent: ResourceExtension

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	4
ccs	2
cts	2
cp	1
rs	10
cc	1
eloc	2
nc	1
nop	0
crap	1

<?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\Bundle\ResourceBundle\Util\ClassUtils;
use Lug\Component\Resource\Model\Resource;
use Lug\Component\Resource\Model\ResourceInterface;
use Lug\Component\Translation\Factory\TranslatableFactory;
use Symfony\Component\DependencyInjection\ContainerBuilder;
use Symfony\Component\DependencyInjection\Definition;
use Symfony\Component\DependencyInjection\Reference;
use Symfony\Component\HttpKernel\DependencyInjection\ConfigurableExtension;

/**
 * @author GeLo <[email protected]>
 */
class ResourceExtension extends ConfigurableExtension
{
    /**
     * @var ResourceBundleInterface
     */
    private $bundle;

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

    /**
     * {@inheritdoc}
     */
    public function getConfiguration(array $config, ContainerBuilder $container)
    {
        if (!class_exists($class = ClassUtils::getNamespace($this).'\\Configuration')) {
            $class = ResourceConfiguration::class;
        }

        return new $class($this->bundle);
    }

    /**
     * {@inheritdoc}
     */
    public function getAlias()
    {
        return $this->bundle->getAlias();
    }

    /**
     * {@inheritdoc}
     */
    protected function loadInternal(array $config, ContainerBuilder $container)
    {
        foreach ($this->bundle->getResources() as $resource) {
            $resourceConfig = $config['resources'][$resource->getName()];

            $this->configureResource($resource, $resourceConfig);
            $this->loadResource($resource, $container);

            if ($resource->getTranslation() !== null) {
                $this->configureResource($resource->getTranslation(), $resourceConfig['translation']);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                $this->loadResource($resource->getTranslation(), $container);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        $this->loadBundle($config, $container);
    }

    /**
     * @param mixed[]          $config
     * @param ContainerBuilder $container
     */
    protected function loadBundle(array $config, ContainerBuilder $container)
    {
    }

    /**
     * @param ResourceInterface $resource
     * @param mixed[]           $config
     */
    private function configureResource(ResourceInterface $resource, array $config)
    {
        $resource->setDriver($config['driver']['name']);
        $resource->setDriverManager($config['driver']['manager']);
        $resource->setDriverMappingPath($config['driver']['mapping']['path']);
        $resource->setDriverMappingFormat($config['driver']['mapping']['format']);
        $resource->setModel($config['model']);
        $resource->setController($config['controller']);
        $resource->setFactory($config['factory']);
        $resource->setRepository($config['repository']);
        $resource->setDomainManager($config['domain_manager']);
        $resource->setForm($config['form']);
        $resource->setChoiceForm($config['choice_form']);
        $resource->setIdPropertyPath($config['id_property_path']);
        $resource->setLabelPropertyPath($config['label_property_path']);
    }

    /**
     * @param ResourceInterface $resource
     * @param ContainerBuilder  $container
     */
    private function loadResource(ResourceInterface $resource, ContainerBuilder $container)
    {
        $name = $resource->getName();
        $controller = 'lug.controller.'.$name;
        $factory = 'lug.factory.'.$name;
        $repository = 'lug.repository.'.$name;
        $domainManager = 'lug.domain_manager.'.$name;
        $form = 'lug.form.type.'.$name;
        $choiceForm = $form.'.choice';

        $container->setDefinition('lug.resource.'.$resource->getName(), $this->createResourceDefinition($resource));
        $container->setAlias('lug.manager.'.$resource->getName(), $this->createManagerAlias($resource));

        if (class_exists($resource->getController())) {

            $container->setDefinition($controller, $this->createControllerDefinition($resource));
        } elseif ($controller !== $resource->getController()) {
            $container->setAlias($container, $resource->getController());
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        if (class_exists($resource->getFactory())) {

            $container->setDefinition($factory, $this->createFactoryDefinition($resource));
        } elseif ($factory !== $resource->getFactory()) {
            $container->setAlias($factory, $resource->getFactory());
        }

        if (class_exists($resource->getRepository())) {
            $container->setDefinition($repository, $this->createRepositoryDefinition($resource));
        } elseif ($repository !== $resource->getRepository()) {
            $container->setAlias($repository, $resource->getRepository());
        }

        if (class_exists($resource->getDomainManager())) {

            $container->setDefinition($domainManager, $this->createDomainManagerDefinition($resource));
        } elseif ($domainManager !== $resource->getDomainManager()) {
            $container->setAlias($domainManager, $resource->getDomainManager());
        }

        if (class_exists($resource->getForm())) {

            $container->setDefinition($form, $this->createFormDefinition($resource));
        } elseif ($form !== $resource->getForm()) {
            $container->setAlias($form, $resource->getForm());
        }

        if (class_exists($resource->getChoiceForm())) {

            $container->setDefinition($choiceForm, $this->createChoiceFormDefinition($resource));
        } elseif ($choiceForm !== $resource->getChoiceForm()) {
            $container->setAlias($choiceForm, $resource->getChoiceForm());
        }

        $container->addClassResource(new \ReflectionClass($resource));
    }

    /**
     * @param ResourceInterface $resource
     *
     * @return Definition
     */
    private function createResourceDefinition(ResourceInterface $resource)
    {
        $arguments = [
            $resource->getName(),
            $resource->getDriver(),
            $resource->getDriverManager(),
            $resource->getDriverMappingPath(),
            $resource->getDriverMappingFormat(),
            $resource->getInterfaces(),
            $resource->getModel(),
            $resource->getController(),
            $resource->getFactory(),
            $resource->getRepository(),
            $resource->getDomainManager(),
            $resource->getForm(),
            $resource->getChoiceForm(),
            $resource->getIdPropertyPath(),
            $resource->getLabelPropertyPath(),
        ];

        if ($resource->getTranslation() !== null) {
            $arguments[] = new Reference('lug.resource.'.$resource->getTranslation()->getName());
        }

        $definition = new Definition(Resource::class, $arguments);
        $definition->addTag('lug.resource');

        return $definition;
    }

    /**
     * @param ResourceInterface $resource
     *
     * @return string
     */
    private function createManagerAlias(ResourceInterface $resource)
    {
        if ($resource->getDriver() === ResourceInterface::DRIVER_DOCTRINE_MONGODB) {
            return 'doctrine_mongodb.odm.'.$resource->getDriverManager().'_document_manager';
        }

        return 'doctrine.orm.'.$resource->getDriverManager().'_entity_manager';
    }

    /**
     * @param ResourceInterface $resource
     *
     * @return Definition
     */
    private function createFactoryDefinition(ResourceInterface $resource)
    {
        $arguments = [new Reference('lug.resource.'.$resource->getName())];

        if (is_a($resource->getFactory(), TranslatableFactory::class, true)) {
            $arguments[] = new Reference('lug.translation.context.locale');
        }

        $definition = new Definition($resource->getFactory(), $arguments);
        $definition->addTag('lug.factory', ['resource' => $resource->getName()]);

        return $definition;
    }

    /**
     * @param ResourceInterface $resource
     *
     * @return Definition
     */
    private function createRepositoryDefinition(ResourceInterface $resource)

    {
        $definition = new Definition($resource->getRepository(), [$resource->getModel()]);
        $definition->setFactory([new Reference('lug.manager.'.$resource->getName()), 'getRepository']);
        $definition->addTag('lug.repository', ['resource' => $resource->getName()]);

        return $definition;
    }

    /**
     * @param ResourceInterface $resource
     *
     * @return Definition
     */
    private function createFormDefinition(ResourceInterface $resource)

    {
        $definition = new Definition($resource->getForm(), [
            new Reference('lug.resource.'.$resource->getName()),
            new Reference('lug.factory.'.$resource->getName()),
        ]);

        $definition->addTag('form.type');

        return $definition;
    }

    /**
     * @param ResourceInterface $resource
     *
     * @return Definition
     */
    private function createChoiceFormDefinition(ResourceInterface $resource)

    {
        $definition = new Definition($resource->getChoiceForm(), [
            new Reference('lug.resource.'.$resource->getName()),
        ]);

        $definition->addTag('form.type');

        return $definition;
    }

    /**
     * @param ResourceInterface $resource
     *
     * @return Definition
     */
    private function createControllerDefinition(ResourceInterface $resource)

    {
        $definition = new Definition($resource->getController(), [
            new Reference('lug.resource.'.$resource->getName()),
        ]);

        $definition->addMethodCall('setContainer', [new Reference('service_container')]);
        $definition->addTag('lug.controller', ['resource' => $resource->getName()]);

        return $definition;
    }

    /**
     * @param ResourceInterface $resource
     *
     * @return Definition
     */
    private function createDomainManagerDefinition(ResourceInterface $resource)
    {
        $definition = new Definition($resource->getDomainManager(), [
            new Reference('lug.resource.'.$resource->getName()),
            new Reference('event_dispatcher'),
            new Reference('lug.manager.'.$resource->getName()),
            new Reference('lug.repository.'.$resource->getName()),
        ]);

        $definition->addTag('lug.domain_manager', ['resource' => $resource->getName()]);

        return $definition;
    }
}


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\Bundle\ResourceBundle\Util\ClassUtils;
16			use Lug\Component\Resource\Model\Resource;
17			use Lug\Component\Resource\Model\ResourceInterface;
18			use Lug\Component\Translation\Factory\TranslatableFactory;
19			use Symfony\Component\DependencyInjection\ContainerBuilder;
20			use Symfony\Component\DependencyInjection\Definition;
21			use Symfony\Component\DependencyInjection\Reference;
22			use Symfony\Component\HttpKernel\DependencyInjection\ConfigurableExtension;
23
24			/**
25			* @author GeLo <[email protected]>
26			*/
27			class ResourceExtension extends ConfigurableExtension
28			{
29			/**
30			* @var ResourceBundleInterface
31			*/
32			private $bundle;
33
34			/**
35			* @param ResourceBundleInterface $bundle
36			*/
37	8		public function __construct(ResourceBundleInterface $bundle)
38			{
39	8		$this->bundle = $bundle;
40	8		}
41
42			/**
43			* {@inheritdoc}
44			*/
45	7		public function getConfiguration(array $config, ContainerBuilder $container)
46			{
47	7		if (!class_exists($class = ClassUtils::getNamespace($this).'\\Configuration')) {
48			$class = ResourceConfiguration::class;
49			}
50
51	7		return new $class($this->bundle);
52			}
53
54			/**
55			* {@inheritdoc}
56			*/
57	8		public function getAlias()
58			{
59	8		return $this->bundle->getAlias();
60			}
61
62			/**
63			* {@inheritdoc}
64			*/
65	7		protected function loadInternal(array $config, ContainerBuilder $container)
66			{
67	7		foreach ($this->bundle->getResources() as $resource) {
68	7		$resourceConfig = $config['resources'][$resource->getName()];
69
70	7		$this->configureResource($resource, $resourceConfig);
71	7		$this->loadResource($resource, $container);
72
73	7		if ($resource->getTranslation() !== null) {
74			$this->configureResource($resource->getTranslation(), $resourceConfig['translation']);
			0 ignored issues – show Documentation introduced 2016-01-19 20:19 UTC by Report Bug Copy Issue Report `$resource->getTranslation()` is of type `null`, but the function expects a `object<Lug\Component\Res...odel\ResourceInterface>`. 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...
75			$this->loadResource($resource->getTranslation(), $container);
			0 ignored issues – show Documentation introduced 2016-01-19 20:19 UTC by Report Bug Copy Issue Report `$resource->getTranslation()` is of type `null`, but the function expects a `object<Lug\Component\Res...odel\ResourceInterface>`. 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...
76			}
77	7		}
78
79	7		$this->loadBundle($config, $container);
80	7		}
81
82			/**
83			* @param mixed[] $config
84			* @param ContainerBuilder $container
85			*/
86			protected function loadBundle(array $config, ContainerBuilder $container)
87			{
88			}
89
90			/**
91			* @param ResourceInterface $resource
92			* @param mixed[] $config
93			*/
94	7		private function configureResource(ResourceInterface $resource, array $config)
95			{
96	7		$resource->setDriver($config['driver']['name']);
97	7		$resource->setDriverManager($config['driver']['manager']);
98	7		$resource->setDriverMappingPath($config['driver']['mapping']['path']);
99	7		$resource->setDriverMappingFormat($config['driver']['mapping']['format']);
100	7		$resource->setModel($config['model']);
101	7		$resource->setController($config['controller']);
102	7		$resource->setFactory($config['factory']);
103	7		$resource->setRepository($config['repository']);
104	7		$resource->setDomainManager($config['domain_manager']);
105	7		$resource->setForm($config['form']);
106	7		$resource->setChoiceForm($config['choice_form']);
107	7		$resource->setIdPropertyPath($config['id_property_path']);
108	7		$resource->setLabelPropertyPath($config['label_property_path']);
109	7		}
110
111			/**
112			* @param ResourceInterface $resource
113			* @param ContainerBuilder $container
114			*/
115	7		private function loadResource(ResourceInterface $resource, ContainerBuilder $container)
116			{
117	7		$name = $resource->getName();
118	7		$controller = 'lug.controller.'.$name;
119	7		$factory = 'lug.factory.'.$name;
120	7		$repository = 'lug.repository.'.$name;
121	7		$domainManager = 'lug.domain_manager.'.$name;
122	7		$form = 'lug.form.type.'.$name;
123	7		$choiceForm = $form.'.choice';
124
125	7		$container->setDefinition('lug.resource.'.$resource->getName(), $this->createResourceDefinition($resource));
126	7		$container->setAlias('lug.manager.'.$resource->getName(), $this->createManagerAlias($resource));
127
128	7	View Code Duplication	if (class_exists($resource->getController())) {
			0 ignored issues – show Duplication introduced 2016-01-19 20:19 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
129	7		$container->setDefinition($controller, $this->createControllerDefinition($resource));
130	7		} elseif ($controller !== $resource->getController()) {
131			$container->setAlias($container, $resource->getController());
			0 ignored issues – show Documentation introduced 2016-01-19 20:19 UTC by Report Bug Copy Issue Report `$container` is of type `object<Symfony\Component...ction\ContainerBuilder>`, but the function expects a `string`. 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...
132			}
133
134	7	View Code Duplication	if (class_exists($resource->getFactory())) {
			0 ignored issues – show Duplication introduced 2016-01-19 20:19 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
135	7		$container->setDefinition($factory, $this->createFactoryDefinition($resource));
136	7		} elseif ($factory !== $resource->getFactory()) {
137			$container->setAlias($factory, $resource->getFactory());
138			}
139
140	7		if (class_exists($resource->getRepository())) {
141	7		$container->setDefinition($repository, $this->createRepositoryDefinition($resource));
142	7		} elseif ($repository !== $resource->getRepository()) {
143			$container->setAlias($repository, $resource->getRepository());
144			}
145
146	7	View Code Duplication	if (class_exists($resource->getDomainManager())) {
			0 ignored issues – show Duplication introduced 2016-01-19 20:19 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
147	7		$container->setDefinition($domainManager, $this->createDomainManagerDefinition($resource));
148	7		} elseif ($domainManager !== $resource->getDomainManager()) {
149			$container->setAlias($domainManager, $resource->getDomainManager());
150			}
151
152	7	View Code Duplication	if (class_exists($resource->getForm())) {
			0 ignored issues – show Duplication introduced 2016-01-19 20:19 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
153	7		$container->setDefinition($form, $this->createFormDefinition($resource));
154	7		} elseif ($form !== $resource->getForm()) {
155			$container->setAlias($form, $resource->getForm());
156			}
157
158	7	View Code Duplication	if (class_exists($resource->getChoiceForm())) {
			0 ignored issues – show Duplication introduced 2016-01-19 20:19 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
159	7		$container->setDefinition($choiceForm, $this->createChoiceFormDefinition($resource));
160	7		} elseif ($choiceForm !== $resource->getChoiceForm()) {
161			$container->setAlias($choiceForm, $resource->getChoiceForm());
162			}
163
164	7		$container->addClassResource(new \ReflectionClass($resource));
165	7		}
166
167			/**
168			* @param ResourceInterface $resource
169			*
170			* @return Definition
171			*/
172	7		private function createResourceDefinition(ResourceInterface $resource)
173			{
174			$arguments = [
175	7		$resource->getName(),
176	7		$resource->getDriver(),
177	7		$resource->getDriverManager(),
178	7		$resource->getDriverMappingPath(),
179	7		$resource->getDriverMappingFormat(),
180	7		$resource->getInterfaces(),
181	7		$resource->getModel(),
182	7		$resource->getController(),
183	7		$resource->getFactory(),
184	7		$resource->getRepository(),
185	7		$resource->getDomainManager(),
186	7		$resource->getForm(),
187	7		$resource->getChoiceForm(),
188	7		$resource->getIdPropertyPath(),
189	7		$resource->getLabelPropertyPath(),
190	7		];
191
192	7		if ($resource->getTranslation() !== null) {
193			$arguments[] = new Reference('lug.resource.'.$resource->getTranslation()->getName());
194			}
195
196	7		$definition = new Definition(Resource::class, $arguments);
197	7		$definition->addTag('lug.resource');
198
199	7		return $definition;
200			}
201
202			/**
203			* @param ResourceInterface $resource
204			*
205			* @return string
206			*/
207	7		private function createManagerAlias(ResourceInterface $resource)
208			{
209	7		if ($resource->getDriver() === ResourceInterface::DRIVER_DOCTRINE_MONGODB) {
210			return 'doctrine_mongodb.odm.'.$resource->getDriverManager().'_document_manager';
211			}
212
213	7		return 'doctrine.orm.'.$resource->getDriverManager().'_entity_manager';
214			}
215
216			/**
217			* @param ResourceInterface $resource
218			*
219			* @return Definition
220			*/
221	7		private function createFactoryDefinition(ResourceInterface $resource)
222			{
223	7		$arguments = [new Reference('lug.resource.'.$resource->getName())];
224
225	7		if (is_a($resource->getFactory(), TranslatableFactory::class, true)) {
226			$arguments[] = new Reference('lug.translation.context.locale');
227			}
228
229	7		$definition = new Definition($resource->getFactory(), $arguments);
230	7		$definition->addTag('lug.factory', ['resource' => $resource->getName()]);
231
232	7		return $definition;
233			}
234
235			/**
236			* @param ResourceInterface $resource
237			*
238			* @return Definition
239			*/
240	7	View Code Duplication	private function createRepositoryDefinition(ResourceInterface $resource)
			0 ignored issues – show Duplication introduced 2016-01-19 20:19 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
241			{
242	7		$definition = new Definition($resource->getRepository(), [$resource->getModel()]);
243	7		$definition->setFactory([new Reference('lug.manager.'.$resource->getName()), 'getRepository']);
244	7		$definition->addTag('lug.repository', ['resource' => $resource->getName()]);
245
246	7		return $definition;
247			}
248
249			/**
250			* @param ResourceInterface $resource
251			*
252			* @return Definition
253			*/
254	7	View Code Duplication	private function createFormDefinition(ResourceInterface $resource)
			0 ignored issues – show Duplication introduced 2016-01-19 20:19 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
255			{
256	7		$definition = new Definition($resource->getForm(), [
257	7		new Reference('lug.resource.'.$resource->getName()),
258	7		new Reference('lug.factory.'.$resource->getName()),
259	7		]);
260
261	7		$definition->addTag('form.type');
262
263	7		return $definition;
264			}
265
266			/**
267			* @param ResourceInterface $resource
268			*
269			* @return Definition
270			*/
271	7	View Code Duplication	private function createChoiceFormDefinition(ResourceInterface $resource)
			0 ignored issues – show Duplication introduced 2016-01-19 20:19 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
272			{
273	7		$definition = new Definition($resource->getChoiceForm(), [
274	7		new Reference('lug.resource.'.$resource->getName()),
275	7		]);
276
277	7		$definition->addTag('form.type');
278
279	7		return $definition;
280			}
281
282			/**
283			* @param ResourceInterface $resource
284			*
285			* @return Definition
286			*/
287	7	View Code Duplication	private function createControllerDefinition(ResourceInterface $resource)
			0 ignored issues – show Duplication introduced 2016-01-19 20:19 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
288			{
289	7		$definition = new Definition($resource->getController(), [
290	7		new Reference('lug.resource.'.$resource->getName()),
291	7		]);
292
293	7		$definition->addMethodCall('setContainer', [new Reference('service_container')]);
294	7		$definition->addTag('lug.controller', ['resource' => $resource->getName()]);
295
296	7		return $definition;
297			}
298
299			/**
300			* @param ResourceInterface $resource
301			*
302			* @return Definition
303			*/
304	7		private function createDomainManagerDefinition(ResourceInterface $resource)
305			{
306	7		$definition = new Definition($resource->getDomainManager(), [
307	7		new Reference('lug.resource.'.$resource->getName()),
308	7		new Reference('event_dispatcher'),
309	7		new Reference('lug.manager.'.$resource->getName()),
310	7		new Reference('lug.repository.'.$resource->getName()),
311	7		]);
312
313	7		$definition->addTag('lug.domain_manager', ['resource' => $resource->getName()]);
314
315	7		return $definition;
316			}
317			}
318

php-lug / lug

Pull Request — master (#36)

ResourceExtension::getAlias() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like