RequestHandler::handle() - Code Metrics - Inspection of "Merge pull request #326 from Schlaefer/feature/1.1..." - PhileCMS/Phile - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — develop ( df587f...f7c208 )

by Schlaefer

created 2018-03-11 13:29 UTC

RequestHandler::handle() A

↳ Parent: RequestHandler

Complexity

Conditions	3
Paths	4

Size

Total Lines	14
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	3.009

Importance

Changes

Metric	Value
dl	0
loc	14
ccs	9
cts	10
cp	0.9
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	10
nc	4
nop	1
crap	3.009

<?php
/**
 * @author  PhileCMS
 * @link    https://philecms.com
 * @license http://opensource.org/licenses/MIT
 */

namespace Phile\Http;

use Interop\Http\Factory\ResponseFactoryInterface;
use Psr\Http\Message\ResponseInterface;
use Psr\Http\Message\ServerRequestInterface;
use Psr\Http\Server\RequestHandlerInterface;

/**
 * Implements a PSR-15 compatible request handler
 */
class RequestHandler implements RequestHandlerInterface
{
    /** @var Iterator the middleware to process */
    protected $queue;

    /** @var ResponseFactoryInterface PSR-17 HTTP response factory */
    protected $responseFactory;

    /** @var integer level */
    protected $level = 0;

    /**
     * Constructor
     *
     * @param ResponseFactoryInterface $responseFactory
     */
    public function __construct(MiddlewareQueue $queue, ResponseFactoryInterface $responseFactory)
    {
        $this->responseFactory = $responseFactory;
        $this->queue = $queue->getIterator();

    }

    /**
     * Handle the request and return a response.
     *
     * @param ServerRequestInterface $request
     * @return ResponseInterface
     */
    public function handle(ServerRequestInterface $request): ResponseInterface
    {
        if ($this->level++ === 0) {
            $this->queue->rewind();
        }
        $current = $this->queue->current();
        if (!$current) {
            return $this->responseFactory->createResponse();
        }
        $this->queue->next();
        $result = call_user_func_array([$current, 'process'], [$request, $this]);
        $this->level--;
        return $result;
    }
}


1		<?php
2		/**
3		* @author PhileCMS
4		* @link https://philecms.com
5		* @license http://opensource.org/licenses/MIT
6		*/
7
8		namespace Phile\Http;
9
10		use Interop\Http\Factory\ResponseFactoryInterface;
11		use Psr\Http\Message\ResponseInterface;
12		use Psr\Http\Message\ServerRequestInterface;
13		use Psr\Http\Server\RequestHandlerInterface;
14
15		/**
16		* Implements a PSR-15 compatible request handler
17		*/
18		class RequestHandler implements RequestHandlerInterface
19		{
20		/** @var Iterator the middleware to process */
21		protected $queue;
22
23		/** @var ResponseFactoryInterface PSR-17 HTTP response factory */
24		protected $responseFactory;
25
26		/** @var integer level */
27		protected $level = 0;
28
29		/**
30		* Constructor
31		*
32		* @param ResponseFactoryInterface $responseFactory
33		*/
34	4	public function __construct(MiddlewareQueue $queue, ResponseFactoryInterface $responseFactory)
35		{
36	4	$this->responseFactory = $responseFactory;
37	4	$this->queue = $queue->getIterator();
		0 ignored issues – show Documentation Bug introduced 2018-03-09 16:25 UTC by Report Bug Copy Issue Report It seems like `$queue->getIterator()` of type `object<Traversable>` is incompatible with the declared type `object<Phile\Http\Iterator>` of property `$queue`. 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...
38	4	}
39
40		/**
41		* Handle the request and return a response.
42		*
43		* @param ServerRequestInterface $request
44		* @return ResponseInterface
45		*/
46	4	public function handle(ServerRequestInterface $request): ResponseInterface
47		{
48	4	if ($this->level++ === 0) {
49	4	$this->queue->rewind();
50		}
51	4	$current = $this->queue->current();
52	4	if (!$current) {
53		return $this->responseFactory->createResponse();
54		}
55	4	$this->queue->next();
56	4	$result = call_user_func_array([$current, 'process'], [$request, $this]);
57	4	$this->level--;
58	4	return $result;
59		}
60		}
61

PhileCMS / Phile

Push — develop ( df587f...f7c208 )

RequestHandler::handle() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like