Spiral\Jobs\Options

Complexity

Total Complexity

9

Size/Duplication

Total Lines	89
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	withDelay()	0	6	1
A	jsonSerialize()	0	5	2
A	delayed()	0	6	1
A	withPipeline()	0	6	1
A	getPriority()	0	3	1
A	__construct()	0	3	1
A	getPipeline()	0	3	1
A	getDelay()	0	3	1

<?php

declare(strict_types=1);

namespace Spiral\Jobs;

use Spiral\RoadRunner\Jobs\Options as RoadRunnerOptions;
use Spiral\RoadRunner\Jobs\OptionsInterface;

/**
 * @deprecated Since 2.9. Please use {@see RoadRunnerOptions} instead.
 */
class Options implements OptionsInterface, \JsonSerializable
{
    /**
     * @var OptionsInterface
     */
    private OptionsInterface $base;

    /**
     * Options constructor
     */
    public function __construct()
    {
        $this->base = new RoadRunnerOptions();
    }

    /**
     * @var string|null
     */
    private ?string $pipeline = null;

    /**
     * {@inheritDoc}
     */
    public function getDelay(): int
    {
        return $this->base->getDelay();
    }

    /**
     * @param int|null $delay
     * @return $this
     */
    public function withDelay(?int $delay): self
    {
        $options = clone $this;
        $options->base = $options->base->withDelay($delay ?? 0);

        return $options;
    }

    /**
     * {@inheritDoc}
     */
    public function getPriority(): int
    {
        return $this->base->getPriority();
    }

    /**
     * @return string|null
     */
    public function getPipeline(): ?string
    {
        return $this->pipeline;
    }

    /**
     * @param string|null $pipeline
     * @return self
     */
    public function withPipeline(?string $pipeline): self
    {
        $options = clone $this;
        $options->pipeline = $pipeline;

        return $options;
    }

    /**
     * @return array
     */
    public function jsonSerialize(): array
    {
        return [
            'delay'    => $this->base->delay ?: null,

Accessing delay on the interface Spiral\RoadRunner\Jobs\OptionsInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

            'pipeline' => $this->pipeline
        ];
    }

    /**
     * @param positive-int|0 $delay

The doc comment positive-int|0 at position 0 could not be parsed: Unknown type name 'positive-int' at position 0 in positive-int|0.

     * @return Options
     */
    public static function delayed(int $delay): self
    {
        $options = new self();
        $options->base = $options->base->withDelay($delay);

        return $options;
    }
}