PortlandLabs\Slackbot\Slack\Api\Payload\ChatUpdatePayload

Complexity

Total Complexity

6

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	addAttachment()	0	3	1
A	getAttachments()	0	6	2
A	beforeSerialize()	0	3	1
A	__construct()	0	3	1
A	getResponseClass()	0	3	1

<?php

namespace PortlandLabs\Slackbot\Slack\Api\Payload;

use CL\Slack\Model\Attachment;
use CL\Slack\Payload\AdvancedSerializeInterface;
use CL\Slack\Payload\ChatUpdatePayloadResponse;
use Doctrine\Common\Collections\ArrayCollection;
use JMS\Serializer\Serializer;

This use statement conflicts with another class in this namespace, PortlandLabs\Slackbot\Slack\Api\Payload\Serializer. Consider defining an alias.

class ChatUpdatePayload extends \CL\Slack\Payload\ChatUpdatePayload implements AdvancedSerializeInterface
{

    /**
     * @var Attachment[]|ArrayCollection
     */
    private $attachments;

    /**
     * @var string
     */
    private $attachmentsJson;

The private property $attachmentsJson is not used, and could be removed.

    public function __construct()
    {
        $this->attachments = new ArrayCollection();
    }

    /**
     * @return Attachment[]|ArrayCollection
     */
    public function getAttachments()
    {
        if (is_string($this->attachments)) {

The condition is_string($this->attachments) is always false.

            $this->attachments = new ArrayCollection(json_decode($this->attachments, true));
        }
        return $this->attachments;
    }

    /**
     * @param Attachment $attachment
     */
    public function addAttachment(Attachment $attachment)
    {
        $this->getAttachments()->add($attachment);
    }

    /**
     * @inheritdoc
     */
    public function beforeSerialize(Serializer $serializer)
    {
        $this->attachments = $serializer->serialize($this->attachments, 'json');
    }

    /**
     * @inheritdoc
     */
    public function getResponseClass()
    {
        return ChatUpdatePayloadResponse::class;
    }

}