Roave\BackwardCompatibility\DetectChanges\BCBreak\MethodBased\MethodConcretenessChanged

Complexity

Total Complexity

3

Size/Duplication

Total Lines	15
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	13	3

<?php

declare(strict_types=1);

namespace Roave\BackwardCompatibility\DetectChanges\BCBreak\MethodBased;

use Roave\BackwardCompatibility\Change;
use Roave\BackwardCompatibility\Changes;
use Roave\BetterReflection\Reflection\ReflectionMethod;
use function Safe\sprintf;

/**
 * A method that changes from concrete to abstract forces all child class
 * implementations to implement it, and is therefore a BC break
 */
final class MethodConcretenessChanged implements MethodBased
{
    public function __invoke(ReflectionMethod $fromMethod, ReflectionMethod $toMethod) : Changes
    {
        if ($fromMethod->isAbstract() || ! $toMethod->isAbstract()) {
            return Changes::empty();
        }

        return Changes::fromList(Change::changed(
            sprintf(
                'Method %s() of class %s changed from concrete to abstract',
                $fromMethod->getName(),
                $fromMethod->getDeclaringClass()->getName()
            ),
            true
        ));
    }
}