StorageFactory::__invoke() - Code Metrics - doctrine/DoctrineModule - Measure and Improve Code Quality continuously with Scrutinizer

StorageFactory::__invoke() A
last analyzed 2020-06-16 07:36 UTC

↳ Parent: StorageFactory

Complexity

Conditions	3
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	3.0123

Importance

Changes

Metric	Value
dl	0
loc	16
ccs	8
cts	9
cp	0.8889
rs	9.7333
c	0
b	0
f	0
cc	3
nc	4
nop	3
crap	3.0123

<?php

declare(strict_types=1);

namespace DoctrineModule\Service\Authentication;

use DoctrineModule\Authentication\Storage\ObjectRepository;
use DoctrineModule\Service\AbstractFactory;
use Interop\Container\ContainerInterface;
use Laminas\ServiceManager\ServiceLocatorInterface;
use function is_string;

/**
 * Factory to create authentication storage object.
 *
 * @link    http://www.doctrine-project.org/
 */
class StorageFactory extends AbstractFactory
{
    /**
     * {@inheritDoc}
     */
    public function __invoke(ContainerInterface $container, $requestedName, ?array $options = null)
    {
        $options = $this->getOptions($container, 'authentication');

        $objectManager = $options->getObjectManager();
        if (is_string($objectManager)) {
            $options->setObjectManager($container->get($objectManager));
        }

        $storage = $options->getStorage();
        if (is_string($storage)) {
            $options->setStorage($container->get($storage));
        }

        return new ObjectRepository($options);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * {@inheritDoc}
     *
     * @return ObjectRepository
     */
    public function createService(ServiceLocatorInterface $container)
    {
        return $this($container, ObjectRepository::class);
    }

    public function getOptionsClass() : string
    {
        return 'DoctrineModule\Options\Authentication';
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace DoctrineModule\Service\Authentication;
6
7		use DoctrineModule\Authentication\Storage\ObjectRepository;
8		use DoctrineModule\Service\AbstractFactory;
9		use Interop\Container\ContainerInterface;
10		use Laminas\ServiceManager\ServiceLocatorInterface;
11		use function is_string;
12
13		/**
14		* Factory to create authentication storage object.
15		*
16		* @link http://www.doctrine-project.org/
17		*/
18		class StorageFactory extends AbstractFactory
19		{
20		/**
21		* {@inheritDoc}
22		*/
23	3	public function __invoke(ContainerInterface $container, $requestedName, ?array $options = null)
24		{
25	3	$options = $this->getOptions($container, 'authentication');
26
27	3	$objectManager = $options->getObjectManager();
28	3	if (is_string($objectManager)) {
29		$options->setObjectManager($container->get($objectManager));
30		}
31
32	3	$storage = $options->getStorage();
33	3	if (is_string($storage)) {
34	3	$options->setStorage($container->get($storage));
35		}
36
37	3	return new ObjectRepository($options);
		0 ignored issues – show Documentation introduced 2020-04-08 18:49 UTC by Report Bug Copy Issue Report `$options` is of type `object<Laminas\Stdlib\AbstractOptions>`, but the function expects a `array<integer,*>\|object<...Options\Authentication>`. 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...
38		}
39
40		/**
41		* {@inheritDoc}
42		*
43		* @return ObjectRepository
44		*/
45	2	public function createService(ServiceLocatorInterface $container)
46		{
47	2	return $this($container, ObjectRepository::class);
48		}
49
50	3	public function getOptionsClass() : string
51		{
52	3	return 'DoctrineModule\Options\Authentication';
53		}
54		}
55

doctrine / DoctrineModule

StorageFactory::__invoke() A last analyzed 2020-06-16 07:36 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

StorageFactory::__invoke() A
last analyzed 2020-06-16 07:36 UTC