edmondscommerce / doctrine-static-meta

src/Entity/Fields/Traits/Boolean/DefaultsEnabledFieldTrait.php

<?php declare(strict_types=1);

namespace EdmondsCommerce\DoctrineStaticMeta\Entity\Fields\Traits\Boolean;

use Doctrine\ORM\Mapping\Builder\ClassMetadataBuilder;
use EdmondsCommerce\DoctrineStaticMeta\Entity\Fields\Interfaces\Boolean\DefaultsEnabledFieldInterface;
use EdmondsCommerce\DoctrineStaticMeta\MappingHelper;

trait DefaultsEnabledFieldTrait
{

    /**
     * @var bool
     */
    private $defaultsEnabled = DefaultsEnabledFieldInterface::DEFAULT_DEFAULTS_ENABLED;

    /**
     * @SuppressWarnings(PHPMD.StaticAccess)
     * @param ClassMetadataBuilder $builder
     */
    public static function metaForDefaultsEnabled(ClassMetadataBuilder $builder): void
    {
        MappingHelper::setSimpleBooleanFields(
            [DefaultsEnabledFieldInterface::PROP_DEFAULTS_ENABLED],
            $builder,
            DefaultsEnabledFieldInterface::DEFAULT_DEFAULTS_ENABLED
        );
    }

    /**
     * @return bool
     */
    public function isDefaultsEnabled(): bool
    {
        if (null === $this->defaultsEnabled) {
            return DefaultsEnabledFieldInterface::DEFAULT_DEFAULTS_ENABLED;
        }

        return $this->defaultsEnabled;
    }

    /**
     * @param bool|null $defaultsEnabled
     *
     * @return self
     */
    private function setDefaultsEnabled(bool $defaultsEnabled): self
    {
        $this->updatePropertyValue(

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

            DefaultsEnabledFieldInterface::PROP_DEFAULTS_ENABLED,
            $defaultsEnabled
        );

        return $this;
    }

    private function initDefaultsEnabled(): void
    {
        $this->defaultsEnabled = DefaultsEnabledFieldInterface::DEFAULT_DEFAULTS_ENABLED;
    }
}