eXpansion\Bundle\CustomChat\Plugins\CustomChat

Complexity

Total Complexity

26

Size/Duplication

Total Lines	243
Duplicated Lines	7.82 %

Coupling/Cohesion

Components	1
Dependencies	8

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	1	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	17	1
D	onPlayerChat()	10	81	17
B	sendChat()	0	41	3
A	setStatus()	0	4	1
A	onApplicationInit()	0	4	1
A	onApplicationReady()	9	9	2
A	onApplicationStop()	0	4	1

<?php

namespace eXpansion\Bundle\CustomChat\Plugins;


use eXpansion\Framework\AdminGroups\Helpers\AdminGroups;
use eXpansion\Framework\Core\DataProviders\Listener\ListenerInterfaceExpApplication;
use eXpansion\Framework\Core\Helpers\ChatNotification;
use eXpansion\Framework\Core\Services\Console;
use eXpansion\Framework\Core\Storage\Data\Player;
use eXpansion\Framework\Core\Storage\PlayerStorage;
use eXpansion\Framework\GameManiaplanet\DataProviders\Listener\ListenerInterfaceMpLegacyChat;
use eXpansion\Framework\Notifications\Services\Notifications;
use Maniaplanet\DedicatedServer\Connection;
use Psr\Log\LoggerInterface;


class CustomChat implements ListenerInterfaceExpApplication, ListenerInterfaceMpLegacyChat
{
    /** @var Connection */
    protected $connection;

    /** @var Console */
    protected $console;

    /** @var AdminGroups */
    protected $adminGroups;

    /** @var bool */
    protected $enabled = true;
    /**
     * @var ChatNotification
     */
    private $chatNotification;
    /**
     * @var PlayerStorage
     */
    private $playerStorage;
    /**
     * @var LoggerInterface
     */
    private $logger;
    /**
     * @var Notifications
     */
    private $notifications;

    /**
     * CustomChat constructor.
     * @param Connection       $connection
     * @param Console          $console
     * @param AdminGroups      $adminGroups
     * @param ChatNotification $chatNotification
     * @param PlayerStorage    $playerStorage
     * @param Notifications    $notifications
     * @param LoggerInterface  $logger
     */
    function __construct(

It is generally recommended to explicitly declare the visibility for methods.

        Connection $connection,
        Console $console,
        AdminGroups $adminGroups,
        ChatNotification $chatNotification,
        PlayerStorage $playerStorage,
        Notifications $notifications,
        LoggerInterface $logger
    ) {
        $this->connection = $connection;
        $this->console = $console;
        $this->adminGroups = $adminGroups;
        $this->chatNotification = $chatNotification;
        $this->playerStorage = $playerStorage;
        $this->logger = $logger;
        $this->notifications = $notifications;
    }

