Asad\ZohoCliq\ZohoCliq

Complexity

Total Complexity

27

Size/Duplication

Total Lines	258
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

19 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	14	4
A	setAuthToken()	0	4	1
A	getAuthToken()	0	4	1
A	setDefaultSendTo()	0	4	1
A	getDefaultSendTo()	0	4	1
A	setDefaultChannel()	0	4	1
A	getDefaultChannel()	0	4	1
A	generateCliqEndpoint()	0	4	1
A	getEndpoint()	0	4	1
A	getMessage()	0	4	1
A	toChannel()	0	5	1
A	toBot()	0	5	1
A	toChat()	0	5	1
A	toBuddy()	0	5	1
A	to()	0	5	1
A	card()	0	13	4
A	send()	0	15	2
A	createBody()	0	8	2
A	prepareAuth()	0	6	1

<?php

namespace Asad\ZohoCliq;

use GuzzleHttp\Client as Guzzle;
use RuntimeException;

class ZohoCliq
{
    /**
     * Zoho Cliq Auth Token
     * @var string
     */
    protected $authtoken;

    /**
     * Default send to
     * @var string
     */
    protected $send_to;

    /**
     * Default Channel to send message
     * @var string
     */
    protected $channel;

    /**
     * Zoho Cliq message endpoint
     * @var string
     */
    protected $endpoint;

    /**
     * Zoho Cliq message
     * @var string
     */
    protected $message;

    /**
     * Zoho Cliq card
     * @var string
     */
    protected $card;

    /**
     * The Guzzle HTTP client instance.
     *
     * @var \GuzzleHttp\Client
     */
    protected $guzzle;

    /**
     * Instantiate a new Client.
     *
     * @param string $endpoint

There is no parameter named $endpoint. Was it maybe removed?

     * @param array $attributes
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct($authtoken, array $attributes = [], Guzzle $guzzle = null)
    {
        $this->authtoken = $authtoken;

        if (isset($attributes['channel'])) {
            $this->setDefaultChannel($attributes['channel']);
        }

        if (isset($attributes['send_to'])) {
            $this->setDefaultSendTo($attributes['send_to']);
        }

        $this->guzzle = $guzzle ?: new Guzzle;
    }

    /**
     * Set Default AuthToken
     * @param string $authtoken
     * @return void
     */
    public function setAuthToken($authtoken)
    {
        $this->authtoken = $authtoken;
    }

    /**
     * Get Default Authtoken
     * @return string
     */
    public function getAuthToken(): string
    {
        return $this->authtoken;
    }

    /**
     * Set Default SendTo
     * @param string $send_to
     * @return void
     */
    public function setDefaultSendTo($send_to)
    {
        $this->send_to = $send_to;
    }

    /**
     * Get Default Send To
     * @return string
     */
    public function getDefaultSendTo(): string
    {
        return $this->send_to;
    }

    /**
     * Set Default Channel
     * @param string $channel
     * @return void
     */
    public function setDefaultChannel($channel)
    {
        $this->channel = $channel;
    }

    /**
     * Get Default Channel
     * @return string
     */
    public function getDefaultChannel(): string
    {
        return $this->channel;
    }

    /**
     * Generate Cliq Endpoint from send_to and channel name
     * @return void
     */

    public function generateCliqEndpoint()
    {
        $this->endpoint = "https://cliq.zoho.com/api/v2/" . $this->getDefaultSendTo() . "/" . $this->getDefaultChannel() . "/message";
    }

    /**
     * Get Default Endpoint
     * @return string
     */
    public function getEndpoint(): string
    {
        return $this->endpoint;
    }

    /**
     * Get Message
     * @return string
     */
    public function getMessage(): string
    {
        return $this->message;
    }

    /**
     * Set Send To onfly
     * @return self
     */

    public function toChannel(): self
    {
        $this->send_to = 'channelsbyname';
        return $this;
    }

    /**
     * Set Send To onfly
     * @return self
     */

    public function toBot(): self
    {
        $this->send_to = 'bots';
        return $this;
    }

    /**
     * Set Send To onfly
     * @return self
     */

    public function toChat(): self
    {
        $this->send_to = 'chats';
        return $this;
    }

    /**
     * Set Send To onfly
     * @return self
     */

    public function toBuddy(): self
    {
        $this->send_to = 'buddies';
        return $this;
    }

    /**
     * Change Channel on fly
     */
    public function to($channel)
    {
        $this->channel = $channel;
        return $this;
    }

    /**
     * Create Message Card
     * @return self
     */
    public function card(array $attributes): self
    {
        $title = isset($attributes['title']) ? $attributes['title'] : "BUG";
        $theme = isset($attributes['theme']) ? $attributes['theme'] : "modern-inline";
        $thumbnail = isset($attributes['thumbnail']) ? $attributes['thumbnail'] : "https://en.gravatar.com/userimage/57826719/1bcf7f90b22897258d2b3e4e84875218.jpg";
        $card = [
            'title' => $title,
            'theme' => $theme,
            'thumbnail' => $thumbnail
        ];
        $this->card = $card;

It seems like $card of type array<string,?,{"title":"?","theme":"?","thumbnail":"?"}> is incompatible with the declared type string of property $card.

        return $this;
    }

    /**
     * Send message to selected destination
     * */
    public function send($message)
    {
        $this->message = $message;
        $this->generateCliqEndpoint();
        $payload = $this->createBody($message);
        $encoded = json_encode($payload, JSON_UNESCAPED_UNICODE);

        if ($encoded === false) {
            throw new RuntimeException(sprintf('JSON encoding error %s: %s', json_last_error(), json_last_error_msg()));
        }
        $this->guzzle->post($this->endpoint, [
            'query' => $this->prepareAuth(),
            'body' => $encoded
        ]);
    }

    public function createBody($message)
    {
        if (gettype($this->card) == 'array') {
            $payload['card'] = $this->card;

$payload was never initialized. Although not strictly required by PHP, it is generally a good practice to add $payload = array(); before regardless.

        }
        $payload['text'] = $message;

The variable $payload does not seem to be defined for all execution paths leading up to this point.

        return $payload;
    }

    public function prepareAuth()
    {
        return [
            'authtoken' => $this->getAuthToken()
        ];
    }
}