Caller::request() - Code Metrics - Inspection of "BWD-00 Log responses" - Bokbasen/php-api-client - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#2)

by Harald

created 2019-06-19 12:52 UTC

Caller::request() A

↳ Parent: Caller

Complexity

Conditions	5
Paths	17

Size

Total Lines	20
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	5

Importance

Changes

Metric	Value
eloc	11
dl	0
loc	20
ccs	11
cts	11
cp	1
rs	9.6111
c	0
b	0
f	0
cc	5
nc	17
nop	4
crap	5

<?php

namespace Bokbasen\ApiClient;

use Bokbasen\ApiClient\Exceptions\BokbasenApiClientException;
use Http\Client\HttpClient;
use Http\Discovery\Psr17FactoryDiscovery;
use Http\Discovery\Psr18ClientDiscovery;
use Http\Message\StreamFactory;
use Psr\Http\Client\ClientExceptionInterface;
use Psr\Http\Client\ClientInterface;
use Psr\Http\Message\RequestFactoryInterface;
use Psr\Http\Message\ResponseInterface;
use Psr\Http\Message\StreamInterface;
use Psr\Http\Message\UriInterface;

class Caller
{
    /**
     * @var HttpClient
     */
    private $httpClient;

    /**
     * @var StreamFactory
     */
    private $streamFactory;

    /**
     * @var RequestFactoryInterface
     */
    private $requestFactory;

    public function __construct(
        ClientInterface $httpClient = null,
        RequestFactoryInterface $requestFactory = null,
        StreamInterface $streamFactory = null
    ) {
        $this->httpClient = $httpClient ?: Psr18ClientDiscovery::find();
class Alien {}

class Dalek extends Alien {}

class Plot
{
    /** @var  Dalek */
    public $villain;
}

$alien = new Alien();
$plot = new Plot();
if ($alien instanceof Dalek) {
    $plot->villain = $alien;
}
        $this->requestFactory = $requestFactory ?: Psr17FactoryDiscovery::findRequestFactory();
        $this->streamFactory = $streamFactory ?: Psr17FactoryDiscovery::findStreamFactory();

    }

    /**
     * @param string                               $method
     * @param string|UriInterface                  $url
     * @param array                                $headers
     * @param resource|string|StreamInterface|null $body
     *
     * @return ResponseInterface
     *
     * @throws BokbasenApiClientException
     */
    public function request(string $method, $url, array $headers = [], $body = null): ResponseInterface
    {
        try {
            $request = $this->requestFactory->createRequest($method, $url);

            if (!empty($headers)) {
                foreach ($headers as $name => $value) {
                    $request = $request->withHeader($name, $value);
                }
            }

            if ($body !== null) {
                $request = $request->withBody(
                    $this->streamFactory->createStream($body)
                );
            }

            return $this->httpClient->sendRequest($request);
        } catch (ClientExceptionInterface | \Exception $e) {
            throw new BokbasenApiClientException($e->getMessage(), $e->getCode(), $e);
        }
    }

    public function setHttpClient(ClientInterface $httpClient): void
    {
        $this->httpClient = $httpClient;
class Alien {}

class Dalek extends Alien {}

class Plot
{
    /** @var  Dalek */
    public $villain;
}

$alien = new Alien();
$plot = new Plot();
if ($alien instanceof Dalek) {
    $plot->villain = $alien;
}
    }
}


