EdmondsCommerce\DoctrineStaticMeta\Entity\Embeddable\Traits\Geo\HasAddressEmbeddableTrait

Complexity

Total Complexity

6

Size/Duplication

Total Lines	68
Duplicated Lines	0 %

Test Coverage

Coverage

88%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	initAddressEmbeddable()	0	3	1
A	metaForAddress()	0	14	1
A	postLoadSetOwningEntityOnAddressEmbeddable()	0	3	1
A	getAddressEmbeddable()	0	3	1
A	setAddressEmbeddable()	0	13	2

<?php declare(strict_types=1);

namespace EdmondsCommerce\DoctrineStaticMeta\Entity\Embeddable\Traits\Geo;

use Doctrine\ORM\Events;
use Doctrine\ORM\Mapping\Builder\ClassMetadataBuilder;
use EdmondsCommerce\DoctrineStaticMeta\Entity\Embeddable\Interfaces\Geo\HasAddressEmbeddableInterface;
use EdmondsCommerce\DoctrineStaticMeta\Entity\Embeddable\Interfaces\Objects\Geo\AddressEmbeddableInterface;
use EdmondsCommerce\DoctrineStaticMeta\Entity\Embeddable\Objects\Geo\AddressEmbeddable;

trait HasAddressEmbeddableTrait
{
    /**
     * @var AddressEmbeddableInterface
     */
    private $addressEmbeddable;

    /**
     * @param ClassMetadataBuilder $builder
     */
    protected static function metaForAddress(ClassMetadataBuilder $builder): void
    {
        $builder->addLifecycleEvent(
            'postLoadSetOwningEntityOnAddressEmbeddable',
            Events::postLoad
        );
        $builder->createEmbedded(
            HasAddressEmbeddableInterface::PROP_ADDRESS_EMBEDDABLE,
            AddressEmbeddable::class
        )
                ->setColumnPrefix(
                    HasAddressEmbeddableInterface::COLUMN_PREFIX_ADDRESS
                )
                ->build();
    }

    /**
     * @return AddressEmbeddableInterface
     */
    public function getAddressEmbeddable(): AddressEmbeddableInterface
    {
        return $this->addressEmbeddable;
    }

    /**
     * @param AddressEmbeddableInterface $address
     *
     * @param bool                       $notify
     *
     * @return $this
     * @SuppressWarnings(PHPMD.BooleanArgumentFlag)
     */
    public function setAddressEmbeddable(
        AddressEmbeddableInterface $address,
        bool $notify = true
    ): self {
        $this->addressEmbeddable = $address;
        $this->addressEmbeddable->setOwningEntity($this);

$this of type EdmondsCommerce\DoctrineStaticMeta\Entity\Embeddable\Traits\Geo\HasAddressEmbeddableTrait is incompatible with the type EdmondsCommerce\DoctrineStaticMeta\Entity\Interfaces\ImplementNotifyChangeTrackingPolicyInterface expected by parameter $entity of EdmondsCommerce\DoctrineStaticMeta\Entity\Embeddable\Interfaces\Objects\AbstractEmbeddableObjectInterface::setOwningEntity().

        if (true === $notify) {
            $this->notifyEmbeddablePrefixedProperties(

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

                HasAddressEmbeddableInterface::PROP_ADDRESS_EMBEDDABLE
            );
        }

        return $this;
    }

    public function postLoadSetOwningEntityOnAddressEmbeddable(): void
    {
        $this->addressEmbeddable->setOwningEntity($this);

$this of type EdmondsCommerce\DoctrineStaticMeta\Entity\Embeddable\Traits\Geo\HasAddressEmbeddableTrait is incompatible with the type EdmondsCommerce\DoctrineStaticMeta\Entity\Interfaces\ImplementNotifyChangeTrackingPolicyInterface expected by parameter $entity of EdmondsCommerce\DoctrineStaticMeta\Entity\Embeddable\Interfaces\Objects\AbstractEmbeddableObjectInterface::setOwningEntity().

    }

    /**
     * Called at construction time
     */
    private function initAddressEmbeddable(): void
    {
        $this->setAddressEmbeddable(new AddressEmbeddable(), false);
    }
}