Factory::getScalarPasses() - Code Metrics - Inspection of "Add SleepUsage analyzer" - ovr/phpsa - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#273)

by Enrico

created 2016-11-18 15:50 UTC

Factory::getScalarPasses() A

↳ Parent: Factory

Complexity

Conditions	1
Paths	1

Size

Total Lines	6
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	1

Importance

Changes

Metric	Value
cc	1
eloc	3
nc	1
nop	0
dl	0
loc	6
ccs	3
cts	3
cp	1
crap	1
rs	9.4285
c	0
b	0
f	0

<?php
/**
 * @author Patsura Dmitry https://github.com/ovr <[email protected]>
 */

namespace PHPSA\Analyzer;

use PHPSA\Analyzer;
use PHPSA\Configuration;
use Symfony\Component\Config\Definition\Builder\NodeDefinition;
use Webiny\Component\EventManager\EventManager;
use PHPSA\Analyzer\Pass as AnalyzerPass;

class Factory
{
    /**
     * @return NodeDefinition[]
     */
    public static function getPassesConfigurations()
    {
        $configs = [];

        foreach (self::getExpressionPasses() as $passClass) {
            $configs[] = $passClass::getMetadata()->getConfiguration();
        }

        foreach (self::getStatementPasses() as $passClass) {
            $configs[] = $passClass::getMetadata()->getConfiguration();
        }

        foreach (self::getScalarPasses() as $passClass) {
            $configs[] = $passClass::getMetadata()->getConfiguration();
        }

        return $configs;
    }

    /**
     * @return \PHPSA\Analyzer\Pass\Metadata[]
     */
    public static function getPassesMetadata()
    {
        $meta = [];

        foreach (self::getExpressionPasses() as $passClass) {
            $meta[] = $passClass::getMetadata();
        }

        foreach (self::getStatementPasses() as $passClass) {
            $meta[] = $passClass::getMetadata();
        }

        foreach (self::getScalarPasses() as $passClass) {
            $meta[] = $passClass::getMetadata();
        }

        return $meta;
    }

    /**
     * @param EventManager $eventManager
     * @param Configuration $config
     * @return Analyzer
     */
    public static function factory(EventManager $eventManager, Configuration $config)

    {
        $analyzersConfig = $config->getValue('analyzers');

        $filterEnabled = function ($passClass) use ($config, $analyzersConfig) {
            /** @var AnalyzerPass\Metadata $passMetadata */
            $passMetadata = $passClass::getMetadata();

            if (!isset($analyzersConfig[$passMetadata->getName()])) {
                return false;
            }

            if (!$analyzersConfig[$passMetadata->getName()]['enabled']) {
                return false;
            }

            if (!$passMetadata->allowsPhpVersion($config->getValue('language_level'))) {
                return false;
            }

            return true;
        };

        $instanciate = function ($passClass) use ($analyzersConfig) {
            $passName = $passClass::getMetadata()->getName();

            return new $passClass($analyzersConfig[$passName]);
        };

        $analyzer = new Analyzer($eventManager);
        $analyzer->registerExpressionPasses(
            array_map($instanciate, array_filter(self::getExpressionPasses(), $filterEnabled))
        );
        $analyzer->registerStatementPasses(
            array_map($instanciate, array_filter(self::getStatementPasses(), $filterEnabled))
        );
        $analyzer->registerScalarPasses(
            array_map($instanciate, array_filter(self::getScalarPasses(), $filterEnabled))
        );
        $analyzer->bind();

        return $analyzer;
    }

    /**
     * @return array
     */
    private static function getStatementPasses()
    {
        return [
            AnalyzerPass\Statement\MagicMethodParameters::class,
            AnalyzerPass\Statement\GotoUsage::class,
            AnalyzerPass\Statement\GlobalUsage::class,
            AnalyzerPass\Statement\HasMoreThanOneProperty::class,
            AnalyzerPass\Statement\MissingBreakStatement::class,
            AnalyzerPass\Statement\MissingVisibility::class,
            AnalyzerPass\Statement\MethodCannotReturn::class,
            AnalyzerPass\Statement\UnexpectedUseOfThis::class,
            AnalyzerPass\Statement\TestAnnotation::class,
            AnalyzerPass\Statement\MissingDocblock::class,
            AnalyzerPass\Statement\OldConstructor::class,
            AnalyzerPass\Statement\ConstantNaming::class,
            AnalyzerPass\Statement\MissingBody::class,
            AnalyzerPass\Statement\InlineHtmlUsage::class,
            AnalyzerPass\Statement\AssignmentInCondition::class,
            AnalyzerPass\Statement\StaticUsage::class,
            AnalyzerPass\Statement\OptionalParamBeforeRequired::class,
            AnalyzerPass\Statement\YodaCondition::class,
            AnalyzerPass\Statement\ForCondition::class,
            AnalyzerPass\Statement\PropertyDefinitionDefaultValue::class,
            AnalyzerPass\Statement\ReturnAndYieldInOneMethod::class,
        ];
    }

