Tmdb - Code Metrics - Inspection of "Active strict mode" - vfalies/tmdb - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — account (184271)

by vincent

created 2017-10-13 14:45 UTC

Tmdb A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	313
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	5

Test Coverage

Coverage

100%

Importance

Changes

Metric	Value
wmc	33
lcom	2
cbo	5
dl	0
loc	313
ccs	98
cts	98
cp	1
rs	9.3999
c	0
b	0
f	0

13 Methods

Rating	Name	Size	Complexity
A	__construct()	8	1
A	getRequest()	6	1
A	postRequest()	6	1
A	deleteRequest()	6	1
B	sendRequest()	26	6
A	buildHTTPUrl()	16	1
A	getConfiguration()	13	3
D	checkOptions()	36	9
A	checkYear()	5	1
A	checkLanguage()	9	3
A	checkSort()	11	3
A	getLogger()	4	1
A	__get()	9	2

<?php declare(strict_types=1);

/**
 * This file is part of the Tmdb package.
 *
 * (c) Vincent Faliès <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * @author Vincent Faliès <[email protected]>
 * @copyright Copyright (c) 2017
 */

namespace VfacTmdb;

use VfacTmdb\Interfaces\TmdbInterface;
use VfacTmdb\Interfaces\HttpRequestInterface;
use Psr\Log\LoggerInterface;
use VfacTmdb\Exceptions\TmdbException;
use VfacTmdb\Exceptions\IncorrectParamException;
use VfacTmdb\Exceptions\ServerErrorException;

/**
 * Tmdb wrapper core class
 * @package Tmdb
 * @author Vincent Faliès <[email protected]>
 * @copyright Copyright (c) 2017
 */
class Tmdb implements TmdbInterface
{

    /**
     * API Key
     * @var string
     */
    private $api_key = null;

    /**
     * Default language for API response
     * @var string
     */
    private $language = 'fr-FR';

    /**
     * Include adult content in search result
     * @var bool
     */
    private $include_adult = false;

    /**
     * API Page result
     * @var int
     */
    private $page = 1;

    /**
     * API configuration
     * @var \stdClass
     */
    protected $configuration = null;

    /**
     * API Genres
     * @var \stdClass
     */
    protected $genres = null;

    /**
     * Base URL of the API
     * @var string
     */
    public $base_api_url = 'https://api.themoviedb.org/';

    /**
     * Logger
     * @var LoggerInterface
     */
    protected $logger = null;

    /**
     * API Version
     * @var int
     */
    protected $version = 3;

    /**
     * Http request object
     * @var HttpRequestInterface
     */
    protected $http_request = null;

    /**
     * Last request url
     * @var string
     */
    protected $url = null;

    /**
     * Constructor
     * @param string $api_key TMDB API Key
     * @param int $version Version of API (Not yet used)
     * @param LoggerInterface $logger Logger used in the class
     * @param HttpRequestInterface $http_request
     */
    public function __construct(string $api_key, int $version = 3, LoggerInterface $logger, HttpRequestInterface $http_request)
    {
        $this->api_key      = $api_key;
        $this->logger       = $logger;
        $this->version      = $version;
        $this->http_request = $http_request;
        $this->request      = new \stdClass;

    }

    /**
     * Send request to TMDB API with GET method
     * @param string $action API action to request
     * @param array $options Array of options of the request (optional)
     * @return \stdClass|null
     */
    public function getRequest(string $action, array $options = array()) : ?\stdClass
    {
        $this->logger->debug('Start sending HTTP request with GET method', array('action' => $action, 'options' => $options));
        $this->url = $this->buildHTTPUrl($action, $options);
        return $this->sendRequest('GET', $this->url);
    }

    /**
     * Send request to TMDB API with POST method
     * @param string $action API action to request
     * @param array $options Array of options of the request (optional)
     * @param array $form_params form_params for request options
     * @return \stdClass|null
     */
    public function postRequest(string $action, array $options = array(), array $form_params = array()) : ?\stdClass
    {
        $this->logger->debug('Start sending HTTP request with POST method', array('action' => $action, 'options' => $options, 'form_params' => $form_params));
        $this->url = $this->buildHTTPUrl($action, $options);
        return $this->sendRequest('POST', $this->url, $form_params);
    }

