RendererFactory::createService() - Code Metrics - bramstroker/zf2-form - Measure and Improve Code Quality continuously with Scrutinizer

RendererFactory::createService() A
last analyzed 2017-06-01 07:23 UTC

↳ Parent: RendererFactory

Complexity

Conditions	3
Paths	3

Size

Total Lines	19
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

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

<?php
/**
 * Description
 *
 * @category  Acsi
 * @package   Acsi\
 * @copyright 2012 Bram Gerritsen
 * @version   SVN: $Id$
 */

namespace StrokerForm\Factory;

use StrokerForm\FormManager;
use StrokerForm\Options\ModuleOptions;
use StrokerForm\Renderer\RendererCollection;
use StrokerForm\Renderer\RendererInterface;
use Zend\ServiceManager\FactoryInterface;
use Zend\ServiceManager\ServiceLocatorInterface;

class RendererFactory implements FactoryInterface
{
    /**
     * Create service
     *
     * @param  ServiceLocatorInterface $serviceLocator
     *
     * @return RendererInterface
     */
    public function createService(ServiceLocatorInterface $serviceLocator)
    {
        /** @var $options ModuleOptions */
        $options = $serviceLocator->get(ModuleOptions::class);
        $rendererCollection = new RendererCollection();
        foreach ($options->getActiveRenderers() as $rendererAlias) {
            /** @var $renderer RendererInterface */
            $renderer = $serviceLocator->get($rendererAlias);
            $renderer->setDefaultOptions($options->getRendererOptions($rendererAlias));
            $renderer->setFormManager($serviceLocator->get(FormManager::class));
            if ($serviceLocator->has('translator')) {
                $renderer->setTranslator($serviceLocator->get('translator'));
            }
            $renderer->setHttpRouter($serviceLocator->get('HttpRouter'));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $rendererCollection->addRenderer($renderer);
        }

        return $rendererCollection;
    }
}


1			<?php
2			/**
3			* Description
4			*
5			* @category Acsi
6			* @package Acsi\
7			* @copyright 2012 Bram Gerritsen
8			* @version SVN: $Id$
9			*/
10
11			namespace StrokerForm\Factory;
12
13			use StrokerForm\FormManager;
14			use StrokerForm\Options\ModuleOptions;
15			use StrokerForm\Renderer\RendererCollection;
16			use StrokerForm\Renderer\RendererInterface;
17			use Zend\ServiceManager\FactoryInterface;
18			use Zend\ServiceManager\ServiceLocatorInterface;
19
20			class RendererFactory implements FactoryInterface
21			{
22			/**
23			* Create service
24			*
25			* @param ServiceLocatorInterface $serviceLocator
26			*
27			* @return RendererInterface
28			*/
29			public function createService(ServiceLocatorInterface $serviceLocator)
30			{
31			/** @var $options ModuleOptions */
32			$options = $serviceLocator->get(ModuleOptions::class);
33			$rendererCollection = new RendererCollection();
34			foreach ($options->getActiveRenderers() as $rendererAlias) {
35			/** @var $renderer RendererInterface */
36			$renderer = $serviceLocator->get($rendererAlias);
37			$renderer->setDefaultOptions($options->getRendererOptions($rendererAlias));
38			$renderer->setFormManager($serviceLocator->get(FormManager::class));
39			if ($serviceLocator->has('translator')) {
40			$renderer->setTranslator($serviceLocator->get('translator'));
41			}
42			$renderer->setHttpRouter($serviceLocator->get('HttpRouter'));
			0 ignored issues – show Documentation introduced 2016-02-20 09:35 UTC by Report Bug Copy Issue Report `$serviceLocator->get('HttpRouter')` is of type `object\|array`, but the function expects a `object<Zend\Mvc\Router\RouteInterface>`. 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...
43			$rendererCollection->addRenderer($renderer);
44			}
45
46			return $rendererCollection;
47			}
48			}
49

bramstroker / zf2-form

GitHub Access Token became invalid

RendererFactory::createService() A last analyzed 2017-06-01 07:23 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

RendererFactory::createService() A
last analyzed 2017-06-01 07:23 UTC