NotificationChannels\FortySixElks\FortySixElksMMS

Complexity

Total Complexity

5

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	image()	0	5	1
A	send()	0	19	3

<?php

namespace NotificationChannels\FortySixElks;

use NotificationChannels\FortySixElks\Exceptions\CouldNotSendNotification;

class FortySixElksMMS extends FortySixElksMedia implements FortySixElksMediaInterface
{
    const ENDPOINT = 'https://api.46elks.com/a1/mms';
    public $type = 'MMS';

    public function __construct()
    {
        return parent::__construct();

Are you sure the usage of parent::__construct() targeting NotificationChannels\FortySixElks\FortySixElksMedia::__construct() seems to always return null.

    }

    public function image($url)
    {
        $this->payload['image'] = $url;

        return $this;
    }

    /**
     * @return $this
     */
    public function send()
    {
        try {
            $response = $this->client->request('POST', self::ENDPOINT, [

The assignment to $response is dead and can be removed.

                'form_params' => [
                    'from'     => $this->from,
                    'message'  => $this->getContent(),
                    'to'       => $this->phone_number,
                    'image'    => isset($this->payload['image']) ? $this->payload['image'] : null,
                ],

            ]);
        } catch (\GuzzleHttp\Exception\BadResponseException $e) {
            $response = $e->getResponse();

            throw CouldNotSendNotification::serviceRespondedWithAnError($response->getBody()->getContents(), $response->getStatusCode());
        }

        return $this;
    }
}