Star\Component\State\Transitions\ManyToOneTransition

Complexity

Total Complexity

5

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Test Coverage

Coverage

86.67%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	getName()	0	3	1
A	getDestinationState()	0	3	1
A	onRegister()	0	7	2

<?php declare(strict_types=1);

This file is missing a doc comment.

The PHP open tag does not have a corresponding PHP close tag

Filename "ManyToOneTransition.php" doesn't match the expected filename "manytoonetransition.php"

namespace Star\Component\State\Transitions;

use Star\Component\State\RegistryBuilder;
use Star\Component\State\StateTransition;
use Webmozart\Assert\Assert;

final class ManyToOneTransition implements StateTransition

Missing class doc comment

{

Opening brace should be on the same line as the declaration for class ManyToOneTransition

    /**

Missing short description in doc comment

     * @var string
     */
    private $name;

Private member variable "name" must contain a leading underscore

Expected 1 blank line before member var; 0 found

Private member variable "name" must be prefixed with an underscore

    /**

Missing short description in doc comment

     * @var string[]
     */
    private $fromStates;

Private member variable "fromStates" must contain a leading underscore

Private member variable "fromStates" must be prefixed with an underscore

    /**

Missing short description in doc comment

     * @var string
     */
    private $to;

Private member variable "to" must contain a leading underscore

Private member variable "to" must be prefixed with an underscore

    public function __construct(string $name, string $to, string ...$fromStates)

Expected 2 blank lines before function; 1 found

Missing function doc comment

    {

Opening brace should be on the same line as the declaration

        $this->name = $name;
        Assert::greaterThanEq(\count($fromStates), 1, 'Expected at least %2$s state. Got: %s');
        $this->fromStates = $fromStates;
        $this->to = $to;

Equals sign not aligned with surrounding assignments; expected 9 spaces but found 1 space

    }

Expected 2 blank lines after function; 1 found

Expected 1 blank line before closing function brace; 0 found

Expected //end __construct()

    public function getName(): string

Missing function doc comment

    {

Opening brace should be on the same line as the declaration

        return $this->name;
    }

Expected 2 blank lines after function; 1 found

Expected 1 blank line before closing function brace; 0 found

Expected //end getName()

    public function onRegister(RegistryBuilder $registry): void

Missing function doc comment

    {

Opening brace should be on the same line as the declaration

        foreach ($this->fromStates as $from) {
            $registry->registerStartingState($this->name, $from, []);

Short array syntax is not allowed

        }

        $registry->registerDestinationState($this->name, $this->to, []);

Short array syntax is not allowed

    }

Expected 2 blank lines after function; 1 found

Expected 1 blank line before closing function brace; 0 found

Expected //end onRegister()

    public function getDestinationState(): string

Missing function doc comment

    {

Opening brace should be on the same line as the declaration

        return $this->to;
    }

Expected 2 blank lines after function; 0 found

Expected 1 blank line before closing function brace; 0 found

Expected //end getDestinationState()

}

Expected //end class

As per coding style, files should not end with a newline character.