EventProfilerListener::detachShared() - Code Metrics - webino/WebinoDebug - Measure and Improve Code Quality continuously with Scrutinizer

EventProfilerListener::detachShared() A
last analyzed 2019-12-25 16:04 UTC

↳ Parent: EventProfilerListener

Complexity

Conditions	3
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	8
rs	10
cc	3
nc	3
nop	1

<?php
/**
 * Webino (http://webino.sk/)
 *
 * @link        https://github.com/webino/WebinoDebug/ for the canonical source repository
 * @copyright   Copyright (c) 2014-2017 Webino, s. r. o. (http://webino.sk/)
 * @license     BSD-3-Clause
 */

namespace WebinoDebug\Listener;

use WebinoDebug\Service\EventProfiler;
use Zend\EventManager\EventInterface;
use Zend\EventManager\SharedEventManagerInterface;
use Zend\EventManager\SharedListenerAggregateInterface;

/**
 * Class EventProfilerListener
 */
class EventProfilerListener implements SharedListenerAggregateInterface

{
    /**
     * Event highest priority
     */
    const HIGHEST_PRIORITY = 9999999999;

    /**
     * @var EventProfiler
     */
    protected $eventProfiler;

    /**
     * @var \Zend\Stdlib\CallbackHandler[]
     */
    protected $listeners = [];

    /**
     * @param object|EventProfiler $eventProfiler
     */
    public function __construct(EventProfiler $eventProfiler)
    {
        $this->eventProfiler = $eventProfiler;
    }

    /**
     * {@inheritdoc}
     */
    public function attachShared(SharedEventManagerInterface $events)
    {
        $this->listeners[] = $events->attach('*', '*', [$this, 'onAnyEvent'], self::HIGHEST_PRIORITY);
    }

    /**
     * {@inheritdoc}
     */
    public function detachShared(SharedEventManagerInterface $events)
    {
        foreach ($this->listeners as $key => $listener) {
            if ($events->detach('*', '*', $listener)) {
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                unset($this->listeners[$key]);
            }
        }
    }

    /**
     * @param EventInterface $event
     */
    public function onAnyEvent(EventInterface $event)
    {
        $this->eventProfiler->setEvent($event);
    }
}


1			<?php
2			/**
3			* Webino (http://webino.sk/)
4			*
5			* @link https://github.com/webino/WebinoDebug/ for the canonical source repository
6			* @copyright Copyright (c) 2014-2017 Webino, s. r. o. (http://webino.sk/)
7			* @license BSD-3-Clause
8			*/
9
10			namespace WebinoDebug\Listener;
11
12			use WebinoDebug\Service\EventProfiler;
13			use Zend\EventManager\EventInterface;
14			use Zend\EventManager\SharedEventManagerInterface;
15			use Zend\EventManager\SharedListenerAggregateInterface;
16
17			/**
18			* Class EventProfilerListener
19			*/
20			class EventProfilerListener implements SharedListenerAggregateInterface
			0 ignored issues – show Deprecated Code introduced 2016-02-15 09:55 UTC by Report Bug Copy Issue Report The interface `Zend\EventManager\SharedListenerAggregateInterface` has been deprecated with message: This interface is deprecated with 2.6.0, and will be removed in 3.0.0. See {@link https://github.com/zendframework/zend-eventmanager/blob/develop/doc/book/migration/removed.md} for details. This class, trait or interface has been deprecated. The supplier of the file has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the type will be removed from the class and what other constant to use instead. Loading history...
21			{
22			/**
23			* Event highest priority
24			*/
25			const HIGHEST_PRIORITY = 9999999999;
26
27			/**
28			* @var EventProfiler
29			*/
30			protected $eventProfiler;
31
32			/**
33			* @var \Zend\Stdlib\CallbackHandler[]
34			*/
35			protected $listeners = [];
36
37			/**
38			* @param object\|EventProfiler $eventProfiler
39			*/
40			public function __construct(EventProfiler $eventProfiler)
41			{
42			$this->eventProfiler = $eventProfiler;
43			}
44
45			/**
46			* {@inheritdoc}
47			*/
48			public function attachShared(SharedEventManagerInterface $events)
49			{
50			$this->listeners[] = $events->attach('', '', [$this, 'onAnyEvent'], self::HIGHEST_PRIORITY);
51			}
52
53			/**
54			* {@inheritdoc}
55			*/
56			public function detachShared(SharedEventManagerInterface $events)
57			{
58			foreach ($this->listeners as $key => $listener) {
59			if ($events->detach('', '', $listener)) {
			0 ignored issues – show Documentation introduced 2016-02-15 09:55 UTC by Report Bug Copy Issue Report `'*'` is of type `string`, but the function expects a `object<Zend\Stdlib\CallbackHandler>`. 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... Unused Code introduced 2016-02-15 09:55 UTC by Report Bug Copy Issue Report The call to `SharedEventManagerInterface::detach()` has too many arguments starting with `$listener`. This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue. If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. In this case you can add the `@ignore` PhpDoc annotation to the duplicate definition and it will be ignored. Loading history...
60			unset($this->listeners[$key]);
61			}
62			}
63			}
64
65			/**
66			* @param EventInterface $event
67			*/
68			public function onAnyEvent(EventInterface $event)
69			{
70			$this->eventProfiler->setEvent($event);
71			}
72			}
73

webino / WebinoDebug

EventProfilerListener::detachShared() A last analyzed 2019-12-25 16:04 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

EventProfilerListener::detachShared() A
last analyzed 2019-12-25 16:04 UTC