NotificationChannels\Twitter\TwitterStatusUpdate

Complexity

Total Complexity

7

Size/Duplication

Total Lines	75
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

0%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	2
A	getContent()	0	4	1
A	getImagePaths()	0	4	1
A	getApiEndpoint()	0	4	1
A	getRequestBody()	0	12	2

<?php

namespace NotificationChannels\Twitter;

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

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

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

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

    /*
     * @param  string $content
     */
    public function __construct(string $content, array $imagePaths = null)
    {
        $this->content = $content;
        $this->imagePaths = $imagePaths ? collect($imagePaths) : null;

It seems like $imagePaths ? collect($imagePaths) : null of type object<Illuminate\Support\Collection> or null is incompatible with the declared type array of property $imagePaths.

    }

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

    /**
     * Get Twitter status update image paths.
     *
     * @return  string
     */
    public function getImagePaths()
    {
        return $this->imagePaths;
    }

    /**
     * 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;
    }
}