Chubbyphp\Deserialization\Denormalizer\DenormalizerObjectMappingRegistry

Complexity

Total Complexity

7

Size/Duplication

Total Lines	61
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	7	7	2
A	addObjectMapping()	4	4	1
B	getObjectMapping()	27	27	4

<?php

declare(strict_types=1);

namespace Chubbyphp\Deserialization\Denormalizer;

use Chubbyphp\Deserialization\DeserializerLogicException;
use Chubbyphp\Deserialization\Doctrine\Denormalizer\DenormalizerObjectMappingRegistry as DoctrineDenormalizerObjectMappingRegistry;
use Chubbyphp\Deserialization\Mapping\DenormalizationObjectMappingInterface;
use Doctrine\Common\Persistence\Proxy;

final class DenormalizerObjectMappingRegistry implements DenormalizerObjectMappingRegistryInterface

This class seems to be duplicated in your project.

{
    /**
     * @var DenormalizationObjectMappingInterface[]
     */
    private $objectMappings;

    /**
     * @param array $objectMappings
     */
    public function __construct(array $objectMappings)
    {
        $this->objectMappings = [];
        foreach ($objectMappings as $objectMapping) {
            $this->addObjectMapping($objectMapping);
        }
    }

    /**
     * @param DenormalizationObjectMappingInterface $objectMapping
     */
    private function addObjectMapping(DenormalizationObjectMappingInterface $objectMapping)
    {
        $this->objectMappings[$objectMapping->getClass()] = $objectMapping;
    }

    /**
     * @param string $class
     *
     * @return DenormalizationObjectMappingInterface
     *
     * @throws DeserializerLogicException
     */
    public function getObjectMapping(string $class): DenormalizationObjectMappingInterface
    {
        $reflectionClass = new \ReflectionClass($class);

        if (interface_exists('Doctrine\Common\Persistence\Proxy')
            && in_array(Proxy::class, $reflectionClass->getInterfaceNames(), true)
        ) {
            $parentClass = $reflectionClass->getParentClass()->name;
            @trigger_error(

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

                sprintf(
                    'Use "%s" instead of "%s" for "%s"',
                    DoctrineDenormalizerObjectMappingRegistry::class,
                    self::class,
                    $parentClass
                ),
                E_USER_DEPRECATED
            );

            $class = $parentClass;
        }

        if (isset($this->objectMappings[$class])) {
            return $this->objectMappings[$class];
        }

        throw DeserializerLogicException::createMissingMapping($class);
    }
}