bytic / models-smart-properties

src/Definitions/Definition/HasPlaces.php

<?php

namespace ByTIC\Models\SmartProperties\Definitions\Definition;

/**
 * Trait HasPlaces
 * @package ByTIC\Models\SmartProperties\Definitions\Traits
 */
trait HasPlaces
{
    protected $places = [];

    /**
     * @return array
     */
    public function getPlaces(): array
    {
        $this->checkBuild();

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

        return $this->places;
    }

    /**
     * @param array $places
     */
    public function setPlaces(array $places): void
    {
        $this->places = $places;
    }

    public function addPlace()
    {
        $this->places = array_unique(array_merge($this->places, func_get_args()));
    }

    /**
     * @return array
     * @deprecated use getPlaces
     */
    public function getItemsNames()
    {
        return $this->getPlaces();
    }
}