ByTIC\DataObjects\Legacy\Behaviors\OldAccessingPatternsTrait

Complexity

Total Complexity

3

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setDataValue()	0	3	1
A	writeData()	0	3	1
A	getAttributeFromArray()	0	3	1

<?php

namespace ByTIC\DataObjects\Legacy\Behaviors;

use ByTIC\DataObjects\Behaviors\PropertyOverloading\PropertyOverloadingTrait;

/**
 * Trait OldAccessingPatternsTrait
 * @package ByTIC\DataObjects\Legacy\Behaviors
 */
trait OldAccessingPatternsTrait
{
    /**
     * @param bool $data
     * @return \ByTIC\DataObjects\BaseDto|PropertyOverloadingTrait
     * @deprecated use fill($data)
     */
    public function writeData($data = false)
    {
        return $this->fill($data);

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

    }

    /**
     * @deprecated use
     * @param $key
     * @param $value
     * @return mixed
     */
    protected function setDataValue($key, $value)
    {
        return $this->setPropertyValue($key, $value);

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

    }

    /**
     * Get an attribute from the $attributes array.
     *
     * @param string $key
     * @return mixed
     */
    protected function getAttributeFromArray(string $key)
    {
        return $this->getPropertyRaw($key);

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

    }
}