NeutrinoApiCom::hydrateOperatingSystem() - Code Metrics - Inspection of "new provider" - ThaDafinser/UserAgentParser - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#46)

by Martin

created 2016-01-08 12:21 UTC

NeutrinoApiCom::hydrateOperatingSystem() B

↳ Parent: NeutrinoApiCom

Complexity

Conditions	5
Paths	4

Size

Total Lines	10
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	5

Importance

Changes	2
Bugs	0	Features	0

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

<?php
namespace UserAgentParser\Provider\Http;

use GuzzleHttp\Client;
use GuzzleHttp\Psr7\Request;
use stdClass;
use UserAgentParser\Exception;
use UserAgentParser\Model;

class NeutrinoApiCom extends AbstractHttpProvider
{
    /**
     * Name of the provider
     *
     * @var string
     */
    protected $name = 'NeutrinoApiCom';

    /**
     * Homepage of the provider
     *
     * @var string
     */
    protected $homepage = 'https://www.neutrinoapi.com/';

    protected $detectionCapabilities = [

        'browser' => [
            'name'    => true,
            'version' => true,
        ],

        'renderingEngine' => [
            'name'    => false,
            'version' => false,
        ],

        'operatingSystem' => [
            'name'    => true,
            'version' => true,
        ],

        'device' => [
            'model'    => true,
            'brand'    => true,
            'type'     => true,
            'isMobile' => true,
            'isTouch'  => false,
        ],

        'bot' => [
            'isBot' => true,
            'name'  => true,
            'type'  => false,
        ],
    ];

    protected $defaultValues = [
        'unknown',
    ];

    private static $uri = 'https://neutrinoapi.com/user-agent-info';

    private $apiUserId;

    private $apiKey;

    public function __construct(Client $client, $apiUserId, $apiKey)
    {
        parent::__construct($client);

        $this->apiUserId = $apiUserId;
        $this->apiKey    = $apiKey;
    }

    public function getVersion()
    {
        return;
    }

    /**
     *
     * @param  string                     $userAgent
     * @param  array                      $headers
     * @return stdClass
     * @throws Exception\RequestException
     */
    protected function getResult($userAgent, array $headers)

    {
        /*
         * an empty UserAgent makes no sense
         */
        if ($userAgent == '') {
            throw new Exception\NoResultFoundException('No result found for user agent: ' . $userAgent);
        }

        $params = [
            'user-id'       => $this->apiUserId,
            'api-key'       => $this->apiKey,
            'output-format' => 'json',
            'output-case'   => 'snake',

            'user-agent' => $userAgent,
        ];

        $body = http_build_query($params, null, '&');

        $request = new Request('POST', self::$uri, [
            'Content-Type' => 'application/x-www-form-urlencoded',
        ], $body);

        try {
            $response = $this->getResponse($request);
        } catch (Exception\RequestException $ex) {
            /* @var $prevEx \GuzzleHttp\Exception\ClientException */
            $prevEx = $ex->getPrevious();

            if ($prevEx->hasResponse() === true && $prevEx->getResponse()->getStatusCode() === 403) {

                throw new Exception\InvalidCredentialsException('Your API userId "' . $this->apiUserId . '" and key "' . $this->apiKey . '" is not valid for ' . $this->getName(), null, $ex);
            }

            throw $ex;
        }

        /*
         * no json returned?
         */
        $contentType = $response->getHeader('Content-Type');
        if (! isset($contentType[0]) || $contentType[0] != 'application/json;charset=UTF-8') {

            throw new Exception\RequestException('Could not get valid "application/json" response from "' . $request->getUri() . '". Response is "' . $response->getBody()->getContents() . '"');
        }

        $content = json_decode($response->getBody()->getContents());

        /*
         * errors
         */
        if (isset($content->api_error)) {
            switch ($content->api_error) {

                case 1:

                    throw new Exception\RequestException('"' . $content->api_error_msg . '" response from "' . $request->getUri() . '". Response is "' . print_r($content, true) . '"');
                    break;
function fx() {
    try {
        doSomething();
        return true;
    }
    catch (\Exception $e) {
        return false;
    }

    return false;
}

                case 2:
                    throw new Exception\LimitationExceededException('Exceeded the maximum number of request with API userId "' . $this->apiUserId . '" and key "' . $this->apiKey . '" for ' . $this->getName());
                    break;
function fx() {
    try {
        doSomething();
        return true;
    }
    catch (\Exception $e) {
        return false;
    }

    return false;
}

                default:

                    throw new Exception\RequestException('"' . $content->api_error_msg . '" response from "' . $request->getUri() . '". Response is "' . print_r($content, true) . '"');
                    break;
function fx() {
    try {
        doSomething();
        return true;
    }
    catch (\Exception $e) {
        return false;
    }

    return false;
}
            }
        }

        /*
         * Missing data?
         */
        if (! $content instanceof stdClass) {
            throw new Exception\RequestException('Could not get valid response from "' . $request->getUri() . '". Response is "' . $response->getBody()->getContents() . '"');
        }

        return $content;
    }

