Application\Model\File

Complexity

Total Complexity

6

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	setCard()	0	3	1
A	getCard()	0	3	1
A	getAcceptedMimeTypes()	0	12	1
A	setFile()	0	12	3

<?php

declare(strict_types=1);

namespace Application\Model;

use Application\Api\FileException;
use Application\Repository\FileRepository;
use Doctrine\ORM\Mapping as ORM;
use Ecodev\Felix\Model\Traits\HasName;
use GraphQL\Doctrine\Attribute as API;
use Psr\Http\Message\UploadedFileInterface;
use Throwable;

/**
 * An uploaded file that is related to a card (pdf/docx/xlsx/etc.).
 */
#[ORM\UniqueConstraint(name: 'unique_name', columns: ['filename'])]
#[ORM\HasLifecycleCallbacks]
#[ORM\Entity(FileRepository::class)]
class File extends AbstractModel implements \Ecodev\Felix\Model\File
{
    #[ORM\JoinColumn(nullable: false, onDelete: 'CASCADE')]
    #[ORM\ManyToOne(targetEntity: Card::class)]
    private ?Card $card = null;

    use \Ecodev\Felix\Model\Traits\File {
        setFile as traitSetFile;
    }
    use HasName;

    public function setCard(Card $card): void
    {
        $this->card = $card;
    }

    public function getCard(): Card
    {
        return $this->card;

The expression return $this->card could return the type null which is incompatible with the type-hinted return Application\Model\Card. Consider adding an additional type-check to rule them out.

    }

    /**
     * Set the file.
     */
    #[API\Input(type: '?GraphQL\Upload\UploadType')]
    public function setFile(UploadedFileInterface $file): void
    {
        try {
            $this->traitSetFile($file);

            // Default to filename as name
            if (!$this->getName()) {
                $this->setName($file->getClientFilename());

It seems like $file->getClientFilename() can also be of type null; however, parameter $name of Application\Model\File::setName() does only seem to accept string, maybe add an additional type check?

            }
        } catch (Throwable $e) {
            throw new FileException($file, $e);
        }
    }

    protected function getAcceptedMimeTypes(): array
    {
        // This list should be kept in sync with FilesComponent.accept
        return [
            'application/msword',
            'application/pdf',
            'application/x-pdf',
            'application/vnd.ms-excel',
            'application/vnd.oasis.opendocument.spreadsheet',
            'application/vnd.oasis.opendocument.text',
            'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet',
            'application/vnd.openxmlformats-officedocument.wordprocessingml.document',
        ];
    }
}