    /**
     * Send request to TMDB API with DELETE method
     * @param  string $action  API action to request
     * @param  array  $options Array of options of the request (optional)
     * @return \stdClass|null
     */
    public function deleteRequest(string $action, array $options = array()) : ?\stdClass
    {
        $this->logger->debug('Start sending HTTP request with DELETE method', array('action' => $action, 'options' => $options));
        $this->url = $this->buildHTTPUrl($action, $options);
        return $this->sendRequest('DELETE', $this->url);
    }

    /**
     * Send request to TMDB API with GET method
     * @param string $method HTTP method (GET, POST)
     * @param string $url API url to request
     * @param array $form_params form params request options
     * @return \stdClass|null
     */
    protected function sendRequest(string $method, string $url, array $form_params = array()) : ?\stdClass
    {
        switch ($method) {
              case 'GET':
switch ($expr) {
    case "A": { //wrong
        doSomething();
        break;
    }
    case "B"; //wrong
        doSomething();
        break;
    case "C": //right
        doSomething();
        break;
}
                  $res = $this->http_request->getResponse($url);
                  break;
              case 'POST':
                  $res = $this->http_request->postResponse($url, [], $form_params);
                  break;
              case 'DELETE':
                  $res = $this->http_request->deleteResponse($url);
                  break;
        }

        if (empty($res->getBody()))
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
        {
            $this->logger->error('Request Body empty', array('method' => $method, 'url' => $url, 'form_params' => $form_params));
            throw new ServerErrorException();
        }
        $response = json_decode($res->getBody());
        if (empty($response)) {
            $this->logger->error('Request Body can not be decode', array('method' => $method, 'url' => $url, 'form_params' => $form_params));
            throw new ServerErrorException();
        }
        return $response;
    }

    /**
     * Build URL for HTTP Call
     * @param string $action API action to request
     * @param array $options Array of options of the request (optional)
     * @return string
     */
    private function buildHTTPUrl(string $action, array $options) : string
    {
        // Url construction
        $url = $this->base_api_url . $this->version . '/' . $action;

        // Parameters
        $params            = [];
        $params['api_key'] = $this->api_key;

        $params = array_merge($params, $options);

        // URL with paramters construction
        $url = $url . '?' . http_build_query($params);

        return $url;
    }

    /**
     * Get API Configuration
     * @return \stdClass
     * @throws TmdbException
     */
    public function getConfiguration() : \stdClass
    {
        try {
            $this->logger->debug('Start getting configuration');
            if (is_null($this->configuration)) {
                $this->logger->debug('No configuration found, sending HTTP request to get it');
                $this->configuration = $this->getRequest('configuration');
            }
            return $this->configuration;
        } catch (TmdbException $ex) {
            throw $ex;
        }
    }

    /**
     * Check options rules before send request
     * @param array $options Array of options to validate
     * @return array
     * @throws IncorrectParamException
     */
    public function checkOptions(array $options) : array
    {
        $params                  = [];
        // Set default options
        // $params['language']      = $this->language;

        // $params['include_adult'] = $this->include_adult;

        // $params['page']          = $this->page;

        // Check options
        foreach ($options as $key => $value) {
            switch ($key) {
                case 'year':
                    $params[$key] = $this->checkYear($value);
                    break;
                case 'language':
                    $params[$key] = $this->checkLanguage($value);
                    break;
                case 'include_adult':
                    $params[$key] = filter_var($value, FILTER_VALIDATE_BOOLEAN);
                    break;
                case 'page':
                    $params[$key] = (int) $value;
                    break;
                case 'sort_by':
                    $params[$key] = $this->checkSort($value);
                    break;
                case 'query':
                case 'session_id':
                    $params[$key] = trim($value);
                    break;
                default:
                    $this->logger->error('Unknown param options', array('options', $options));
                    throw new IncorrectParamException;
            }
        }
        return $params;
    }

    /**
     * Check year format
     * @param mixed $year year to validate
     * @return int year validated
     */
    private function checkYear($year) : int
    {
        $year = (int) $year;
        return $year;
    }

