RecursiveCheckTrait::isRecursive()   A
last analyzed

Complexity

Conditions 1
Paths 1

Size

Total Lines 3
Code Lines 1

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 2
CRAP Score 1

Importance

Changes 1
Bugs 0 Features 0
Metric Value
cc 1
eloc 1
c 1
b 0
f 0
nc 1
nop 2
dl 0
loc 3
ccs 2
cts 2
cp 1
crap 1
rs 10
1
<?php
2
3
namespace Maiorano\ObjectHydrator\Strategies;
4
5
use ReflectionType;
6
7
trait RecursiveCheckTrait
8
{
9
    /**
10
     * @param string $key
11
     * @param mixed  $value
12
     *
13
     * @return bool
14
     */
15 10
    public function isRecursive(string $key, mixed $value): bool
16
    {
17 10
        return $this->checkTypeRecursion($this->getMapping($key)->getType(), $value);
0 ignored issues
show
Bug introduced by
It seems like getMapping() must be provided by classes using this trait. How about adding it as abstract method to this trait? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

17
        return $this->checkTypeRecursion($this->/** @scrutinizer ignore-call */ getMapping($key)->getType(), $value);
Loading history...
18
    }
19
20
    /**
21
     * @param ReflectionType|null $type
22
     * @param mixed               $value
23
     *
24
     * @return bool
25
     */
26 10
    private function checkTypeRecursion(?ReflectionType $type, mixed $value): bool
27
    {
28 10
        if (!$type) {
29 4
            return false;
30
        }
31
32 9
        $allowsNull = $type->allowsNull() ?? true;
33 9
        if ($allowsNull && is_null($value)) {
34 3
            return false;
35
        }
36
37 7
        return !is_scalar($value) && !$type->isBuiltin();
38
    }
39
}
40