HnrAzevedo\Datamanager\MagicsTrait

Complexity

Total Complexity

7

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__get()	0	11	3
A	__set()	0	22	4

<?php

namespace HnrAzevedo\Datamanager;

trait MagicsTrait{

    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;
            }
        }

        $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'] = true;
        $this->data[$prop]['value'] = $value;
        
        return $this;
    }

    public function __get(string $field)
    {
        $this->isSettable($field);

        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[$field]['type']){
                case 'date': $this->data[$field]['value'] = (date_format( date_create_from_format('Y-m-d' , $this->data[$field]['value'] ) , DATAMANAGER_CONFIG['dateformat']));

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

            }
        }

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