ByTIC\MediaLibrary\Models\MediaRecords\MediaRecord

Complexity

Total Complexity

4

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	3
Bugs	1	Features	2

3 Methods

Rating	Name	Duplication	Size	Complexity
A	populateFromModel()	0	4	1
A	populateFromFile()	0	4	1
A	populateFromCollection()	0	8	2

<?php

namespace ByTIC\MediaLibrary\Models\MediaRecords;

use ByTIC\MediaLibrary\Collections\Collection;
use Nip\Filesystem\File;
use Nip\Records\Record;

/**
 * Class MediaRecord
 * @package ByTIC\MediaLibrary\Models\MediaRecords
 *
 * @property string $model
 * @property int $model_id
 * @property int $collection_name
 * @property int $file_name
 * @property int $path
 * @property int $disk
 */
class MediaRecord extends Record
{
    /**
     * @param \Nip\Records\AbstractModels\Record $model
     */
    public function populateFromFile(File $file)
    {
        $this->file_name = $file->getName();

The property $file_name was declared of type integer, but $file->getName() is of type string. Maybe add a type cast?

        $this->path = $file->getPath();

The property $path was declared of type integer, but $file->getPath() is of type string. Maybe add a type cast?

    }

    /**
     * @param \Nip\Records\AbstractModels\Record $model
     */
    public function populateFromModel(Record $model)
    {
        $this->model = $model->getManager()->getMorphName();

It seems like $model->getManager()->getMorphName() of type Nip\Records\AbstractModels\RecordManager or Nip\Records\Collections\Collection or Nip\Records\Traits\Relations\HasRelationsRecordsTrait or true is incompatible with the declared type string of property $model.

It seems like getMorphName() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $this->model_id = $model->getPrimaryKey();
    }

    /**
     * @param Collection $collection
     */
    public function populateFromCollection($collection)
    {
        if (is_object($collection)) {
            $this->collection_name = $collection->getName();

The property $collection_name was declared of type integer, but $collection->getName() is of type string. Maybe add a type cast?

            $this->disk = $collection->getMediaFilesystemDiskName();

The property $disk was declared of type integer, but $collection->getMediaFilesystemDiskName() is of type string. Maybe add a type cast?

            return;
        }
        $this->collection_name = $collection;
    }
}