Doctrine\ORM\Mapping\TransientMetadata

Complexity

Total Complexity

10

Size/Duplication

Total Lines	82
Duplicated Lines	0 %

Test Coverage

Coverage

88.46%

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	getName()	0	3	1
A	setDeclaringClass()	0	3	1
A	setReflectionProperty()	0	3	1
A	getValue()	0	3	1
A	__construct()	0	3	1
A	getDeclaringClass()	0	3	1
A	setValue()	0	3	1
A	wakeupReflection()	0	4	1
A	isPrimaryKey()	0	3	1
A	getValueGenerationExecutor()	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;

class TransientMetadata implements Property
{
    /** @var ClassMetadata */
    protected $declaringClass;

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

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

    public function __construct(string $name)
    {
        $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 isPrimaryKey() : bool
    {
        return false;
    }

    /**
     * {@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\TransientMetadata::setReflectionProperty() does only seem to accept ReflectionProperty, maybe add an additional type check?

        );
    }

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