Request::__invoke() - Code Metrics - Inspection of "Moved origin builder to dedicated class" - ekinhbayar/gitamp - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 50dac6...4872ea )

by Pieter

created 2017-03-04 17:37 UTC

Request::__invoke() A

↳ Parent: Request

Complexity

Conditions	1
Paths	1

Size

Total Lines	10
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	0

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

<?php declare(strict_types = 1);

namespace ekinhbayar\GitAmp\Log;

use Aerys\Bootable;
use Aerys\Request as AerysRequest;
use Aerys\Response;use Aerys\Server;
// Bad
$a = 5; $b = 6; $c = 7;

// Good
$a = 5;
$b = 6;
$c = 7;
use Psr\Log\LoggerInterface;

class Request implements Bootable
{
    private $logger;

    public function boot(Server $server, LoggerInterface $logger)
    {
        $this->logger = $logger;
    }

    public function __invoke(AerysRequest $request, Response $response)
    {
        $this->logger->debug('Incoming request', [
            'method'     => $request->getMethod(),
            'uri'        => $request->getUri(),
            'headers'    => $request->getAllHeaders(),
            'parameters' => $request->getAllParams(),
            'body'       => $request->getBody(),
        ]);
    }
}


1		<?php declare(strict_types = 1);
2
3		namespace ekinhbayar\GitAmp\Log;
4
5		use Aerys\Bootable;
6		use Aerys\Request as AerysRequest;
7		use Aerys\Response;use Aerys\Server;
		0 ignored issues – show Coding Style introduced 2017-03-04 17:40 UTC by Report Bug Copy Issue Report It is generally recommended to place each PHP statement on a line by itself. Let’s take a look at an example: // Bad $a = 5; $b = 6; $c = 7; // Good $a = 5; $b = 6; $c = 7; Loading history...
8		use Psr\Log\LoggerInterface;
9
10		class Request implements Bootable
11		{
12		private $logger;
13
14	1	public function boot(Server $server, LoggerInterface $logger)
15		{
16	1	$this->logger = $logger;
17		}
18
19	1	public function __invoke(AerysRequest $request, Response $response)
20		{
21	1	$this->logger->debug('Incoming request', [
22	1	'method' => $request->getMethod(),
23	1	'uri' => $request->getUri(),
24	1	'headers' => $request->getAllHeaders(),
25	1	'parameters' => $request->getAllParams(),
26	1	'body' => $request->getBody(),
27		]);
28		}
29		}
30

ekinhbayar / gitamp

Push — master ( 50dac6...4872ea )

Request::__invoke() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like