    /**
     * Called when a player chats.
     *
     * @param Player $player
     * @param        $text
     *
     * @return void
     */
    public function onPlayerChat(Player $player, $text)
    {
        $text = trim($text);
        $nick = trim($player->getNickName());

        if ($player->getPlayerId() == 0) {
            return;
        }

        if ($player->getPlayerId() != 0 && substr($text, 0, 1) != "/") {
            $matches = [];

            if ($this->enabled || $this->adminGroups->isAdmin($player->getLogin())) {
                $matchFound = false;
                $matchLogin = [];

                //match urls and shorten them to fit in chat.
                if (preg_match_all('/(\$l\[?){0,1}(https?:\/\/(www\.)?[-a-zA-Z0-9@:%._\+~#=]{2,256}\.[a-z]{2,6})\b([-a-zA-Z0-9@:%_\+.~#?&\/\/=]*)(\]?)/',
                    $text, $urls)) {
                    foreach ($urls[0] as $k => $url) {
                        if ($urls[1][$k] != '$l[') {
                            $url = str_replace('$l', '', $url);
                            $text = str_replace('$l', '', $text);

                            if (strlen($url) >= 33) {
                                $short = substr($url, 0, 30)."...";
                                $text = str_replace($url, '$l['.$url.']'.$short.'$l', $text);
                            } else {
                                $text = str_replace($url, '$l'.$url.'$l', $text);
                            }
                        }

                    }

                }

                if (preg_match_all("/(\s|\G)(\@(?P<login>[\w-\._]+)[\s]{0,1})/", $text, $matches)) {
                    $group = [];

                    foreach ($matches['login'] as $login) {
                        foreach ($this->playerStorage->getOnline() as $player2) {
                            if ($player2->getLogin() == $login) {
                                $matchFound = true;
                                $matchLogin[$player2->getLogin()] = $player2->getLogin();
                            } else {
                                if (!in_array($player->getLogin(), $matchLogin)) {
                                    $group[$player2->getLogin()] = $player2->getLogin();
                                }
                            }
                        }
                    }

                    $diff = array_diff($group, $matchLogin);

                    if ($matchFound) {
                        $this->sendChat($player, $text, '$f90', $matchLogin);
                        $this->notifications->notice($text, [], "Chat Notification", 0, $matchLogin);

$matchLogin is of type array, but the function expects a string|object<eXpansion\Framework\Core\Model\UserGroups\Group>|null.

                        if (count($diff) > 0) {
                            $this->sendChat($player, $text, '$ff0', $group);
                        }
                        $this->console->writeln('$ff0['.$nick.'$ff0] '.$text);

                        return;
                    } else {

This code seems to be duplicated across your project.

                        $this->sendChat($player, $text, '$ff0', null);
                        $this->console->writeln('$ff0['.$nick.'$ff0] '.$text);

                        return;
                    }
                } else {

This code seems to be duplicated across your project.

                    $this->sendChat($player, $text, '$ff0', null);
                    $this->console->writeln('$ff0['.$nick.'$ff0] '.$text);
                }
            } else {
                $this->console->writeln('$333['.$nick.'$333] '.$text);
                $this->chatNotification->sendMessage('expansion_customchat.chat.disabledstate',
                    $player->getLogin());
            }
        }

    }

    /**
     * @param Player        $player
     * @param        string $text
     * @param        string $color
     * @param null          $group
     */
    private function sendChat(Player $player, $text, $color, $group = null)
    {
        $nick = trim($player->getNickName());
        $nick = str_ireplace('$w', '', $nick);
        $nick = str_ireplace('$z', '$z$s', $nick);
        $replacements = [
            "(y)" => "",
            ":yes:" => "",
            ":thumbsup:" => "",
            ":no:" => "",
            ":thumbsdown:" => "",
            "(n)" => "",
            ":happy:" => "",
            ":smile:" => "",
            ":sad:" => "",
            ":heart:" => '$d00$z$s',
            "<3" => '$d00$z$s',

        ];
        // fix for chat...
        $nick = str_replace(['$<', '$>'], '', $nick);
        $text = str_replace(['$<', '$>'], '', $text);
        $text = str_replace(array_keys($replacements), array_values($replacements), $text);
        $separator = '$aaa⏵';
        $prefix = '';
        $postfix = '';
        if ($this->adminGroups->isAdmin($player->getLogin())) {
            $separator = ' $eed$n►$z$s';
            $prefix = '';
            $postfix = '';
        }

        try {
            $this->connection->chatSendServerMessage(
                $prefix.'$fff$<'.$nick.'$>$z$s'.$postfix.$separator.' '.$color.$text, $group
            );
        } catch (\Exception $e) {
            $this->console->writeln('$ff0 error while sending chat: $fff'.$e->getMessage());
            $this->logger->error("Error while sending custom chat", ['exception' => $e]);
        }
    }

    /**
     * Set the status of the plugin
     *
     * @param boolean $status
     *
     * @return void
     */
    public function setStatus($status)
    {
        $this->enabled = $status;
    }

    /**
     * called at eXpansion init
     *
     * @return void
     */
    public function onApplicationInit()
    {
        // do nothing
    }

    /**
     * called when init is done and callbacks are enabled
     *
     * @return void
     */
    public function onApplicationReady()

This method seems to be duplicated in your project.

    {
        try {
            $this->connection->chatEnableManualRouting();
        } catch (\Exception $e) {
            $this->console->writeln('Error while enabling custom chat: $f00'.$e->getMessage());
            $this->logger->error("Error enabling custom chat", ['exception' => $e]);
        }
    }

    /**
     * called when requesting application stop
     *
     * @return void
     */
    public function onApplicationStop()
    {
        // do nothing
    }
}