Darkilliant\ProcessBundle\Configuration\ConfigurationStep

Complexity

Total Complexity

6

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	create()	0	3	1
A	__construct()	0	6	1
A	getChildren()	0	3	1
A	getService()	0	3	1
A	getOptions()	0	3	1
A	isEnabled()	0	3	1

<?php
/**
 * Created by PhpStorm.
 * User: darkilliant
 * Date: 5/8/18
 * Time: 11:04 AM.
 */

namespace Darkilliant\ProcessBundle\Configuration;

class ConfigurationStep
{
    private $next;

The private property $next is not used, and could be removed.

    private $options;

    private $service;

    /** @var bool */
    private $enabled;

    private function __construct(string $service, array $options, array $children, bool $enabled = true)
    {
        $this->service = $service;
        $this->options = $options;
        $this->children = array_map([ConfigurationStep::class, 'create'], $children);

The property children does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->enabled = $enabled;
    }

    public static function create(array $config)
    {
        return new self($config['service'], $config['options'] ?? [], $config['children'] ?? [], $config['enabled'] ?? true);
    }

    public function getOptions(): array
    {
        return $this->options;
    }

    public function getService(): string
    {
        return $this->service;
    }

    public function getChildren(): array
    {
        return $this->children;
    }

    public function isEnabled()
    {
        return $this->enabled;
    }
}