ArtistHandler::handle() - Code Metrics - Inspection of "Perform automatic refactoring to php 8.1" - uxmp/core - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — main ( 1d1c5b...83bc52 )

by Daniel

created 2022-05-01 11:51 UTC

ArtistHandler::handle() A

↳ Parent: Project

Complexity

Conditions	5
Paths	5

Size

Total Lines	8
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	5

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	5
eloc	6
c	1
b	0
f	0
nc	5
nop	2
dl	0
loc	8
ccs	7
cts	7
cp	1
crap	5
rs	9.6111

<?php

declare(strict_types=1);

namespace Uxmp\Core\Component\Playlist\MediaAddition\Handler;

use Uxmp\Core\Orm\Repository\ArtistRepositoryInterface;

/**
 * Adds songs of an artist to the list of songs
 */
final class ArtistHandler implements HandlerInterface
{
    public function __construct(
        private readonly ArtistRepositoryInterface $artistRepository,

    ) {
    }

    /**
     * Adds all eligible song ids of the artists' album to the songList array
     *
     * @param array<int> $songList
     */
    public function handle(int $mediaId, array &$songList): void
    {
        $artist = $this->artistRepository->find($mediaId);
        if ($artist !== null) {
            foreach ($artist->getAlbums() as $album) {
                foreach ($album->getDiscs() as $disc) {
                    foreach ($disc->getSongs() as $song) {
                        $songList[] = $song->getId();
                    }
                }
            }
        }
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Uxmp\Core\Component\Playlist\MediaAddition\Handler;
6
7		use Uxmp\Core\Orm\Repository\ArtistRepositoryInterface;
8
9		/**
10		* Adds songs of an artist to the list of songs
11		*/
12		final class ArtistHandler implements HandlerInterface
13		{
14	1	public function __construct(
15		private readonly ArtistRepositoryInterface $artistRepository,
		0 ignored issues – show Bug introduced 2022-05-01 11:55 UTC by Report Bug Copy Issue Report A parse error occurred: Syntax error, unexpected T_STRING, expecting T_VARIABLE on line 15 at column 25 Loading history...
16		) {
17		}
18
19		/**
20		* Adds all eligible song ids of the artists' album to the songList array
21		*
22		* @param array<int> $songList
23		*/
24	1	public function handle(int $mediaId, array &$songList): void
25		{
26	1	$artist = $this->artistRepository->find($mediaId);
27	1	if ($artist !== null) {
28	1	foreach ($artist->getAlbums() as $album) {
29	1	foreach ($album->getDiscs() as $disc) {
30	1	foreach ($disc->getSongs() as $song) {
31	1	$songList[] = $song->getId();
32		}
33		}
34		}
35		}
36		}
37		}
38

uxmp / core

Push — main ( 1d1c5b...83bc52 )

ArtistHandler::handle() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like