AbterPhp\Files\Orm\FileDownloadRepo

Complexity

Total Complexity

3

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getByUser()	0	4	1
A	getByFile()	0	4	1
A	getPage()	0	4	1

<?php

declare(strict_types=1);

namespace AbterPhp\Files\Orm;

use AbterPhp\Admin\Domain\Entities\User;
use AbterPhp\Files\Domain\Entities\File;
use AbterPhp\Files\Domain\Entities\FileDownload as Entity;
use AbterPhp\Files\Orm\DataMappers\FileDownloadSqlDataMapper; // @phan-suppress-current-line PhanUnreferencedUseNormal
use AbterPhp\Framework\Orm\IGridRepo;
use Opulence\Orm\Repositories\Repository;

class FileDownloadRepo extends Repository implements IGridRepo
{
    /**
     * @param int      $limitFrom
     * @param int      $pageSize
     * @param string[] $orders
     * @param array    $conditions
     * @param array    $params
     *
     * @return Entity[]
     * @throws \Opulence\Orm\OrmException
     */
    public function getPage(int $limitFrom, int $pageSize, array $orders, array $conditions, array $params): array
    {
        /** @see FileDownloadSqlDataMapper::getPage() */
        return $this->getFromDataMapper('getPage', [$limitFrom, $pageSize, $orders, $conditions, $params]);

The expression return $this->getFromDataMapper('getPage', array($limitFrom, $pageSize, $orders, $conditions, $params)) could return the type object which is incompatible with the type-hinted return array. Consider adding an additional type-check to rule them out.

    }

    /**
     * @param File $file
     *
     * @return Entity[]
     * @throws \Opulence\Orm\OrmException
     */
    public function getByFile(File $file): array
    {
        /** @see FileDownloadSqlDataMapper::getByFileId() */
        return $this->getFromDataMapper('getByFileId', [$file->getId()]);

The expression return $this->getFromDataMapper('getByFileId', array($file->getId())) could return the type object which is incompatible with the type-hinted return array. Consider adding an additional type-check to rule them out.

    }

    /**
     * @param User $user
     *
     * @return Entity[]
     * @throws \Opulence\Orm\OrmException
     */
    public function getByUser(User $user): array
    {
        /** @see FileDownloadSqlDataMapper::getByUserId() */
        return $this->getFromDataMapper('getByUserId', [$user->getId()]);

The expression return $this->getFromDataMapper('getByUserId', array($user->getId())) could return the type object which is incompatible with the type-hinted return array. Consider adding an additional type-check to rule them out.

    }
}