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

Test Failed

Pull Request — master (#2)

by Harald

created 2019-06-19 12:36 UTC

Caller::getMessageFactory() A

↳ Parent: Caller::request()

Complexity

Conditions	2
Paths	2

Size

Total Lines	7
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	2

Importance

Changes

Metric	Value
eloc	3
dl	0
loc	7
ccs	4
cts	4
cp	1
rs	10
c	0
b	0
f	0
cc	2
nc	2
nop	0
crap	2

<?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;
filter:
    dependency_paths: ["lib/*"]
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();
class Id
{
    public $id;

    public function __construct($id)
    {
        $this->id = $id;
    }

}

class Account
{
    /** @var  Id $id */
    public $id;
}

$account_id = false;

if (starsAreRight()) {
    $account_id = new Id(42);
}

$account = new Account();
if ($account instanceof Id)
{
    $account->id = $account_id;
}
    }

    /**
     * @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;
		0 ignored issues – show Bug introduced 2019-06-19 12:41 UTC by Report Bug Copy Issue Report The type `Psr\Http\Message\RequestFactoryInterface` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
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	2	public function __construct(
35		ClientInterface $httpClient = null,
36		RequestFactoryInterface $requestFactory = null,
37		StreamInterface $streamFactory = null
38		) {
39	2	$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		$this->requestFactory = $requestFactory ?: Psr17FactoryDiscovery::findRequestFactory();
41		$this->streamFactory = $streamFactory ?: Psr17FactoryDiscovery::findStreamFactory();
		0 ignored issues – show Documentation Bug introduced 2019-06-19 12:41 UTC by Report Bug Copy Issue Report It seems like `$streamFactory ?: Http\D...ry::findStreamFactory()` can also be of type `Psr\Http\Message\StreamInterface`. However, the property `$streamFactory` is declared as type `Http\Message\StreamFactory`. Maybe add an additional type 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 mixed type is assigned to a property that is type hinted more strictly. For example, imagine you have a variable `$accountId` that can either hold an Id object or false (if there is no account id yet). Your code now assigns that value to the `id` property of an instance of the `Account` class. This class holds a proper account, so the id value must no longer be false. Either this assignment is in error or a type check should be added for that assignment. class Id { public $id; public function __construct($id) { $this->id = $id; } } class Account { /** @var Id $id */ public $id; } $account_id = false; if (starsAreRight()) { $account_id = new Id(42); } $account = new Account(); if ($account instanceof Id) { $account->id = $account_id; } Loading history...
42		}
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		public function request(string $method, $url, array $headers = [], $body = null): ResponseInterface
55		{
56		try {
57		$request = $this->requestFactory->createRequest($method, $url);
58
59		if (!empty($headers)) {
60		foreach ($headers as $name => $value) {
61		$request = $request->withHeader($name, $value);
62		}
63		}
64
65		if ($body !== null) {
66		$request = $request->withBody(
67		$this->streamFactory->createStream($body)
68		);
69		}
70
71		return $this->httpClient->sendRequest($request);
72		} catch (ClientExceptionInterface \| \Exception $e) {
73		throw new BokbasenApiClientException($e->getMessage(), $e->getCode(), $e);
74		}
75		}
76
77		public function setHttpClient(ClientInterface $httpClient): void
78		{
79		$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		}
81		}
82

Bokbasen / php-api-client

GitHub Access Token became invalid

Pull Request — master (#2)

Caller::getMessageFactory() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like