yiicod\listener\Listener

Complexity

Total Complexity

9

Size/Duplication

Total Lines	69
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
C	init()	0	22	7
A	bootstrap()	0	4	1
A	hasHandler()	0	15	1

<?php

namespace yiicod\listener;

use Yii;
use yii\base\BootstrapInterface;
use yii\base\Component;
use yii\base\Event;
use yii\helpers\ArrayHelper;

/**
 * Class Listener
 *
 * @package yiicod\listener\components
 */
class Listener extends Component implements BootstrapInterface
{
    public $eventAliases = [
        '@app/config/listeners.php',
    ];

    /**
     * @var array
     */
    private static $_events;

Consider using a different property name as you override a private property of the parent class.

    public function init()
    {
        $events = [];
        static::$_events = [];

Since $_events is declared private, accessing it with static will lead to errors in possible sub-classes; consider using self, or increasing the visibility of $_events to at least protected.

        foreach ($this->eventAliases as $alias) {
            if (file_exists(Yii::getAlias($alias))) {
                $events = ArrayHelper::merge($events, include(Yii::getAlias($alias)));
            }
        }
        ksort($events);
        foreach ($events as $priority => $keys) {
            foreach ($keys as $event => $listeners) {
                foreach ($listeners as $handler) {
                    list($class, $name) = explode('@', $event);
                    if (false === self::hasHandler($class, $name, $handler)) {
                        static::$_events[$class][$name][] = $handler;

Since $_events is declared private, accessing it with static will lead to errors in possible sub-classes; consider using self, or increasing the visibility of $_events to at least protected.

                        Event::on($class, $name, $handler);
                    }
                }
            }
        }
    }

    /**
     * Register events
     *
     * @param \yii\base\Application $app
     */
    public function bootstrap($app)
    {
        // Called in the init method
    }

    /**
     * Check event list if handler exists for class.
     *
     * @param $class
     * @param $name
     * @param $handler
     *
     * @return bool
     */
    public static function hasHandler(string $class, string $name, array $handler): bool
    {
        $result = array_filter((static::$_events[$class][$name] ?? []), function ($item) use ($handler) {

Since $_events is declared private, accessing it with static will lead to errors in possible sub-classes; consider using self, or increasing the visibility of $_events to at least protected.

            $item = array_map(function ($value) {
                return trim($value, '\\');
            }, $item);
            $handler = array_map(function ($value) {

Consider using a different name than the imported variable $handler, or did you forget to import by reference?

                return trim($value, '\\');
            }, $handler);

            return $item === $handler;
        });

        return count($result) > 0;
    }
}