Application\Repository\CardRepository

Complexity

Total Complexity

11

Size/Duplication

Total Lines	127
Duplicated Lines	0 %

Test Coverage

Coverage

73.52%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getOneByLegacyId()	0	4	1
A	getNextCodeAvailable()	0	19	2
A	getFilenamesForDimensionUpdate()	0	17	2
A	getAccessibleSubQuery()	0	28	5
A	getExportCards()	0	23	1

<?php

declare(strict_types=1);

namespace Application\Repository;

use Application\Enum\CardVisibility;
use Application\Model\Card;
use Application\Model\Collection;
use Application\Model\Export;
use Application\Model\User;
use Doctrine\ORM\Query\Expr\Join;
use Ecodev\Felix\Repository\LimitedAccessSubQuery;
use Ecodev\Felix\Utility;

/**
 * @extends AbstractRepository<Card>
 */
class CardRepository extends AbstractRepository implements LimitedAccessSubQuery
{
    /**
     * Returns pure SQL to get ID of all cards that are accessible to given user.
     * A card is accessible if:
     * - card is public
     * - card is member and user is logged in
     * - card owner or creator is the user
     * - card's collection responsible is the user.
     */
    public function getAccessibleSubQuery(?\Ecodev\Felix\Model\User $user): string
    {
        if ($user && $user->getRole() === User::ROLE_ADMINISTRATOR) {
            return '';
        }

        $visibility = [CardVisibility::Public->value];
        if ($user) {
            $visibility[] = CardVisibility::Member->value;
        }

        $qb = $this->getEntityManager()
            ->getConnection()
            ->createQueryBuilder()
            ->select('card.id')
            ->from('card')
            ->where('card.visibility IN (' . Utility::quoteArray($visibility) . ')');

        if ($user) {
            $userId = $user->getId() ?? -1;
            $qb->leftJoin('card', 'card_collection', 'card_collection', 'card_collection.card_id = card.id')
                ->leftJoin('card_collection', 'collection_user', 'collection_user', 'card_collection.collection_id = collection_user.collection_id')
                ->orWhere('card.owner_id = ' . $userId)
                ->orWhere('card.creator_id = ' . $userId)
                ->orWhere('collection_user.user_id = ' . $userId);
        }

        return $qb->getSQL();
    }

    /**
     * Returns all filename in DB and their id and sizes.
     *
     * @return string[][]
     */
    public function getFilenamesForDimensionUpdate(?string $site = null): array
    {
        $filenames = $this->getEntityManager()->getConnection()->createQueryBuilder()
            ->from('card')
            ->addSelect('id')
            ->addSelect('width')
            ->addSelect('height')
            ->addSelect('CONCAT("data/images/", filename) AS filename')
            ->where('filename != ""')
            ->orderBy('filename');

        if ($site) {
            $filenames
                ->where('site = "' . $site . '"');
        }

        return $filenames->fetchAllAssociative();
    }

    /**
     * Return the next available code.
     */
    public function getNextCodeAvailable(Collection $collection): string
    {
        static $nextId = null;

        if (!$nextId) {
            $connection = $this->getEntityManager()->getConnection();
            $database = $connection->quote($connection->getDatabase());

It seems like $connection->getDatabase() can also be of type null; however, parameter $value of Doctrine\DBAL\Connection::quote() does only seem to accept string, maybe add an additional type check?

            $sql = "SELECT `AUTO_INCREMENT`
                FROM INFORMATION_SCHEMA.TABLES
                WHERE TABLE_SCHEMA = $database
                AND TABLE_NAME = 'card'";

            $nextId = (int) $connection->fetchOne($sql);
        } else {
            ++$nextId;
        }

        return $collection->getName() . '-' . $nextId;
    }

    /**
     * Get a card from it's legacy id.
     */
    public function getOneByLegacyId(int $legacy_id): ?Card
    {
        return $this->getAclFilter()->runWithoutAcl(fn () => $this->findOneBy([
            'legacyId' => $legacy_id,
        ]));
    }

    /**
     * Returns **some** cards for the given export, starting at $firstResult.
     *
     * This method has to be called repeatedly with a different $firstResult in order
     * to iterate over **all** cards of a given export
     */
    public function getExportCards(Export $export, int $lastCard): array
    {
        $cardIds = $this->getEntityManager()->getConnection()->fetchFirstColumn(
            'SELECT card_id FROM export_card WHERE export_id = :export AND card_id > :lastCard ORDER BY card_id LIMIT 250',
            [
                'export' => $export->getId(),
                'lastCard' => $lastCard,
            ],
        );

        $qb = $this->createQueryBuilder('card');
        $qb->select('card, artist, country, documentType, domain, institution, period')
            ->leftJoin('card.artists', 'artist', Join::WITH)
            ->leftJoin('card.country', 'country', Join::WITH)
            ->leftJoin('card.documentType', 'documentType', Join::WITH)
            ->leftJoin('card.domains', 'domain', Join::WITH)
            ->leftJoin('card.institution', 'institution', Join::WITH)
            ->leftJoin('card.periods', 'period', Join::WITH)
            ->andWhere('card.id IN (:cards)')
            ->setParameter('cards', $cardIds)
            ->orderBy('card.id');

        return $this->getAclFilter()->runWithoutAcl(fn () => $qb->getQuery()->getResult());
    }
}