ByTIC\Models\SmartProperties\RecordsTraits\HasSmartProperties\RecordTrait

Complexity

Total Complexity

12

Size/Duplication

Total Lines	106
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	1	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getNewSmartPropertyFromValue()	0	6	1
A	setSmartProperty()	0	19	4
A	getNewSmartPropertyFromDefinition()	0	6	1
A	updateSmartProperty()	0	10	2
A	getSmartPropertyValueFromDefinition()	0	10	2
A	getSmartProperty()	0	8	2

<?php

namespace ByTIC\Models\SmartProperties\RecordsTraits\HasSmartProperties;

use ByTIC\Models\SmartProperties\Properties\AbstractProperty\Generic as PropertyValue;
use ByTIC\Models\SmartProperties\Properties\Definitions\Definition;
use ByTIC\Models\SmartProperties\Properties\PropertiesRegistry;
use Nip\Records\RecordManager;

/**
 * Class RecordTrait
 * @package ByTIC\Models\SmartProperties\RecordsTraits\HasStatus
 *
 * @property string $status
 * @method RecordManager|RecordsTrait getManager()
 *
 */
trait RecordTrait
{
    use \ByTIC\Models\SmartProperties\RecordsTraits\AbstractTrait\RecordTrait;

    /**
     * @param $name
     * @return PropertyValue
     */
    public function getSmartProperty($name)
    {
        if ($this->getManager()->hasSmartPropertyDefinition($name) === false) {
            return null;
        }
        $definition = $this->getManager()->getSmartPropertyDefinition($name);
        return PropertiesRegistry::getWithInit($this, $definition, function () use ($definition) {
            return $this->getNewSmartPropertyFromDefinition($definition);
        });
    }

    /**
     * @param $name
     * @param $value
     * @return bool
     * @throws \Exception
     */
    public function updateSmartProperty($name, $value)
    {
        if (empty($value)) {
            return false;
        }

        $newStatus = $this->getNewSmartPropertyFromValue($name, $value);
        $return = $newStatus->update();
        $this->setSmartProperty($name, $newStatus);
        return $return;
    }

    /**
     * @param Definition $definition
     * @return PropertyValue
     * @internal Do not use
     */
    public function getNewSmartPropertyFromDefinition($definition)
    {
        $name = $definition->getName();
        $value = $this->getSmartPropertyValueFromDefinition($definition);

        return $this->getNewSmartPropertyFromValue($name, $value);
    }

    /**
     * @param Definition $definition
     * @return mixed
     * @internal Do not use
     */
    public function getSmartPropertyValueFromDefinition($definition)
    {
        $field = $definition->getField();
        $value = $this->getAttributeFromArray($field);

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

        if (empty($value)) {
            $value = $definition->getDefaultValue();
            $this->setAttribute($field, $value);

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

        }

        return $value;
    }

    /**
     * @param $name
     * @param $value
     * @return PropertyValue
     * @throws \Exception
     * @internal Do not use
     */
    public function getNewSmartPropertyFromValue($name, $value)
    {
        $object = clone $this->getManager()->getSmartPropertyItem($name, $value);
        $object->setItem($this);

The method setItem() does not exist on Nip\Records\Collections\Collection. Did you maybe mean setItems()?

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

        return $object;

The expression return $object also could return the type ByTIC\Models\SmartProperties\RecordsTraits\HasSmartProperties\RecordsTrait|Nip\Records\Collections\Collection|Nip\Records\RecordManager which is incompatible with the documented return type ByTIC\Models\SmartProperties\Properties\AbstractProperty\Generic.

    }

    /**
     * @param string $name
     * @param PropertyValue|string $value
     * @throws \Exception
     */
    protected function setSmartProperty($name, $value)
    {
        if (empty($value)) {
            return;
        }
        $definition = $this->getManager()->getSmartPropertyDefinition($name);
        $field = $definition->getField();

The method getField() does not exist on Nip\Records\Collections\Collection.

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

        if (is_string($value)) {
            $value = $this->getNewSmartPropertyFromValue($name, $value);
        }

        $this->setPropertyValue($field, $value->getName());

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

        
        $currentProperty = $this->getSmartProperty($name);
        if ($currentProperty->getName() === $value->getName()) {
            return;
        }
        PropertiesRegistry::set($this, $definition, $value);
    }
}