Slack\Message\Message

Complexity

Total Complexity

12

Size/Duplication

Total Lines	76
Duplicated Lines	13.16 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

52.63%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getText()	0	4	1
A	isMarkdownEnabled()	0	4	2
A	hasAttachments()	0	4	2
A	getAttachments()	0	4	2
A	getChannel()	0	4	1
A	getUser()	0	4	1
A	jsonUnserialize()	10	10	3

<?php
namespace Slack\Message;

use Slack\ClientObject;

/**
 * Represents a chat message and its data.
 */
class Message extends ClientObject
{
    /**
     * Gets the message text.
     *
     * @return string
     */
    public function getText()
    {
        return $this->data['text'];
    }

    /**
     * Checks if Markdown is enabled for the message text.
     *
     * @return bool
     */
    public function isMarkdownEnabled()
    {
        return isset($this->data['mrkdwn']) ? $this->data['mrkdwn'] == true : true;
    }

    /**
     * Checks if the message has attachments.
     *
     * @return bool True if the message has attachments, otherwise false.
     */
    public function hasAttachments()
    {
        return isset($this->data['attachments']) && count($this->data['attachments']) > 0;
    }

    /**
     * Gets all message attachments.
     *
     * @return Attachment[]
     */
    public function getAttachments()
    {
        return isset($this->data['attachments']) ? $this->data['attachments'] : [];
    }

    /**
     * Gets the channel the message is in.
     *
     * @return \React\Promise\PromiseInterface
     */
    public function getChannel()
    {
        return $this->client->getChannelById($this->data['channel']);
    }

    /**
     * Gets the user that sent the message.
     *
     * @return \React\Promise\PromiseInterface
     */
    public function getUser()
    {
        return $this->client->getUserById($this->data['user']);
    }

    /**
     * {@inheritDoc}
     */
    public function jsonUnserialize(array $data)
    {
        if (!isset($this->data['attachments'])) {
            return;
        }

        for ($i = 0; $i < count($this->data['attachments']); $i++) {

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

            $this->data['attachments'][$i] = Attachment::fromData($this->data['attachments'][$i]);
        }
    }
}