Maiorano\ObjectHydrator\Strategies\RecursiveCheckTrait

Complexity

Total Complexity

6

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	1	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	checkTypeRecursion()	0	12	5
A	isRecursive()	0	3	1

<?php

namespace Maiorano\ObjectHydrator\Strategies;

use ReflectionType;

trait RecursiveCheckTrait
{
    /**
     * @param string $key
     * @param mixed  $value
     *
     * @return bool
     */
    public function isRecursive(string $key, mixed $value): bool
    {
        return $this->checkTypeRecursion($this->getMapping($key)->getType(), $value);

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

    }

    /**
     * @param ReflectionType|null $type
     * @param mixed               $value
     *
     * @return bool
     */
    private function checkTypeRecursion(?ReflectionType $type, mixed $value): bool
    {
        if (!$type) {
            return false;
        }

        $allowsNull = $type->allowsNull() ?? true;
        if ($allowsNull && is_null($value)) {
            return false;
        }

        return !is_scalar($value) && !$type->isBuiltin();
    }
}