ByTIC\Models\SmartProperties\Definitions\Definition\HasName

Complexity

Total Complexity

5

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setName()	0	3	1
A	getName()	0	7	2
A	initName()	0	8	2

<?php

namespace ByTIC\Models\SmartProperties\Definitions\Definition;

/**
 * Trait HasName
 * @package ByTIC\Models\SmartProperties\Definitions\Traits
 */
trait HasName
{

    /**
     * @var string
     */
    protected $name = null;


    /**
     * @return mixed
     */
    public function getName(): ?string
    {
        if ($this->name === null) {
            $this->initName();
        }

        return $this->name;
    }

    /**
     * @param mixed $name
     */
    public function setName($name)
    {
        $this->name = $name;
    }

    protected function initName()
    {
        $name = $this->getField();

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

        if (empty($name)) {
            return;
        }
        $name = inflector()->classify($this->getField());
        $this->setName($name);
    }

}