ByTIC\MediaLibrary\Models\MediaProperties\MediaProperty

Complexity

Total Complexity

20

Size/Duplication

Total Lines	134
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	0	Features	1

12 Methods

Rating	Name	Duplication	Size	Complexity
A	setDefaultMedia()	0	4	1
A	dbLoaded()	0	8	2
A	setCustomPropertiesAttribute()	0	9	2
A	getCustomProperty()	0	4	2
A	initCustomProperties()	0	5	2
A	populateFromCollection()	0	4	2
A	setCustomPropery()	0	5	1
A	populateFromModel()	0	4	1
A	getCustomProperties()	0	4	1
A	defaultMedia()	0	13	3
A	saveDbLoaded()	0	4	1
A	checkCustomProperties()	0	4	2

<?php

namespace ByTIC\MediaLibrary\Models\MediaProperties;

use ByTIC\MediaLibrary\Collections\Collection;
use ByTIC\MediaLibrary\HasMedia\HasMediaTrait;
use Nip\Records\Record;

/**
 * Class MediaProperty
 * @package ByTIC\MediaLibrary\Models\MediaRecords
 *
 * @property string $model
 * @property int $model_id
 * @property string $collection_name
 * @property string $custom_properties
 *
 * @method HasMediaTrait getModel
 */
class MediaProperty extends Record
{
    /**
     * @var null|array
     */
    protected $customPropertiesArray = null;

    /**
     * @param 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|string $collection
     */
    public function populateFromCollection($collection)
    {
        $name = is_object($collection) ? $collection->getName() : $collection;
        $this->collection_name = $name;
    }

    /**
     * @return array
     */
    public function getCustomProperties()
    {
        $this->checkCustomProperties();
        return $this->customPropertiesArray;
    }

    protected function checkCustomProperties()
    {
        if ($this->customPropertiesArray == null) {
            $this->initCustomProperties();
        }
    }


    public function initCustomProperties()
    {
        $properties = json_decode($this->custom_properties, true);
        $properties = (is_array($properties)) ? $properties : [];
        $this->customPropertiesArray = $properties;
    }

    /**
     * @param $name
     * @param null $default

Are you sure the doc-type for parameter $default is correct as it would always require null to be passed?

     * @return mixed
     */
    public function getCustomProperty($name, $default = null)
    {
        $this->checkCustomProperties();
        return isset($this->customPropertiesArray[$name]) ? $this->customPropertiesArray[$name] : $default;
    }

    /**
     * @param $name
     * @param $value
     */
    public function setCustomPropery($name, $value)
    {
        $this->checkCustomProperties();
        $this->customPropertiesArray[$name] = $value;
        $this->setCustomPropertiesAttribute($this->getCustomProperties());
    }

    /**
     * @param bool $value
     */
    public function saveDbLoaded($value)
    {
        $this->dbLoaded($value);
        $this->save();
    }

    /**
     * @param bool|null $value
     * @return bool
     */
    public function dbLoaded($value = null)
    {
        if ($value == null) {
            return $this->getCustomProperty('dbLoaded', false);
        }
        $value = ($value == true);

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

        $this->setCustomPropery('dbLoaded', $value);
        return $value;
    }

    /**
     * @param null $value

Are you sure the doc-type for parameter $value is correct as it would always require null to be passed?

     * @return mixed|null
     */
    public function defaultMedia($value = null)
    {
        if ($value == null) {

The condition $value == null is always true.

            return $this->getCustomProperty('defaultMedia', $this->getModel()->default_image);
        }
        $this->setCustomPropery('defaultMedia', $value);
        $this->save();

        if ($this->collection_name == 'images') {
            $this->getModel()->default_image = $value;
            $this->getModel()->update();
        }
        return $value;
    }

    /**
     * @param $value
     */
    public function setDefaultMedia($value)
    {
        $this->dbLoaded($value);
        $this->save();
    }

    /**
     * @param $value
     */
    public function setCustomPropertiesAttribute($value)
    {
        if (is_string($value)) {
            $this->customPropertiesArray = null;
        } else {
            $this->customPropertiesArray = $value;
            $value = json_encode($value);
        }
        $this->setDataValue('custom_properties', $value);
    }
}