OmnideskBundle\Service\RequestService

Complexity

Total Complexity

22

Size/Duplication

Total Lines	193
Duplicated Lines	15.54 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

11 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getCallsLeft()	0	4	1
A	post()	10	10	1
B	postMultipart()	0	38	10
A	get()	10	10	1
A	put()	10	10	1
A	delete()	0	9	1
A	getClient()	0	16	3
A	getUrl()	0	4	1
A	getHeaders()	0	7	1
A	getAuth()	0	7	1

<?php

namespace OmnideskBundle\Service;

use GuzzleHttp\Client;
use GuzzleHttp\HandlerStack;
use GuzzleHttp\Middleware;
use Psr\Http\Message\ResponseInterface;

/**
 * Class RequestService
 * @package OmnideskBundle\Service
 */
class RequestService
{
    /**
     * @var array
     */
    protected $config;

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

    /**
     * @var int|null
     */
    protected $callsLeft;

    /**
     * RequestService constructor.
     * @param array  $config
     */
    public function __construct(array $config)
    {
        $this->client = $this->getClient();
        $this->config = $config;
    }

    /**
     * @return int|null
     */
    public function getCallsLeft()
    {
        return $this->callsLeft;
    }

    /**
     * @param string $url
     * @param array  $params
     * @return array
     */
    public function post($url, array $params = [])

This method seems to be duplicated in your project.

    {
        $response = $this->client->post($this->getUrl($url), [
            'headers' => $this->getHeaders(),
            'auth' => $this->getAuth(),
            'json' => $params,
        ]);

        return json_decode((string) $response->getBody(), true);
    }

    /**
     * @param string $url
     * @param string $name
     * @param array  $params
     * @return array
     */
    public function postMultipart($url, $name, array $params = [])
    {
        $multipart = [];

        foreach ($params as $key => $param) {
            if ($key !== 'attachments' && $key !== 'custom_fields') {
                $multipart[] = [
                    'name' => "{$name}[{$key}]",
                    'contents' => $param,
                ];
            }
        }

        if (isset($params['attachments']) && $attachments = (array) $params['attachments']) {
            foreach ($attachments as $key => $attachment) {
                $multipart[] = [
                    'name' => "{$name}[attachments][{$key}]",
                    'contents' => fopen($attachment, 'rb'),
                ];
            }
        }

        if (isset($params['custom_fields']) && $fields = (array) $params['custom_fields']) {
            foreach ($fields as $key => $field) {
                $multipart[] = [
                    'name' => "{$name}[custom_fields][{$key}]",
                    'contents' => $field,
                ];
            }
        }

        $response = $this->client->post($this->getUrl($url), [
            'auth' => $this->getAuth(),
            'multipart' => $multipart,
        ]);

        return json_decode((string) $response->getBody(), true);
    }

    /**
     * @param string $url
     * @param array  $params
     * @return array
     */
    public function get($url, array $params = [])

This method seems to be duplicated in your project.

    {
        $response = $this->client->get($this->getUrl($url), [
            'headers' => $this->getHeaders(),
            'auth' => $this->getAuth(),
            'query' => $params,
        ]);

        return json_decode((string) $response->getBody(), true);
    }

    /**
     * @param string $url
     * @param array  $params
     * @return array
     */
    public function put($url, array $params = [])

This method seems to be duplicated in your project.

    {
        $response = $this->client->put($this->getUrl($url), [
            'headers' => $this->getHeaders(),
            'auth' => $this->getAuth(),
            'json' => $params,
        ]);

        return json_decode((string) $response->getBody(), true);
    }

    /**
     * @param string $url
     * @return int
     */
    public function delete($url)
    {
        $response = $this->client->delete($this->getUrl($url), [
            'headers' => $this->getHeaders(),
            'auth' => $this->getAuth(),
        ]);

        return $response->getStatusCode();
    }

    /**
     * @return Client
     */
    protected function getClient()
    {
        if (!$this->client) {
            $stack = HandlerStack::create();

            $stack->push(Middleware::mapResponse(function (ResponseInterface $response) {
                $this->callsLeft = $response->getHeaderLine('api_calls_left') ?: null;

It seems like $response->getHeaderLine('api_calls_left') ?: null can also be of type string. However, the property $callsLeft is declared as type integer|null. Maybe add an additional type check?

                return $response;
            }));

            $this->client = new Client(['handler' => $stack]);
        }

        return $this->client;
    }

    /**
     * @param string $url
     * @return string
     */
    protected function getUrl($url)
    {
        return sprintf('https://%s.omnidesk.ru/api/%s.json', $this->config['domain'], $url);
    }

    /**
     * @return array
     */
    protected function getHeaders()
    {
        return [
            'Content-Type' => 'application/json',
            'Content-Length' => 0,
        ];
    }

    /**
     * @return array
     */
    protected function getAuth()
    {
        return [
            $this->config['email'],
            $this->config['key'],
        ];
    }
}