HeadlessChromeClient::init() - Code Metrics - Inspection of "merge" - phmLabs/Smoke - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 9abf36...7af07c )

by Nils

created 2017-12-13 13:44 UTC

HeadlessChromeClient::init() A

↳ Parent: HeadlessChromeClient

Complexity

Conditions	2
Paths	2

Size

Total Lines	10
Code Lines	7

Duplication

Lines	10
Ratio	100 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	10
loc	10
rs	9.4285
cc	2
eloc	7
nc	2
nop	2

<?php

namespace whm\Smoke\Http;

use phm\HttpWebdriverClient\Http\Client\Decorator\FileCacheDecorator;
use phm\HttpWebdriverClient\Http\Client\Decorator\LoggerDecorator;
use phm\HttpWebdriverClient\Http\Client\HeadlessChrome\HeadlessChromeClient as HeadlessChrome;
use phm\HttpWebdriverClient\Http\Client\HttpClient;
use Psr\Http\Message\RequestInterface;

class HeadlessChromeClient implements HttpClient
{
    /**
     * @var HeadlessChrome
     */
    private $chromeClient;

    public function init($nocache = false, $clientTimeout = 31000)
    {
        if ($nocache) {
            $this->chromeClient = new HeadlessChrome($clientTimeout);
        } else {
            $chromeClient = new HeadlessChrome($clientTimeout);
            $cachedClient = new FileCacheDecorator($chromeClient);
            $this->chromeClient = new LoggerDecorator($cachedClient);

        }
    }

    public function sendRequest(RequestInterface $request)
    {
        return $this->chromeClient->sendRequest($request);
    }

    public function sendRequests(array $requests)
    {
        return $this->chromeClient->sendRequests($requests);
    }

    public function getClientType()
    {
        return $this->chromeClient->getClientType();
    }

    public function close()
    {
        $this->chromeClient->close();
    }

    public function setOption($key, $value)
    {
        $this->chromeClient->setOption($key, $value);

    }
}


1		<?php
2
3		namespace whm\Smoke\Http;
4
5		use phm\HttpWebdriverClient\Http\Client\Decorator\FileCacheDecorator;
6		use phm\HttpWebdriverClient\Http\Client\Decorator\LoggerDecorator;
7		use phm\HttpWebdriverClient\Http\Client\HeadlessChrome\HeadlessChromeClient as HeadlessChrome;
8		use phm\HttpWebdriverClient\Http\Client\HttpClient;
9		use Psr\Http\Message\RequestInterface;
10
11	View Code Duplication	class HeadlessChromeClient implements HttpClient
12		{
13		/**
14		* @var HeadlessChrome
15		*/
16		private $chromeClient;
17
18		public function init($nocache = false, $clientTimeout = 31000)
19		{
20		if ($nocache) {
21		$this->chromeClient = new HeadlessChrome($clientTimeout);
22		} else {
23		$chromeClient = new HeadlessChrome($clientTimeout);
24		$cachedClient = new FileCacheDecorator($chromeClient);
25		$this->chromeClient = new LoggerDecorator($cachedClient);
		0 ignored issues – show Documentation Bug introduced 2017-11-15 13:27 UTC by Report Bug Copy Issue Report It seems like `new \phm\HttpWebdriverCl...ecorator($cachedClient)` of type `object<phm\HttpWebdriver...orator\LoggerDecorator>` is incompatible with the declared type `object<phm\HttpWebdriver...e\HeadlessChromeClient>` of property `$chromeClient`. 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...
26		}
27		}
28
29		public function sendRequest(RequestInterface $request)
30		{
31		return $this->chromeClient->sendRequest($request);
32		}
33
34		public function sendRequests(array $requests)
35		{
36		return $this->chromeClient->sendRequests($requests);
37		}
38
39		public function getClientType()
40		{
41		return $this->chromeClient->getClientType();
42		}
43
44		public function close()
45		{
46		$this->chromeClient->close();
47		}
48
49		public function setOption($key, $value)
50		{
51		$this->chromeClient->setOption($key, $value);
		0 ignored issues – show Bug introduced 2017-12-13 13:46 UTC by Report Bug Copy Issue Report The method `setOption()` does not seem to exist on `object<phm\HttpWebdriver...e\HeadlessChromeClient>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
52		}
53		}
54

phmLabs / Smoke

Push — master ( 9abf36...7af07c )

HeadlessChromeClient::init() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like