    /**
     *
     * @param stdClass $resultRaw
     *
     * @return bool
     */
    private function hasResult(stdClass $resultRaw)

    {
        if (isset($resultRaw->type) && $this->isRealResult($resultRaw->type)) {
            return true;
        }

        return false;
    }

    /**
     *
     * @param  stdClass $resultRaw
     * @return boolean
     */
    private function isBot(stdClass $resultRaw)

    {
        if (isset($resultRaw->type) && $resultRaw->type === 'robot') {
            return true;
        }

        return false;
    }

    /**
     *
     * @param Model\Bot $bot
     * @param stdClass  $resultRaw
     */
    private function hydrateBot(Model\Bot $bot, stdClass $resultRaw)

    {
        $bot->setIsBot(true);

        if (isset($resultRaw->browser_name) && $this->isRealResult($resultRaw->browser_name) === true) {
            $bot->setName($resultRaw->browser_name);
        }
    }

    /**
     *
     * @param Model\Browser $browser
     * @param stdClass      $resultRaw
     */
    private function hydrateBrowser(Model\Browser $browser, stdClass $resultRaw)
    {
        if (isset($resultRaw->browser_name) && $this->isRealResult($resultRaw->browser_name) === true) {
            $browser->setName($resultRaw->browser_name);
        }

        if (isset($resultRaw->version) && $this->isRealResult($resultRaw->version) === true) {
            $browser->getVersion()->setComplete($resultRaw->version);
        }
    }

    /**
     *
     * @param Model\OperatingSystem $os
     * @param stdClass              $resultRaw
     */
    private function hydrateOperatingSystem(Model\OperatingSystem $os, stdClass $resultRaw)
    {
        if (isset($resultRaw->operating_system_family) && $this->isRealResult($resultRaw->operating_system_family) === true) {
            $os->setName($resultRaw->operating_system_family);
        }

        if (isset($resultRaw->operating_system_version) && $this->isRealResult($resultRaw->operating_system_version) === true) {
            $os->getVersion()->setComplete($resultRaw->operating_system_version);
        }
    }

    /**
     *
     * @param Model\Device $device
     * @param stdClass     $resultRaw
     */
    private function hydrateDevice(Model\Device $device, stdClass $resultRaw)
    {
        if (isset($resultRaw->mobile_model) && $this->isRealResult($resultRaw->mobile_model) === true) {
            $device->setModel($resultRaw->mobile_model);
        }

        if (isset($resultRaw->mobile_brand) && $this->isRealResult($resultRaw->mobile_brand) === true) {

            $device->setBrand($resultRaw->mobile_brand);
        }

        if (isset($resultRaw->type) && $this->isRealResult($resultRaw->type) === true) {

            $device->setType($resultRaw->type);
        }

        if (isset($resultRaw->is_mobile) && $resultRaw->is_mobile === true) {
            $device->setIsMobile(true);
        }
    }

    public function parse($userAgent, array $headers = [])

    {
        $resultRaw = $this->getResult($userAgent, $headers);

        /*
         * No result found?
         */
        if ($this->hasResult($resultRaw) !== true) {
            throw new Exception\NoResultFoundException('No result found for user agent: ' . $userAgent);
        }

        /*
         * Hydrate the model
         */
        $result = new Model\UserAgent();
        $result->setProviderResultRaw($resultRaw);

        /*
         * Bot detection
         */
        if ($this->isBot($resultRaw) === true) {
            $this->hydrateBot($result->getBot(), $resultRaw);

            return $result;
        }

        /*
         * hydrate the result
         */
        $this->hydrateBrowser($result->getBrowser(), $resultRaw);
        $this->hydrateOperatingSystem($result->getOperatingSystem(), $resultRaw);
        $this->hydrateDevice($result->getDevice(), $resultRaw);

        return $result;
    }
}


