TraitObservable::on() - Code Metrics - web-complete/core - Measure and Improve Code Quality continuously with Scrutinizer

TraitObservable::on() A
last analyzed 2018-04-25 08:52 UTC

↳ Parent: TraitObservable

Complexity

Conditions	2
Paths	2

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	9
rs	9.6666
c	0
b	0
f	0
cc	2
eloc	5
nc	2
nop	3

<?php

namespace WebComplete\core\utils\traits;

trait TraitObservable
{

    private $listeners = [];

    /**
     * @param string $eventName
     * @param array $eventData
     */
    public function trigger(string $eventName, array $eventData)
    {
        if ($this->listeners) {

            $listeners = $this->listeners[$eventName] ?? [];
            foreach ($listeners as $listener) {
                \call_user_func($listener[1], $eventData);
            }
        }
    }

    /**
     * @param string $eventName
     * @param callable $callable
     * @param int $priority
     */
    public function on(string $eventName, callable $callable, int $priority = 100)
    {
        if (!isset($this->listeners[$eventName])) {
            $this->listeners[$eventName] = [];
        }

        $this->listeners[$eventName][] = [$priority, $callable];
        \uasort($this->listeners[$eventName], function ($event1, $event2) {
            return $event1[0] <=> $event2[0];
        });
    }

    /**
     * @return array
     */
    public function getListeners(): array
    {
        return $this->listeners;
    }

    /**
     * @param array $listeners
     */
    public function setListeners(array $listeners)
    {
        $this->listeners = $listeners;
    }
}


1			<?php
2
3			namespace WebComplete\core\utils\traits;
4
5			trait TraitObservable
6			{
7
8			private $listeners = [];
9
10			/**
11			* @param string $eventName
12			* @param array $eventData
13			*/
14			public function trigger(string $eventName, array $eventData)
15			{
16			if ($this->listeners) {
			0 ignored issues – show Bug Best Practice introduced 2017-12-23 18:16 UTC by Report Bug Copy Issue Report The expression `$this->listeners` of type `array` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `! empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
17			$listeners = $this->listeners[$eventName] ?? [];
18			foreach ($listeners as $listener) {
19			\call_user_func($listener[1], $eventData);
20			}
21			}
22			}
23
24			/**
25			* @param string $eventName
26			* @param callable $callable
27			* @param int $priority
28			*/
29			public function on(string $eventName, callable $callable, int $priority = 100)
30			{
31			if (!isset($this->listeners[$eventName])) {
32			$this->listeners[$eventName] = [];
33			}
34
35			$this->listeners[$eventName][] = [$priority, $callable];
36			\uasort($this->listeners[$eventName], function ($event1, $event2) {
37			return $event1[0] <=> $event2[0];
38			});
39			}
40
41			/**
42			* @return array
43			*/
44			public function getListeners(): array
45			{
46			return $this->listeners;
47			}
48
49			/**
50			* @param array $listeners
51			*/
52			public function setListeners(array $listeners)
53			{
54			$this->listeners = $listeners;
55			}
56			}
57

web-complete / core

TraitObservable::on() A last analyzed 2018-04-25 08:52 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

TraitObservable::on() A
last analyzed 2018-04-25 08:52 UTC