PullRequestEvent::__construct() - Code Metrics - Inspection of "[WIP] Upgrade gitamp to use latest amphp packages" - ekinhbayar/gitamp - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Pull Request — master (#46)

by Ekin

created 2020-09-06 14:29 UTC

PullRequestEvent::__construct() A

↳ Parent: Project

Complexity

Conditions	1
Paths	1

Size

Total Lines	14
Code Lines	10

Duplication

Lines	14
Ratio	100 %

Code Coverage

Tests	10
CRAP Score	1

Importance

Changes

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

<?php declare(strict_types=1);

namespace ekinhbayar\GitAmp\Event\GitHub;

use ekinhbayar\GitAmp\Event\BaseEvent;
use ekinhbayar\GitAmp\Presentation\Information;
use ekinhbayar\GitAmp\Presentation\Type;
use ekinhbayar\GitAmp\Presentation\Ring;
use ekinhbayar\GitAmp\Presentation\Sound\BaseSound;
use ekinhbayar\GitAmp\Presentation\Sound\Swell;
use ekinhbayar\GitAmp\Presentation\Sound\SwellEgg;

class PullRequestEvent extends BaseEvent
{
    private const SPECIAL_REPOSITORIES = [
        'ekinhbayar/gitamp',
        'amphp/amp',
    ];

    public function __construct(array $event)
    {
        parent::__construct(
            (int) $event['id'],
            new Type(2),
            new Information(
                $event['payload']['pull_request']['html_url'],
                $event['payload']['pull_request']['title'],
                $this->buildMessage($event)
            ),
            new Ring(10000, 600),
            $this->buildSound($event),
        );

    }

    private function buildMessage(array $event): string
    {
        return \sprintf(
            '%s %s a PR for %s',
            $event['actor']['login'],
            $event['payload']['action'],
            $event['repo']['name'],
        );
    }

    private function buildSound(array $event): BaseSound
    {
        if (\in_array($event['repo']['name'], self::SPECIAL_REPOSITORIES, true)) {
            return new SwellEgg();
        }

        return new Swell();
    }
}


1		<?php declare(strict_types=1);
2
3		namespace ekinhbayar\GitAmp\Event\GitHub;
4
5		use ekinhbayar\GitAmp\Event\BaseEvent;
6		use ekinhbayar\GitAmp\Presentation\Information;
7		use ekinhbayar\GitAmp\Presentation\Type;
8		use ekinhbayar\GitAmp\Presentation\Ring;
9		use ekinhbayar\GitAmp\Presentation\Sound\BaseSound;
10		use ekinhbayar\GitAmp\Presentation\Sound\Swell;
11		use ekinhbayar\GitAmp\Presentation\Sound\SwellEgg;
12
13		class PullRequestEvent extends BaseEvent
14		{
15		private const SPECIAL_REPOSITORIES = [
16		'ekinhbayar/gitamp',
17		'amphp/amp',
18		];
19
20	2	public function __construct(array $event)
21		{
22	2	parent::__construct(
23	2	(int) $event['id'],
24	2	new Type(2),
25	2	new Information(
26	2	$event['payload']['pull_request']['html_url'],
27	2	$event['payload']['pull_request']['title'],
28	2	$this->buildMessage($event)
29		),
30	2	new Ring(10000, 600),
31	2	$this->buildSound($event),
32		);
		0 ignored issues – show Bug introduced 2020-09-06 14:35 UTC by Report Bug Copy Issue Report This code did not parse for me. Apparently, there is an error somewhere around this line: Syntax error, unexpected ')' Loading history...
33		}
34
35	2	private function buildMessage(array $event): string
36		{
37	2	return \sprintf(
38	2	'%s %s a PR for %s',
39	2	$event['actor']['login'],
40	2	$event['payload']['action'],
41	2	$event['repo']['name'],
42		);
43		}
44
45	2	private function buildSound(array $event): BaseSound
46		{
47	2	if (\in_array($event['repo']['name'], self::SPECIAL_REPOSITORIES, true)) {
48	1	return new SwellEgg();
49		}
50
51	1	return new Swell();
52		}
53		}
54

ekinhbayar / gitamp

Pull Request — master (#46)

PullRequestEvent::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like