Factory::createFromGuzzleClient() - Code Metrics - Inspection of "Updated most tests" - php-api-clients/foundation - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( f9a93a...d93906 )

by Cees-Jan

created 2016-08-19 15:57 UTC

Factory::createFromGuzzleClient() A

↳ Parent: Factory

Complexity

Conditions	1
Paths	1

Size

Total Lines	11
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

Metric	Value
dl	0
loc	11
ccs	2
cts	2
cp	1
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	9
nc	1
nop	3
crap	1

<?php
declare(strict_types=1);

namespace ApiClients\Foundation\Transport;

use GuzzleHttp\Client as GuzzleClient;
use GuzzleHttp\HandlerStack;
use React\Dns\Resolver\Resolver;
use React\EventLoop\Factory as LoopFactory;
use React\EventLoop\LoopInterface;
use React\HttpClient\Client as HttpClient;
use React\HttpClient\Factory as HttpClientFactory;
use React\Dns\Resolver\Factory as ResolverFactory;
use WyriHaximus\React\GuzzlePsr7\HttpClientAdapter;

class Factory
{
    /**
     * @param LoopInterface|null $loop
     * @param array $options
     * @return Client
     */
    public static function create(LoopInterface $loop = null, array $options = []): Client
    {
        if (!($loop instanceof LoopInterface)) {
            $loop = LoopFactory::create();
        }

        if (!isset($options[Options::DNS])) {
            $options[Options::DNS] = '8.8.8.8';
        }

        $resolver = (new ResolverFactory())->createCached($options[Options::DNS], $loop);
        $httpClient = (new HttpClientFactory())->create($loop, $resolver);

        return self::createFromReactHttpClient(
            $httpClient,
            $resolver,
            $loop,
            $options
        );
    }

    /**
     * @param HttpClient $httpClient
     * @param Resolver $resolver
     * @param LoopInterface|null $loop
     * @param array $options
     * @return Client
     */
    public static function createFromReactHttpClient(
        HttpClient $httpClient,
        Resolver $resolver,
        LoopInterface $loop = null,
        array $options = []
    ): Client {
        return self::createFromGuzzleClient(
            $loop,
function notNullable(stdClass $x) { }

// Unsafe
function withoutCheck(stdClass $x = null) {
    notNullable($x);
}

// Safe - Alternative 1: Adding Additional Type-Check
function withCheck(stdClass $x = null) {
    if ($x instanceof stdClass) {
        notNullable($x);
    }
}

// Safe - Alternative 2: Changing Parameter
function withNonNullableParam(stdClass $x) {
    notNullable($x);
}
            new GuzzleClient(
                [
                    'handler' => HandlerStack::create(
                        new HttpClientAdapter(
                            $loop,
function notNullable(stdClass $x) { }

// Unsafe
function withoutCheck(stdClass $x = null) {
    notNullable($x);
}

// Safe - Alternative 1: Adding Additional Type-Check
function withCheck(stdClass $x = null) {
    if ($x instanceof stdClass) {
        notNullable($x);
    }
}

// Safe - Alternative 2: Changing Parameter
function withNonNullableParam(stdClass $x) {
    notNullable($x);
}
                            $httpClient,
                            $resolver
                        )
                    ),
                ]
            ),
            $options
        );
    }

    /**
     * @param LoopInterface $loop
     * @param GuzzleClient $guzzle
     * @param array $options
     * @return Client
     */
    public static function createFromGuzzleClient(
        LoopInterface $loop,
        GuzzleClient $guzzle,
        array $options = []
    ): Client {
        return new Client(
            $loop,
            $guzzle,
            $options
        );
    }
}


