NotificationChannels\Twitter\TwitterStatusUpdate

Complexity

Total Complexity

7

Size/Duplication

Total Lines	86
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	withImage()	0	6	1
A	getContent()	0	4	1
A	getImages()	0	4	1
A	getApiEndpoint()	0	4	1
A	getRequestBody()	0	12	2

<?php

namespace NotificationChannels\Twitter;

use NotificationChannels\Twitter\TwitterImage;

class TwitterStatusUpdate
{
    /** @var string */
    protected $content;

    /**
     * @var  array
     */
    private $images;

    /**
     * @var  array
     */
    public $imageIds;

    /**
     * @var  string
     */
    private $apiEndpoint = 'statuses/update';

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

    /**
     * Set Twitter media files.
     *
     * @return  $this
     */
    public function withImage($images){
        collect($images)->each(function($image){
            $this->images[] = new TwitterImage($image);
        });
        return $this;
    }

    /**
     * Get Twitter status update content.
     *
     * @return  string
     */
    public function getContent()
    {
        return $this->content;
    }

    /**
     * Get Twitter images list.
     *
     * @return  string
     */
    public function getImages()
    {
        return $this->images;
    }

    /**
     * Return Twitter status update api endpoint.
     * @return  string
     */
    public function getApiEndpoint()
    {
        return $this->apiEndpoint;
    }

    /**
     * Build Twitter request body.
     * @return  array
     */
    public function getRequestBody()
    {
        $body = [
            'status' => $this->getContent(),
        ];

        if ($this->imageIds) {

The expression $this->imageIds of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $body['media_ids'] = $this->imageIds->implode(',');

The method implode cannot be called on $this->imageIds (of type array).

        }

        return $body;
    }
}