eXpansion\Framework\Core\Helpers\ChatNotification

Complexity

Total Complexity

4

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	14	2
A	sendMessage()	0	13	2

<?php

namespace eXpansion\Framework\Core\Helpers;

use eXpansion\Framework\Core\Model\Helpers\ChatNotificationInterface;
use eXpansion\Framework\Core\Storage\PlayerStorage;
use Maniaplanet\DedicatedServer\Connection;

/**
 * Class ChatNotification
 *
 * @package eXpansion\Framework\Core\Helpers;
 * @author oliver de Cramer <oliverde8@gmail.com>
 */
class ChatNotification implements ChatNotificationInterface
{
    /** @var  Connection */
    protected $connection;

    /** @var Translations */
    protected $translations;

    /** @var PlayerStorage */
    protected $playerStorage;

    protected $colorCodes = [];

    /**
     * ChatNotification constructor.
     *
     * @param Connection $connection
     * @param Translations $translations
     */
    public function __construct(
        Connection $connection,
        Translations $translations,
        PlayerStorage $playerStorage,
        $colorCodes
    ){
        $this->connection = $connection;
        $this->translations = $translations;
        $this->playerStorage = $playerStorage;

        foreach ($colorCodes as $code => $colorCode) {
            $this->colorCodes["{" . $code . "}"] = '$z' . $colorCode;
        }
    }

    /**
     * Send message.
     *
     * @param string $messageId
     * @param string|string[]|null $to
     * @param string[] $parameters
     */
    public function sendMessage($messageId, $to = null, $parameters = [])
    {
        $parameters = array_merge($this->colorCodes, $parameters);

        if (is_string($to)) {
            $player = $this->playerStorage->getPlayerInfo($to);
            $message = $this->translations->getTranslation($messageId, $parameters, strtolower($player->getLanguage()));
        } else {
            $message = $this->translations->getTranslations($messageId, $parameters);
        }

        $this->connection->chatSendServerMessage($message, $to);

It seems like $message defined by $this->translations->getTranslations($messageId, $parameters) on line 64 can also be of type array; however, Maniaplanet\DedicatedServer\Connection::chatSendServerMessage() does only seem to accept string|array<integer,array<integer,string>>, maybe add an additional type check?

    }
}