1			<?php
2			namespace UserAgentParser\Provider\Http;
3
4			use GuzzleHttp\Client;
5			use GuzzleHttp\Psr7\Request;
6			use stdClass;
7			use UserAgentParser\Exception;
8			use UserAgentParser\Model;
9
10			class NeutrinoApiCom extends AbstractHttpProvider
11			{
12			/**
13			* Name of the provider
14			*
15			* @var string
16			*/
17			protected $name = 'NeutrinoApiCom';
18
19			/**
20			* Homepage of the provider
21			*
22			* @var string
23			*/
24			protected $homepage = 'https://www.neutrinoapi.com/';
25
26			protected $detectionCapabilities = [
27
28			'browser' => [
29			'name' => true,
30			'version' => true,
31			],
32
33			'renderingEngine' => [
34			'name' => false,
35			'version' => false,
36			],
37
38			'operatingSystem' => [
39			'name' => true,
40			'version' => true,
41			],
42
43			'device' => [
44			'model' => true,
45			'brand' => true,
46			'type' => true,
47			'isMobile' => true,
48			'isTouch' => false,
49			],
50
51			'bot' => [
52			'isBot' => true,
53			'name' => true,
54			'type' => false,
55			],
56			];
57
58			protected $defaultValues = [
59			'unknown',
60			];
61
62			private static $uri = 'https://neutrinoapi.com/user-agent-info';
63
64			private $apiUserId;
65
66			private $apiKey;
67
68	18		public function __construct(Client $client, $apiUserId, $apiKey)
69			{
70	18		parent::__construct($client);
71
72	18		$this->apiUserId = $apiUserId;
73	18		$this->apiKey = $apiKey;
74	18		}
75
76	1		public function getVersion()
77			{
78	1		return;
79			}
80
81			/**
82			*
83			* @param string $userAgent
84			* @param array $headers
85			* @return stdClass
86			* @throws Exception\RequestException
87			*/
88	13		protected function getResult($userAgent, array $headers)
			0 ignored issues – show Unused Code introduced 2015-12-11 22:04 UTC by Report Bug Copy Issue Report The parameter `$headers` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
89			{
90			/*
91			* an empty UserAgent makes no sense
92			*/
93	13		if ($userAgent == '') {
94	1		throw new Exception\NoResultFoundException('No result found for user agent: ' . $userAgent);
95			}
96
97			$params = [
98	12		'user-id' => $this->apiUserId,
99	12		'api-key' => $this->apiKey,
100	12		'output-format' => 'json',
101	12		'output-case' => 'snake',
102
103	12		'user-agent' => $userAgent,
104	12		];
105
106	12		$body = http_build_query($params, null, '&');
107
108	12		$request = new Request('POST', self::$uri, [
109	12		'Content-Type' => 'application/x-www-form-urlencoded',
110	12		], $body);
111
112			try {
113	12		$response = $this->getResponse($request);
114	12		} catch (Exception\RequestException $ex) {
115			/* @var $prevEx \GuzzleHttp\Exception\ClientException */
116	2		$prevEx = $ex->getPrevious();
117
118	2	View Code Duplication	if ($prevEx->hasResponse() === true && $prevEx->getResponse()->getStatusCode() === 403) {
			0 ignored issues – show Duplication introduced 2016-01-08 12:27 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
119	1		throw new Exception\InvalidCredentialsException('Your API userId "' . $this->apiUserId . '" and key "' . $this->apiKey . '" is not valid for ' . $this->getName(), null, $ex);
120			}
121
122	1		throw $ex;
123			}
124
125			/*
126			* no json returned?
127			*/
128	10		$contentType = $response->getHeader('Content-Type');
129	10	View Code Duplication	if (! isset($contentType[0]) \|\| $contentType[0] != 'application/json;charset=UTF-8') {
			0 ignored issues – show Duplication introduced 2015-12-11 22:04 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
130	1		throw new Exception\RequestException('Could not get valid "application/json" response from "' . $request->getUri() . '". Response is "' . $response->getBody()->getContents() . '"');
131			}
132
133	9		$content = json_decode($response->getBody()->getContents());
134
135			/*
136			* errors
137			*/
138	9		if (isset($content->api_error)) {
139	3		switch ($content->api_error) {
140
141	3	View Code Duplication	case 1:
			0 ignored issues – show Duplication introduced 2015-12-11 22:04 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
142	1		throw new Exception\RequestException('"' . $content->api_error_msg . '" response from "' . $request->getUri() . '". Response is "' . print_r($content, true) . '"');
143			break;
			0 ignored issues – show Unused Code introduced 2015-12-11 22:04 UTC by Report Bug Copy Issue Report `break;` does not seem to be reachable. This check looks for unreachable code. It uses sophisticated control flow analysis techniques to find statements which will never be executed. Unreachable code is most often the result of `return`, `die` or `exit` statements that have been added for debug purposes. function fx() { try { doSomething(); return true; } catch (\Exception $e) { return false; } return false; } In the above example, the last `return false` will never be executed, because a return statement has already been met in every possible execution path. Loading history...
144
145	2		case 2:
146	1		throw new Exception\LimitationExceededException('Exceeded the maximum number of request with API userId "' . $this->apiUserId . '" and key "' . $this->apiKey . '" for ' . $this->getName());
147			break;
			0 ignored issues – show Unused Code introduced 2015-12-11 22:04 UTC by Report Bug Copy Issue Report `break;` does not seem to be reachable. This check looks for unreachable code. It uses sophisticated control flow analysis techniques to find statements which will never be executed. Unreachable code is most often the result of `return`, `die` or `exit` statements that have been added for debug purposes. function fx() { try { doSomething(); return true; } catch (\Exception $e) { return false; } return false; } In the above example, the last `return false` will never be executed, because a return statement has already been met in every possible execution path. Loading history...
148
149	1	View Code Duplication	default:
			0 ignored issues – show Duplication introduced 2015-12-11 22:04 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
150	1		throw new Exception\RequestException('"' . $content->api_error_msg . '" response from "' . $request->getUri() . '". Response is "' . print_r($content, true) . '"');
151			break;
			0 ignored issues – show Unused Code introduced 2015-12-11 22:04 UTC by Report Bug Copy Issue Report `break;` does not seem to be reachable. This check looks for unreachable code. It uses sophisticated control flow analysis techniques to find statements which will never be executed. Unreachable code is most often the result of `return`, `die` or `exit` statements that have been added for debug purposes. function fx() { try { doSomething(); return true; } catch (\Exception $e) { return false; } return false; } In the above example, the last `return false` will never be executed, because a return statement has already been met in every possible execution path. Loading history...
152	1		}
153			}
154
155			/*
156			* Missing data?
157			*/
158	6		if (! $content instanceof stdClass) {
159	1		throw new Exception\RequestException('Could not get valid response from "' . $request->getUri() . '". Response is "' . $response->getBody()->getContents() . '"');
160			}
161
162	5		return $content;
163			}
164
165			/**
166			*
167			* @param stdClass $resultRaw
168			*
169			* @return bool
170			*/
171	5	View Code Duplication	private function hasResult(stdClass $resultRaw)
			0 ignored issues – show Duplication introduced 2015-12-11 22:04 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
172			{
173	5		if (isset($resultRaw->type) && $this->isRealResult($resultRaw->type)) {
174	4		return true;
175			}
176
177	1		return false;
178			}
179
180			/**
181			*
182			* @param stdClass $resultRaw
183			* @return boolean
184			*/
185	4	View Code Duplication	private function isBot(stdClass $resultRaw)
			0 ignored issues – show Duplication introduced 2015-12-11 22:04 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
186			{
187	4		if (isset($resultRaw->type) && $resultRaw->type === 'robot') {
188	1		return true;
189			}
190
191	3		return false;
192			}
193
194			/**
195			*
196			* @param Model\Bot $bot
197			* @param stdClass $resultRaw
198			*/
199	1	View Code Duplication	private function hydrateBot(Model\Bot $bot, stdClass $resultRaw)
			0 ignored issues – show Duplication introduced 2015-12-11 22:04 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
200			{
201	1		$bot->setIsBot(true);
202
203	1		if (isset($resultRaw->browser_name) && $this->isRealResult($resultRaw->browser_name) === true) {
204	1		$bot->setName($resultRaw->browser_name);
205	1		}
206	1		}
207
208			/**
209			*
210			* @param Model\Browser $browser
211			* @param stdClass $resultRaw
212			*/
213	3		private function hydrateBrowser(Model\Browser $browser, stdClass $resultRaw)
214			{
215	3		if (isset($resultRaw->browser_name) && $this->isRealResult($resultRaw->browser_name) === true) {
216	1		$browser->setName($resultRaw->browser_name);
217	1		}
218
219	3		if (isset($resultRaw->version) && $this->isRealResult($resultRaw->version) === true) {
220	1		$browser->getVersion()->setComplete($resultRaw->version);
221	1		}
222	3		}
223
224			/**
225			*
226			* @param Model\OperatingSystem $os
227			* @param stdClass $resultRaw
228			*/
229	3		private function hydrateOperatingSystem(Model\OperatingSystem $os, stdClass $resultRaw)
230			{
231	3		if (isset($resultRaw->operating_system_family) && $this->isRealResult($resultRaw->operating_system_family) === true) {
232	1		$os->setName($resultRaw->operating_system_family);
233	1		}
234
235	3		if (isset($resultRaw->operating_system_version) && $this->isRealResult($resultRaw->operating_system_version) === true) {
236	1		$os->getVersion()->setComplete($resultRaw->operating_system_version);
237	1		}
238	3		}
239
240			/**
241			*
242			* @param Model\Device $device
243			* @param stdClass $resultRaw
244			*/
245	3		private function hydrateDevice(Model\Device $device, stdClass $resultRaw)
246			{
247	3		if (isset($resultRaw->mobile_model) && $this->isRealResult($resultRaw->mobile_model) === true) {
248	1		$device->setModel($resultRaw->mobile_model);
249	1		}
250
251	3	View Code Duplication	if (isset($resultRaw->mobile_brand) && $this->isRealResult($resultRaw->mobile_brand) === true) {
			0 ignored issues – show Duplication introduced 2015-12-11 22:04 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
252	1		$device->setBrand($resultRaw->mobile_brand);
253	1		}
254
255	3	View Code Duplication	if (isset($resultRaw->type) && $this->isRealResult($resultRaw->type) === true) {
			0 ignored issues – show Duplication introduced 2015-12-11 22:04 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
256	3		$device->setType($resultRaw->type);
257	3		}
258
259	3		if (isset($resultRaw->is_mobile) && $resultRaw->is_mobile === true) {
260	1		$device->setIsMobile(true);
261	1		}
262	3		}
263
264	13	View Code Duplication	public function parse($userAgent, array $headers = [])
			0 ignored issues – show Duplication introduced 2015-12-11 22:04 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
265			{
266	13		$resultRaw = $this->getResult($userAgent, $headers);
267
268			/*
269			* No result found?
270			*/
271	5		if ($this->hasResult($resultRaw) !== true) {
272	1		throw new Exception\NoResultFoundException('No result found for user agent: ' . $userAgent);
273			}
274
275			/*
276			* Hydrate the model
277			*/
278	4		$result = new Model\UserAgent();
279	4		$result->setProviderResultRaw($resultRaw);
280
281			/*
282			* Bot detection
283			*/
284	4		if ($this->isBot($resultRaw) === true) {
285	1		$this->hydrateBot($result->getBot(), $resultRaw);
286
287	1		return $result;
288			}
289
290			/*
291			* hydrate the result
292			*/
293	3		$this->hydrateBrowser($result->getBrowser(), $resultRaw);
294	3		$this->hydrateOperatingSystem($result->getOperatingSystem(), $resultRaw);
295	3		$this->hydrateDevice($result->getDevice(), $resultRaw);
296
297	3		return $result;
298			}
299			}
300

ThaDafinser / UserAgentParser

GitHub Access Token became invalid

Pull Request — master (#46)

NeutrinoApiCom::hydrateOperatingSystem() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like