ByTIC\Models\SmartProperties\RecordsTraits\HasSmartProperties\RecordTrait

Complexity

Total Complexity

16

Size/Duplication

Total Lines	123
Duplicated Lines	0 %

Test Coverage

Coverage

97.92%

Importance

Changes	1
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	checkSmartProperty()	0	4	2
A	getNewSmartPropertyFromDefinition()	0	6	1
A	getSmartPropertyValueFromDefinition()	0	9	2
A	getSmartProperty()	0	5	1
A	getNewSmartPropertyFromValue()	0	6	1
A	initSmartProperty()	0	6	2
A	setSmartProperty()	0	16	5
A	updateSmartProperty()	0	10	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 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;

    /**
     * @var PropertyValue[]
     */
    protected $smartProperties = [];

    /**
     * @param $name
     * @return PropertyValue
     */
    public function getSmartProperty($name)
    {
        $this->checkSmartProperty($name);

        return $this->smartProperties[$name];
    }

    /**
     * @param string $name
     */
    public function checkSmartProperty($name)
    {
        if (!isset($this->smartProperties[$name])) {
            $this->initSmartProperty($name);
        }
    }

    /**
     * @param string $name
     */
    public function initSmartProperty($name)
    {
        if ($this->getManager()->hasSmartPropertyDefinition($name)) {
            $definition = $this->getManager()->getSmartPropertyDefinition($name);
            $property = $this->getNewSmartPropertyFromDefinition($definition);
            $this->setSmartProperty($name, $property);
        }
    }

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

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

    /**
     * @param Definition $definition
     * @return mixed
     */
    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 ($value === null) {
            $value = $definition->getDefaultValue();
        }

        return $value;
    }

    /**
     * @param $name
     * @param $value
     * @return PropertyValue
     * @throws \Exception
     */
    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)
    {
        $definition = $this->getManager()->getSmartPropertyDefinition($name);
        $field = $definition->getField();

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

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

        if ($value instanceof PropertyValue) {
            $this->setDataValue($field, $value->getName());

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

            $this->smartProperties[$name] = $value;
            return;
        }
        if (!empty($value)) {
            $currentProperty = $this->getSmartProperty($name);
            if (is_object($currentProperty) && $currentProperty->getName() === $value) {
                return;
            }
            $this->setDataValue($field, $value);
            $this->smartProperties[$name] = $this->getNewSmartPropertyFromValue($name, $value);
        }
    }

    /**
     * @param $name
     * @param $value
     * @return bool
     * @throws \Exception
     */
    public function updateSmartProperty($name, $value)
    {
        if (!empty($value)) {
            $newStatus = $this->getNewSmartPropertyFromValue($name, $value);
            $this->setSmartProperty($name, $newStatus);
            $return = $newStatus->update();
            return $return;
        }

        return false;
    }
}