These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
1 | <?php |
||
2 | /** |
||
3 | * PHP Smart Analysis project 2015-2016 |
||
4 | * |
||
5 | * @author Patsura Dmitry https://github.com/ovr <[email protected]> |
||
6 | */ |
||
7 | |||
8 | namespace PHPSA\Compiler\Expression; |
||
9 | |||
10 | use PHPSA\CompiledExpression; |
||
11 | use PHPSA\Context; |
||
12 | use PHPSA\Definition\ClassDefinition; |
||
13 | |||
14 | class StaticPropertyFetch extends AbstractExpressionCompiler |
||
15 | { |
||
16 | protected $name = 'PhpParser\Node\Expr\StaticPropertyFetch'; |
||
17 | |||
18 | /** |
||
19 | * {expr}::${expr}; |
||
20 | * |
||
21 | * @param \PhpParser\Node\Expr\StaticPropertyFetch $expr |
||
22 | * @param Context $context |
||
23 | * @return CompiledExpression |
||
24 | */ |
||
25 | protected function compile($expr, Context $context) |
||
26 | { |
||
27 | $expressionCompiler = $context->getExpressionCompiler(); |
||
28 | $leftCE = $expressionCompiler->compile($expr->class); |
||
29 | |||
30 | if ($leftCE->isObject()) { |
||
31 | $name = $expr->name; |
||
32 | |||
33 | /** @var ClassDefinition $classDefinition */ |
||
34 | $classDefinition = $context->scope; |
||
35 | if (!$classDefinition->hasProperty($name, true)) { |
||
36 | $context->notice( |
||
37 | 'language_error', |
||
38 | sprintf('Static property $%s does not exist in %s scope', $name, $expr->class), |
||
39 | $expr |
||
40 | ); |
||
41 | |||
42 | return new CompiledExpression(); |
||
43 | } |
||
44 | |||
45 | $property = $classDefinition->getPropertyStatement($name, true); |
||
46 | if (!$property->isStatic()) { |
||
47 | $context->notice( |
||
48 | 'language_error', |
||
49 | sprintf('Property $%s is not static but was called in a static way', $name), |
||
50 | $expr |
||
51 | ); |
||
52 | } |
||
53 | |||
54 | return $expressionCompiler->compile($property); |
||
0 ignored issues
–
show
|
|||
55 | } |
||
56 | |||
57 | $context->debug('Unknown static property fetch', $expr); |
||
58 | return new CompiledExpression(); |
||
59 | } |
||
60 | } |
||
61 |
Unless you are absolutely sure that the expression can never be null because of other conditions, we strongly recommend to add an additional type check to your code: