Doctrine\ORM\Mapping\EmbeddedClassMetadata

Complexity

Total Complexity

12

Size/Duplication

Total Lines	97
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

12 Methods

Rating	Name	Duplication	Size	Complexity
A	getValue()	0	3	1
A	getValueGenerationExecutor()	0	3	1
A	wakeupReflection()	0	4	1
A	__construct()	0	5	1
A	setName()	0	3	1
A	isPrimaryKey()	0	3	1
A	setPrimaryKey()	0	3	1
A	setValue()	0	3	1
A	setReflectionProperty()	0	3	1
A	setDeclaringClass()	0	3	1
A	getDeclaringClass()	0	3	1
A	getName()	0	3	1

<?php

declare(strict_types=1);

namespace Doctrine\ORM\Mapping;

use Doctrine\DBAL\Platforms\AbstractPlatform;
use Doctrine\ORM\Reflection\ReflectionService;
use Doctrine\ORM\Sequencing\Planning\ValueGenerationExecutor;
use ReflectionProperty;

/**
 * @property MappedSuperClassMetadata $parent
 */
class EmbeddedClassMetadata extends ComponentMetadata implements Property
{
    /** @var ClassMetadata */
    private $declaringClass;

    /** @var ReflectionProperty */
    private $reflection;

    /** @var string */
    private $name;

    /** @var bool */
    protected $primaryKey = false;

    public function __construct(string $name, string $className, ?MappedSuperClassMetadata $parent = null)
    {
        parent::__construct($className, $parent);

$parent of type Doctrine\ORM\Mapping\MappedSuperClassMetadata|null is incompatible with the type Doctrine\ORM\Mapping\ClassMetadataBuildingContext expected by parameter $metadataBuildingContext of Doctrine\ORM\Mapping\ComponentMetadata::__construct().

        $this->name = $name;
    }

    /**
     * {@inheritdoc}
     */
    public function getDeclaringClass() : ComponentMetadata
    {
        return $this->declaringClass;
    }

    public function setDeclaringClass(ComponentMetadata $declaringClass) : void
    {
        $this->declaringClass = $declaringClass;

$declaringClass is of type Doctrine\ORM\Mapping\ComponentMetadata, but the property $declaringClass was declared to be of type Doctrine\ORM\Mapping\ClassMetadata. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

    }

    /**
     * {@inheritdoc}
     */
    public function getName() : string
    {
        return $this->name;
    }

    /**
     * {@inheritdoc}
     */
    public function setName($name) : void
    {
        $this->name = $name;
    }

    public function setPrimaryKey(bool $isPrimaryKey) : void
    {
        $this->primaryKey = $isPrimaryKey;
    }

    public function isPrimaryKey() : bool
    {
        return $this->primaryKey;
    }

    /**
     * {@inheritdoc}
     */
    public function setValue($object, $value) : void
    {
        $this->reflection->setValue($object, $value);
    }

    /**
     * {@inheritdoc}
     */
    public function getValue($object)
    {
        return $this->reflection->getValue($object);
    }

    /**
     * {@inheritdoc}
     */
    public function setReflectionProperty(ReflectionProperty $reflectionProperty) : void
    {
        $this->reflection = $reflectionProperty;
    }

    /**
     * {@inheritdoc}
     */
    public function wakeupReflection(ReflectionService $reflectionService) : void
    {
        $this->setReflectionProperty(
            $reflectionService->getAccessibleProperty($this->declaringClass->getClassName(), $this->name)

It seems like $reflectionService->getAccessibleProperty($this->declaringClass->getClassName(), $this->name) can also be of type null; however, parameter $reflectionProperty of Doctrine\ORM\Mapping\EmbeddedClassMetadata::setReflectionProperty() does only seem to accept ReflectionProperty, maybe add an additional type check?

        );
    }

    public function getValueGenerationExecutor(AbstractPlatform $platform) : ?ValueGenerationExecutor
    {
        return null;
    }
}