These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
1 | <?php |
||
2 | declare(strict_types=1); |
||
3 | |||
4 | namespace Moka\Generator\Template; |
||
5 | |||
6 | /** |
||
7 | * Class PropertyTemplate |
||
8 | * @package Moka\Generator\Template |
||
9 | */ |
||
10 | class PropertyTemplate implements TemplateInterface |
||
11 | { |
||
12 | use VisibilityTrait; |
||
13 | |||
14 | const TEMPLATE = ' |
||
15 | %s %s $%s; |
||
16 | '; |
||
17 | |||
18 | /** |
||
19 | * @param \ReflectionProperty $property |
||
20 | * @return string |
||
21 | */ |
||
22 | 4 | public static function generate(\Reflector $property): string |
|
23 | { |
||
24 | 4 | return static::doGenerate($property); |
|
1 ignored issue
–
show
|
|||
25 | } |
||
26 | |||
27 | /** |
||
28 | * @param \ReflectionProperty $property |
||
29 | * @return string |
||
30 | */ |
||
31 | 4 | protected static function doGenerate(\ReflectionProperty $property): string |
|
32 | { |
||
33 | 4 | $visibility = static::getVisibility($property); |
|
34 | |||
35 | 4 | $static = $property->isStatic() ? 'static' : ''; |
|
36 | |||
37 | 4 | $name = $property->name; |
|
38 | |||
39 | 4 | return sprintf( |
|
40 | 4 | self::TEMPLATE, |
|
41 | 4 | $visibility, |
|
42 | 4 | $static, |
|
43 | 4 | $name |
|
44 | ); |
||
45 | } |
||
46 | } |
||
47 |
This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass.
Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type.