einfach\operation\step\AbstractStep

Complexity

Total Complexity

3

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

0%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	isSkipped()	0	4	1
A	skip()	0	4	1
	__invoke()	0	1	?

<?php

namespace einfach\operation\step;

abstract class AbstractStep
{
    /**
     * @var callable
     */
    public $function;
    public $signature;
    public $track;
    /**
     * Indicates was this step performed or not
     *
     * @var bool
     */
    protected $skipped;

    public function __construct(callable $callable, string $signature)
    {
        $this->function = $callable;
        $this->signature = $signature;
        $this->skipped = false;
    }

    public function isSkipped()
    {
        return true == $this->skipped;

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

    }

    public function skip()
    {
        return $this->skipped = true;
    }

    abstract public function __invoke(&$params, string $track);
}