ByTIC\DocumentGenerator\Tests\Fixtures\Models\PdfLetters\PdfLetter

Complexity

Total Complexity

6

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getItemsManager()	0	2	1
A	getMediaFilesystemDisk()	0	10	2
A	getIssueDate()	0	3	1
A	registerMediaCollections()	0	3	1
A	getModelExample()	0	2	1

<?php

namespace ByTIC\DocumentGenerator\Tests\Fixtures\Models\PdfLetters;

use ByTIC\DocumentGenerator\PdfLetters\Models\PdfLetters\PdfLetterTrait;
use ByTIC\MediaLibrary\Collections\Collection;
use ByTIC\MediaLibrary\HasMedia\Interfaces\HasMedia;
use ByTIC\MediaLibrary\Loaders\Filesystem;
use ByTIC\MediaLibrary\MediaRepository\MediaRepository;
use DateTime;
use Nip\Filesystem\FileDisk;
use League\Flysystem\Adapter\Local as LocalAdapter;
use Nip\Records\Record;
use Nip\Records\Traits\AbstractTrait\RecordTrait as AbstractRecordTrait;

/**
 * Class PdfLetter
 * @package ByTIC\DocumentGenerator\Tests\Fixtures\Models\PdfLetters
 */
class PdfLetter extends Record implements HasMedia
{
    use PdfLetterTrait;

The trait ByTIC\DocumentGenerator\PdfLetters\Models\PdfLetters\PdfLetterTrait requires the property $violations which is not provided by ByTIC\DocumentGenerator\Tests\Fixtures\Models\PdfLetters\PdfLetter.

    /**
     * @return FileDisk
     */
    public function getMediaFilesystemDisk()
    {
        if (!isset($this->disk)) {
            $this->disk = new FileDisk(

The property disk does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

                new LocalAdapter(
                    TEST_FIXTURE_PATH
                )
            );
        }
        return $this->disk;
    }

    public function getItemsManager()
    {
    }

    /**
     * @param MediaRepository $mediaCollection
     */
    public function registerMediaCollections($mediaRepository)
    {
        $mediaRepository->getCollection('files')->setLoaderClass(Filesystem::class);
    }

    /**
     * @return AbstractRecordTrait
     */
    public function getModelExample()
    {
    }

    public function getIssueDate(): DateTime
    {
        return $this->issueDate;

The expression return $this->issueDate could return the type null which is incompatible with the type-hinted return DateTime. Consider adding an additional type-check to rule them out.

The property issueDate does not exist on ByTIC\DocumentGenerator\Tests\Fixtures\Models\PdfLetters\PdfLetter. Since you implemented __get, consider adding a @property annotation.

    }
}