AbstractExtension::subjectInstanceOf() - Code Metrics - Inspection of "feat: facades of managers proxy to drivers" - nunomaduro/laravel-code-analyse - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( f235c1...2c36df )

by Nuno

created 2018-07-04 22:56 UTC

AbstractExtension::subjectInstanceOf() A

↳ Parent: AbstractExtension

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
nc	2
nop	1
dl	0
loc	3
rs	10
c	0
b	0
f	0

<?php

declare(strict_types=1);

/**
 * This file is part of Laravel Code Analyse.
 *
 * (c) Nuno Maduro <[email protected]>
 *
 *  For the full copyright and license information, please view the LICENSE
 *  file that was distributed with this source code.
 */

namespace NunoMaduro\LaravelCodeAnalyse;

use Mockery;
use PHPStan\Reflection\ClassReflection;
use PHPStan\Reflection\MethodReflection;
use PHPStan\Reflection\BrokerAwareExtension;
use PHPStan\Reflection\MethodsClassReflectionExtension;

/**
 * @internal
 */
abstract class AbstractExtension implements MethodsClassReflectionExtension, BrokerAwareExtension
{
    use Concerns\HasBroker;

    /**
     * Whether the methods can be accessed statically
     */
    protected $staticAccess = false;

    /**
     * Holds already discovered methods.
     *
     * @var array
     */
    private $cache = [];

    /**
     * {@inheritdoc}
     */
    public function hasMethod(ClassReflection $classReflection, string $methodName): bool
    {
        $hasMethod = false;

        if ($this->subjectInstanceOf($classReflection)) {
            foreach ($this->searchIn($classReflection) as $toBeSearchClass) {
                $hasMethod = $this->broker->getClass($toBeSearchClass)
                    ->hasNativeMethod($methodName);

                if ($hasMethod) {
                    $this->pushToCache($classReflection, $methodName, $toBeSearchClass);
                    break;
                }
            }
        }

        return $hasMethod;
    }

    /**
     * {@inheritdoc}
     */
    public function getMethod(ClassReflection $classReflection, string $methodName): MethodReflection
    {
        $methodReflection = $this->broker->getClass($this->cache[$classReflection->getName()][$methodName])
            ->getNativeMethod($methodName);

        if ($this->staticAccess) {
            $methodReflection = Mockery::mock($methodReflection);
            $methodReflection->shouldReceive('isStatic')
                ->andReturn(true);
        }

        return $methodReflection;

    }

    /**
     * @param \PHPStan\Reflection\ClassReflection $classReflection
     *
     * @return bool
     */
    protected function subjectInstanceOf(ClassReflection $classReflection): bool
    {
        return $classReflection->getName() === $this->subject() || $classReflection->isSubclassOf($this->subject());
    }

    /**
     * @param \PHPStan\Reflection\ClassReflection $classReflection
     * @param string $methodName
     * @param string $toBeSearchClass
     */
    protected function pushToCache(ClassReflection $classReflection, string $methodName, string $toBeSearchClass): void
    {
        if (! array_key_exists($classReflection->getName(), $this->cache)) {
            $this->cache[$classReflection->getName()] = [];
        }

        $this->cache[$classReflection->getName()][$methodName] = $toBeSearchClass;
    }

    /**
     * Returns the class under analyse.
     *
     * @return string
     */
    abstract protected function subject(): string;

    /**
     * Returns the classes where the native method should be search for.
     *
     * @param \PHPStan\Reflection\ClassReflection $classReflection
     *
     * @return array
     */
    abstract protected function searchIn(ClassReflection $classReflection): array;
}


1			<?php
2
3			declare(strict_types=1);
4
5			/**
6			* This file is part of Laravel Code Analyse.
7			*
8			* (c) Nuno Maduro <[email protected]>
9			*
10			* For the full copyright and license information, please view the LICENSE
11			* file that was distributed with this source code.
12			*/
13
14			namespace NunoMaduro\LaravelCodeAnalyse;
15
16			use Mockery;
17			use PHPStan\Reflection\ClassReflection;
18			use PHPStan\Reflection\MethodReflection;
19			use PHPStan\Reflection\BrokerAwareExtension;
20			use PHPStan\Reflection\MethodsClassReflectionExtension;
21
22			/**
23			* @internal
24			*/
25			abstract class AbstractExtension implements MethodsClassReflectionExtension, BrokerAwareExtension
26			{
27			use Concerns\HasBroker;
28
29			/**
30			* Whether the methods can be accessed statically
31			*/
32			protected $staticAccess = false;
33
34			/**
35			* Holds already discovered methods.
36			*
37			* @var array
38			*/
39			private $cache = [];
40
41			/**
42			* {@inheritdoc}
43			*/
44			public function hasMethod(ClassReflection $classReflection, string $methodName): bool
45			{
46			$hasMethod = false;
47
48			if ($this->subjectInstanceOf($classReflection)) {
49			foreach ($this->searchIn($classReflection) as $toBeSearchClass) {
50			$hasMethod = $this->broker->getClass($toBeSearchClass)
51			->hasNativeMethod($methodName);
52
53			if ($hasMethod) {
54			$this->pushToCache($classReflection, $methodName, $toBeSearchClass);
55			break;
56			}
57			}
58			}
59
60			return $hasMethod;
61			}
62
63			/**
64			* {@inheritdoc}
65			*/
66			public function getMethod(ClassReflection $classReflection, string $methodName): MethodReflection
67			{
68			$methodReflection = $this->broker->getClass($this->cache[$classReflection->getName()][$methodName])
69			->getNativeMethod($methodName);
70
71			if ($this->staticAccess) {
72			$methodReflection = Mockery::mock($methodReflection);
73			$methodReflection->shouldReceive('isStatic')
74			->andReturn(true);
75			}
76
77			return $methodReflection;
			0 ignored issues – show Bug Best Practice introduced 2018-07-04 19:49 UTC by Report Bug Copy Issue Report The expression `return $methodReflection` could return the type `Mockery\MockInterface` which is incompatible with the type-hinted return `PHPStan\Reflection\MethodReflection`. Consider adding an additional type-check to rule them out. Loading history...
78			}
79
80			/**
81			* @param \PHPStan\Reflection\ClassReflection $classReflection
82			*
83			* @return bool
84			*/
85			protected function subjectInstanceOf(ClassReflection $classReflection): bool
86			{
87			return $classReflection->getName() === $this->subject() \|\| $classReflection->isSubclassOf($this->subject());
88			}
89
90			/**
91			* @param \PHPStan\Reflection\ClassReflection $classReflection
92			* @param string $methodName
93			* @param string $toBeSearchClass
94			*/
95			protected function pushToCache(ClassReflection $classReflection, string $methodName, string $toBeSearchClass): void
96			{
97			if (! array_key_exists($classReflection->getName(), $this->cache)) {
98			$this->cache[$classReflection->getName()] = [];
99			}
100
101			$this->cache[$classReflection->getName()][$methodName] = $toBeSearchClass;
102			}
103
104			/**
105			* Returns the class under analyse.
106			*
107			* @return string
108			*/
109			abstract protected function subject(): string;
110
111			/**
112			* Returns the classes where the native method should be search for.
113			*
114			* @param \PHPStan\Reflection\ClassReflection $classReflection
115			*
116			* @return array
117			*/
118			abstract protected function searchIn(ClassReflection $classReflection): array;
119			}
120

nunomaduro / laravel-code-analyse

Push — master ( f235c1...2c36df )

AbstractExtension::subjectInstanceOf() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like