PropertyBecameInternal::__invoke() - Code Metrics - Inspection of "Merge pull request #141 from Roave/feature/#51-sup..." - Roave/BackwardCompatibilityCheck - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 85853a...5df06c )

by James

created 2019-07-17 08:16 UTC

PropertyBecameInternal::__invoke() A

↳ Parent: PropertyBecameInternal

Complexity

Conditions	3
Paths	2

Size

Total Lines	15
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

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

<?php

declare(strict_types=1);

namespace Roave\BackwardCompatibility\DetectChanges\BCBreak\PropertyBased;

use Roave\BackwardCompatibility\Change;
use Roave\BackwardCompatibility\Changes;
use Roave\BackwardCompatibility\Formatter\ReflectionPropertyName;
use Roave\BetterReflection\Reflection\ReflectionProperty;
use function Safe\preg_match;
use function Safe\sprintf;

/**
 * A property that is marked internal is no available to downstream consumers.
 */
final class PropertyBecameInternal implements PropertyBased
{
    /** @var ReflectionPropertyName */
    private $formatProperty;

    public function __construct()
    {
        $this->formatProperty = new ReflectionPropertyName();
    }

    public function __invoke(ReflectionProperty $fromProperty, ReflectionProperty $toProperty) : Changes
    {
        if ($this->isInternalDocComment($toProperty->getDocComment())
            && ! $this->isInternalDocComment($fromProperty->getDocComment())
        ) {
            return Changes::fromList(Change::changed(
                sprintf(
                    'Property %s was marked "@internal"',
                    $this->formatProperty->__invoke($fromProperty),
                ),
                true
            ));
        }

        return Changes::empty();
    }

    private function isInternalDocComment(string $comment) : bool
    {
        return preg_match('/\s+@internal\s+/', $comment) === 1;
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Roave\BackwardCompatibility\DetectChanges\BCBreak\PropertyBased;
6
7			use Roave\BackwardCompatibility\Change;
8			use Roave\BackwardCompatibility\Changes;
9			use Roave\BackwardCompatibility\Formatter\ReflectionPropertyName;
10			use Roave\BetterReflection\Reflection\ReflectionProperty;
11			use function Safe\preg_match;
12			use function Safe\sprintf;
13
14			/**
15			* A property that is marked internal is no available to downstream consumers.
16			*/
17			final class PropertyBecameInternal implements PropertyBased
18			{
19			/** @var ReflectionPropertyName */
20			private $formatProperty;
21
22			public function __construct()
23			{
24			$this->formatProperty = new ReflectionPropertyName();
25			}
26
27			public function __invoke(ReflectionProperty $fromProperty, ReflectionProperty $toProperty) : Changes
28			{
29			if ($this->isInternalDocComment($toProperty->getDocComment())
30			&& ! $this->isInternalDocComment($fromProperty->getDocComment())
31			) {
32			return Changes::fromList(Change::changed(
33			sprintf(
34			'Property %s was marked "@internal"',
35			$this->formatProperty->__invoke($fromProperty),
36			),
37			true
38			));
39			}
40
41			return Changes::empty();
42			}
43
44			private function isInternalDocComment(string $comment) : bool
45			{
46			return preg_match('/\s+@internal\s+/', $comment) === 1;
47			}
48			}
49

Roave / BackwardCompatibilityCheck

Push — master ( 85853a...5df06c )

PropertyBecameInternal::__invoke() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like