PhpTek\Exodus\Job\StaticSiteRewriteLinksJob

Complexity

Total Complexity

4

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	2
A	process()	0	7	1
A	getJobType()	0	4	1

<?php

namespace PhpTek\Exodus\Job;

use PhpTek\Exodus\Task\StaticSiteRewriteLinksTask;
use SilverStripe\Core\Injector\Injectable;
use Symbiote\QueuedJobs\Jobs\AbstractQueuedJob;

The type Symbiote\QueuedJobs\Jobs\AbstractQueuedJob was not found. Maybe you did not declare it correctly or list all dependencies?

use Symbiote\QueuedJobs\Jobs\QueuedJob;

The type Symbiote\QueuedJobs\Jobs\QueuedJob was not found. Maybe you did not declare it correctly or list all dependencies?

/**
 *
 * A Queued jobs wrapper for StaticSiteRewriteLinksTask.
 *
 * @package phptek/silverstripe-exodus
 * @author Sam Minee <sam@silverstripe.com>
 * @author Russell Michell <russ@theruss.com>
 */

if (!class_exists(AbstractQueuedJob::class)) {
    return;
}

class StaticSiteRewriteLinksJob extends AbstractQueuedJob implements QueuedJob
{
    use Injectable;

    /**
     * The ID number of the StaticSiteContentSource which has the links to be rewritten
     *
     * @var int
     */
    protected $contentSourceID;

    /**
     *
     * Sets the content source id
     *
     * @param number $contentSourceID
     */
    public function __construct($contentSourceID = null)
    {
        if ($contentSourceID) {
            $this->contentSourceID = $contentSourceID;
        }
    }

    /**
     *
     * @return string
     */
    public function getJobType()
    {
        $this->totalSteps = 1;

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

        return QueuedJob::QUEUED;
    }

    /**
     * Starts the rewrite links task
     *
     * @return void
     */
    public function process()
    {
        $task = singleton(StaticSiteRewriteLinksTask::class);
        $task->setContentSourceID($this->contentSourceID);
        $task->process();
        $this->currentStep = 1;

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

        $this->isComplete = true;

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

    }
}