IssuesEvent::buildSound() - 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

IssuesEvent::buildSound() A

↳ Parent: Project

Complexity

Conditions	2
Paths	2

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	2

Importance

Changes

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

<?php declare(strict_types=1);

namespace ekinhbayar\GitAmp\Event\GitHub;

use ekinhbayar\GitAmp\Event\BaseEvent;
use ekinhbayar\GitAmp\Presentation\Information;
use ekinhbayar\GitAmp\Presentation\Ring;
use ekinhbayar\GitAmp\Presentation\Sound\BaseSound;
use ekinhbayar\GitAmp\Presentation\Sound\Clav;
use ekinhbayar\GitAmp\Presentation\Sound\ClavEgg;
use ekinhbayar\GitAmp\Presentation\Type;

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

    public function __construct(array $event)
    {
        parent::__construct(
            (int) $event['id'],
            new Type(3),
            new Information(
                $event['payload']['issue']['html_url'],
                $event['payload']['issue']['title'],
                $this->buildMessage($event)
            ),
            new Ring(3000, 80),
            $this->buildSound($event),
        );

    }

    private function buildMessage(array $event): string
    {
        return \sprintf(
            '%s %s an issue in %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 ClavEgg();
        }

        return new Clav(\strlen($event['payload']['issue']['title']) * 1.1);
    }
}


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\Ring;
8		use ekinhbayar\GitAmp\Presentation\Sound\BaseSound;
9		use ekinhbayar\GitAmp\Presentation\Sound\Clav;
10		use ekinhbayar\GitAmp\Presentation\Sound\ClavEgg;
11		use ekinhbayar\GitAmp\Presentation\Type;
12
13		class IssuesEvent 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(3),
25	2	new Information(
26	2	$event['payload']['issue']['html_url'],
27	2	$event['payload']['issue']['title'],
28	2	$this->buildMessage($event)
29		),
30	2	new Ring(3000, 80),
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 an issue in %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 ClavEgg();
49		}
50
51	1	return new Clav(\strlen($event['payload']['issue']['title']) * 1.1);
52		}
53		}
54

ekinhbayar / gitamp

Pull Request — master (#46)

IssuesEvent::buildSound() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like