Request - Code Metrics - seregazhuk/php-smsintel-api - Measure and Improve Code Quality continuously with Scrutinizer

Request A
last analyzed 2019-10-02 13:46 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	88
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

Metric	Value
wmc	4
lcom	1
cbo	1
dl	0
loc	88
rs	10
c	0
b	0
f	0

7 Methods

Rating	Name	Size	Complexity
A	__construct()	4	1
A	setCredentials()	7	1
A	exec()	8	1
A	createRequestBody()	12	1
	formatRequestBody()	1	?
	parseResponse()	1	?
	makeEndPoint()	1	?

<?php

namespace seregazhuk\SmsIntel\Api\Requests;

use GuzzleHttp\Client;
use seregazhuk\SmsIntel\Api\GuzzleHttpClient;

abstract class Request
{

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

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

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

    public function __construct(GuzzleHttpClient $http)
    {
        $this->client = $http;

    }


    /**
     * @param string $login
     * @param string $password
     * @return $this
     */
    public function setCredentials($login, $password)
    {
        $this->login = $login;
        $this->password = $password;

        return $this;
    }

    /**
     * Make the request to API
     *
     * @param string $action
     * @param array $params
     * @return array|null
     */
    public function exec($action, $params = [])
    {
        $endPoint = $this->makeEndPoint($action);
        $requestBody = $this->createRequestBody($params);

        $response = $this->client->post($endPoint, $requestBody);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        return $this->parseResponse($response);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * @param array $params
     * @return string
     */
    protected function createRequestBody(array $params)
    {
        $params = array_merge(
            [
                'login'    => $this->login,
                'password' => $this->password,
            ],
            $params
        );

        return $this->formatRequestBody($params);
    }

    /**
     * @param array $requestBody
     * @return mixed
     */
    abstract protected function formatRequestBody(array $requestBody);

    /**
     * @param string $response
     * @return array
     */
    abstract protected function parseResponse($response);

    /**
     * @param string $action
     * @return string
     */
    abstract protected function makeEndPoint($action);
}


1			<?php
2
3			namespace seregazhuk\SmsIntel\Api\Requests;
4
5			use GuzzleHttp\Client;
6			use seregazhuk\SmsIntel\Api\GuzzleHttpClient;
7
8			abstract class Request
9			{
10
11			/**
12			* @var Client
13			*/
14			protected $client;
15
16			/**
17			* @var string
18			*/
19			protected $login;
20
21			/**
22			* @var string
23			*/
24			protected $password;
25
26			public function __construct(GuzzleHttpClient $http)
27			{
28			$this->client = $http;
			0 ignored issues – show Documentation Bug introduced 2019-10-02 13:30 UTC by Report Bug Copy Issue Report It seems like `$http` of type `object<seregazhuk\SmsIntel\Api\GuzzleHttpClient>` is incompatible with the declared type `object<GuzzleHttp\Client>` of property `$client`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
29			}
30
31
32			/**
33			* @param string $login
34			* @param string $password
35			* @return $this
36			*/
37			public function setCredentials($login, $password)
38			{
39			$this->login = $login;
40			$this->password = $password;
41
42			return $this;
43			}
44
45			/**
46			* Make the request to API
47			*
48			* @param string $action
49			* @param array $params
50			* @return array\|null
51			*/
52			public function exec($action, $params = [])
53			{
54			$endPoint = $this->makeEndPoint($action);
55			$requestBody = $this->createRequestBody($params);
56
57			$response = $this->client->post($endPoint, $requestBody);
			0 ignored issues – show Documentation introduced 2016-08-04 04:36 UTC by Report Bug Copy Issue Report `$requestBody` is of type `string`, but the function expects a `array`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
58			return $this->parseResponse($response);
			0 ignored issues – show Documentation introduced 2019-10-02 13:30 UTC by Report Bug Copy Issue Report `$response` is of type `object<Psr\Http\Message\ResponseInterface>`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
59			}
60
61			/**
62			* @param array $params
63			* @return string
64			*/
65			protected function createRequestBody(array $params)
66			{
67			$params = array_merge(
68			[
69			'login' => $this->login,
70			'password' => $this->password,
71			],
72			$params
73			);
74
75			return $this->formatRequestBody($params);
76			}
77
78			/**
79			* @param array $requestBody
80			* @return mixed
81			*/
82			abstract protected function formatRequestBody(array $requestBody);
83
84			/**
85			* @param string $response
86			* @return array
87			*/
88			abstract protected function parseResponse($response);
89
90			/**
91			* @param string $action
92			* @return string
93			*/
94			abstract protected function makeEndPoint($action);
95			}
96

seregazhuk / php-smsintel-api

Request A last analyzed 2019-10-02 13:46 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

7 Methods

Duplication Side-by-Side

Filter issues like

Request A
last analyzed 2019-10-02 13:46 UTC