ContainerHandlerProvider::__construct() - Code Metrics - Inspection of "Replace broken doc links with internal ones" - ICanBoogie/MessageBus - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 3e3d3c...053aa5 )

by Olivier

created 2020-11-16 04:52 UTC

ContainerHandlerProvider::__construct() A

↳ Parent: ContainerHandlerProvider

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	5
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	2

<?php

/*
 * This file is part of the ICanBoogie package.
 *
 * (c) Olivier Laviale <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace ICanBoogie\MessageBus\PSR;

use ICanBoogie\MessageBus\HandlerProvider;
use ICanBoogie\MessageBus\NotFound;
use Psr\Container\ContainerInterface;
use Psr\Container\NotFoundExceptionInterface;

use function get_class;

class ContainerHandlerProvider implements HandlerProvider
{
    /**
     * @var ContainerInterface
     */
    private $container;

    /**
     * @var array<string, string>
     */
    private $handlers;

    /**
     * @param array<string, string> $mapping
     *   An array of key/value pairs where _key_ is a message class and _value_ the service
     *   identifier of its handler.
     */
    public function __construct(ContainerInterface $container, array $mapping)
    {
        $this->handlers = $mapping;
        $this->container = $container;
    }

    public function getHandlerForMessage(object $message): callable
    {
        $class = get_class($message);
        $id = $this->handlers[$class] ?? null;

        if (!$id) {
            throw new NotFound("No handler for messages of type `$class`.");
        }

        try {
            return $this->container->get($id);
        } catch (NotFoundExceptionInterface $e) {
            throw new NotFound("No handler for messages of type `$class`.", $e);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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


1			<?php
2
3			/*
4			* This file is part of the ICanBoogie package.
5			*
6			* (c) Olivier Laviale <[email protected]>
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			namespace ICanBoogie\MessageBus\PSR;
13
14			use ICanBoogie\MessageBus\HandlerProvider;
15			use ICanBoogie\MessageBus\NotFound;
16			use Psr\Container\ContainerInterface;
17			use Psr\Container\NotFoundExceptionInterface;
18
19			use function get_class;
20
21			class ContainerHandlerProvider implements HandlerProvider
22			{
23			/**
24			* @var ContainerInterface
25			*/
26			private $container;
27
28			/**
29			* @var array<string, string>
30			*/
31			private $handlers;
32
33			/**
34			* @param array<string, string> $mapping
35			* An array of key/value pairs where _key_ is a message class and _value_ the service
36			* identifier of its handler.
37			*/
38			public function __construct(ContainerInterface $container, array $mapping)
39			{
40			$this->handlers = $mapping;
41			$this->container = $container;
42			}
43
44			public function getHandlerForMessage(object $message): callable
45			{
46			$class = get_class($message);
47			$id = $this->handlers[$class] ?? null;
48
49			if (!$id) {
50			throw new NotFound("No handler for messages of type `$class`.");
51			}
52
53			try {
54			return $this->container->get($id);
55			} catch (NotFoundExceptionInterface $e) {
56			throw new NotFound("No handler for messages of type `$class`.", $e);
			0 ignored issues – show Documentation introduced 2020-11-16 04:14 UTC by Report Bug Copy Issue Report `$e` is of type `object<Psr\Container\NotFoundExceptionInterface>`, but the function expects a `null\|object<Throwable>`. 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...
57			}
58			}
59			}
60

ICanBoogie / MessageBus

Push — master ( 3e3d3c...053aa5 )

ContainerHandlerProvider::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like