Factory::isCli() - Code Metrics - Inspection of "[#11] - add unit tests for Json request + Request..." - Logi-CE/fmup - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — issue_11 (3b88d6)

by Jay

created 2017-08-25 13:24 UTC

Factory::isCli() A

↳ Parent: Factory

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

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

<?php
namespace FMUP\Request;

class Factory
{
    use \FMUP\Sapi\OptionalTrait;

    const CONTENT_TYPE = 'Content-Type';
    const JSON_HEADER = 'application/json';

    /**
     * Build the correct request object
     * @return Cli|Http|Json
     */
    public function get()
    {
        if ($this->isCli()) {
            return new \FMUP\Request\Cli();
        }
        if ($this->isJson()) {
            return new \FMUP\Request\Json();
        }
        return new \FMUP\Request\Http();
    }

    /**
     * Checks if request should be cli
     * @return bool
     */
    private function isCli()
    {
        return $this->getSapi()->get() == \FMUP\Sapi::CLI;
    }

    /**
     * Checks if request should be json
     * @return bool
     */
    private function isJson()
    {
        $headers = !$this->isCli() ? (array)$this->getHeaders() : [];
        return isset($headers[self::CONTENT_TYPE]) && in_array(self::JSON_HEADER, (array)$headers[self::CONTENT_TYPE]);
    }

    /**
     * Retrieve headers of the request
     * @codeCoverageIgnore
     * @return array|false
     */
    protected function getHeaders()
    {
        return function_exists('getallheaders') ? getallheaders() : false;
    }
}


1		<?php
2		namespace FMUP\Request;
3
4		class Factory
5		{
6		use \FMUP\Sapi\OptionalTrait;
7
8		const CONTENT_TYPE = 'Content-Type';
9		const JSON_HEADER = 'application/json';
10
11		/**
12		* Build the correct request object
13		* @return Cli\|Http\|Json
14		*/
15	7	public function get()
16		{
17	7	if ($this->isCli()) {
18	2	return new \FMUP\Request\Cli();
19		}
20	5	if ($this->isJson()) {
21	1	return new \FMUP\Request\Json();
22		}
23	4	return new \FMUP\Request\Http();
24		}
25
26		/**
27		* Checks if request should be cli
28		* @return bool
29		*/
30	7	private function isCli()
31		{
32	7	return $this->getSapi()->get() == \FMUP\Sapi::CLI;
33		}
34
35		/**
36		* Checks if request should be json
37		* @return bool
38		*/
39	5	private function isJson()
40		{
41	5	$headers = !$this->isCli() ? (array)$this->getHeaders() : [];
42	5	return isset($headers[self::CONTENT_TYPE]) && in_array(self::JSON_HEADER, (array)$headers[self::CONTENT_TYPE]);
43		}
44
45		/**
46		* Retrieve headers of the request
47		* @codeCoverageIgnore
48		* @return array\|false
49		*/
50		protected function getHeaders()
51		{
52		return function_exists('getallheaders') ? getallheaders() : false;
53		}
54		}
55

Logi-CE / fmup

Branch — issue_11 (3b88d6)

Factory::isCli() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like