T3Bot\Commands\TellCommand

Complexity

Total Complexity

19

Size/Duplication

Total Lines	152
Duplicated Lines	7.24 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

100%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	11	11	1
A	process()	0	11	2
B	processPresenceChange()	0	30	5
A	processReviewMessage()	0	18	4
A	processForgeMessage()	0	11	2
A	processTextMessage()	0	5	1
A	processTell()	0	20	4

<?php
/**
 * T3Bot.
 *
 * @author Frank Nägler <frank.naegler@typo3.org>
 *
 * @link http://www.t3bot.de
 * @link http://wiki.typo3.org/T3Bot
 */
namespace T3Bot\Commands;

use Slack\Payload;
use Slack\RealTimeClient;

/**
 * Class TellCommand.
 *
 * @property array helpCommands
 * @property string commandName
 */
class TellCommand extends AbstractCommand
{
    const PRESENCE_ACTIVE = 'active';
    const PRESENCE_AWAY = 'away';

    /**
     * AbstractCommand constructor.
     *
     * @param Payload        $payload
     * @param RealTimeClient $client
     */
    public function __construct(Payload $payload, RealTimeClient $client)

This method seems to be duplicated in your project.

    {
        $this->commandName = 'tell';
        $this->helpCommands = [
            'help' => 'shows this help',
            'tell [@to-user] about review:[Gerrit-ID]' => 'tell the target user about the given review',
            'tell [@to-user] about forge:[Issue-ID]' => 'tell the target user about the given forge issue',
            'tell [@to-user] [your message]' => 'tell the target user your message',
        ];
        parent::__construct($payload, $client);
    }

    /**
     * @return bool|string
     *
     * @throws \Doctrine\DBAL\DBALException
     */
    public function process()
    {
        $message = $this->payload->getData()['text'];
        $this->params = array_map('trim', explode(' ', $message));
        $result = false;
        if ($this->params[0] === 'tell') {
            $result = $this->processTell();
        }

        return $result;
    }

    /**
     * @param string $user
     * @param string $presence
     *
     * @throws \Doctrine\DBAL\DBALException
     */
    public function processPresenceChange($user, $presence)
    {
        if ($presence === self::PRESENCE_ACTIVE) {
            $queryBuilder = $this->getDatabaseConnection()->createQueryBuilder();
            $notifications = $queryBuilder
                ->select('*')
                ->from('notifications')
                ->where($queryBuilder->expr()->eq('to_user', $queryBuilder->createNamedParameter($user)))
                ->andWhere($queryBuilder->expr()->eq('delivered', $queryBuilder->createNamedParameter('0000-00-00 00:00:00')))
                ->execute()
                ->fetchAll();
            foreach ($notifications as $notification) {
                if (strpos($notification['message'], 'review:') === 0) {
                    $this->processReviewMessage($notification, $user);
                } elseif (strpos($notification['message'], 'forge:') === 0) {
                    $this->processForgeMessage($notification, $user);
                } else {
                    $this->processTextMessage($notification, $user);
                }
                $now = new \DateTime();
                $now->setTimestamp(time());
                $this->getDatabaseConnection()->update(
                    'notifications',
                    ['delivered' => $now],
                    ['id' => $notification['id']],
                    ['datetime']
                );
            }
        }
    }

    /**
     * @param array $notification
     * @param string $user
     */
    protected function processReviewMessage(array $notification, string $user)
    {
        $parts = explode(':', $notification['message']);
        $refId = (int) trim($parts[1]);
        $result = $this->queryGerrit('change:' . $refId);
        $msg = '*Hi <@' . $user . '>, <@' . $notification['from_user'] . '>'
            . ' ask you to look at this patch:*';

        if (is_array($result)) {
            foreach ($result as $item) {
                if ((int) $item->_number === $refId) {
                    $message = $this->buildReviewMessage($item);
                    $message->setText($msg);
                    $this->sendResponse($message, $user);
                }
            }
        }
    }

    /**
     * @param array $notification
     * @param string $user
     */
    protected function processForgeMessage(array $notification, string $user)
    {
        $parts = explode(':', $notification['message']);
        $issueNumber = (int) trim($parts[1]);
        $result = $this->queryForge('issues/' . $issueNumber);
        if ($result) {
            $msg = '*Hi <@' . $user . '>, <@' . $notification['from_user'] . '>'
                . ' ask you to look at this issue:*';
            $this->sendResponse($msg . "\n" . $this->buildIssueMessage($result->issue), $user);
        }
    }

    /**
     * @param array $notification
     * @param string $user
     */
    protected function processTextMessage(array $notification, string $user)
    {
        $msg = '*Hi <@' . $user . '>, here is a message from <@' . $notification['from_user'] . '>' . ' for you:*';
        $this->sendResponse($msg . chr(10) . $notification['message'], $user);
    }

    /**
     * @return string
     *
     * @throws \Doctrine\DBAL\DBALException
     */
    protected function processTell() : string
    {
        $params = $this->params;
        array_shift($params);
        $toUser = array_shift($params);
        $toUser = str_replace(['<', '>', '@'], '', $toUser);
        if ($params[0] === 'about'
            && (strpos($params[1], 'review:') !== false || strpos($params[1], 'forge:') !== false)) {
            $message = $params[1];
        } else {
            $message = implode(' ', $params);
        }
        $this->getDatabaseConnection()->insert('notifications', [
            'from_user' => $this->payload->getData()['user'],
            'to_user' => $toUser,
            'message' => $message,
        ]);

        return 'OK, I will tell <@' . $toUser . '> about your message';
    }
}