    /**
     * @return array
     */
    private static function getExpressionPasses()
    {
        return [
            // Another
            AnalyzerPass\Expression\ErrorSuppression::class,
            AnalyzerPass\Expression\MultipleUnaryOperators::class,
            AnalyzerPass\Expression\StupidUnaryOperators::class,
            AnalyzerPass\Expression\VariableVariableUsage::class,
            AnalyzerPass\Expression\Casts::class,
            AnalyzerPass\Expression\EvalUsage::class,
            AnalyzerPass\Expression\FinalStaticUsage::class,
            AnalyzerPass\Expression\CompareWithArray::class,
            AnalyzerPass\Expression\DivisionFromZero::class,
            AnalyzerPass\Expression\DivisionByOne::class,
            AnalyzerPass\Expression\BacktickUsage::class,
            AnalyzerPass\Expression\LogicInversion::class,
            AnalyzerPass\Expression\ExitUsage::class,
            // Arrays
            AnalyzerPass\Expression\ArrayShortDefinition::class,
            AnalyzerPass\Expression\ArrayDuplicateKeys::class,
            AnalyzerPass\Expression\ArrayIllegalOffsetType::class,
            // Function call
            AnalyzerPass\Expression\FunctionCall\AliasCheck::class,
            AnalyzerPass\Expression\FunctionCall\DebugCode::class,
            AnalyzerPass\Expression\FunctionCall\RandomApiMigration::class,
            AnalyzerPass\Expression\FunctionCall\SleepUsage::class,
            AnalyzerPass\Expression\FunctionCall\UseCast::class,
            AnalyzerPass\Expression\FunctionCall\DeprecatedIniOptions::class,
            AnalyzerPass\Expression\FunctionCall\RegularExpressions::class,
            AnalyzerPass\Expression\FunctionCall\ArgumentUnpacking::class,
            AnalyzerPass\Expression\FunctionCall\DeprecatedFunctions::class,
        ];
    }

    /**
     * @return array
     */
    private static function getScalarPasses()
    {
        return [
            AnalyzerPass\Scalar\CheckLNumberKind::class,
        ];
    }
}


