Maiorano\ObjectHydrator\Mappings\MethodMapping

Complexity

Total Complexity

4

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	1	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getKey()	0	4	1
A	__construct()	0	4	1
A	setValue()	0	3	1
A	getType()	0	6	1

<?php

namespace Maiorano\ObjectHydrator\Mappings;

use JetBrains\PhpStorm\Pure;
use Maiorano\ObjectHydrator\Attributes\HydrationKey;
use ReflectionException;
use ReflectionMethod;
use ReflectionNamedType;

class MethodMapping implements HydrationMappingInterface
{
    /**
     * @var ReflectionMethod
     */
    private ReflectionMethod $reflector;
    /**
     * @var HydrationKey
     */
    private HydrationKey $key;

    /**
     * @param ReflectionMethod $reflector
     * @param HydrationKey     $key
     */
    public function __construct(ReflectionMethod $reflector, HydrationKey $key)
    {
        $this->reflector = $reflector;
        $this->key = $key;
    }

    /**
     * @return string
     */
    #[Pure]
    public function getKey(): string
    {
        return $this->key->getKey();
    }

    /**
     * @return ?ReflectionNamedType
     */
    #[Pure]
    public function getType(): ?ReflectionNamedType
    {
        $parameters = $this->reflector->getParameters();

        return $parameters[0]?->getType();

The expression return $parameters[0]->getType() could return the type ReflectionType which includes types incompatible with the type-hinted return ReflectionNamedType|null. Consider adding an additional type-check to rule them out.

    }

    /**
     * @param object $object
     * @param mixed  $value
     *
     * @throws ReflectionException
     *
     * @return void
     */
    public function setValue(object $object, mixed $value): void
    {
        $this->reflector->invoke($object, $value);
    }
}