Jcid\Voys\Voys

Complexity

Total Complexity

6

Size/Duplication

Total Lines	111
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	callPhonenumer()	0	19	1
A	getCallStatus()	0	15	1
A	validateLoginAndPassword()	0	12	1
A	validatePhonenumbers()	0	5	1
A	validateCallId()	0	4	1

<?php

namespace Jcid\Voys;

use Assert\Assertion;
use GuzzleHttp\Client;
use GuzzleHttp\RequestOptions;

final class Voys
{
    const API_URL = 'https://api.voipgrid.nl/api/clicktodial/';

    /**
     * @var Client
     */
    private $client;

    /**
     * @var string
     */
    private $login;

    /**
     * @var string
     */
    private $password;

    /**
     * @param string $login
     * @param string $password
     */
    public function __construct(string $login, string $password)
    {
        $this->client = new Client();
        $this->validateLoginAndPassword($login, $password);
    }

    /**
     * @param string $fromPhonenumer
     * @param string $toPhonenumer
     *
     * @return string
     */
    public function callPhonenumer(string $toPhonenumer, string $fromPhonenumer): string
    {
        $this->validatePhonenumbers($toPhonenumer, $fromPhonenumer);

        $response = $this->client->request(
            'POST',
            self::API_URL,
            [
                RequestOptions::AUTH => [$this->login, $this->password],
                RequestOptions::JSON => [
                    VoysOptionsInterface::B_NUMBER => $toPhonenumer,
                    VoysOptionsInterface::A_NUMBER => $fromPhonenumer,
                ],
            ]
        );
        $voysResponse = new VoysResponse($response);

$response of type object<Psr\Http\Message\ResponseInterface> is not a sub-type of object<GuzzleHttp\Psr7\Response>. It seems like you assume a concrete implementation of the interface Psr\Http\Message\ResponseInterface to be always present.

        return $voysResponse->getCallid();
    }

    /**
     * @param string $callid
     *
     * @return string
     */
    public function getCallStatus(string $callid): string
    {
        $this->validateCallId($callid);

        $response = $this->client->request(
            'GET',
            sprintf('%s%s/', self::API_URL, $callid),
            [
                RequestOptions::AUTH => [$this->login, $this->password],
            ]
        );
        $voysResponse = new VoysResponse($response);

$response of type object<Psr\Http\Message\ResponseInterface> is not a sub-type of object<GuzzleHttp\Psr7\Response>. It seems like you assume a concrete implementation of the interface Psr\Http\Message\ResponseInterface to be always present.

        return $voysResponse->getStatus();
    }

    /**
     * @param string $login
     * @param string $password
     */
    private function validateLoginAndPassword(string $login, string $password): void
    {
        // Login validation
        Assertion::string($login, 'Login: Value "%s" expected to be string, type %s given.');
        Assertion::email($login, 'Login: Value "%s" was expected to be a valid e-mail address.');

        // Password validation
        Assertion::string($password, 'Password: Value "%s" expected to be string, type %s given.');

        $this->login = $login;
        $this->password = $password;
    }

    /**
     * @param string $toPhonenumer
     * @param string $fromPhonenumer
     */
    private function validatePhonenumbers(string $toPhonenumer, string $fromPhonenumer): void
    {
        Assertion::notNull($toPhonenumer);
        Assertion::notNull($fromPhonenumer);
    }

    /**
     * @param string $callid
     */
    private function validateCallId($callid): void
    {
        Assertion::string($callid);
    }
}