PimpleFactory::setFactories() - Code Metrics - Inspection of "Foundation -> Phoundation rename." - nikolaposa/phoundation - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( f4be61...d5c601 )

by Nikola

created 2017-02-19 14:17 UTC

PimpleFactory::setFactories() A

↳ Parent: PimpleFactory

Complexity

Conditions	3
Paths	2

Size

Total Lines	17
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	3.1406

Importance

Changes

Metric	Value
dl	0
loc	17
ccs	3
cts	4
cp	0.75
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	9
nc	2
nop	0
crap	3.1406

<?php
/**
 * This file is part of the Phoundation package.
 *
 * Copyright (c) Nikola Posa
 *
 * For full copyright and license information, please refer to the LICENSE file,
 * located at the package root folder.
 */

declare(strict_types=1);

namespace Phoundation\Di\Container\Factory;

use Xtreamwayz\Pimple\Container;

/**
 * @author Nikola Posa <[email protected]>
 */
final class PimpleFactory extends AbstractFactory
{
    /**
     * @var Container
     */
    private $container;

    protected function createContainer()
    {
        return new Container();
    }

    protected function configure($container)
    {
        $this->container = $container;

        $this->setFactories();
        $this->setInvokables();

        $this->container[$this->getConfigServiceName()] = $this->getConfig();
    }

    private function setFactories()
    {
        foreach ($this->getDiConfig('factories') as $name => $factory) {
            $this->container[$name] = function ($container) use ($factory, $name) {
                /* @var $container Container */

                if ($container->has($factory)) {
                    $factory = $container->get($factory);
$x = 1;
$callable = function() use ($x) {
    $x = 2; // Not visible in outer scope. If you would like this, how
            // about using a different variable name than $x?
};

$callable();
var_dump($x); // integer(1)
                } else {
                    $factory = new $factory();
$x = 1;
$callable = function() use ($x) {
    $x = 2; // Not visible in outer scope. If you would like this, how
            // about using a different variable name than $x?
};

$callable();
var_dump($x); // integer(1)
                    $container[$factory] = $container->protect($factory);
                }

                return $factory($container, $name);
            };
        }
    }

    private function setInvokables()
    {
        foreach ($this->getDiConfig('invokables') as $name => $className) {
            $this->container[$name] = function () use ($className) {
                return new $className();
            };
        }
    }
}


1		<?php
2		/**
3		* This file is part of the Phoundation package.
4		*
5		* Copyright (c) Nikola Posa
6		*
7		* For full copyright and license information, please refer to the LICENSE file,
8		* located at the package root folder.
9		*/
10
11		declare(strict_types=1);
12
13		namespace Phoundation\Di\Container\Factory;
14
15		use Xtreamwayz\Pimple\Container;
16
17		/**
18		* @author Nikola Posa <[email protected]>
19		*/
20		final class PimpleFactory extends AbstractFactory
21		{
22		/**
23		* @var Container
24		*/
25		private $container;
26
27	2	protected function createContainer()
28		{
29	2	return new Container();
30		}
31
32	2	protected function configure($container)
33		{
34	2	$this->container = $container;
35
36	2	$this->setFactories();
37	2	$this->setInvokables();
38
39	2	$this->container[$this->getConfigServiceName()] = $this->getConfig();
40	2	}
41
42	2	private function setFactories()
43		{
44	2	foreach ($this->getDiConfig('factories') as $name => $factory) {
45		$this->container[$name] = function ($container) use ($factory, $name) {
46		/* @var $container Container */
47
48		if ($container->has($factory)) {
49		$factory = $container->get($factory);
		0 ignored issues – show Bug introduced 2017-02-19 10:34 UTC by Report Bug Copy Issue Report Consider using a different name than the imported variable `$factory`, or did you forget to import by reference? It seems like you are assigning to a variable which was imported through a `use` statement which was not imported by reference. For clarity, we suggest to use a different name or import by reference depending on whether you would like to have the change visibile in outer-scope. Change not visible in outer-scope $x = 1; $callable = function() use ($x) { $x = 2; // Not visible in outer scope. If you would like this, how // about using a different variable name than $x? }; $callable(); var_dump($x); // integer(1) Change visible in outer-scope $x = 1; $callable = function() use (&$x) { $x = 2; }; $callable(); var_dump($x); // integer(2) Loading history...
50		} else {
51		$factory = new $factory();
		0 ignored issues – show Bug introduced 2017-02-19 10:34 UTC by Report Bug Copy Issue Report Consider using a different name than the imported variable `$factory`, or did you forget to import by reference? It seems like you are assigning to a variable which was imported through a `use` statement which was not imported by reference. For clarity, we suggest to use a different name or import by reference depending on whether you would like to have the change visibile in outer-scope. Change not visible in outer-scope $x = 1; $callable = function() use ($x) { $x = 2; // Not visible in outer scope. If you would like this, how // about using a different variable name than $x? }; $callable(); var_dump($x); // integer(1) Change visible in outer-scope $x = 1; $callable = function() use (&$x) { $x = 2; }; $callable(); var_dump($x); // integer(2) Loading history...
52		$container[$factory] = $container->protect($factory);
53		}
54
55		return $factory($container, $name);
56		};
57		}
58	2	}
59
60	2	private function setInvokables()
61		{
62	2	foreach ($this->getDiConfig('invokables') as $name => $className) {
63		$this->container[$name] = function () use ($className) {
64		return new $className();
65		};
66		}
67	2	}
68		}
69

nikolaposa / phoundation

Push — master ( f4be61...d5c601 )

PimpleFactory::setFactories() A

Complexity

Size

Duplication

Code Coverage

Importance

Change not visible in outer-scope

Change visible in outer-scope

Change not visible in outer-scope

Change visible in outer-scope

Duplication Side-by-Side

Filter issues like