Erdemkeren\JetSms\JetSmsService

Complexity

Total Complexity

8

Size/Duplication

Total Lines	136
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	17	1
A	sendShortMessage()	0	16	3
B	sendShortMessages()	0	23	4

<?php

namespace Erdemkeren\JetSms;

use Erdemkeren\JetSms\Http\Clients\JetSmsClientInterface;
use Erdemkeren\JetSms\Http\Responses\JetSmsResponseInterface;

/**
 * Class JetSmsService.
 */
final class JetSmsService
{
    /**
     * The jet sms client implementation.
     *
     * @var JetSmsClientInterface
     */
    private $client;

    /**
     * The short message factory implementation.
     *
     * @var ShortMessageFactoryInterface
     */
    private $factory;

    /**
     * The short message collection factory implementation.
     *
     * @var ShortMessageFactoryInterface
     */
    private $collectionFactory;

    /**
     * The before callback which will be called before sending single messages.
     *
     * @var callable|null
     */
    private $beforeSingleShortMessageCallback;

    /**
     * The after callback which will be called before sending single messages.
     *
     * @var callable|null
     */
    private $afterSingleShortMessageCallback;

    /**
     * The before callback which will be called before sending multiple messages.
     *
     * @var callable|null
     */
    private $beforeMultipleShortMessageCallback;

    /**
     * The after callback which will be called after sending multiple messages.
     *
     * @var callable|null
     */
    private $afterMultipleShortMessageCallback;

    /**
     * JetSmsService constructor.
     *
     * @param  JetSmsClientInterface                  $jetSmsClient
     * @param  ShortMessageFactoryInterface           $shortMessageFactory
     * @param  ShortMessageCollectionFactoryInterface $shortMessageCollectionFactory
     * @param  callable|null                          $beforeSingleShortMessageCallback
     * @param  callable|null                          $afterSingleShortMessageCallback
     * @param  callable|null                          $beforeMultipleShortMessageCallback
     * @param  callable|null                          $afterMultipleShortMessageCallback
     */
    public function __construct(
        JetSmsClientInterface $jetSmsClient,
        ShortMessageFactoryInterface $shortMessageFactory,
        ShortMessageCollectionFactoryInterface $shortMessageCollectionFactory,
        $beforeSingleShortMessageCallback = null,
        $afterSingleShortMessageCallback = null,
        $beforeMultipleShortMessageCallback = null,
        $afterMultipleShortMessageCallback = null
    ) {
        $this->client = $jetSmsClient;
        $this->factory = $shortMessageFactory;
        $this->collectionFactory = $shortMessageCollectionFactory;

It seems like $shortMessageCollectionFactory of type object<Erdemkeren\JetSms\ShortMessageCollectionFactoryInterface> is incompatible with the declared type object<Erdemkeren\JetSms\ShortMessageFactoryInterface> of property $collectionFactory.

        $this->beforeSingleShortMessageCallback = $beforeSingleShortMessageCallback;
        $this->afterSingleShortMessageCallback = $afterSingleShortMessageCallback;
        $this->beforeMultipleShortMessageCallback = $beforeMultipleShortMessageCallback;
        $this->afterMultipleShortMessageCallback = $afterMultipleShortMessageCallback;
    }

    /**
     * Send the given body to the given receivers.
     *
     * @param  string       $body      The body of the short message.
     * @param  array|string $receivers The receiver(s) of the message.
     *
     * @return JetSmsResponseInterface The parsed JetSms response object.
     */
    public function sendShortMessage($receivers, $body)
    {
        $shortMessage = $this->factory->create($receivers, $body);

        if(is_callable($this->beforeSingleShortMessageCallback)) {
            ($this->beforeSingleShortMessageCallback)($shortMessage);
        }

        $response = $this->client->sendShortMessage($shortMessage);

        if(is_callable($this->afterSingleShortMessageCallback)) {
            ($this->afterSingleShortMessageCallback)($response, $shortMessage);
        }

        return $response;
    }

    /**
     * Send the given short messages.
     *
     * @param  array $messages         An array containing short message arrays.
     *
     * @return JetSmsResponseInterface The parsed JetSms response object.
     */
    public function sendShortMessages(array $messages)
    {
        $collection = $this->collectionFactory->create();

The call to create() misses some required arguments starting with $receivers.

        if(is_callable($this->beforeMultipleShortMessageCallback)) {
            ($this->beforeMultipleShortMessageCallback)($collection);
        }

        foreach ($messages as $message) {
            $collection->push($this->factory->create(

The method push() does not seem to exist on object<Erdemkeren\JetSms\ShortMessage>.

                $message['recipient'],
                $message['message']
            ));
        }

        $response = $this->client->sendShortMessages($collection);

$collection is of type object<Erdemkeren\JetSms\ShortMessage>, but the function expects a object<Erdemkeren\JetSms\ShortMessageCollection>.

        if(is_callable($this->afterMultipleShortMessageCallback)) {
            ($this->afterMultipleShortMessageCallback)($response, $collection);
        }

        return $response;
    }
}