HnrAzevedo\Datamanager\MagicsTrait::__set()

Complexity

Conditions	6
Paths	9

Size

Total Lines	27
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	1

<?php
    public function __set(string $prop,$value)
    {
        if(is_array($value)){
            $attr = array_keys($value)[0];
            $this->data[$prop][$attr] = $value[$attr];

The property data does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            return $this;
        }

        if($this->full){

The property full does not exist on HnrAzevedo\Datamanager\MagicsTrait. Since you implemented __get, consider adding a @property annotation.

            switch($this->data[$prop]['type']){
                case 'date':
                    $value = (date_format( date_create_from_format(DATAMANAGER_CONFIG['dateformat'],$value) , 'Y-m-d'));
                    break;
                case 'datetime':
                    $value = (date_format( date_create_from_format(DATAMANAGER_CONFIG['datetimeformat'],$value) , 'Y-m-d H:i:s'));
                    break;
            }
        }

        $this->isSettable($prop);

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

        $this->data[$prop]['changed'] = ($prop === $this->primary) ? false : true;

The property primary does not exist on HnrAzevedo\Datamanager\MagicsTrait. Since you implemented __get, consider adding a @property annotation.

        $this->data[$prop]['value'] = $value;
        
        $this->$prop = $value;
        
        return $this;
    }
}