Nopolabs\Yabot\Plugin\PluginMatcher

Complexity

Total Complexity

10

Size/Duplication

Total Lines	53
Duplicated Lines	22.64 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	13	1
C	matches()	12	24	8
A	debug()	0	4	1

<?php

namespace Nopolabs\Yabot\Plugin;


use Nopolabs\Yabot\Helpers\LogTrait;
use Nopolabs\Yabot\Message\Message;
use Psr\Log\LoggerInterface;
use Psr\Log\LogLevel;

class PluginMatcher
{
    use LogTrait;

    private $pluginId;
    private $isBot;
    private $channels;
    private $users;

    public function __construct(
        string $pluginId,
        bool $isBot = null,
        array $channels,
        array $users,
        LoggerInterface $logger)
    {
        $this->pluginId = $pluginId;
        $this->isBot = $isBot;
        $this->channels = $channels;
        $this->users = $users;
        $this->setLog($logger);
    }

    public function matches(Message $message) : bool
    {
        if ($message->isHandled()) {
            $this->debug('message already handled');
            return false;
        }

        if ($this->isBot !== null && $this->isBot !== $message->isBot()) {

This code seems to be duplicated across your project.

            $this->debug('isBot match failed');
            return false;
        }

        if ($this->channels && !in_array($message->getChannelName(), $this->channels)) {

The expression $this->channels 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 code seems to be duplicated across your project.

            $this->debug('channels match failed '.json_encode($this->channels));
            return false;
        }

        if ($this->users && !in_array($message->getUsername(), $this->users)) {

The expression $this->users 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 code seems to be duplicated across your project.

            $this->debug('users match failed '.json_encode($this->users));
            return false;
        }

        return true;
    }

    private function debug($msg)
    {
        $this->log(LogLevel::DEBUG, "PluginMatcher {$this->pluginId}: $msg");
    }
}