1		<?php
2		/**
3		* @author Patsura Dmitry https://github.com/ovr <[email protected]>
4		*/
5
6		namespace PHPSA\Analyzer;
7
8		use PHPSA\Analyzer;
9		use PHPSA\Configuration;
10		use Symfony\Component\Config\Definition\Builder\NodeDefinition;
11		use Webiny\Component\EventManager\EventManager;
12		use PHPSA\Analyzer\Pass as AnalyzerPass;
13
14		class Factory
15		{
16		/**
17		* @return NodeDefinition[]
18		*/
19	1	public static function getPassesConfigurations()
20		{
21		$configs = [];
22
23		foreach (self::getExpressionPasses() as $passClass) {
24		$configs[] = $passClass::getMetadata()->getConfiguration();
25		}
26
27	1	foreach (self::getStatementPasses() as $passClass) {
28		$configs[] = $passClass::getMetadata()->getConfiguration();
29		}
30
31		foreach (self::getScalarPasses() as $passClass) {
32		$configs[] = $passClass::getMetadata()->getConfiguration();
33		}
34
35		return $configs;
36		}
37
38		/**
39		* @return \PHPSA\Analyzer\Pass\Metadata[]
40		*/
41		public static function getPassesMetadata()
42		{
43		$meta = [];
44
45		foreach (self::getExpressionPasses() as $passClass) {
46		$meta[] = $passClass::getMetadata();
47		}
48
49		foreach (self::getStatementPasses() as $passClass) {
50		$meta[] = $passClass::getMetadata();
51		}
52
53		foreach (self::getScalarPasses() as $passClass) {
54		$meta[] = $passClass::getMetadata();
55		}
56
57		return $meta;
58		}
59
60		/**
61		* @param EventManager $eventManager
62		* @param Configuration $config
63		* @return Analyzer
64		*/
65	47	public static function factory(EventManager $eventManager, Configuration $config)
		0 ignored issues – show Best Practice introduced 2016-09-17 17:35 UTC by Report Bug Copy Issue Report Using PHP4-style constructors that are named like the class is not recommend; better use the more explicit `__construct` method. Loading history...
66		{
67	47	$analyzersConfig = $config->getValue('analyzers');
68
69		$filterEnabled = function ($passClass) use ($config, $analyzersConfig) {
70		/** @var AnalyzerPass\Metadata $passMetadata */
71	47	$passMetadata = $passClass::getMetadata();
72
73	47	if (!isset($analyzersConfig[$passMetadata->getName()])) {
74	47	return false;
75		}
76
77	47	if (!$analyzersConfig[$passMetadata->getName()]['enabled']) {
78		return false;
79		}
80
81	47	if (!$passMetadata->allowsPhpVersion($config->getValue('language_level'))) {
82		return false;
83		}
84
85	47	return true;
86	47	};
87
88	47	$instanciate = function ($passClass) use ($analyzersConfig) {
89	47	$passName = $passClass::getMetadata()->getName();
90
91	47	return new $passClass($analyzersConfig[$passName]);
92	47	};
93
94	47	$analyzer = new Analyzer($eventManager);
95	47	$analyzer->registerExpressionPasses(
96	47	array_map($instanciate, array_filter(self::getExpressionPasses(), $filterEnabled))
97	47	);
98	47	$analyzer->registerStatementPasses(
99	47	array_map($instanciate, array_filter(self::getStatementPasses(), $filterEnabled))
100	47	);
101	47	$analyzer->registerScalarPasses(
102	47	array_map($instanciate, array_filter(self::getScalarPasses(), $filterEnabled))
103	47	);
104	47	$analyzer->bind();
105
106	47	return $analyzer;
107		}
108
109		/**
110		* @return array
111		*/
112	47	private static function getStatementPasses()
113		{
114		return [
115	47	AnalyzerPass\Statement\MagicMethodParameters::class,
116	47	AnalyzerPass\Statement\GotoUsage::class,
117	47	AnalyzerPass\Statement\GlobalUsage::class,
118	47	AnalyzerPass\Statement\HasMoreThanOneProperty::class,
119	47	AnalyzerPass\Statement\MissingBreakStatement::class,
120	47	AnalyzerPass\Statement\MissingVisibility::class,
121	47	AnalyzerPass\Statement\MethodCannotReturn::class,
122	47	AnalyzerPass\Statement\UnexpectedUseOfThis::class,
123	47	AnalyzerPass\Statement\TestAnnotation::class,
124	47	AnalyzerPass\Statement\MissingDocblock::class,
125	47	AnalyzerPass\Statement\OldConstructor::class,
126	47	AnalyzerPass\Statement\ConstantNaming::class,
127	47	AnalyzerPass\Statement\MissingBody::class,
128	47	AnalyzerPass\Statement\InlineHtmlUsage::class,
129	47	AnalyzerPass\Statement\AssignmentInCondition::class,
130	47	AnalyzerPass\Statement\StaticUsage::class,
131	47	AnalyzerPass\Statement\OptionalParamBeforeRequired::class,
132	47	AnalyzerPass\Statement\YodaCondition::class,
133	47	AnalyzerPass\Statement\ForCondition::class,
134	47	AnalyzerPass\Statement\PropertyDefinitionDefaultValue::class,
135	47	AnalyzerPass\Statement\ReturnAndYieldInOneMethod::class,
136	47	];
137		}
138
139		/**
140		* @return array
141		*/
142	47	private static function getExpressionPasses()
143		{
144		return [
145		// Another
146	47	AnalyzerPass\Expression\ErrorSuppression::class,
147	47	AnalyzerPass\Expression\MultipleUnaryOperators::class,
148	47	AnalyzerPass\Expression\StupidUnaryOperators::class,
149	47	AnalyzerPass\Expression\VariableVariableUsage::class,
150	47	AnalyzerPass\Expression\Casts::class,
151	47	AnalyzerPass\Expression\EvalUsage::class,
152	47	AnalyzerPass\Expression\FinalStaticUsage::class,
153	47	AnalyzerPass\Expression\CompareWithArray::class,
154	47	AnalyzerPass\Expression\DivisionFromZero::class,
155	47	AnalyzerPass\Expression\DivisionByOne::class,
156	47	AnalyzerPass\Expression\BacktickUsage::class,
157	47	AnalyzerPass\Expression\LogicInversion::class,
158	47	AnalyzerPass\Expression\ExitUsage::class,
159		// Arrays
160	47	AnalyzerPass\Expression\ArrayShortDefinition::class,
161	47	AnalyzerPass\Expression\ArrayDuplicateKeys::class,
162	47	AnalyzerPass\Expression\ArrayIllegalOffsetType::class,
163		// Function call
164	47	AnalyzerPass\Expression\FunctionCall\AliasCheck::class,
165	47	AnalyzerPass\Expression\FunctionCall\DebugCode::class,
166	47	AnalyzerPass\Expression\FunctionCall\RandomApiMigration::class,
167	47	AnalyzerPass\Expression\FunctionCall\SleepUsage::class,
168	47	AnalyzerPass\Expression\FunctionCall\UseCast::class,
169	47	AnalyzerPass\Expression\FunctionCall\DeprecatedIniOptions::class,
170	47	AnalyzerPass\Expression\FunctionCall\RegularExpressions::class,
171	47	AnalyzerPass\Expression\FunctionCall\ArgumentUnpacking::class,
172	47	AnalyzerPass\Expression\FunctionCall\DeprecatedFunctions::class,
173	47	];
174		}
175
176		/**
177		* @return array
178		*/
179	47	private static function getScalarPasses()
180		{
181		return [
182	47	AnalyzerPass\Scalar\CheckLNumberKind::class,
183	47	];
184		}
185		}
186

ovr / phpsa

Pull Request — master (#273)

Factory::getScalarPasses() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like