Application\Repository\ArtistRepository

Complexity

Total Complexity

4

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Test Coverage

Coverage

94.74%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	getOrCreateByNames()	0	35	4

<?php

declare(strict_types=1);

namespace Application\Repository;

use Application\Enum\Site;
use Application\Model\Artist;

/**
 * @extends AbstractRepository<Artist>
 */
class ArtistRepository extends AbstractRepository
{
    /**
     * Get or create artists by their given names.
     *
     * @param string[] $names
     *
     * @return Artist[]
     */
    public function getOrCreateByNames(array $names, Site $site): array
    {
        if (!$names) {

The expression $names of type string[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            return [];
        }

        // Dedup and trim whitespaces
        $names = array_unique(array_map(fn ($value) => trim($value), $names));

        $artists = $this->findBy([
            'name' => $names,
            'site' => $site->value,
        ]);

        $found = [];
        foreach ($artists as $artist) {
            // Names are not trimmed when saved from thesaurus interface, so we
            // have names with trailing whitespace in the database.
            // With the collation we use, MariaDB trims whitespaces when doing
            // comparison (affecting UNIQUE constraint and SELECT queries).
            // So we must be sure to handle whitespaces in the same way as
            // MariaDB when doing comparisons.
            $found[] = trim($artist->getName());
        }

        $notFound = array_diff($names, $found);
        foreach ($notFound as $name) {
            $artist = new Artist();
            $artist->setSite($site);
            $this->getEntityManager()->persist($artist);
            $artist->setName($name);
            $artists[] = $artist;
        }

        return $artists;
    }
}