    /**
     * Check language
     * @param string $language Language string with format ISO 639-1
     * @return string Language string validated
     * @throws IncorrectParamException
     */
    private function checkLanguage(string $language) : string
    {
        $check = preg_match("#([a-z]{2})-([A-Z]{2})#", $language);
        if ($check === 0 || $check === false) {
            $this->logger->error('Incorrect language param option', array('language' => $language));
            throw new IncorrectParamException;
        }
        return $language;
    }

    /**
     * Check sort direction
     * @param  string $direction direction of sorting
     * @return string            Sort string validated
     * @throws IncorrectParamException
     */
    private function checkSort(string $direction) : string
    {
        switch ($direction) {
            case 'asc':
            case 'desc':
                break;
            default:
                throw new IncorrectParamException;
        }
        return 'created_at.'.$direction;
    }

    /**
     * Get logger
     * @return LoggerInterface
     */
    public function getLogger() : LoggerInterface
    {
        return $this->logger;
    }

    /**
     * Magical property getter
     * @param  string $name Name of the property
     * @return string       Value of the property
     */
    public function __get(string $name) : string
    {
        switch ($name) {
            case 'url':
                return $this->$name;
            default:
                throw new IncorrectParamException;
        }
    }
}


1		<?php declare(strict_types=1);
2
3		/**
4		* This file is part of the Tmdb package.
5		*
6		* (c) Vincent Faliès <[email protected]>
7		*
8		* For the full copyright and license information, please view the LICENSE
9		* file that was distributed with this source code.
10		*
11		* @author Vincent Faliès <[email protected]>
12		* @copyright Copyright (c) 2017
13		*/
14
15		namespace VfacTmdb;
16
17		use VfacTmdb\Interfaces\TmdbInterface;
18		use VfacTmdb\Interfaces\HttpRequestInterface;
19		use Psr\Log\LoggerInterface;
20		use VfacTmdb\Exceptions\TmdbException;
21		use VfacTmdb\Exceptions\IncorrectParamException;
22		use VfacTmdb\Exceptions\ServerErrorException;
23
24		/**
25		* Tmdb wrapper core class
26		* @package Tmdb
27		* @author Vincent Faliès <[email protected]>
28		* @copyright Copyright (c) 2017
29		*/
30		class Tmdb implements TmdbInterface
31		{
32
33		/**
34		* API Key
35		* @var string
36		*/
37		private $api_key = null;
38
39		/**
40		* Default language for API response
41		* @var string
42		*/
43		private $language = 'fr-FR';
44
45		/**
46		* Include adult content in search result
47		* @var bool
48		*/
49		private $include_adult = false;
50
51		/**
52		* API Page result
53		* @var int
54		*/
55		private $page = 1;
56
57		/**
58		* API configuration
59		* @var \stdClass
60		*/
61		protected $configuration = null;
62
63		/**
64		* API Genres
65		* @var \stdClass
66		*/
67		protected $genres = null;
68
69		/**
70		* Base URL of the API
71		* @var string
72		*/
73		public $base_api_url = 'https://api.themoviedb.org/';
74
75		/**
76		* Logger
77		* @var LoggerInterface
78		*/
79		protected $logger = null;
80
81		/**
82		* API Version
83		* @var int
84		*/
85		protected $version = 3;
86
87		/**
88		* Http request object
89		* @var HttpRequestInterface
90		*/
91		protected $http_request = null;
92
93		/**
94		* Last request url
95		* @var string
96		*/
97		protected $url = null;
98
99		/**
100		* Constructor
101		* @param string $api_key TMDB API Key
102		* @param int $version Version of API (Not yet used)
103		* @param LoggerInterface $logger Logger used in the class
104		* @param HttpRequestInterface $http_request
105		*/
106	359	public function __construct(string $api_key, int $version = 3, LoggerInterface $logger, HttpRequestInterface $http_request)
107		{
108	359	$this->api_key = $api_key;
109	359	$this->logger = $logger;
110	359	$this->version = $version;
111	359	$this->http_request = $http_request;
112	359	$this->request = new \stdClass;
		0 ignored issues – show Bug introduced 2017-10-12 15:47 UTC by Report Bug Copy Issue Report The property `request` does not seem to exist. Did you mean `http_request`? An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
113	359	}
114
115		/**
116		* Send request to TMDB API with GET method
117		* @param string $action API action to request
118		* @param array $options Array of options of the request (optional)
119		* @return \stdClass\|null
120		*/
121	317	public function getRequest(string $action, array $options = array()) : ?\stdClass
122		{
123	317	$this->logger->debug('Start sending HTTP request with GET method', array('action' => $action, 'options' => $options));
124	317	$this->url = $this->buildHTTPUrl($action, $options);
125	317	return $this->sendRequest('GET', $this->url);
126		}
127
128		/**
129		* Send request to TMDB API with POST method
130		* @param string $action API action to request
131		* @param array $options Array of options of the request (optional)
132		* @param array $form_params form_params for request options
133		* @return \stdClass\|null
134		*/
135	15	public function postRequest(string $action, array $options = array(), array $form_params = array()) : ?\stdClass
136		{
137	15	$this->logger->debug('Start sending HTTP request with POST method', array('action' => $action, 'options' => $options, 'form_params' => $form_params));
138	15	$this->url = $this->buildHTTPUrl($action, $options);
139	15	return $this->sendRequest('POST', $this->url, $form_params);
140		}
141
142		/**
143		* Send request to TMDB API with DELETE method
144		* @param string $action API action to request
145		* @param array $options Array of options of the request (optional)
146		* @return \stdClass\|null
147		*/
148	5	public function deleteRequest(string $action, array $options = array()) : ?\stdClass
149		{
150	5	$this->logger->debug('Start sending HTTP request with DELETE method', array('action' => $action, 'options' => $options));
151	5	$this->url = $this->buildHTTPUrl($action, $options);
152	5	return $this->sendRequest('DELETE', $this->url);
153		}
154
155		/**
156		* Send request to TMDB API with GET method
157		* @param string $method HTTP method (GET, POST)
158		* @param string $url API url to request
159		* @param array $form_params form params request options
160		* @return \stdClass\|null
161		*/
162	6	protected function sendRequest(string $method, string $url, array $form_params = array()) : ?\stdClass
163		{
164		switch ($method) {
165	6	case 'GET':
		0 ignored issues – show Coding Style introduced 2017-10-12 15:47 UTC by Report Bug Copy Issue Report `case` statements should be defined using a colon. As per the PSR-2 coding standard, case statements should not be wrapped in curly braces. There is no need for braces, since each case is terminated by the next `break`. There is also the option to use a semicolon instead of a colon, this is discouraged because many programmers do not even know it works and the colon is universal between programming languages. switch ($expr) { case "A": { //wrong doSomething(); break; } case "B"; //wrong doSomething(); break; case "C": //right doSomething(); break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
166	4	$res = $this->http_request->getResponse($url);
167	4	break;
168	2	case 'POST':
169	1	$res = $this->http_request->postResponse($url, [], $form_params);
170	1	break;
171	1	case 'DELETE':
172	1	$res = $this->http_request->deleteResponse($url);
173	1	break;
174		}
175
176	6	if (empty($res->getBody()))
		0 ignored issues – show Bug introduced 2017-10-13 14:48 UTC by Report Bug Copy Issue Report The variable `$res` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
177		{
178	2	$this->logger->error('Request Body empty', array('method' => $method, 'url' => $url, 'form_params' => $form_params));
179	2	throw new ServerErrorException();
180		}
181	4	$response = json_decode($res->getBody());
182	4	if (empty($response)) {
183	1	$this->logger->error('Request Body can not be decode', array('method' => $method, 'url' => $url, 'form_params' => $form_params));
184	1	throw new ServerErrorException();
185		}
186	3	return $response;
187		}
188
189		/**
190		* Build URL for HTTP Call
191		* @param string $action API action to request
192		* @param array $options Array of options of the request (optional)
193		* @return string
194		*/
195	319	private function buildHTTPUrl(string $action, array $options) : string
196		{
197		// Url construction
198	319	$url = $this->base_api_url . $this->version . '/' . $action;
199
200		// Parameters
201	319	$params = [];
202	319	$params['api_key'] = $this->api_key;
203
204	319	$params = array_merge($params, $options);
205
206		// URL with paramters construction
207	319	$url = $url . '?' . http_build_query($params);
208
209	319	return $url;
210		}
211
212		/**
213		* Get API Configuration
214		* @return \stdClass
215		* @throws TmdbException
216		*/
217	50	public function getConfiguration() : \stdClass
218		{
219		try {
220	50	$this->logger->debug('Start getting configuration');
221	50	if (is_null($this->configuration)) {
222	50	$this->logger->debug('No configuration found, sending HTTP request to get it');
223	50	$this->configuration = $this->getRequest('configuration');
224		}
225	49	return $this->configuration;
226	1	} catch (TmdbException $ex) {
227	1	throw $ex;
228		}
229		}
230
231		/**
232		* Check options rules before send request
233		* @param array $options Array of options to validate
234		* @return array
235		* @throws IncorrectParamException
236		*/
237	308	public function checkOptions(array $options) : array
238		{
239	308	$params = [];
240		// Set default options
241		// $params['language'] = $this->language;
		0 ignored issues – show Unused Code Comprehensibility introduced 2017-10-13 14:48 UTC by Report Bug Copy Issue Report `59%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
242		// $params['include_adult'] = $this->include_adult;
		0 ignored issues – show Unused Code Comprehensibility introduced 2017-10-13 14:48 UTC by Report Bug Copy Issue Report `59%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
243		// $params['page'] = $this->page;
		0 ignored issues – show Unused Code Comprehensibility introduced 2017-10-13 14:48 UTC by Report Bug Copy Issue Report `59%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
244		// Check options
245	308	foreach ($options as $key => $value) {
246		switch ($key) {
247	73	case 'year':
248	2	$params[$key] = $this->checkYear($value);
249	2	break;
250	72	case 'language':
251	37	$params[$key] = $this->checkLanguage($value);
252	36	break;
253	64	case 'include_adult':
254	1	$params[$key] = filter_var($value, FILTER_VALIDATE_BOOLEAN);
255	1	break;
256	64	case 'page':
257	1	$params[$key] = (int) $value;
258	1	break;
259	63	case 'sort_by':
260	2	$params[$key] = $this->checkSort($value);
261	1	break;
262	61	case 'query':
263	33	case 'session_id':
264	56	$params[$key] = trim($value);
265	56	break;
266		default:
267	5	$this->logger->error('Unknown param options', array('options', $options));
268	5	throw new IncorrectParamException;
269		}
270		}
271	301	return $params;
272		}
273
274		/**
275		* Check year format
276		* @param mixed $year year to validate
277		* @return int year validated
278		*/
279	2	private function checkYear($year) : int
280		{
281	2	$year = (int) $year;
282	2	return $year;
283		}
284
285		/**
286		* Check language
287		* @param string $language Language string with format ISO 639-1
288		* @return string Language string validated
289		* @throws IncorrectParamException
290		*/
291	37	private function checkLanguage(string $language) : string
292		{
293	37	$check = preg_match("#([a-z]{2})-([A-Z]{2})#", $language);
294	37	if ($check === 0 \|\| $check === false) {
295	1	$this->logger->error('Incorrect language param option', array('language' => $language));
296	1	throw new IncorrectParamException;
297		}
298	36	return $language;
299		}
300
301		/**
302		* Check sort direction
303		* @param string $direction direction of sorting
304		* @return string Sort string validated
305		* @throws IncorrectParamException
306		*/
307	2	private function checkSort(string $direction) : string
308		{
309		switch ($direction) {
310	2	case 'asc':
311	1	case 'desc':
312	1	break;
313		default:
314	1	throw new IncorrectParamException;
315		}
316	1	return 'created_at.'.$direction;
317		}
318
319		/**
320		* Get logger
321		* @return LoggerInterface
322		*/
323	329	public function getLogger() : LoggerInterface
324		{
325	329	return $this->logger;
326		}
327
328		/**
329		* Magical property getter
330		* @param string $name Name of the property
331		* @return string Value of the property
332		*/
333	73	public function __get(string $name) : string
334		{
335		switch ($name) {
336	73	case 'url':
337	72	return $this->$name;
338		default:
339	1	throw new IncorrectParamException;
340		}
341		}
342		}
343

vfalies / tmdb

Branch — account (184271)

Tmdb A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

13 Methods

Available Fixes

Duplication Side-by-Side

Filter issues like