SilverStripe\QueuedJobs\Tasks\Engines\DoormanRunner

Complexity

Total Complexity

10

Size/Duplication

Total Lines	92
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	setDefaultRules()	0	4	1
A	getDefaultRules()	0	4	1
B	runQueue()	0	33	6
A	getNextJobDescriptorWithoutMutex()	0	22	2

<?php

namespace SilverStripe\QueuedJobs\Tasks\Engines;

use AsyncPHP\Doorman\Rule;
use SilverStripe\ORM\FieldType\DBDatetime;
use SilverStripe\QueuedJobs\DataObjects\QueuedJobDescriptor;
use SilverStripe\QueuedJobs\Jobs\DoormanProcessManager;
use SilverStripe\QueuedJobs\Jobs\DoormanQueuedJobTask;
use SilverStripe\QueuedJobs\Services\QueuedJob;

/**
 * Runs all jobs through the doorman engine
 */
class DoormanRunner extends BaseRunner implements TaskRunnerEngine
{

    /**
     * @var string
     */
    protected $defaultRules = array();

    /**
     * Assign default rules for this task
     *
     * @param array $rules
     */
    public function setDefaultRules($rules)
    {
        $this->defaultRules = $rules;

It seems like $rules of type array is incompatible with the declared type string of property $defaultRules.

    }

    /**
     * @return array List of rules
     */
    public function getDefaultRules()
    {
        return $this->defaultRules;
    }

    /**
     * Run tasks on the given queue
     *
     * @param string $queue
     */
    public function runQueue($queue)
    {

        // split jobs out into multiple tasks...

        $manager = new DoormanProcessManager();
        // $manager->setLogPath(__DIR__);

75% of this comment could be valid code. Did you maybe forget this after debugging?

        // Assign default rules
        $defaultRules = $this->getDefaultRules();
        if ($defaultRules) {
            foreach ($defaultRules as $rule) {

The expression $defaultRules of type string is not traversable.

                $manager->addRule($rule);
            }
        }

        $descriptor = $this->getNextJobDescriptorWithoutMutex($queue);

        while ($manager->tick() || $descriptor) {
            $this->logDescriptorStatus($descriptor, $queue);

It seems like $descriptor can also be of type object<SilverStripe\ORM\DataObject>; however, SilverStripe\QueuedJobs\Tasks\Engines\BaseRunner::logDescriptorStatus() does only seem to accept boolean|null|object<SilverStripe\QueuedJobs\DataObjects\QueuedJobDescriptor>, maybe add an additional type check?

            if ($descriptor instanceof QueuedJobDescriptor) {
                $descriptor->JobStatus = QueuedJob::STATUS_INIT;
                $descriptor->write();

                $manager->addTask(new DoormanQueuedJobTask($descriptor));
            }

            sleep(1);

            $descriptor = $this->getNextJobDescriptorWithoutMutex($queue);
        }
    }

    /**
     * @param string $queue
     * @return null|QueuedJobDescriptor
     */
    protected function getNextJobDescriptorWithoutMutex($queue)
    {
        $list = QueuedJobDescriptor::get()
            ->filter('JobType', $queue)
            ->sort('ID', 'ASC');

        $descriptor = $list
            ->filter('JobStatus', QueuedJob::STATUS_WAIT)
            ->first();

        if ($descriptor) {
            return $descriptor;
        }

        return $list
            ->filter('JobStatus', QueuedJob::STATUS_NEW)
            ->where(sprintf(
                '"StartAfter" < \'%s\' OR "StartAfter" IS NULL',
                DBDatetime::now()->getValue()
            ))
            ->first();
    }
}