Skobkin\Bundle\PointToolsBundle\Service\Telegram\SimpleSender

Complexity

Total Complexity

3

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

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

<?php

namespace Skobkin\Bundle\PointToolsBundle\Service\Telegram;

use GuzzleHttp\Exception\ClientException;
use unreal4u\TelegramAPI\Telegram\Methods\SendMessage;
use unreal4u\TelegramAPI\TgLog;

/**
 * Service which sends simple messages to Telegram users
 */
class SimpleSender
{
    /**
     * @var TgLog
     */
    private $client;

    /**
     * @param TgLog $client
     */
    public function __construct(TgLog $client)
    {
        $this->client = $client;
    }

    /**
     * Send simple message
     *
     * @param int $chatId
     * @param string $text
     *
     * @return bool
     */
    public function sendMessage(int $chatId, string $text): bool
    {
        $sendMessage = new SendMessage();
        $sendMessage->chat_id = $chatId;

The property $chat_id was declared of type string, but $chatId is of type integer. Maybe add a type cast?

        $sendMessage->text = $text;

        try {
            $this->client->performApiRequest($sendMessage);

            return true;
        } catch (ClientException $e) {
            return false;
        }
    }
}