1		<?php
2
3		namespace Bokbasen\ApiClient;
4
5		use Bokbasen\ApiClient\Exceptions\BokbasenApiClientException;
6		use Http\Client\HttpClient;
7		use Http\Discovery\Psr17FactoryDiscovery;
8		use Http\Discovery\Psr18ClientDiscovery;
9		use Http\Message\StreamFactory;
10		use Psr\Http\Client\ClientExceptionInterface;
11		use Psr\Http\Client\ClientInterface;
12		use Psr\Http\Message\RequestFactoryInterface;
13		use Psr\Http\Message\ResponseInterface;
14		use Psr\Http\Message\StreamInterface;
15		use Psr\Http\Message\UriInterface;
16
17		class Caller
18		{
19		/**
20		* @var HttpClient
21		*/
22		private $httpClient;
23
24		/**
25		* @var StreamFactory
26		*/
27		private $streamFactory;
28
29		/**
30		* @var RequestFactoryInterface
31		*/
32		private $requestFactory;
33
34	21	public function __construct(
35		ClientInterface $httpClient = null,
36		RequestFactoryInterface $requestFactory = null,
37		StreamInterface $streamFactory = null
38		) {
39	21	$this->httpClient = $httpClient ?: Psr18ClientDiscovery::find();
		0 ignored issues – show Documentation Bug introduced 2019-06-19 12:41 UTC by Report Bug Copy Issue Report `$httpClient ?: Http\Disc...ClientDiscovery::find()` is of type `Psr\Http\Client\ClientInterface`, but the property `$httpClient` was declared to be of type `Http\Client\HttpClient`. Are you sure that you always receive this specific sub-class here, or does it make sense to add an `instanceof` check? Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a given class or a super-class is assigned to a property that is type hinted more strictly. Either this assignment is in error or an instanceof check should be added for that assignment. class Alien {} class Dalek extends Alien {} class Plot { /** @var Dalek */ public $villain; } $alien = new Alien(); $plot = new Plot(); if ($alien instanceof Dalek) { $plot->villain = $alien; } Loading history...
40	21	$this->requestFactory = $requestFactory ?: Psr17FactoryDiscovery::findRequestFactory();
41	21	$this->streamFactory = $streamFactory ?: Psr17FactoryDiscovery::findStreamFactory();
		0 ignored issues – show Documentation Bug introduced 2019-06-19 12:52 UTC by Report Bug Copy Issue Report It seems like `$streamFactory ?: Http\D...ry::findStreamFactory()` of type `Psr\Http\Message\StreamFactoryInterface` or `Psr\Http\Message\StreamInterface` is incompatible with the declared type `Http\Message\StreamFactory` of property `$streamFactory`. 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...
42	21	}
43
44		/**
45		* @param string $method
46		* @param string\|UriInterface $url
47		* @param array $headers
48		* @param resource\|string\|StreamInterface\|null $body
49		*
50		* @return ResponseInterface
51		*
52		* @throws BokbasenApiClientException
53		*/
54	21	public function request(string $method, $url, array $headers = [], $body = null): ResponseInterface
55		{
56		try {
57	21	$request = $this->requestFactory->createRequest($method, $url);
58
59	21	if (!empty($headers)) {
60	21	foreach ($headers as $name => $value) {
61	21	$request = $request->withHeader($name, $value);
62		}
63		}
64
65	21	if ($body !== null) {
66	15	$request = $request->withBody(
67	15	$this->streamFactory->createStream($body)
68		);
69		}
70
71	21	return $this->httpClient->sendRequest($request);
72	3	} catch (ClientExceptionInterface \| \Exception $e) {
73	3	throw new BokbasenApiClientException($e->getMessage(), $e->getCode(), $e);
74		}
75		}
76
77	21	public function setHttpClient(ClientInterface $httpClient): void
78		{
79	21	$this->httpClient = $httpClient;
		0 ignored issues – show Documentation Bug introduced 2019-06-19 12:41 UTC by Report Bug Copy Issue Report `$httpClient` is of type `Psr\Http\Client\ClientInterface`, but the property `$httpClient` was declared to be of type `Http\Client\HttpClient`. Are you sure that you always receive this specific sub-class here, or does it make sense to add an `instanceof` check? Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a given class or a super-class is assigned to a property that is type hinted more strictly. Either this assignment is in error or an instanceof check should be added for that assignment. class Alien {} class Dalek extends Alien {} class Plot { /** @var Dalek */ public $villain; } $alien = new Alien(); $plot = new Plot(); if ($alien instanceof Dalek) { $plot->villain = $alien; } Loading history...
80	21	}
81		}
82

Bokbasen / php-api-client

GitHub Access Token became invalid

Pull Request — master (#2)

Caller::request() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like