Setono\SyliusSchedulerPlugin\Factory\JobFactory

Complexity

Total Complexity

4

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	createFromSchedule()	0	10	1
A	createRetryJob()	0	16	1
A	createNew()	0	3	1
A	__construct()	0	4	1

<?php

declare(strict_types=1);

namespace Setono\SyliusSchedulerPlugin\Factory;

use Setono\SyliusSchedulerPlugin\Model\JobInterface;
use Setono\SyliusSchedulerPlugin\Model\ScheduleInterface;
use Sylius\Component\Resource\Factory\FactoryInterface;

class JobFactory implements JobFactoryInterface
{
    /**
     * @var FactoryInterface
     */
    private $factory;

    /**
     * @param FactoryInterface $factory
     */
    public function __construct(
        FactoryInterface $factory
    ) {
        $this->factory = $factory;
    }

    /**
     * {@inheritdoc}
     */
    public function createNew()
    {
        return $this->factory->createNew();
    }

    /**
     * {@inheritdoc}
     */
    public function createRetryJob(JobInterface $job): JobInterface
    {
        /** @var JobInterface $retryJob */
        $retryJob = $this->createNew();
        $retryJob->setState(JobInterface::STATE_PENDING);
        $retryJob->setOriginalJob($job);
        $retryJob->setSchedule($job->getSchedule());
        $retryJob->setCommand($job->getCommand());

It seems like $job->getCommand() can also be of type null; however, parameter $command of Setono\SyliusSchedulerPlugin\Model\JobInterface::setCommand() does only seem to accept string, maybe add an additional type check?

        $retryJob->setArgs($job->getArgs());

        $retryJob->setQueue($job->getQueue());
        $retryJob->setPriority($job->getPriority());

        $retryJob->setMaxRuntime($job->getMaxRuntime());

        return $retryJob;
    }

    /**
     * {@inheritdoc}
     */
    public function createFromSchedule(ScheduleInterface $schedule, $currentTime = 'now'): JobInterface
    {
        /** @var JobInterface $job */
        $job = $this->createNew();
        $job->setSchedule($schedule);
        $job->setCommand($schedule->getCommand());

It seems like $schedule->getCommand() can also be of type null; however, parameter $command of Setono\SyliusSchedulerPlugin\Model\JobInterface::setCommand() does only seem to accept string, maybe add an additional type check?

        $job->setArgs($schedule->getArgs());
        $job->setExecuteAfter($schedule->getNextRunDate($currentTime));

        return $job;
    }
}