SignpostMarv\DaftObject\DaftNestedObject\PHPStan\ClassReflectionExtension

Complexity

Total Complexity

4

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	hasProperty()	0	13	2
A	setBroker()	0	3	1
A	getProperty()	0	3	1

<?php
/**
* Base daft objects.
*
* @author SignpostMarv
*/
declare(strict_types=1);

namespace SignpostMarv\DaftObject\DaftNestedObject\PHPStan;

use PHPStan\Broker\Broker;
use PHPStan\Reflection\BrokerAwareExtension;
use PHPStan\Reflection\ClassReflection;
use PHPStan\Reflection\PropertiesClassReflectionExtension;
use PHPStan\Reflection\PropertyReflection;
use SignpostMarv\DaftObject\DaftNestedObject;
use SignpostMarv\DaftObject\PHPStan\ClassReflectionExtension as Base;

class ClassReflectionExtension extends Base
{
    /**
    * @var Broker|null
    */
    private $broker;

    public function setBroker(Broker $broker) : void
    {
        $this->broker = $broker;
    }

    public function hasProperty(ClassReflection $classReflection, string $propertyName) : bool
    {
        $className = $classReflection->getName();

The assignment to $className is dead and can be removed.

        $property = ucfirst($propertyName);

        return
            parent::hasProperty($classReflection, $property) ||
            in_array($property, [
                'intNestedLeft',
                'intNestedRight',
                'intNestedLevel',
                'intNestedParentId'
            ]);
    }

    public function getProperty(ClassReflection $ref, string $propertyName) : PropertyReflection
    {
        return new PropertyReflectionExtension($ref, $this->broker, $propertyName);

It seems like $this->broker can also be of type null; however, parameter $broker of SignpostMarv\DaftObject\DaftNestedObject\PHPStan\PropertyReflectionExtension::__construct() does only seem to accept PHPStan\Broker\Broker, maybe add an additional type check?

    }
}