PropertyInitializationTemplate::generate() - Code Metrics - Inspection of "Merge pull request #37 from facile-it/feature/prox..." - facile-it/moka - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 7ee214...32fd8d )

by Alberto

created 2017-07-12 15:12 UTC

PropertyInitializationTemplate::generate() A

↳ Parent: PropertyInitializationTemplate

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

Metric	Value
cc	1
eloc	2
nc	1
nop	1
dl	0
loc	4
ccs	2
cts	2
cp	1
crap	1
rs	10
c	0
b	0
f	0

<?php
declare(strict_types=1);

namespace Moka\Generator\Template;

/**
 * Class PropertyInitializationTemplate
 * @package Moka\Generator\Template
 */
class PropertyInitializationTemplate implements TemplateInterface
{
    const TEMPLATE = '
            %s%s = function () {
                return $this->doGet("%s");
            };
    ';

    /**
     * @param \ReflectionProperty $property
     * @return string
     */
    public static function generate(\Reflector $property): string
    {
        return self::doGenerate($property);

    }

    /**
     * @param \ReflectionProperty $property
     * @return string
     */
    protected static function doGenerate(\ReflectionProperty $property): string
    {
        $scope = $property->isStatic() ? 'self::$' : '$this->';

        $name = $property->name;

        return sprintf(
            self::TEMPLATE,
            $scope,
            $name,
            $name
        );
    }
}


1		<?php
2		declare(strict_types=1);
3
4		namespace Moka\Generator\Template;
5
6		/**
7		* Class PropertyInitializationTemplate
8		* @package Moka\Generator\Template
9		*/
10		class PropertyInitializationTemplate implements TemplateInterface
11		{
12		const TEMPLATE = '
13		%s%s = function () {
14		return $this->doGet("%s");
15		};
16		';
17
18		/**
19		* @param \ReflectionProperty $property
20		* @return string
21		*/
22	4	public static function generate(\Reflector $property): string
23		{
24	4	return self::doGenerate($property);
		1 ignored issue – show Compatibility introduced 2017-07-06 12:13 UTC by Report Bug Copy Issue Report `$property` of type `object<Reflector>` is not a sub-type of `object<ReflectionProperty>`. It seems like you assume a concrete implementation of the interface `Reflector` to be always present. 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. Loading history...
25		}
26
27		/**
28		* @param \ReflectionProperty $property
29		* @return string
30		*/
31	4	protected static function doGenerate(\ReflectionProperty $property): string
32		{
33	4	$scope = $property->isStatic() ? 'self::$' : '$this->';
34
35	4	$name = $property->name;
36
37	4	return sprintf(
38	4	self::TEMPLATE,
39	4	$scope,
40	4	$name,
41	4	$name
42		);
43		}
44		}
45

facile-it / moka

Push — master ( 7ee214...32fd8d )

PropertyInitializationTemplate::generate() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like