1		<?php
2		declare(strict_types=1);
3
4		namespace ApiClients\Foundation\Transport;
5
6		use GuzzleHttp\Client as GuzzleClient;
7		use GuzzleHttp\HandlerStack;
8		use React\Dns\Resolver\Resolver;
9		use React\EventLoop\Factory as LoopFactory;
10		use React\EventLoop\LoopInterface;
11		use React\HttpClient\Client as HttpClient;
12		use React\HttpClient\Factory as HttpClientFactory;
13		use React\Dns\Resolver\Factory as ResolverFactory;
14		use WyriHaximus\React\GuzzlePsr7\HttpClientAdapter;
15
16		class Factory
17		{
18		/**
19		* @param LoopInterface\|null $loop
20		* @param array $options
21		* @return Client
22		*/
23	2	public static function create(LoopInterface $loop = null, array $options = []): Client
24		{
25	2	if (!($loop instanceof LoopInterface)) {
26	1	$loop = LoopFactory::create();
27		}
28
29	2	if (!isset($options[Options::DNS])) {
30	2	$options[Options::DNS] = '8.8.8.8';
31		}
32
33	2	$resolver = (new ResolverFactory())->createCached($options[Options::DNS], $loop);
34	2	$httpClient = (new HttpClientFactory())->create($loop, $resolver);
35
36	2	return self::createFromReactHttpClient(
37		$httpClient,
38		$resolver,
39		$loop,
40		$options
41		);
42		}
43
44		/**
45		* @param HttpClient $httpClient
46		* @param Resolver $resolver
47		* @param LoopInterface\|null $loop
48		* @param array $options
49		* @return Client
50		*/
51	2	public static function createFromReactHttpClient(
52		HttpClient $httpClient,
53		Resolver $resolver,
54		LoopInterface $loop = null,
55		array $options = []
56		): Client {
57	2	return self::createFromGuzzleClient(
58		$loop,
		0 ignored issues – show Bug introduced 2016-08-19 16:05 UTC by Report Bug Copy Issue Report It seems like `$loop` defined by parameter `$loop` on line `54` can be `null`; however, `ApiClients\Foundation\Tr...reateFromGuzzleClient()` does not accept `null`, maybe add an additional type check? It seems like you allow that null is being passed for a parameter, however the function which is called does not seem to accept null. We recommend to add an additional type check (or disallow null for the parameter): function notNullable(stdClass $x) { } // Unsafe function withoutCheck(stdClass $x = null) { notNullable($x); } // Safe - Alternative 1: Adding Additional Type-Check function withCheck(stdClass $x = null) { if ($x instanceof stdClass) { notNullable($x); } } // Safe - Alternative 2: Changing Parameter function withNonNullableParam(stdClass $x) { notNullable($x); } Loading history...
59	2	new GuzzleClient(
60		[
61	2	'handler' => HandlerStack::create(
62	2	new HttpClientAdapter(
63		$loop,
		0 ignored issues – show Bug introduced 2016-08-07 20:12 UTC by Report Bug Copy Issue Report It seems like `$loop` defined by parameter `$loop` on line `54` can be `null`; however, `WyriHaximus\React\Guzzle...tAdapter::__construct()` does not accept `null`, maybe add an additional type check? It seems like you allow that null is being passed for a parameter, however the function which is called does not seem to accept null. We recommend to add an additional type check (or disallow null for the parameter): function notNullable(stdClass $x) { } // Unsafe function withoutCheck(stdClass $x = null) { notNullable($x); } // Safe - Alternative 1: Adding Additional Type-Check function withCheck(stdClass $x = null) { if ($x instanceof stdClass) { notNullable($x); } } // Safe - Alternative 2: Changing Parameter function withNonNullableParam(stdClass $x) { notNullable($x); } Loading history...
64		$httpClient,
65		$resolver
66		)
67		),
68		]
69		),
70		$options
71		);
72		}
73
74		/**
75		* @param LoopInterface $loop
76		* @param GuzzleClient $guzzle
77		* @param array $options
78		* @return Client
79		*/
80	2	public static function createFromGuzzleClient(
81		LoopInterface $loop,
82		GuzzleClient $guzzle,
83		array $options = []
84		): Client {
85	2	return new Client(
86		$loop,
87		$guzzle,
88		$options
89		);
90		}
91		}
92

php-api-clients / foundation

GitHub Access Token became invalid

Push — master ( f9a93a...d93906 )

Factory::createFromGuzzleClient() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like