ContainerAwareEventBusFactory::make() - Code Metrics - Innmind/EventBusBundle - Measure and Improve Code Quality continuously with Scrutinizer

ContainerAwareEventBusFactory::make() A
last analyzed 2017-02-12 13:32 UTC

↳ Parent: ContainerAwareEventBusFactory

Complexity

Conditions	3
Paths	3

Size

Total Lines	19
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	3

Importance

Changes

Metric	Value
dl	0
loc	19
ccs	8
cts	8
cp	1
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	12
nc	3
nop	3
crap	3

<?php
declare(strict_types = 1);

namespace Innmind\EventBusBundle\Factory;

use Innmind\EventBusBundle\ContainerAwareEventBus;
use Innmind\EventBus\ClassName\ExtractorInterface;
use Innmind\Immutable\{
    Map,
    SetInterface,
    Set
};
use Symfony\Component\DependencyInjection\ContainerInterface;

final class ContainerAwareEventBusFactory
{
    public static function make(
        ContainerInterface $container,
        array $services,
        ExtractorInterface $extractor
    ): ContainerAwareEventBus {
        $map = new Map('string', SetInterface::class);

        foreach ($services as $class => $listeners) {
            $set = new Set('string');

            foreach ($listeners as $listener) {
                $set = $set->add($listener);
            }

            $map = $map->put($class, $set);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        return new ContainerAwareEventBus($container, $map, $extractor);
    }
}


1		<?php
2		declare(strict_types = 1);
3
4		namespace Innmind\EventBusBundle\Factory;
5
6		use Innmind\EventBusBundle\ContainerAwareEventBus;
7		use Innmind\EventBus\ClassName\ExtractorInterface;
8		use Innmind\Immutable\{
9		Map,
10		SetInterface,
11		Set
12		};
13		use Symfony\Component\DependencyInjection\ContainerInterface;
14
15		final class ContainerAwareEventBusFactory
16		{
17	3	public static function make(
18		ContainerInterface $container,
19		array $services,
20		ExtractorInterface $extractor
21		): ContainerAwareEventBus {
22	3	$map = new Map('string', SetInterface::class);
23
24	3	foreach ($services as $class => $listeners) {
25	3	$set = new Set('string');
26
27	3	foreach ($listeners as $listener) {
28	3	$set = $set->add($listener);
29		}
30
31	3	$map = $map->put($class, $set);
		2 ignored issues – show Documentation introduced 2017-02-12 13:28 UTC by Report Bug Copy Issue Report `$class` is of type `integer\|string`, but the function expects a `object<Innmind\Immutable\T>`. 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... Documentation introduced 2017-02-12 13:28 UTC by Report Bug Copy Issue Report `$set` is of type `object<Innmind\Immutable\Set>`, but the function expects a `object<Innmind\Immutable\S>`. 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...
32		}
33
34	3	return new ContainerAwareEventBus($container, $map, $extractor);
35		}
36		}
37

Innmind / EventBusBundle

ContainerAwareEventBusFactory::make() A last analyzed 2017-02-12 13:32 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

ContainerAwareEventBusFactory::make() A
last analyzed 2017-02-12 13:32 UTC